<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40"> <head> <meta http-equiv="Content-Type" content="text/html; charset=us-ascii"> <meta name="Generator" content="Microsoft Word 15 (filtered medium)"> <style></style> </head> <body lang="EN-US" link="blue" vlink="purple"> <div class="WordSection1"> Hi Yong,<o:p></o:p> <o:p> </o:p> That has been submitted. <o:p></o:p> <o:p> </o:p> MC address is the address sent to memory controller bus to address the physical memory. In many places it is also called physical address.<o:p></o:p> <o:p> </o:p> Logical address is the address before it is translated to physical address. The translation can be either linear or through page table. Under VMID0, logical address of some apertures (AGP, FB) are linearly mapped to MC address. (I think in our driver implementation, logical address == MC address). But Gart aperture is mapped through gpuvm (gart) page table – the mapped address can also be called virtual address.<o:p></o:p> <o:p> </o:p> Under VMID1~15, it is user space virtual address (can also be called logical address), mapped through gpuvm page tables. <o:p></o:p> <o:p> </o:p> So virtual address is one type of logical address. Virtual address is usually mapped through page table/vm.<o:p></o:p> <o:p> </o:p> Just my understanding.<o:p></o:p> <o:p> </o:p> <div> Regards,<o:p></o:p> Oak<o:p></o:p> </div> <o:p> </o:p> <div> <div style="border:none;border-top:solid #E1E1E1 1.0pt;padding:3.0pt 0in 0in 0in"> From: Zhao, Yong <Yong.Zhao@amd.com> Sent: Thursday, November 7, 2019 12:16 PM To: Alex Deucher <alexdeucher@gmail.com>; Zeng, Oak <Oak.Zeng@amd.com> Cc: Kuehling, Felix <Felix.Kuehling@amd.com>; Koenig, Christian <Christian.Koenig@amd.com>; amd-gfx@lists.freedesktop.org Subject: Re: [PATCH] drm/amdgpu: Add comments to gmc structure<o:p></o:p> </div> </div> <o:p> </o:p> <div> If this is not submitted, I would like to see some comments regarding the explanation of MC address and logical address, which I prefer to mention as GPU physical/virtual address.<o:p></o:p> </div> <div> <o:p> </o:p> </div> <div> Regards,<o:p></o:p> </div> <div> Yong<o:p></o:p> </div> <div> <o:p> </o:p> </div> <div> <o:p> </o:p> </div> <div class="MsoNormal" align="center" style="text-align:center"> <hr size="2" width="98%" align="center"> </div> <div id="divRplyFwdMsg"> From: amd-gfx <<a href="mailto:amd-gfx-bounces@lists.freedesktop.org">amd-gfx-bounces@lists.freedesktop.org</a>> on behalf of Alex Deucher <<a href="mailto:alexdeucher@gmail.com">alexdeucher@gmail.com</a>> Sent: Thursday, November 7, 2019 9:02 AM To: Zeng, Oak <<a href="mailto:Oak.Zeng@amd.com">Oak.Zeng@amd.com</a>> Cc: Kuehling, Felix <<a href="mailto:Felix.Kuehling@amd.com">Felix.Kuehling@amd.com</a>>; Koenig, Christian <<a href="mailto:Christian.Koenig@amd.com">Christian.Koenig@amd.com</a>>; <a href="mailto:amd-gfx@lists.freedesktop.org">amd-gfx@lists.freedesktop.org</a> <<a href="mailto:amd-gfx@lists.freedesktop.org">amd-gfx@lists.freedesktop.org</a>> Subject: Re: [PATCH] drm/amdgpu: Add comments to gmc structure <o:p> </o:p> <div> <o:p></o:p> </div> </div> <div> <div> On Wed, Nov 6, 2019 at 12:27 PM Zeng, Oak <<a href="mailto:Oak.Zeng@amd.com">Oak.Zeng@amd.com</a>> wrote: > > Explain fields like aper_base, agp_start etc. The definition > of those fields are confusing as they are from different view > (CPU or GPU). Add comments for easier understand. > > Change-Id: I02c2a27cd0dbc205498eb86aafa722f2e0c25fe6 > Signed-off-by: Oak Zeng <<a href="mailto:Oak.Zeng@amd.com">Oak.Zeng@amd.com</a>> Same comments as the previous version. With those addressed, Reviewed-by: Alex Deucher <<a href="mailto:alexander.deucher@amd.com">alexander.deucher@amd.com</a>> > --- > drivers/gpu/drm/amd/amdgpu/amdgpu_gmc.h | 26 ++++++++++++++++++++++++++ > 1 file changed, 26 insertions(+) > > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_gmc.h b/drivers/gpu/drm/amd/amdgpu/amdgpu_gmc.h > index 555d8e5..1356ff9 100644 > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_gmc.h > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_gmc.h > @@ -127,18 +127,44 @@ struct amdgpu_xgmi { > }; > > struct amdgpu_gmc { > + /* FB's physical address in MMIO space (for CPU to > + * map FB). This is different compared to the apg/ > + * gart/vram_start/end field as the later is from > + * GPU's view and aper_base is from CPU's view. > + */ > resource_size_t aper_size; > resource_size_t aper_base; > /* for some chips with <= 32MB we need to lie > * about vram size near mc fb location */ > u64 mc_vram_size; > u64 visible_vram_size; > + /* APG aperture start and end in MC address space > + * Driver find a hole in the MC address space > + * to place AGP by setting MC_VM_AGP_BOT/TOP registers > + * Under VMID0, logical address == MC address > + * AGP aperture is used to simulate FB in ZFB case > + */ > u64 agp_size; > u64 agp_start; > u64 agp_end; > + /* GART aperture start and end in MC address space > + * Driver find a hole in the MC address space > + * to place GART by setting VM_CONTEXT0_PAGE_TABLE_START/END_ADDR > + * registers > + * Under VMID0, logical address inside GART aperture will > + * be translated through gpuvm gart page table to access > + * paged system memory > + */ > u64 gart_size; > u64 gart_start; > u64 gart_end; > + /* Frame buffer aperture of this GPU device. Different from > + * fb_start (see below), this only covers the local GPU device. > + * Driver get fb_start from MC_VM_FB_LOCATION_BASE (set by vbios) > + * and calculate vram_start of this local device by adding an > + * offset inside the XGMI hive. > + * Under VMID0, logical address == MC address > + */ > u64 vram_start; > u64 vram_end; > /* FB region , it's same as local vram region in single GPU, in XGMI > -- > 2.7.4 > > _______________________________________________ > amd-gfx mailing list > <a href="mailto:amd-gfx@lists.freedesktop.org">amd-gfx@lists.freedesktop.org</a> > <a href="https://lists.freedesktop.org/mailman/listinfo/amd-gfx">https://lists.freedesktop.org/mailman/listinfo/amd-gfx</a> _______________________________________________ amd-gfx mailing list <a href="mailto:amd-gfx@lists.freedesktop.org">amd-gfx@lists.freedesktop.org</a> <a href="https://lists.freedesktop.org/mailman/listinfo/amd-gfx">https://lists.freedesktop.org/mailman/listinfo/amd-gfx</a><o:p></o:p> </div> </div> </div> </body> </html>