<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:dt="uuid:C2F41010-65B3-11d1-A29F-00AA00C14882" 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=gb2312"> <meta name="Generator" content="Microsoft Word 15 (filtered medium)"> <style></style> </head> <body lang="EN-US" link="#0563C1" vlink="#954F72"> <div class="WordSection1"> Hi Ray,<o:p></o:p> Could you help to merge the patch of support 5 level paging iommu to edk2platform branch?<o:p></o:p> Do you need more check ?<o:p></o:p> Thank you.<o:p></o:p> BR<o:p></o:p> Sheng Wei<o:p></o:p> <o:p> </o:p> <div style="border:none;border-left:solid blue 1.5pt;padding:0cm 0cm 0cm 4.0pt"> <div> <div style="border:none;border-top:solid #E1E1E1 1.0pt;padding:3.0pt 0cm 0cm 0cm"> From: Sheng, W Sent: 2020年12月4日 11:44 To: Huang, Jenny <jenny.huang@intel.com>; Yao, Jiewen <jiewen.yao@intel.com>; devel@edk2.groups.io Cc: Ni, Ray <ray.ni@intel.com>; Chaganty, Rangasai V <rangasai.v.chaganty@intel.com>; Feng, Roger <roger.feng@intel.com> Subject: RE: [PATCH v5] IntelSiliconPkg/VTd: Add iommu 5 level paging support<o:p></o:p> </div> </div> <o:p> </o:p> Hi Jenny,<o:p></o:p> Thank you for the review.<o:p></o:p> I add your name under " Reviewed-by " to the patch.<o:p></o:p> <o:p> </o:p> <a id="OWAAMDAC086C12E27431687362E9BB699D829" href="mailto:jiewen.yao@intel.com">@Yao, Jiewen</a> <o:p></o:p> Do you have more comments about this patch?<o:p></o:p> Thank you.<o:p></o:p> BR<o:p></o:p> Sheng Wei<o:p></o:p> <o:p> </o:p> > -----Original Message-----<o:p></o:p> > From: Huang, Jenny <<a href="mailto:jenny.huang@intel.com">jenny.huang@intel.com</a>><o:p></o:p> > Sent: 2020年12月4日 11:33<o:p></o:p> > To: Sheng, W <<a href="mailto:w.sheng@intel.com">w.sheng@intel.com</a>>; Yao, Jiewen <<a href="mailto:jiewen.yao@intel.com">jiewen.yao@intel.com</a>>;<o:p></o:p> > <a href="mailto:devel@edk2.groups.io">devel@edk2.groups.io</a><o:p></o:p> > Cc: Ni, Ray <<a href="mailto:ray.ni@intel.com">ray.ni@intel.com</a>>; Chaganty, Rangasai V<o:p></o:p> > <<a href="mailto:rangasai.v.chaganty@intel.com">rangasai.v.chaganty@intel.com</a>>; Feng, Roger <<a href="mailto:roger.feng@intel.com">roger.feng@intel.com</a>><o:p></o:p> > Subject: RE: [PATCH v5] IntelSiliconPkg/VTd: Add iommu 5 level paging support<o:p></o:p> > <o:p></o:p> > Reviewed-by: Jenny Huang < <a href="mailto:jenny.huang@intel.com"> jenny.huang@intel.com</a>><o:p></o:p> > <o:p></o:p> > -----Original Message-----<o:p></o:p> > From: Sheng, W <<a href="mailto:w.sheng@intel.com">w.sheng@intel.com</a>><o:p></o:p> > Sent: Tuesday, December 1, 2020 7:03 PM<o:p></o:p> > To: Yao, Jiewen <<a href="mailto:jiewen.yao@intel.com">jiewen.yao@intel.com</a>>; <a href="mailto:devel@edk2.groups.io">devel@edk2.groups.io</a><o:p></o:p> > Cc: Ni, Ray <<a href="mailto:ray.ni@intel.com">ray.ni@intel.com</a>>; Chaganty, Rangasai V<o:p></o:p> > <<a href="mailto:rangasai.v.chaganty@intel.com">rangasai.v.chaganty@intel.com</a>>; Huang, Jenny <<a href="mailto:jenny.huang@intel.com">jenny.huang@intel.com</a>>;<o:p></o:p> > Feng, Roger <<a href="mailto:roger.feng@intel.com">roger.feng@intel.com</a>><o:p></o:p> > Subject: [PATCH v5] IntelSiliconPkg/VTd: Add iommu 5 level paging support<o:p></o:p> > <o:p></o:p> > Hi Jiewen,<o:p></o:p> > About the patch of support 5 level paging iommu.<o:p></o:p> > Thank you for giving the review comments.<o:p></o:p> > I have done all the update.<o:p></o:p> > Could you give "review by" on this patch ?<o:p></o:p> > Thank you.<o:p></o:p> > BR<o:p></o:p> > Sheng Wei<o:p></o:p> > <o:p></o:p> > <o:p></o:p> > > -----Original Message-----<o:p></o:p> > > From: Sheng, W<o:p></o:p> > > Sent: 2020年11月24日 13:44<o:p></o:p> > > To: Yao, Jiewen <<a href="mailto:jiewen.yao@intel.com">jiewen.yao@intel.com</a>>; <a href="mailto:devel@edk2.groups.io">devel@edk2.groups.io</a><o:p></o:p> > > Cc: Ni, Ray <<a href="mailto:ray.ni@intel.com">ray.ni@intel.com</a>>; Chaganty, Rangasai V<o:p></o:p> > > <<a href="mailto:rangasai.v.chaganty@intel.com">rangasai.v.chaganty@intel.com</a>>; Huang, Jenny <<a href="mailto:jenny.huang@intel.com">jenny.huang@intel.com</a>><o:p></o:p> > > Subject: RE: [PATCH v4] IntelSiliconPkg/VTd: Add iommu 5 level paging<o:p></o:p> > > support<o:p></o:p> > ><o:p></o:p> > > Hi Jiewen, All,<o:p></o:p> > > Thank you for the review. I just check and update the patch.<o:p></o:p> > > <a href="https://edk2.groups.io/g/devel/message/67865?p=,,,20,0,0,0::relevance"> https://edk2.groups.io/g/devel/message/67865?p=,,,20,0,0,0::relevance</a>,<o:p></o:p> > > ,posteri<o:p></o:p> > > d%3A2558558,20,2,0,78471874<o:p></o:p> > > Could we continue the patch review ?<o:p></o:p> > > BR<o:p></o:p> > > Sheng Wei<o:p></o:p> > ><o:p></o:p> > > > -----Original Message-----<o:p></o:p> > > > From: Yao, Jiewen <<a href="mailto:jiewen.yao@intel.com">jiewen.yao@intel.com</a>><o:p></o:p> > > > Sent: 2020年11月23日 16:44<o:p></o:p> > > > To: Sheng, W <<a href="mailto:w.sheng@intel.com">w.sheng@intel.com</a>>; <a href="mailto:devel@edk2.groups.io">devel@edk2.groups.io</a><o:p></o:p> > > > Cc: Ni, Ray <<a href="mailto:ray.ni@intel.com">ray.ni@intel.com</a>>; Chaganty, Rangasai V<o:p></o:p> > > > <<a href="mailto:rangasai.v.chaganty@intel.com">rangasai.v.chaganty@intel.com</a>>; Huang, Jenny<o:p></o:p> > > > <<a href="mailto:jenny.huang@intel.com">jenny.huang@intel.com</a>><o:p></o:p> > > > Subject: RE: [PATCH v4] IntelSiliconPkg/VTd: Add iommu 5 level<o:p></o:p> > > > paging support<o:p></o:p> > > ><o:p></o:p> > > > Thanks.<o:p></o:p> > > ><o:p></o:p> > > > I only reviewed the policy part. Comment below:<o:p></o:p> > > ><o:p></o:p> > > > 1) I recommend you can merge below 2 if into one - if<o:p></o:p> > > > ((mAcpiDmarTable-<o:p></o:p> > > > >HostAddressWidth <= 48) &&<o:p></o:p> > > > (mVtdUnitInformation[VtdIndex].CapReg.Bits.SAGAW & BIT2) != 0)) {<o:p></o:p> > > > You can use 2 lines, but there is no need to use 2 if.<o:p></o:p> > > ><o:p></o:p> > > > + if (mAcpiDmarTable->HostAddressWidth <= 48) {<o:p></o:p> > > > + if ((mVtdUnitInformation[VtdIndex].CapReg.Bits.SAGAW & BIT2) != 0)<o:p></o:p> > {<o:p></o:p> > > > + mVtdUnitInformation[VtdIndex].Is5LevelPaging = FALSE;<o:p></o:p> > > > + }<o:p></o:p> > > > + }<o:p></o:p> > > ><o:p></o:p> > > > 2) I think below code has typo.<o:p></o:p> > > > The DEBUG message about 4-level and 5-level should be reversed.<o:p></o:p> > > > Also we should use DEBUG_INFO instead of DEBUG_ERROR.<o:p></o:p> > > ><o:p></o:p> > > > + if (mVtdUnitInformation[VtdIndex].Is5LevelPaging) {<o:p></o:p> > > > + ContextEntry->Bits.AddressWidth = 0x3;<o:p></o:p> > > > + DEBUG((DEBUG_ERROR, "Using 4-level page-table on VTD %d\n",<o:p></o:p> > > > VtdIndex));<o:p></o:p> > > > + } else {<o:p></o:p> > > > + ContextEntry->Bits.AddressWidth = 0x2;<o:p></o:p> > > > + DEBUG((DEBUG_ERROR, "Using 5-level page-table on VTD %d\n",<o:p></o:p> > > > VtdIndex));<o:p></o:p> > > > + }<o:p></o:p> > > ><o:p></o:p> > > ><o:p></o:p> > > ><o:p></o:p> > > ><o:p></o:p> > > > > -----Original Message-----<o:p></o:p> > > > > From: Sheng, W <<a href="mailto:w.sheng@intel.com">w.sheng@intel.com</a>><o:p></o:p> > > > > Sent: Monday, November 23, 2020 4:04 PM<o:p></o:p> > > > > To: <a href="mailto:devel@edk2.groups.io">devel@edk2.groups.io</a><o:p></o:p> > > > > Cc: Ni, Ray <<a href="mailto:ray.ni@intel.com">ray.ni@intel.com</a>>; Chaganty, Rangasai V<o:p></o:p> > > > > <<a href="mailto:rangasai.v.chaganty@intel.com">rangasai.v.chaganty@intel.com</a>>; Yao, Jiewen<o:p></o:p> > > > > <<a href="mailto:jiewen.yao@intel.com">jiewen.yao@intel.com</a>>; Huang, Jenny <<a href="mailto:jenny.huang@intel.com">jenny.huang@intel.com</a>><o:p></o:p> > > > > Subject: [PATCH v4] IntelSiliconPkg/VTd: Add iommu 5 level paging<o:p></o:p> > > > > support<o:p></o:p> > > > ><o:p></o:p> > > > > Support iommu 5 level paging for translation table.<o:p></o:p> > > > ><o:p></o:p> > > > > REF: <a href="https://bugzilla.tianocore.org/show_bug.cgi?id=3067"> https://bugzilla.tianocore.org/show_bug.cgi?id=3067</a><o:p></o:p> > > > ><o:p></o:p> > > > > Signed-off-by: Sheng Wei <<a href="mailto:w.sheng@intel.com">w.sheng@intel.com</a>><o:p></o:p> > > > > Cc: Ray Ni <<a href="mailto:ray.ni@intel.com">ray.ni@intel.com</a>><o:p></o:p> > > > > Cc: Rangasai V Chaganty <<a href="mailto:rangasai.v.chaganty@intel.com">rangasai.v.chaganty@intel.com</a>><o:p></o:p> > > > > Cc: Jiewen Yao <<a href="mailto:jiewen.yao@intel.com">jiewen.yao@intel.com</a>><o:p></o:p> > > > > Cc: Jenny Huang <<a href="mailto:jenny.huang@intel.com">jenny.huang@intel.com</a>><o:p></o:p> > > > > ---<o:p></o:p> > > > > .../Feature/VTd/IntelVTdDxe/DmaProtection.c | 4 +-<o:p></o:p> > > > > .../Feature/VTd/IntelVTdDxe/DmaProtection.h | 19 +-<o:p></o:p> > > > > .../Feature/VTd/IntelVTdDxe/TranslationTable.c | 281<o:p></o:p> > > +++++++++++++++--<o:p></o:p> > > > > ----<o:p></o:p> > > > > .../Feature/VTd/IntelVTdDxe/TranslationTableEx.c | 31 ++-<o:p></o:p> > > > > .../Feature/VTd/IntelVTdDxe/VtdReg.c | 10 +-<o:p></o:p> > > > > 5 files changed, 245 insertions(+), 100 deletions(-)<o:p></o:p> > > > ><o:p></o:p> > > > > diff --git<o:p></o:p> > > > > a/Silicon/Intel/IntelSiliconPkg/Feature/VTd/IntelVTdDxe/DmaProtection.<o:p></o:p> > > > > c<o:p></o:p> > > > > b/Silicon/Intel/IntelSiliconPkg/Feature/VTd/IntelVTdDxe/DmaProtection.<o:p></o:p> > > > > c<o:p></o:p> > > > > index 9b6135ef..628565ee 100644<o:p></o:p> > > > > ---<o:p></o:p> > > > > a/Silicon/Intel/IntelSiliconPkg/Feature/VTd/IntelVTdDxe/DmaProtection.<o:p></o:p> > > > > c<o:p></o:p> > > > > +++<o:p></o:p> > > > > b/Silicon/Intel/IntelSiliconPkg/Feature/VTd/IntelVTdDxe/DmaProtection.<o:p></o:p> > > > > c<o:p></o:p> > > > > @@ -523,10 +523,10 @@ SetupVtd (<o:p></o:p> > > > > for (Index = 0; Index < mVtdUnitNumber; Index++) {<o:p></o:p> > > > > DEBUG ((DEBUG_INFO,"VTD Unit %d (Segment: %04x)\n", Index,<o:p></o:p> > > > > mVtdUnitInformation[Index].Segment));<o:p></o:p> > > > > if (mVtdUnitInformation[Index].ExtRootEntryTable != NULL) {<o:p></o:p> > > > > - DumpDmarExtContextEntryTable<o:p></o:p> > > > > (mVtdUnitInformation[Index].ExtRootEntryTable);<o:p></o:p> > > > > + DumpDmarExtContextEntryTable<o:p></o:p> > > > > (mVtdUnitInformation[Index].ExtRootEntryTable,<o:p></o:p> > > > > mVtdUnitInformation[Index].Is5LevelPaging);<o:p></o:p> > > > > }<o:p></o:p> > > > > if (mVtdUnitInformation[Index].RootEntryTable != NULL) {<o:p></o:p> > > > > - DumpDmarContextEntryTable<o:p></o:p> > > > > (mVtdUnitInformation[Index].RootEntryTable);<o:p></o:p> > > > > + DumpDmarContextEntryTable<o:p></o:p> > > > > (mVtdUnitInformation[Index].RootEntryTable,<o:p></o:p> > > > > mVtdUnitInformation[Index].Is5LevelPaging);<o:p></o:p> > > > > }<o:p></o:p> > > > > }<o:p></o:p> > > > ><o:p></o:p> > > > > diff --git<o:p></o:p> > > > > a/Silicon/Intel/IntelSiliconPkg/Feature/VTd/IntelVTdDxe/DmaProtection.<o:p></o:p> > > > > h<o:p></o:p> > > > > b/Silicon/Intel/IntelSiliconPkg/Feature/VTd/IntelVTdDxe/DmaProtection.<o:p></o:p> > > > > h<o:p></o:p> > > > > index a3331db8..f641cea0 100644<o:p></o:p> > > > > ---<o:p></o:p> > > > > a/Silicon/Intel/IntelSiliconPkg/Feature/VTd/IntelVTdDxe/DmaProtection.<o:p></o:p> > > > > h<o:p></o:p> > > > > +++<o:p></o:p> > > > > b/Silicon/Intel/IntelSiliconPkg/Feature/VTd/IntelVTdDxe/DmaProtection.<o:p></o:p> > > > > h<o:p></o:p> > > > > @@ -77,6 +77,7 @@ typedef struct {<o:p></o:p> > > > > BOOLEAN HasDirtyContext;<o:p></o:p> > > > > BOOLEAN HasDirtyPages;<o:p></o:p> > > > > PCI_DEVICE_INFORMATION PciDeviceInfo;<o:p></o:p> > > > > + BOOLEAN Is5LevelPaging;<o:p></o:p> > > > > } VTD_UNIT_INFORMATION;<o:p></o:p> > > > ><o:p></o:p> > > > > //<o:p></o:p> > > > > @@ -375,31 +376,37 @@ ParseDmarAcpiTableRmrr (<o:p></o:p> > > > > /**<o:p></o:p> > > > > Dump DMAR context entry table.<o:p></o:p> > > > ><o:p></o:p> > > > > - @param[in] RootEntry DMAR root entry.<o:p></o:p> > > > > + @param[in] RootEntry DMAR root entry.<o:p></o:p> > > > > + @param[in] Is5LevelPaging If it is the 5 level paging.<o:p></o:p> > > > > **/<o:p></o:p> > > > > VOID<o:p></o:p> > > > > DumpDmarContextEntryTable (<o:p></o:p> > > > > - IN VTD_ROOT_ENTRY *RootEntry<o:p></o:p> > > > > + IN VTD_ROOT_ENTRY *RootEntry,<o:p></o:p> > > > > + IN BOOLEAN Is5LevelPaging<o:p></o:p> > > > > );<o:p></o:p> > > > ><o:p></o:p> > > > > /**<o:p></o:p> > > > > Dump DMAR extended context entry table.<o:p></o:p> > > > ><o:p></o:p> > > > > - @param[in] ExtRootEntry DMAR extended root entry.<o:p></o:p> > > > > + @param[in] ExtRootEntry DMAR extended root entry.<o:p></o:p> > > > > + @param[in] Is5LevelPaging If it is the 5 level paging.<o:p></o:p> > > > > **/<o:p></o:p> > > > > VOID<o:p></o:p> > > > > DumpDmarExtContextEntryTable (<o:p></o:p> > > > > - IN VTD_EXT_ROOT_ENTRY *ExtRootEntry<o:p></o:p> > > > > + IN VTD_EXT_ROOT_ENTRY *ExtRootEntry, IN BOOLEAN Is5LevelPaging<o:p></o:p> > > > > );<o:p></o:p> > > > ><o:p></o:p> > > > > /**<o:p></o:p> > > > > Dump DMAR second level paging entry.<o:p></o:p> > > > ><o:p></o:p> > > > > - @param[in] SecondLevelPagingEntry The second level paging entry.<o:p></o:p> > > > > + @param[in] SecondLevelPagingEntry The second level paging entry.<o:p></o:p> > > > > + @param[in] Is5LevelPaging If it is the 5 level paging.<o:p></o:p> > > > > **/<o:p></o:p> > > > > VOID<o:p></o:p> > > > > DumpSecondLevelPagingEntry (<o:p></o:p> > > > > - IN VOID *SecondLevelPagingEntry<o:p></o:p> > > > > + IN VOID *SecondLevelPagingEntry, IN BOOLEAN Is5LevelPaging<o:p></o:p> > > > > );<o:p></o:p> > > > ><o:p></o:p> > > > > /**<o:p></o:p> > > > > diff --git<o:p></o:p> > > > > a/Silicon/Intel/IntelSiliconPkg/Feature/VTd/IntelVTdDxe/Translatio<o:p></o:p> > > > > nT<o:p></o:p> > > > > ab<o:p></o:p> > > > > le.c<o:p></o:p> > > > > b/Silicon/Intel/IntelSiliconPkg/Feature/VTd/IntelVTdDxe/Translatio<o:p></o:p> > > > > nT<o:p></o:p> > > > > ab<o:p></o:p> > > > > le.c<o:p></o:p> > > > > index 201d663d..6c786b40 100644<o:p></o:p> > > > > ---<o:p></o:p> > > > > a/Silicon/Intel/IntelSiliconPkg/Feature/VTd/IntelVTdDxe/Translatio<o:p></o:p> > > > > nT<o:p></o:p> > > > > ab<o:p></o:p> > > > > le.c<o:p></o:p> > > > > +++<o:p></o:p> > > > > b/Silicon/Intel/IntelSiliconPkg/Feature/VTd/IntelVTdDxe/Translatio<o:p></o:p> > > > > nT ab le.c @@ -128,11 +128,26 @@ CreateContextEntry (<o:p></o:p> > > > ><o:p></o:p> > > > > DEBUG ((DEBUG_INFO,"Source: S%04x B%02x D%02x F%02x\n",<o:p></o:p> > > > > mVtdUnitInformation[VtdIndex].Segment, SourceId.Bits.Bus,<o:p></o:p> > > > > SourceId.Bits.Device, SourceId.Bits.Function));<o:p></o:p> > > > ><o:p></o:p> > > > > - if ((mVtdUnitInformation[VtdIndex].CapReg.Bits.SAGAW & BIT2) == 0)<o:p></o:p> > {<o:p></o:p> > > > > - DEBUG((DEBUG_ERROR, "!!!! 4-level page-table is not supported on<o:p></o:p> > > > > VTD %d !!!!\n", VtdIndex));<o:p></o:p> > > > > + mVtdUnitInformation[VtdIndex].Is5LevelPaging = FALSE;<o:p></o:p> > > > > + if ((mVtdUnitInformation[VtdIndex].CapReg.Bits.SAGAW & BIT3) != 0)<o:p></o:p> > {<o:p></o:p> > > > > + mVtdUnitInformation[VtdIndex].Is5LevelPaging = TRUE;<o:p></o:p> > > > > + if (mAcpiDmarTable->HostAddressWidth <= 48) {<o:p></o:p> > > > > + if ((mVtdUnitInformation[VtdIndex].CapReg.Bits.SAGAW &<o:p></o:p> > > > > + BIT2) != 0)<o:p></o:p> > > {<o:p></o:p> > > > > + mVtdUnitInformation[VtdIndex].Is5LevelPaging = FALSE;<o:p></o:p> > > > > + }<o:p></o:p> > > > > + }<o:p></o:p> > > > > + } else if ((mVtdUnitInformation[VtdIndex].CapReg.Bits.SAGAW &<o:p></o:p> > > > > + BIT2) ==<o:p></o:p> > > > > 0) {<o:p></o:p> > > > > + DEBUG((DEBUG_ERROR, "!!!! Page-table type is not supported<o:p></o:p> > > > > + on<o:p></o:p> > > > > VTD %d !!!!\n", VtdIndex));<o:p></o:p> > > > > return EFI_UNSUPPORTED;<o:p></o:p> > > > > }<o:p></o:p> > > > > - ContextEntry->Bits.AddressWidth = 0x2;<o:p></o:p> > > > > +<o:p></o:p> > > > > + if (mVtdUnitInformation[VtdIndex].Is5LevelPaging) {<o:p></o:p> > > > > + ContextEntry->Bits.AddressWidth = 0x3;<o:p></o:p> > > > > + DEBUG((DEBUG_ERROR, "Using 4-level page-table on VTD %d\n",<o:p></o:p> > > > > VtdIndex));<o:p></o:p> > > > > + } else {<o:p></o:p> > > > > + ContextEntry->Bits.AddressWidth = 0x2;<o:p></o:p> > > > > + DEBUG((DEBUG_ERROR, "Using 5-level page-table on VTD %d\n",<o:p></o:p> > > > > VtdIndex));<o:p></o:p> > > > > + }<o:p></o:p> > > > > }<o:p></o:p> > > > ><o:p></o:p> > > > > FlushPageTableMemory (VtdIndex,<o:p></o:p> > > > > (UINTN)mVtdUnitInformation[VtdIndex].RootEntryTable,<o:p></o:p> > > > > EFI_PAGES_TO_SIZE(EntryTablePages));<o:p></o:p> > > > > @@ -148,6 +163,7 @@ CreateContextEntry (<o:p></o:p> > > > > @param[in] MemoryBase The base of the memory.<o:p></o:p> > > > > @param[in] MemoryLimit The limit of the memory.<o:p></o:p> > > > > @param[in] IoMmuAccess The IOMMU access.<o:p></o:p> > > > > + @param[in] Is5LevelPaging If it is the 5 level paging.<o:p></o:p> > > > ><o:p></o:p> > > > > @return The second level paging entry.<o:p></o:p> > > > > **/<o:p></o:p> > > > > @@ -157,16 +173,23 @@ CreateSecondLevelPagingEntryTable (<o:p></o:p> > > > > IN VTD_SECOND_LEVEL_PAGING_ENTRY *SecondLevelPagingEntry,<o:p></o:p> > > > > IN UINT64 MemoryBase,<o:p></o:p> > > > > IN UINT64 MemoryLimit,<o:p></o:p> > > > > - IN UINT64 IoMmuAccess<o:p></o:p> > > > > + IN UINT64 IoMmuAccess,<o:p></o:p> > > > > + IN BOOLEAN Is5LevelPaging<o:p></o:p> > > > > )<o:p></o:p> > > > > {<o:p></o:p> > > > > + UINTN Index5;<o:p></o:p> > > > > UINTN Index4;<o:p></o:p> > > > > UINTN Index3;<o:p></o:p> > > > > UINTN Index2;<o:p></o:p> > > > > + UINTN Lvl5Start;<o:p></o:p> > > > > + UINTN Lvl5End;<o:p></o:p> > > > > + UINTN Lvl4PagesStart;<o:p></o:p> > > > > + UINTN Lvl4PagesEnd;<o:p></o:p> > > > > UINTN Lvl4Start;<o:p></o:p> > > > > UINTN Lvl4End;<o:p></o:p> > > > > UINTN Lvl3Start;<o:p></o:p> > > > > UINTN Lvl3End;<o:p></o:p> > > > > + VTD_SECOND_LEVEL_PAGING_ENTRY *Lvl5PtEntry;<o:p></o:p> > > > > VTD_SECOND_LEVEL_PAGING_ENTRY *Lvl4PtEntry;<o:p></o:p> > > > > VTD_SECOND_LEVEL_PAGING_ENTRY *Lvl3PtEntry;<o:p></o:p> > > > > VTD_SECOND_LEVEL_PAGING_ENTRY *Lvl2PtEntry; @@ -184,7 +207,7<o:p></o:p> > > > @@<o:p></o:p> > > > > CreateSecondLevelPagingEntryTable (<o:p></o:p> > > > > if (SecondLevelPagingEntry == NULL) {<o:p></o:p> > > > > SecondLevelPagingEntry = AllocateZeroPages (1);<o:p></o:p> > > > > if (SecondLevelPagingEntry == NULL) {<o:p></o:p> > > > > - DEBUG ((DEBUG_ERROR,"Could not Alloc LVL4 PT. \n"));<o:p></o:p> > > > > + DEBUG ((DEBUG_ERROR,"Could not Alloc LVL4 or LVL5 PT.<o:p></o:p> > > > > + \n"));<o:p></o:p> > > > > return NULL;<o:p></o:p> > > > > }<o:p></o:p> > > > > FlushPageTableMemory (VtdIndex,<o:p></o:p> > > > > (UINTN)SecondLevelPagingEntry, EFI_PAGES_TO_SIZE(1)); @@ -197,66<o:p></o:p> > > > > +220,109 @@ CreateSecondLevelPagingEntryTable (<o:p></o:p> > > > > return SecondLevelPagingEntry;<o:p></o:p> > > > > }<o:p></o:p> > > > ><o:p></o:p> > > > > - Lvl4Start = RShiftU64 (BaseAddress, 39) & 0x1FF;<o:p></o:p> > > > > - Lvl4End = RShiftU64 (EndAddress - 1, 39) & 0x1FF;<o:p></o:p> > > > > + if (Is5LevelPaging) {<o:p></o:p> > > > > + Lvl5Start = RShiftU64 (BaseAddress, 48) & 0x1FF;<o:p></o:p> > > > > + Lvl5End = RShiftU64 (EndAddress - 1, 48) & 0x1FF;<o:p></o:p> > > > > + DEBUG ((DEBUG_INFO," Lvl5Start - 0x%x, Lvl5End - 0x%x\n",<o:p></o:p> > > > > + Lvl5Start,<o:p></o:p> > > > > Lvl5End));<o:p></o:p> > > > ><o:p></o:p> > > > > - DEBUG ((DEBUG_INFO," Lvl4Start - 0x%x, Lvl4End - 0x%x\n",<o:p></o:p> > > > > Lvl4Start, Lvl4End));<o:p></o:p> > > > > + Lvl4Start = RShiftU64 (BaseAddress, 39) & 0x1FF;<o:p></o:p> > > > > + Lvl4End = RShiftU64 (EndAddress - 1, 39) & 0x1FF;<o:p></o:p> > > > ><o:p></o:p> > > > > - Lvl4PtEntry = (VTD_SECOND_LEVEL_PAGING_ENTRY<o:p></o:p> > > > > *)SecondLevelPagingEntry;<o:p></o:p> > > > > - for (Index4 = Lvl4Start; Index4 <= Lvl4End; Index4++) {<o:p></o:p> > > > > - if (Lvl4PtEntry[Index4].Uint64 == 0) {<o:p></o:p> > > > > - Lvl4PtEntry[Index4].Uint64 = (UINT64)(UINTN)AllocateZeroPages (1);<o:p></o:p> > > > > - if (Lvl4PtEntry[Index4].Uint64 == 0) {<o:p></o:p> > > > > - DEBUG ((DEBUG_ERROR,"!!!!!! ALLOCATE LVL4 PAGE FAIL<o:p></o:p> > > > > (0x%x)!!!!!!\n", Index4));<o:p></o:p> > > > > - ASSERT(FALSE);<o:p></o:p> > > > > - return NULL;<o:p></o:p> > > > > - }<o:p></o:p> > > > > - FlushPageTableMemory (VtdIndex,<o:p></o:p> > (UINTN)Lvl4PtEntry[Index4].Uint64,<o:p></o:p> > > > > SIZE_4KB);<o:p></o:p> > > > > - SetSecondLevelPagingEntryAttribute (&Lvl4PtEntry[Index4],<o:p></o:p> > > > > EDKII_IOMMU_ACCESS_READ | EDKII_IOMMU_ACCESS_WRITE);<o:p></o:p> > > > > - }<o:p></o:p> > > > > + Lvl4PagesStart = (Lvl5Start<<9) | Lvl4Start;<o:p></o:p> > > > > + Lvl4PagesEnd = (Lvl5End<<9) | Lvl4End;<o:p></o:p> > > > > + DEBUG ((DEBUG_INFO," Lvl4PagesStart - 0x%x, Lvl4PagesEnd -<o:p></o:p> > > > > + 0x%x\n",<o:p></o:p> > > > > Lvl4PagesStart, Lvl4PagesEnd));<o:p></o:p> > > > ><o:p></o:p> > > > > - Lvl3Start = RShiftU64 (BaseAddress, 30) & 0x1FF;<o:p></o:p> > > > > - if (ALIGN_VALUE_LOW(BaseAddress + SIZE_1GB, SIZE_1GB) <=<o:p></o:p> > > > > EndAddress) {<o:p></o:p> > > > > - Lvl3End = SIZE_4KB/sizeof(VTD_SECOND_LEVEL_PAGING_ENTRY) - 1;<o:p></o:p> > > > > - } else {<o:p></o:p> > > > > - Lvl3End = RShiftU64 (EndAddress - 1, 30) & 0x1FF;<o:p></o:p> > > > > + Lvl5PtEntry = (VTD_SECOND_LEVEL_PAGING_ENTRY<o:p></o:p> > > > > *)SecondLevelPagingEntry;<o:p></o:p> > > > > + } else {<o:p></o:p> > > > > + Lvl5Start = RShiftU64 (BaseAddress, 48) & 0x1FF;<o:p></o:p> > > > > + Lvl5End = Lvl5Start;<o:p></o:p> > > > > +<o:p></o:p> > > > > + Lvl4Start = RShiftU64 (BaseAddress, 39) & 0x1FF;<o:p></o:p> > > > > + Lvl4End = RShiftU64 (EndAddress - 1, 39) & 0x1FF;<o:p></o:p> > > > > + DEBUG ((DEBUG_INFO," Lvl4Start - 0x%x, Lvl4End - 0x%x\n",<o:p></o:p> > > > > + Lvl4Start,<o:p></o:p> > > > > Lvl4End));<o:p></o:p> > > > > +<o:p></o:p> > > > > + Lvl4PtEntry = (VTD_SECOND_LEVEL_PAGING_ENTRY<o:p></o:p> > > > > *)SecondLevelPagingEntry;<o:p></o:p> > > > > + }<o:p></o:p> > > > > +<o:p></o:p> > > > > + for (Index5 = Lvl5Start; Index5 <= Lvl5End; Index5++) {<o:p></o:p> > > > > + if (Is5LevelPaging) {<o:p></o:p> > > > > + if (Lvl5PtEntry[Index5].Uint64 == 0) {<o:p></o:p> > > > > + Lvl5PtEntry[Index5].Uint64 = (UINT64)(UINTN)AllocateZeroPages (1);<o:p></o:p> > > > > + if (Lvl5PtEntry[Index5].Uint64 == 0) {<o:p></o:p> > > > > + DEBUG ((DEBUG_ERROR,"!!!!!! ALLOCATE LVL4 PAGE FAIL<o:p></o:p> > > > > (0x%x)!!!!!!\n", Index5));<o:p></o:p> > > > > + ASSERT(FALSE);<o:p></o:p> > > > > + return NULL;<o:p></o:p> > > > > + }<o:p></o:p> > > > > + FlushPageTableMemory (VtdIndex,<o:p></o:p> > > > > + (UINTN)Lvl5PtEntry[Index5].Uint64,<o:p></o:p> > > > > SIZE_4KB);<o:p></o:p> > > > > + SetSecondLevelPagingEntryAttribute (&Lvl5PtEntry[Index5],<o:p></o:p> > > > > EDKII_IOMMU_ACCESS_READ | EDKII_IOMMU_ACCESS_WRITE);<o:p></o:p> > > > > + }<o:p></o:p> > > > > + Lvl4Start = Lvl4PagesStart & 0x1FF;<o:p></o:p> > > > > + if (((Index5+1)<<9) > Lvl4PagesEnd) {<o:p></o:p> > > > > + Lvl4End = SIZE_4KB/sizeof(VTD_SECOND_LEVEL_PAGING_ENTRY) -<o:p></o:p> > 1;;<o:p></o:p> > > > > + Lvl4PagesStart = (Index5+1)<<9;<o:p></o:p> > > > > + } else {<o:p></o:p> > > > > + Lvl4End = Lvl4PagesEnd & 0x1FF;<o:p></o:p> > > > > + }<o:p></o:p> > > > > + DEBUG ((DEBUG_INFO," Lvl5(0x%x): Lvl4Start - 0x%x, Lvl4End<o:p></o:p> > > > > + - 0x%x\n",<o:p></o:p> > > > > Index5, Lvl4Start, Lvl4End));<o:p></o:p> > > > > + Lvl4PtEntry = (VTD_SECOND_LEVEL_PAGING_ENTRY<o:p></o:p> > > > > *)(UINTN)VTD_64BITS_ADDRESS(Lvl5PtEntry[Index5].Bits.AddressLo,<o:p></o:p> > > > > Lvl5PtEntry[Index5].Bits.AddressHi);<o:p></o:p> > > > > }<o:p></o:p> > > > > - DEBUG ((DEBUG_INFO," Lvl4(0x%x): Lvl3Start - 0x%x, Lvl3End -<o:p></o:p> > 0x%x\n",<o:p></o:p> > > > > Index4, Lvl3Start, Lvl3End));<o:p></o:p> > > > ><o:p></o:p> > > > > - Lvl3PtEntry = (VTD_SECOND_LEVEL_PAGING_ENTRY<o:p></o:p> > > > > *)(UINTN)VTD_64BITS_ADDRESS(Lvl4PtEntry[Index4].Bits.AddressLo,<o:p></o:p> > > > > Lvl4PtEntry[Index4].Bits.AddressHi);<o:p></o:p> > > > > - for (Index3 = Lvl3Start; Index3 <= Lvl3End; Index3++) {<o:p></o:p> > > > > - if (Lvl3PtEntry[Index3].Uint64 == 0) {<o:p></o:p> > > > > - Lvl3PtEntry[Index3].Uint64 = (UINT64)(UINTN)AllocateZeroPages (1);<o:p></o:p> > > > > - if (Lvl3PtEntry[Index3].Uint64 == 0) {<o:p></o:p> > > > > - DEBUG ((DEBUG_ERROR,"!!!!!! ALLOCATE LVL3 PAGE FAIL (0x%x,<o:p></o:p> > > > > 0x%x)!!!!!!\n", Index4, Index3));<o:p></o:p> > > > > + for (Index4 = Lvl4Start; Index4 <= Lvl4End; Index4++) {<o:p></o:p> > > > > + if (Lvl4PtEntry[Index4].Uint64 == 0) {<o:p></o:p> > > > > + Lvl4PtEntry[Index4].Uint64 = (UINT64)(UINTN)AllocateZeroPages (1);<o:p></o:p> > > > > + if (Lvl4PtEntry[Index4].Uint64 == 0) {<o:p></o:p> > > > > + DEBUG ((DEBUG_ERROR,"!!!!!! ALLOCATE LVL4 PAGE FAIL<o:p></o:p> > > > > (0x%x)!!!!!!\n", Index4));<o:p></o:p> > > > > ASSERT(FALSE);<o:p></o:p> > > > > return NULL;<o:p></o:p> > > > > }<o:p></o:p> > > > > - FlushPageTableMemory (VtdIndex,<o:p></o:p> > (UINTN)Lvl3PtEntry[Index3].Uint64,<o:p></o:p> > > > > SIZE_4KB);<o:p></o:p> > > > > - SetSecondLevelPagingEntryAttribute (&Lvl3PtEntry[Index3],<o:p></o:p> > > > > EDKII_IOMMU_ACCESS_READ | EDKII_IOMMU_ACCESS_WRITE);<o:p></o:p> > > > > + FlushPageTableMemory (VtdIndex,<o:p></o:p> > > > > + (UINTN)Lvl4PtEntry[Index4].Uint64,<o:p></o:p> > > > > SIZE_4KB);<o:p></o:p> > > > > + SetSecondLevelPagingEntryAttribute (&Lvl4PtEntry[Index4],<o:p></o:p> > > > > EDKII_IOMMU_ACCESS_READ | EDKII_IOMMU_ACCESS_WRITE);<o:p></o:p> > > > > + }<o:p></o:p> > > > > +<o:p></o:p> > > > > + Lvl3Start = RShiftU64 (BaseAddress, 30) & 0x1FF;<o:p></o:p> > > > > + if (ALIGN_VALUE_LOW(BaseAddress + SIZE_1GB, SIZE_1GB) <=<o:p></o:p> > > > > EndAddress) {<o:p></o:p> > > > > + Lvl3End = SIZE_4KB/sizeof(VTD_SECOND_LEVEL_PAGING_ENTRY) -<o:p></o:p> > 1;<o:p></o:p> > > > > + } else {<o:p></o:p> > > > > + Lvl3End = RShiftU64 (EndAddress - 1, 30) & 0x1FF;<o:p></o:p> > > > > }<o:p></o:p> > > > > + DEBUG ((DEBUG_INFO," Lvl4(0x%x): Lvl3Start - 0x%x, Lvl3End<o:p></o:p> > > > > + - 0x%x\n",<o:p></o:p> > > > > Index4, Lvl3Start, Lvl3End));<o:p></o:p> > > > ><o:p></o:p> > > > > - Lvl2PtEntry = (VTD_SECOND_LEVEL_PAGING_ENTRY<o:p></o:p> > > > > *)(UINTN)VTD_64BITS_ADDRESS(Lvl3PtEntry[Index3].Bits.AddressLo,<o:p></o:p> > > > > Lvl3PtEntry[Index3].Bits.AddressHi);<o:p></o:p> > > > > - for (Index2 = 0; Index2 <<o:p></o:p> > > > > SIZE_4KB/sizeof(VTD_SECOND_LEVEL_PAGING_ENTRY); Index2++) {<o:p></o:p> > > > > - Lvl2PtEntry[Index2].Uint64 = BaseAddress;<o:p></o:p> > > > > - SetSecondLevelPagingEntryAttribute (&Lvl2PtEntry[Index2],<o:p></o:p> > > > > IoMmuAccess);<o:p></o:p> > > > > - Lvl2PtEntry[Index2].Bits.PageSize = 1;<o:p></o:p> > > > > - BaseAddress += SIZE_2MB;<o:p></o:p> > > > > + Lvl3PtEntry = (VTD_SECOND_LEVEL_PAGING_ENTRY<o:p></o:p> > > > > *)(UINTN)VTD_64BITS_ADDRESS(Lvl4PtEntry[Index4].Bits.AddressLo,<o:p></o:p> > > > > Lvl4PtEntry[Index4].Bits.AddressHi);<o:p></o:p> > > > > + for (Index3 = Lvl3Start; Index3 <= Lvl3End; Index3++) {<o:p></o:p> > > > > + if (Lvl3PtEntry[Index3].Uint64 == 0) {<o:p></o:p> > > > > + Lvl3PtEntry[Index3].Uint64 = (UINT64)(UINTN)AllocateZeroPages<o:p></o:p> > (1);<o:p></o:p> > > > > + if (Lvl3PtEntry[Index3].Uint64 == 0) {<o:p></o:p> > > > > + DEBUG ((DEBUG_ERROR,"!!!!!! ALLOCATE LVL3 PAGE FAIL<o:p></o:p> > > > > + (0x%x,<o:p></o:p> > > > > 0x%x)!!!!!!\n", Index4, Index3));<o:p></o:p> > > > > + ASSERT(FALSE);<o:p></o:p> > > > > + return NULL;<o:p></o:p> > > > > + }<o:p></o:p> > > > > + FlushPageTableMemory (VtdIndex,<o:p></o:p> > > > > + (UINTN)Lvl3PtEntry[Index3].Uint64,<o:p></o:p> > > > > SIZE_4KB);<o:p></o:p> > > > > + SetSecondLevelPagingEntryAttribute<o:p></o:p> > > > > + (&Lvl3PtEntry[Index3],<o:p></o:p> > > > > EDKII_IOMMU_ACCESS_READ | EDKII_IOMMU_ACCESS_WRITE);<o:p></o:p> > > > > + }<o:p></o:p> > > > > +<o:p></o:p> > > > > + Lvl2PtEntry = (VTD_SECOND_LEVEL_PAGING_ENTRY<o:p></o:p> > > > > *)(UINTN)VTD_64BITS_ADDRESS(Lvl3PtEntry[Index3].Bits.AddressLo,<o:p></o:p> > > > > Lvl3PtEntry[Index3].Bits.AddressHi);<o:p></o:p> > > > > + for (Index2 = 0; Index2 <<o:p></o:p> > > > > SIZE_4KB/sizeof(VTD_SECOND_LEVEL_PAGING_ENTRY); Index2++) {<o:p></o:p> > > > > + Lvl2PtEntry[Index2].Uint64 = BaseAddress;<o:p></o:p> > > > > + SetSecondLevelPagingEntryAttribute<o:p></o:p> > > > > + (&Lvl2PtEntry[Index2],<o:p></o:p> > > > > IoMmuAccess);<o:p></o:p> > > > > + Lvl2PtEntry[Index2].Bits.PageSize = 1;<o:p></o:p> > > > > + BaseAddress += SIZE_2MB;<o:p></o:p> > > > > + if (BaseAddress >= MemoryLimit) {<o:p></o:p> > > > > + break;<o:p></o:p> > > > > + }<o:p></o:p> > > > > + }<o:p></o:p> > > > > + FlushPageTableMemory (VtdIndex, (UINTN)Lvl2PtEntry,<o:p></o:p> > > > > + SIZE_4KB);<o:p></o:p> > > > > if (BaseAddress >= MemoryLimit) {<o:p></o:p> > > > > break;<o:p></o:p> > > > > }<o:p></o:p> > > > > }<o:p></o:p> > > > > - FlushPageTableMemory (VtdIndex, (UINTN)Lvl2PtEntry, SIZE_4KB);<o:p></o:p> > > > > + FlushPageTableMemory (VtdIndex,<o:p></o:p> > > > > + (UINTN)&Lvl3PtEntry[Lvl3Start],<o:p></o:p> > > > > (UINTN)&Lvl3PtEntry[Lvl3End + 1] - (UINTN)&Lvl3PtEntry[Lvl3Start]);<o:p></o:p> > > > > if (BaseAddress >= MemoryLimit) {<o:p></o:p> > > > > break;<o:p></o:p> > > > > }<o:p></o:p> > > > > }<o:p></o:p> > > > > - FlushPageTableMemory (VtdIndex, (UINTN)&Lvl3PtEntry[Lvl3Start],<o:p></o:p> > > > > (UINTN)&Lvl3PtEntry[Lvl3End + 1] - (UINTN)&Lvl3PtEntry[Lvl3Start]);<o:p></o:p> > > > > - if (BaseAddress >= MemoryLimit) {<o:p></o:p> > > > > - break;<o:p></o:p> > > > > - }<o:p></o:p> > > > > + FlushPageTableMemory (VtdIndex,<o:p></o:p> > > > > + (UINTN)&Lvl4PtEntry[Lvl4Start],<o:p></o:p> > > > > (UINTN)&Lvl4PtEntry[Lvl4End + 1] - (UINTN)&Lvl4PtEntry[Lvl4Start]);<o:p></o:p> > > > > }<o:p></o:p> > > > > - FlushPageTableMemory (VtdIndex, (UINTN)&Lvl4PtEntry[Lvl4Start],<o:p></o:p> > > > > (UINTN)&Lvl4PtEntry[Lvl4End + 1] -<o:p></o:p> > > > > (UINTN)&Lvl4PtEntry[Lvl4Start]);<o:p></o:p> > > > > + FlushPageTableMemory (VtdIndex, (UINTN)&Lvl5PtEntry[Lvl5Start],<o:p></o:p> > > > > (UINTN)&Lvl5PtEntry[Lvl5End + 1] -<o:p></o:p> > > > > (UINTN)&Lvl5PtEntry[Lvl5Start]);<o:p></o:p> > > > ><o:p></o:p> > > > > return SecondLevelPagingEntry;<o:p></o:p> > > > > }<o:p></o:p> > > > > @@ -266,26 +332,28 @@ CreateSecondLevelPagingEntryTable (<o:p></o:p> > > > ><o:p></o:p> > > > > @param[in] VtdIndex The index of the VTd engine.<o:p></o:p> > > > > @param[in] IoMmuAccess The IOMMU access.<o:p></o:p> > > > > + @param[in] Is5LevelPaging If it is the 5 level paging.<o:p></o:p> > > > ><o:p></o:p> > > > > @return The second level paging entry.<o:p></o:p> > > > > **/<o:p></o:p> > > > > VTD_SECOND_LEVEL_PAGING_ENTRY *<o:p></o:p> > > > > CreateSecondLevelPagingEntry (<o:p></o:p> > > > > IN UINTN VtdIndex,<o:p></o:p> > > > > - IN UINT64 IoMmuAccess<o:p></o:p> > > > > + IN UINT64 IoMmuAccess,<o:p></o:p> > > > > + IN BOOLEAN Is5LevelPaging<o:p></o:p> > > > > )<o:p></o:p> > > > > {<o:p></o:p> > > > > VTD_SECOND_LEVEL_PAGING_ENTRY *SecondLevelPagingEntry;<o:p></o:p> > > > ><o:p></o:p> > > > > SecondLevelPagingEntry = NULL;<o:p></o:p> > > > > - SecondLevelPagingEntry = CreateSecondLevelPagingEntryTable<o:p></o:p> > > > > (VtdIndex, SecondLevelPagingEntry, 0, mBelow4GMemoryLimit,<o:p></o:p> > > > > IoMmuAccess);<o:p></o:p> > > > > + SecondLevelPagingEntry = CreateSecondLevelPagingEntryTable<o:p></o:p> > > > > + (VtdIndex,<o:p></o:p> > > > > SecondLevelPagingEntry, 0, mBelow4GMemoryLimit, IoMmuAccess,<o:p></o:p> > > > > Is5LevelPaging);<o:p></o:p> > > > > if (SecondLevelPagingEntry == NULL) {<o:p></o:p> > > > > return NULL;<o:p></o:p> > > > > }<o:p></o:p> > > > ><o:p></o:p> > > > > if (mAbove4GMemoryLimit != 0) {<o:p></o:p> > > > > ASSERT (mAbove4GMemoryLimit > BASE_4GB);<o:p></o:p> > > > > - SecondLevelPagingEntry = CreateSecondLevelPagingEntryTable<o:p></o:p> > > (VtdIndex,<o:p></o:p> > > > > SecondLevelPagingEntry, SIZE_4GB, mAbove4GMemoryLimit,<o:p></o:p> > > IoMmuAccess);<o:p></o:p> > > > > + SecondLevelPagingEntry = CreateSecondLevelPagingEntryTable<o:p></o:p> > > > > + (VtdIndex,<o:p></o:p> > > > > SecondLevelPagingEntry, SIZE_4GB, mAbove4GMemoryLimit,<o:p></o:p> > > > > IoMmuAccess, Is5LevelPaging);<o:p></o:p> > > > > if (SecondLevelPagingEntry == NULL) {<o:p></o:p> > > > > return NULL;<o:p></o:p> > > > > }<o:p></o:p> > > > > @@ -326,11 +394,13 @@ SetupTranslationTable (<o:p></o:p> > > > > /**<o:p></o:p> > > > > Dump DMAR context entry table.<o:p></o:p> > > > ><o:p></o:p> > > > > - @param[in] RootEntry DMAR root entry.<o:p></o:p> > > > > + @param[in] RootEntry DMAR root entry.<o:p></o:p> > > > > + @param[in] Is5LevelPaging If it is the 5 level paging.<o:p></o:p> > > > > **/<o:p></o:p> > > > > VOID<o:p></o:p> > > > > DumpDmarContextEntryTable (<o:p></o:p> > > > > - IN VTD_ROOT_ENTRY *RootEntry<o:p></o:p> > > > > + IN VTD_ROOT_ENTRY *RootEntry,<o:p></o:p> > > > > + IN BOOLEAN Is5LevelPaging<o:p></o:p> > > > > )<o:p></o:p> > > > > {<o:p></o:p> > > > > UINTN Index;<o:p></o:p> > > > > @@ -359,7 +429,7 @@ DumpDmarContextEntryTable (<o:p></o:p> > > > > if (ContextEntry[Index2].Bits.Present == 0) {<o:p></o:p> > > > > continue;<o:p></o:p> > > > > }<o:p></o:p> > > > > - DumpSecondLevelPagingEntry ((VOID<o:p></o:p> > > > > *)(UINTN)VTD_64BITS_ADDRESS(ContextEntry[Index2].Bits.SecondLevelP<o:p></o:p> > > > > ag<o:p></o:p> > > > > e<o:p></o:p> > > > > TranslationPointerLo,<o:p></o:p> > > > > ContextEntry[Index2].Bits.SecondLevelPageTranslationPointerHi));<o:p></o:p> > > > > + DumpSecondLevelPagingEntry ((VOID<o:p></o:p> > > > > *)(UINTN)VTD_64BITS_ADDRESS(ContextEntry[Index2].Bits.SecondLevelP<o:p></o:p> > > > > ag<o:p></o:p> > > > > e<o:p></o:p> > > > > TranslationPointerLo,<o:p></o:p> > > > > ContextEntry[Index2].Bits.SecondLevelPageTranslationPointerHi),<o:p></o:p> > > > > Is5LevelPaging);<o:p></o:p> > > > > }<o:p></o:p> > > > > }<o:p></o:p> > > > > DEBUG ((DEBUG_INFO,"=========================\n"));<o:p></o:p> > > > > @@ -368,17 +438,22 @@ DumpDmarContextEntryTable (<o:p></o:p> > > > > /**<o:p></o:p> > > > > Dump DMAR second level paging entry.<o:p></o:p> > > > ><o:p></o:p> > > > > - @param[in] SecondLevelPagingEntry The second level paging entry.<o:p></o:p> > > > > + @param[in] SecondLevelPagingEntry The second level paging entry.<o:p></o:p> > > > > + @param[in] Is5LevelPaging If it is the 5 level paging.<o:p></o:p> > > > > **/<o:p></o:p> > > > > VOID<o:p></o:p> > > > > DumpSecondLevelPagingEntry (<o:p></o:p> > > > > - IN VOID *SecondLevelPagingEntry<o:p></o:p> > > > > + IN VOID *SecondLevelPagingEntry, IN BOOLEAN Is5LevelPaging<o:p></o:p> > > > > )<o:p></o:p> > > > > {<o:p></o:p> > > > > + UINTN Index5;<o:p></o:p> > > > > UINTN Index4;<o:p></o:p> > > > > UINTN Index3;<o:p></o:p> > > > > UINTN Index2;<o:p></o:p> > > > > UINTN Index1;<o:p></o:p> > > > > + UINTN Lvl5IndexEnd;<o:p></o:p> > > > > + VTD_SECOND_LEVEL_PAGING_ENTRY *Lvl5PtEntry;<o:p></o:p> > > > > VTD_SECOND_LEVEL_PAGING_ENTRY *Lvl4PtEntry;<o:p></o:p> > > > > VTD_SECOND_LEVEL_PAGING_ENTRY *Lvl3PtEntry;<o:p></o:p> > > > > VTD_SECOND_LEVEL_PAGING_ENTRY *Lvl2PtEntry; @@ -386,38<o:p></o:p> > +461,53<o:p></o:p> > > > @@<o:p></o:p> > > > > DumpSecondLevelPagingEntry (<o:p></o:p> > > > ><o:p></o:p> > > > > DEBUG ((DEBUG_VERBOSE,"================\n"));<o:p></o:p> > > > > DEBUG ((DEBUG_VERBOSE,"DMAR Second Level Page Table:\n"));<o:p></o:p> > > > > + DEBUG ((DEBUG_VERBOSE,"SecondLevelPagingEntry Base - 0x%x,<o:p></o:p> > > > > Is5LevelPaging - %d\n", SecondLevelPagingEntry, Is5LevelPaging));<o:p></o:p> > > > ><o:p></o:p> > > > > - DEBUG ((DEBUG_VERBOSE,"SecondLevelPagingEntry Base - 0x%x\n",<o:p></o:p> > > > > SecondLevelPagingEntry));<o:p></o:p> > > > > + Lvl5IndexEnd = Is5LevelPaging ?<o:p></o:p> > > > > SIZE_4KB/sizeof(VTD_SECOND_LEVEL_PAGING_ENTRY) : 1;<o:p></o:p> > > > > Lvl4PtEntry = (VTD_SECOND_LEVEL_PAGING_ENTRY<o:p></o:p> > > > > *)SecondLevelPagingEntry;<o:p></o:p> > > > > - for (Index4 = 0; Index4 <<o:p></o:p> > > > > SIZE_4KB/sizeof(VTD_SECOND_LEVEL_PAGING_ENTRY); Index4++) {<o:p></o:p> > > > > - if (Lvl4PtEntry[Index4].Uint64 != 0) {<o:p></o:p> > > > > - DEBUG ((DEBUG_VERBOSE," Lvl4Pt Entry(0x%03x) - 0x%016lx\n",<o:p></o:p> > > Index4,<o:p></o:p> > > > > Lvl4PtEntry[Index4].Uint64));<o:p></o:p> > > > > - }<o:p></o:p> > > > > - if (Lvl4PtEntry[Index4].Uint64 == 0) {<o:p></o:p> > > > > - continue;<o:p></o:p> > > > > - }<o:p></o:p> > > > > - Lvl3PtEntry = (VTD_SECOND_LEVEL_PAGING_ENTRY<o:p></o:p> > > > > *)(UINTN)VTD_64BITS_ADDRESS(Lvl4PtEntry[Index4].Bits.AddressLo,<o:p></o:p> > > > > Lvl4PtEntry[Index4].Bits.AddressHi);<o:p></o:p> > > > > - for (Index3 = 0; Index3 <<o:p></o:p> > > > > SIZE_4KB/sizeof(VTD_SECOND_LEVEL_PAGING_ENTRY); Index3++) {<o:p></o:p> > > > > - if (Lvl3PtEntry[Index3].Uint64 != 0) {<o:p></o:p> > > > > - DEBUG ((DEBUG_VERBOSE," Lvl3Pt Entry(0x%03x) - 0x%016lx\n",<o:p></o:p> > > > > Index3, Lvl3PtEntry[Index3].Uint64));<o:p></o:p> > > > > + Lvl5PtEntry = (VTD_SECOND_LEVEL_PAGING_ENTRY<o:p></o:p> > > > > *)SecondLevelPagingEntry;<o:p></o:p> > > > > +<o:p></o:p> > > > > + for (Index5 = 0; Index5 < Lvl5IndexEnd; Index5++) {<o:p></o:p> > > > > + if (Is5LevelPaging) {<o:p></o:p> > > > > + if (Lvl5PtEntry[Index5].Uint64 != 0) {<o:p></o:p> > > > > + DEBUG ((DEBUG_VERBOSE," Lvl5Pt Entry(0x%03x) -<o:p></o:p> > > > > + 0x%016lx\n",<o:p></o:p> > > > > Index5, Lvl5PtEntry[Index5].Uint64));<o:p></o:p> > > > > }<o:p></o:p> > > > > - if (Lvl3PtEntry[Index3].Uint64 == 0) {<o:p></o:p> > > > > + if (Lvl5PtEntry[Index5].Uint64 == 0) {<o:p></o:p> > > > > continue;<o:p></o:p> > > > > }<o:p></o:p> > > > > + Lvl4PtEntry = (VTD_SECOND_LEVEL_PAGING_ENTRY<o:p></o:p> > > > > *)(UINTN)VTD_64BITS_ADDRESS(Lvl5PtEntry[Index5].Bits.AddressLo,<o:p></o:p> > > > > Lvl5PtEntry[Index5].Bits.AddressHi);<o:p></o:p> > > > > + }<o:p></o:p> > > > ><o:p></o:p> > > > > - Lvl2PtEntry = (VTD_SECOND_LEVEL_PAGING_ENTRY<o:p></o:p> > > > > *)(UINTN)VTD_64BITS_ADDRESS(Lvl3PtEntry[Index3].Bits.AddressLo,<o:p></o:p> > > > > Lvl3PtEntry[Index3].Bits.AddressHi);<o:p></o:p> > > > > - for (Index2 = 0; Index2 <<o:p></o:p> > > > > SIZE_4KB/sizeof(VTD_SECOND_LEVEL_PAGING_ENTRY); Index2++) {<o:p></o:p> > > > > - if (Lvl2PtEntry[Index2].Uint64 != 0) {<o:p></o:p> > > > > - DEBUG ((DEBUG_VERBOSE," Lvl2Pt Entry(0x%03x) -<o:p></o:p> > 0x%016lx\n",<o:p></o:p> > > > > Index2, Lvl2PtEntry[Index2].Uint64));<o:p></o:p> > > > > + for (Index4 = 0; Index4 <<o:p></o:p> > > > > SIZE_4KB/sizeof(VTD_SECOND_LEVEL_PAGING_ENTRY); Index4++) {<o:p></o:p> > > > > + if (Lvl4PtEntry[Index4].Uint64 != 0) {<o:p></o:p> > > > > + DEBUG ((DEBUG_VERBOSE," Lvl4Pt Entry(0x%03x) -<o:p></o:p> > > > > + 0x%016lx\n",<o:p></o:p> > > > > Index4, Lvl4PtEntry[Index4].Uint64));<o:p></o:p> > > > > + }<o:p></o:p> > > > > + if (Lvl4PtEntry[Index4].Uint64 == 0) {<o:p></o:p> > > > > + continue;<o:p></o:p> > > > > + }<o:p></o:p> > > > > + Lvl3PtEntry = (VTD_SECOND_LEVEL_PAGING_ENTRY<o:p></o:p> > > > > *)(UINTN)VTD_64BITS_ADDRESS(Lvl4PtEntry[Index4].Bits.AddressLo,<o:p></o:p> > > > > Lvl4PtEntry[Index4].Bits.AddressHi);<o:p></o:p> > > > > + for (Index3 = 0; Index3 <<o:p></o:p> > > > > SIZE_4KB/sizeof(VTD_SECOND_LEVEL_PAGING_ENTRY); Index3++) {<o:p></o:p> > > > > + if (Lvl3PtEntry[Index3].Uint64 != 0) {<o:p></o:p> > > > > + DEBUG ((DEBUG_VERBOSE," Lvl3Pt Entry(0x%03x) - 0x%016lx\n",<o:p></o:p> > > > > Index3, Lvl3PtEntry[Index3].Uint64));<o:p></o:p> > > > > }<o:p></o:p> > > > > - if (Lvl2PtEntry[Index2].Uint64 == 0) {<o:p></o:p> > > > > + if (Lvl3PtEntry[Index3].Uint64 == 0) {<o:p></o:p> > > > > continue;<o:p></o:p> > > > > }<o:p></o:p> > > > > - if (Lvl2PtEntry[Index2].Bits.PageSize == 0) {<o:p></o:p> > > > > - Lvl1PtEntry = (VTD_SECOND_LEVEL_PAGING_ENTRY<o:p></o:p> > > > > *)(UINTN)VTD_64BITS_ADDRESS(Lvl2PtEntry[Index2].Bits.AddressLo,<o:p></o:p> > > > > Lvl2PtEntry[Index2].Bits.AddressHi);<o:p></o:p> > > > > - for (Index1 = 0; Index1 <<o:p></o:p> > > > > SIZE_4KB/sizeof(VTD_SECOND_LEVEL_PAGING_ENTRY); Index1++) {<o:p></o:p> > > > > - if (Lvl1PtEntry[Index1].Uint64 != 0) {<o:p></o:p> > > > > - DEBUG ((DEBUG_VERBOSE," Lvl1Pt Entry(0x%03x) -<o:p></o:p> > > > 0x%016lx\n",<o:p></o:p> > > > > Index1, Lvl1PtEntry[Index1].Uint64));<o:p></o:p> > > > > +<o:p></o:p> > > > > + Lvl2PtEntry = (VTD_SECOND_LEVEL_PAGING_ENTRY<o:p></o:p> > > > > *)(UINTN)VTD_64BITS_ADDRESS(Lvl3PtEntry[Index3].Bits.AddressLo,<o:p></o:p> > > > > Lvl3PtEntry[Index3].Bits.AddressHi);<o:p></o:p> > > > > + for (Index2 = 0; Index2 <<o:p></o:p> > > > > SIZE_4KB/sizeof(VTD_SECOND_LEVEL_PAGING_ENTRY); Index2++) {<o:p></o:p> > > > > + if (Lvl2PtEntry[Index2].Uint64 != 0) {<o:p></o:p> > > > > + DEBUG ((DEBUG_VERBOSE," Lvl2Pt Entry(0x%03x) -<o:p></o:p> > 0x%016lx\n",<o:p></o:p> > > > > Index2, Lvl2PtEntry[Index2].Uint64));<o:p></o:p> > > > > + }<o:p></o:p> > > > > + if (Lvl2PtEntry[Index2].Uint64 == 0) {<o:p></o:p> > > > > + continue;<o:p></o:p> > > > > + }<o:p></o:p> > > > > + if (Lvl2PtEntry[Index2].Bits.PageSize == 0) {<o:p></o:p> > > > > + Lvl1PtEntry = (VTD_SECOND_LEVEL_PAGING_ENTRY<o:p></o:p> > > > > *)(UINTN)VTD_64BITS_ADDRESS(Lvl2PtEntry[Index2].Bits.AddressLo,<o:p></o:p> > > > > Lvl2PtEntry[Index2].Bits.AddressHi);<o:p></o:p> > > > > + for (Index1 = 0; Index1 <<o:p></o:p> > > > > SIZE_4KB/sizeof(VTD_SECOND_LEVEL_PAGING_ENTRY); Index1++) {<o:p></o:p> > > > > + if (Lvl1PtEntry[Index1].Uint64 != 0) {<o:p></o:p> > > > > + DEBUG ((DEBUG_VERBOSE," Lvl1Pt Entry(0x%03x) -<o:p></o:p> > > > 0x%016lx\n",<o:p></o:p> > > > > Index1, Lvl1PtEntry[Index1].Uint64));<o:p></o:p> > > > > + }<o:p></o:p> > > > > }<o:p></o:p> > > > > }<o:p></o:p> > > > > }<o:p></o:p> > > > > @@ -510,6 +600,7 @@ PageAttributeToLength (<o:p></o:p> > > > > @param[in] VtdIndex The index used to identify a VTd<o:p></o:p> > engine.<o:p></o:p> > > > > @param[in] SecondLevelPagingEntry The second level paging entry in<o:p></o:p> > > > > VTd table for the device.<o:p></o:p> > > > > @param[in] Address The address to be checked.<o:p></o:p> > > > > + @param[in] Is5LevelPaging If it is the 5 level paging.<o:p></o:p> > > > > @param[out] PageAttributes The page attribute of the page<o:p></o:p> > entry.<o:p></o:p> > > > ><o:p></o:p> > > > > @return The page entry.<o:p></o:p> > > > > @@ -519,6 +610,7 @@ GetSecondLevelPageTableEntry (<o:p></o:p> > > > > IN UINTN VtdIndex,<o:p></o:p> > > > > IN VTD_SECOND_LEVEL_PAGING_ENTRY *SecondLevelPagingEntry,<o:p></o:p> > > > > IN PHYSICAL_ADDRESS Address,<o:p></o:p> > > > > + IN BOOLEAN Is5LevelPaging,<o:p></o:p> > > > > OUT PAGE_ATTRIBUTE *PageAttribute<o:p></o:p> > > > > )<o:p></o:p> > > > > {<o:p></o:p> > > > > @@ -526,17 +618,38 @@ GetSecondLevelPageTableEntry (<o:p></o:p> > > > > UINTN Index2;<o:p></o:p> > > > > UINTN Index3;<o:p></o:p> > > > > UINTN Index4;<o:p></o:p> > > > > + UINTN Index5;<o:p></o:p> > > > > UINT64 *L1PageTable;<o:p></o:p> > > > > UINT64 *L2PageTable;<o:p></o:p> > > > > UINT64 *L3PageTable;<o:p></o:p> > > > > UINT64 *L4PageTable;<o:p></o:p> > > > > + UINT64 *L5PageTable;<o:p></o:p> > > > ><o:p></o:p> > > > > + Index5 = ((UINTN)RShiftU64 (Address, 48)) &<o:p></o:p> > > > > + PAGING_VTD_INDEX_MASK;<o:p></o:p> > > > > Index4 = ((UINTN)RShiftU64 (Address, 39)) &<o:p></o:p> > PAGING_VTD_INDEX_MASK;<o:p></o:p> > > > > Index3 = ((UINTN)Address >> 30) & PAGING_VTD_INDEX_MASK;<o:p></o:p> > > > > Index2 = ((UINTN)Address >> 21) & PAGING_VTD_INDEX_MASK;<o:p></o:p> > > > > Index1 = ((UINTN)Address >> 12) & PAGING_VTD_INDEX_MASK;<o:p></o:p> > > > ><o:p></o:p> > > > > - L4PageTable = (UINT64 *)SecondLevelPagingEntry;<o:p></o:p> > > > > + if (Is5LevelPaging) {<o:p></o:p> > > > > + L5PageTable = (UINT64 *)SecondLevelPagingEntry;<o:p></o:p> > > > > + if (L5PageTable[Index5] == 0) {<o:p></o:p> > > > > + L5PageTable[Index5] = (UINT64)(UINTN)AllocateZeroPages (1);<o:p></o:p> > > > > + if (L5PageTable[Index5] == 0) {<o:p></o:p> > > > > + DEBUG ((DEBUG_ERROR,"!!!!!! ALLOCATE LVL5 PAGE FAIL<o:p></o:p> > > > > (0x%x)!!!!!!\n", Index4));<o:p></o:p> > > > > + ASSERT(FALSE);<o:p></o:p> > > > > + *PageAttribute = PageNone;<o:p></o:p> > > > > + return NULL;<o:p></o:p> > > > > + }<o:p></o:p> > > > > + FlushPageTableMemory (VtdIndex, (UINTN)L5PageTable[Index5],<o:p></o:p> > > > > SIZE_4KB);<o:p></o:p> > > > > + SetSecondLevelPagingEntryAttribute<o:p></o:p> > > > > ((VTD_SECOND_LEVEL_PAGING_ENTRY *)&L5PageTable[Index5],<o:p></o:p> > > > > EDKII_IOMMU_ACCESS_READ | EDKII_IOMMU_ACCESS_WRITE);<o:p></o:p> > > > > + FlushPageTableMemory (VtdIndex,<o:p></o:p> > > > > + (UINTN)&L5PageTable[Index5],<o:p></o:p> > > > > sizeof(L5PageTable[Index5]));<o:p></o:p> > > > > + }<o:p></o:p> > > > > + L4PageTable = (UINT64 *)(UINTN)(L5PageTable[Index5] &<o:p></o:p> > > > > PAGING_4K_ADDRESS_MASK_64);<o:p></o:p> > > > > + } else {<o:p></o:p> > > > > + L4PageTable = (UINT64 *)SecondLevelPagingEntry; }<o:p></o:p> > > > > +<o:p></o:p> > > > > if (L4PageTable[Index4] == 0) {<o:p></o:p> > > > > L4PageTable[Index4] = (UINT64)(UINTN)AllocateZeroPages (1);<o:p></o:p> > > > > if (L4PageTable[Index4] == 0) { @@ -785,7 +898,7 @@<o:p></o:p> > > > > SetSecondLevelPagingAttribute (<o:p></o:p> > > > > }<o:p></o:p> > > > ><o:p></o:p> > > > > while (Length != 0) {<o:p></o:p> > > > > - PageEntry = GetSecondLevelPageTableEntry (VtdIndex,<o:p></o:p> > > > > SecondLevelPagingEntry, BaseAddress, &PageAttribute);<o:p></o:p> > > > > + PageEntry = GetSecondLevelPageTableEntry (VtdIndex,<o:p></o:p> > > > > SecondLevelPagingEntry, BaseAddress,<o:p></o:p> > > > > mVtdUnitInformation[VtdIndex].Is5LevelPaging, &PageAttribute);<o:p></o:p> > > > > if (PageEntry == NULL) {<o:p></o:p> > > > > DEBUG ((DEBUG_ERROR, "PageEntry - NULL\n"));<o:p></o:p> > > > > return RETURN_UNSUPPORTED;<o:p></o:p> > > > > @@ -913,7 +1026,7 @@ SetAccessAttribute (<o:p></o:p> > > > ><o:p></o:p> > > > > if (ExtContextEntry != NULL) {<o:p></o:p> > > > > if (ExtContextEntry->Bits.Present == 0) {<o:p></o:p> > > > > - SecondLevelPagingEntry = CreateSecondLevelPagingEntry (VtdIndex,<o:p></o:p> > 0);<o:p></o:p> > > > > + SecondLevelPagingEntry = CreateSecondLevelPagingEntry<o:p></o:p> > > > > + (VtdIndex, 0,<o:p></o:p> > > > > mVtdUnitInformation[VtdIndex].Is5LevelPaging);<o:p></o:p> > > > > DEBUG ((DEBUG_VERBOSE,"SecondLevelPagingEntry - 0x%x (S%04x<o:p></o:p> > > > > B%02x D%02x F%02x) New\n", SecondLevelPagingEntry, Segment,<o:p></o:p> > > > > SourceId.Bits.Bus, SourceId.Bits.Device, SourceId.Bits.Function));<o:p></o:p> > > > > Pt = (UINT64)RShiftU64<o:p></o:p> > > > > ((UINT64)(UINTN)SecondLevelPagingEntry,<o:p></o:p> > > > > 12);<o:p></o:p> > > > ><o:p></o:p> > > > > @@ -922,7 +1035,7 @@ SetAccessAttribute (<o:p></o:p> > > > > ExtContextEntry->Bits.DomainIdentifier = DomainIdentifier;<o:p></o:p> > > > > ExtContextEntry->Bits.Present = 1;<o:p></o:p> > > > > FlushPageTableMemory (VtdIndex, (UINTN)ExtContextEntry,<o:p></o:p> > > > > sizeof(*ExtContextEntry));<o:p></o:p> > > > > - DumpDmarExtContextEntryTable<o:p></o:p> > > > > (mVtdUnitInformation[VtdIndex].ExtRootEntryTable);<o:p></o:p> > > > > + DumpDmarExtContextEntryTable<o:p></o:p> > > > > (mVtdUnitInformation[VtdIndex].ExtRootEntryTable,<o:p></o:p> > > > > mVtdUnitInformation[VtdIndex].Is5LevelPaging);<o:p></o:p> > > > > mVtdUnitInformation[VtdIndex].HasDirtyContext = TRUE;<o:p></o:p> > > > > } else {<o:p></o:p> > > > > SecondLevelPagingEntry = (VOID<o:p></o:p> > > > > *)(UINTN)VTD_64BITS_ADDRESS(ExtContextEntry-<o:p></o:p> > > > > >Bits.SecondLevelPageTranslationPointerLo, ExtContextEntry-<o:p></o:p> > > > > >Bits.SecondLevelPageTranslationPointerHi);<o:p></o:p> > > > > @@ -930,7 +1043,7 @@ SetAccessAttribute (<o:p></o:p> > > > > }<o:p></o:p> > > > > } else if (ContextEntry != NULL) {<o:p></o:p> > > > > if (ContextEntry->Bits.Present == 0) {<o:p></o:p> > > > > - SecondLevelPagingEntry = CreateSecondLevelPagingEntry (VtdIndex,<o:p></o:p> > 0);<o:p></o:p> > > > > + SecondLevelPagingEntry = CreateSecondLevelPagingEntry<o:p></o:p> > > > > + (VtdIndex, 0,<o:p></o:p> > > > > mVtdUnitInformation[VtdIndex].Is5LevelPaging);<o:p></o:p> > > > > DEBUG ((DEBUG_VERBOSE,"SecondLevelPagingEntry - 0x%x (S%04x<o:p></o:p> > > > > B%02x D%02x F%02x) New\n", SecondLevelPagingEntry, Segment,<o:p></o:p> > > > > SourceId.Bits.Bus, SourceId.Bits.Device, SourceId.Bits.Function));<o:p></o:p> > > > > Pt = (UINT64)RShiftU64<o:p></o:p> > > > > ((UINT64)(UINTN)SecondLevelPagingEntry,<o:p></o:p> > > > > 12);<o:p></o:p> > > > ><o:p></o:p> > > > > @@ -939,7 +1052,7 @@ SetAccessAttribute (<o:p></o:p> > > > > ContextEntry->Bits.DomainIdentifier = DomainIdentifier;<o:p></o:p> > > > > ContextEntry->Bits.Present = 1;<o:p></o:p> > > > > FlushPageTableMemory (VtdIndex, (UINTN)ContextEntry,<o:p></o:p> > > > > sizeof(*ContextEntry));<o:p></o:p> > > > > - DumpDmarContextEntryTable<o:p></o:p> > > > > (mVtdUnitInformation[VtdIndex].RootEntryTable);<o:p></o:p> > > > > + DumpDmarContextEntryTable<o:p></o:p> > > > > (mVtdUnitInformation[VtdIndex].RootEntryTable,<o:p></o:p> > > > > mVtdUnitInformation[VtdIndex].Is5LevelPaging);<o:p></o:p> > > > > mVtdUnitInformation[VtdIndex].HasDirtyContext = TRUE;<o:p></o:p> > > > > } else {<o:p></o:p> > > > > SecondLevelPagingEntry = (VOID<o:p></o:p> > > > > *)(UINTN)VTD_64BITS_ADDRESS(ContextEntry-<o:p></o:p> > > > > >Bits.SecondLevelPageTranslationPointerLo, ContextEntry-<o:p></o:p> > > > > >Bits.SecondLevelPageTranslationPointerHi);<o:p></o:p> > > > > @@ -1000,7 +1113,7 @@ AlwaysEnablePageAttribute (<o:p></o:p> > > > ><o:p></o:p> > > > > if (mVtdUnitInformation[VtdIndex].FixedSecondLevelPagingEntry == 0) {<o:p></o:p> > > > > DEBUG((DEBUG_INFO, "CreateSecondLevelPagingEntry - %d\n",<o:p></o:p> > > > > VtdIndex));<o:p></o:p> > > > > - mVtdUnitInformation[VtdIndex].FixedSecondLevelPagingEntry =<o:p></o:p> > > > > CreateSecondLevelPagingEntry (VtdIndex, EDKII_IOMMU_ACCESS_READ |<o:p></o:p> > > > > EDKII_IOMMU_ACCESS_WRITE);<o:p></o:p> > > > > + mVtdUnitInformation[VtdIndex].FixedSecondLevelPagingEntry =<o:p></o:p> > > > > CreateSecondLevelPagingEntry (VtdIndex, EDKII_IOMMU_ACCESS_READ |<o:p></o:p> > > > > EDKII_IOMMU_ACCESS_WRITE,<o:p></o:p> > > > > mVtdUnitInformation[VtdIndex].Is5LevelPaging);<o:p></o:p> > > > > }<o:p></o:p> > > > ><o:p></o:p> > > > > SecondLevelPagingEntry =<o:p></o:p> > > > > mVtdUnitInformation[VtdIndex].FixedSecondLevelPagingEntry;<o:p></o:p> > > > > diff --git<o:p></o:p> > > > ><o:p></o:p> > a/Silicon/Intel/IntelSiliconPkg/Feature/VTd/IntelVTdDxe/TranslationTableEx.<o:p></o:p> > > > > c<o:p></o:p> > > > ><o:p></o:p> > b/Silicon/Intel/IntelSiliconPkg/Feature/VTd/IntelVTdDxe/TranslationTableEx.<o:p></o:p> > > > > c<o:p></o:p> > > > > index 0ed9e3ca..a4466891 100644<o:p></o:p> > > > > ---<o:p></o:p> > > > ><o:p></o:p> > a/Silicon/Intel/IntelSiliconPkg/Feature/VTd/IntelVTdDxe/TranslationTableEx.<o:p></o:p> > > > > c<o:p></o:p> > > > > +++<o:p></o:p> > > > ><o:p></o:p> > b/Silicon/Intel/IntelSiliconPkg/Feature/VTd/IntelVTdDxe/TranslationTableEx.<o:p></o:p> > > > > c<o:p></o:p> > > > > @@ -78,11 +78,28 @@ CreateExtContextEntry (<o:p></o:p> > > > ><o:p></o:p> > > > > DEBUG ((DEBUG_INFO,"DOMAIN: S%04x, B%02x D%02x F%02x\n",<o:p></o:p> > > > > mVtdUnitInformation[VtdIndex].Segment, SourceId.Bits.Bus,<o:p></o:p> > > > > SourceId.Bits.Device, SourceId.Bits.Function));<o:p></o:p> > > > ><o:p></o:p> > > > > - if ((mVtdUnitInformation[VtdIndex].CapReg.Bits.SAGAW & BIT2) == 0)<o:p></o:p> > {<o:p></o:p> > > > > - DEBUG((DEBUG_ERROR, "!!!! 4-level page-table is not supported on<o:p></o:p> > > > > VTD %d !!!!\n", VtdIndex));<o:p></o:p> > > > > + mVtdUnitInformation[VtdIndex].Is5LevelPaging = FALSE;<o:p></o:p> > > > > + if ((mVtdUnitInformation[VtdIndex].CapReg.Bits.SAGAW & BIT3) != 0)<o:p></o:p> > {<o:p></o:p> > > > > + mVtdUnitInformation[VtdIndex].Is5LevelPaging = TRUE;<o:p></o:p> > > > > + if (mAcpiDmarTable->HostAddressWidth <= 48) {<o:p></o:p> > > > > + if ((mVtdUnitInformation[VtdIndex].CapReg.Bits.SAGAW &<o:p></o:p> > > > > + BIT2) != 0)<o:p></o:p> > > {<o:p></o:p> > > > > + mVtdUnitInformation[VtdIndex].Is5LevelPaging = FALSE;<o:p></o:p> > > > > + }<o:p></o:p> > > > > + }<o:p></o:p> > > > > + } else if ((mVtdUnitInformation[VtdIndex].CapReg.Bits.SAGAW &<o:p></o:p> > > > > + BIT2) ==<o:p></o:p> > > > > 0) {<o:p></o:p> > > > > + DEBUG((DEBUG_ERROR, "!!!! Page-table type is not supported<o:p></o:p> > > > > + on<o:p></o:p> > > > > VTD %d !!!!\n", VtdIndex));<o:p></o:p> > > > > return EFI_UNSUPPORTED;<o:p></o:p> > > > > }<o:p></o:p> > > > > - ExtContextEntry->Bits.AddressWidth = 0x2;<o:p></o:p> > > > > +<o:p></o:p> > > > > + if (mVtdUnitInformation[VtdIndex].Is5LevelPaging) {<o:p></o:p> > > > > + ExtContextEntry->Bits.AddressWidth = 0x3;<o:p></o:p> > > > > + DEBUG((DEBUG_ERROR, "Using 4-level page-table on VTD %d\n",<o:p></o:p> > > > > VtdIndex));<o:p></o:p> > > > > + } else {<o:p></o:p> > > > > + ExtContextEntry->Bits.AddressWidth = 0x2;<o:p></o:p> > > > > + DEBUG((DEBUG_ERROR, "Using 5-level page-table on VTD %d\n",<o:p></o:p> > > > > VtdIndex));<o:p></o:p> > > > > + }<o:p></o:p> > > > > +<o:p></o:p> > > > > +<o:p></o:p> > > > > }<o:p></o:p> > > > ><o:p></o:p> > > > > FlushPageTableMemory (VtdIndex,<o:p></o:p> > > > > (UINTN)mVtdUnitInformation[VtdIndex].ExtRootEntryTable,<o:p></o:p> > > > > EFI_PAGES_TO_SIZE(EntryTablePages));<o:p></o:p> > > > > @@ -93,11 +110,13 @@ CreateExtContextEntry (<o:p></o:p> > > > > /**<o:p></o:p> > > > > Dump DMAR extended context entry table.<o:p></o:p> > > > ><o:p></o:p> > > > > - @param[in] ExtRootEntry DMAR extended root entry.<o:p></o:p> > > > > + @param[in] ExtRootEntry DMAR extended root entry.<o:p></o:p> > > > > + @param[in] Is5LevelPaging If it is the 5 level paging.<o:p></o:p> > > > > **/<o:p></o:p> > > > > VOID<o:p></o:p> > > > > DumpDmarExtContextEntryTable (<o:p></o:p> > > > > - IN VTD_EXT_ROOT_ENTRY *ExtRootEntry<o:p></o:p> > > > > + IN VTD_EXT_ROOT_ENTRY *ExtRootEntry, IN BOOLEAN Is5LevelPaging<o:p></o:p> > > > > )<o:p></o:p> > > > > {<o:p></o:p> > > > > UINTN Index;<o:p></o:p> > > > > @@ -127,7 +146,7 @@ DumpDmarExtContextEntryTable (<o:p></o:p> > > > > if (ExtContextEntry[Index2].Bits.Present == 0) {<o:p></o:p> > > > > continue;<o:p></o:p> > > > > }<o:p></o:p> > > > > - DumpSecondLevelPagingEntry ((VOID<o:p></o:p> > > > > *)(UINTN)VTD_64BITS_ADDRESS(ExtContextEntry[Index2].Bits.SecondLev<o:p></o:p> > > > > el<o:p></o:p> > > > > Pa<o:p></o:p> > > > > geTranslationPointerLo,<o:p></o:p> > > > > ExtContextEntry[Index2].Bits.SecondLevelPageTranslationPointerHi))<o:p></o:p> > > > > ;<o:p></o:p> > > > > + DumpSecondLevelPagingEntry ((VOID<o:p></o:p> > > > > *)(UINTN)VTD_64BITS_ADDRESS(ExtContextEntry[Index2].Bits.SecondLev<o:p></o:p> > > > > el<o:p></o:p> > > > > Pa<o:p></o:p> > > > > geTranslationPointerLo,<o:p></o:p> > > > > ExtContextEntry[Index2].Bits.SecondLevelPageTranslationPointerHi),<o:p></o:p> > > > > Is5LevelPaging);<o:p></o:p> > > > > }<o:p></o:p> > > > ><o:p></o:p> > > > > if (ExtRootEntry[Index].Bits.UpperPresent == 0) { diff --git<o:p></o:p> > > > > a/Silicon/Intel/IntelSiliconPkg/Feature/VTd/IntelVTdDxe/VtdReg.c<o:p></o:p> > > > > b/Silicon/Intel/IntelSiliconPkg/Feature/VTd/IntelVTdDxe/VtdReg.c<o:p></o:p> > > > > index 699639ba..686d235f 100644<o:p></o:p> > > > > ---<o:p></o:p> > > > > a/Silicon/Intel/IntelSiliconPkg/Feature/VTd/IntelVTdDxe/VtdReg.c<o:p></o:p> > > > > +++ b/Silicon/Intel/IntelSiliconPkg/Feature/VTd/IntelVTdDxe/VtdReg<o:p></o:p> > > > > +++ .c<o:p></o:p> > > > > @@ -174,8 +174,14 @@ PrepareVtdConfig (<o:p></o:p> > > > > if ((mVtdUnitInformation[Index].CapReg.Bits.SLLPS & BIT0) == 0) {<o:p></o:p> > > > > DEBUG((DEBUG_WARN, "!!!! 2MB super page is not supported on<o:p></o:p> > > > > VTD %d !!!!\n", Index));<o:p></o:p> > > > > }<o:p></o:p> > > > > - if ((mVtdUnitInformation[Index].CapReg.Bits.SAGAW & BIT2) == 0) {<o:p></o:p> > > > > - DEBUG((DEBUG_ERROR, "!!!! 4-level page-table is not supported on<o:p></o:p> > > > > VTD %d !!!!\n", Index));<o:p></o:p> > > > > + if ((mVtdUnitInformation[Index].CapReg.Bits.SAGAW & BIT3) != 0) {<o:p></o:p> > > > > + DEBUG((DEBUG_INFO, "Support 5-level page-table on VTD<o:p></o:p> > > > > + %d\n",<o:p></o:p> > > > > Index));<o:p></o:p> > > > > + }<o:p></o:p> > > > > + if ((mVtdUnitInformation[Index].CapReg.Bits.SAGAW & BIT2) != 0) {<o:p></o:p> > > > > + DEBUG((DEBUG_INFO, "Support 4-level page-table on VTD<o:p></o:p> > > > > + %d\n",<o:p></o:p> > > > > Index));<o:p></o:p> > > > > + }<o:p></o:p> > > > > + if ((mVtdUnitInformation[Index].CapReg.Bits.SAGAW & (BIT3 |<o:p></o:p> > > > > + BIT2)) == 0)<o:p></o:p> > > > > {<o:p></o:p> > > > > + DEBUG((DEBUG_ERROR, "!!!! Page-table type 0x%X is not<o:p></o:p> > > > > + supported on<o:p></o:p> > > > > VTD %d !!!!\n", Index, mVtdUnitInformation[Index].CapReg.Bits.SAGAW));<o:p></o:p> > > > > return ;<o:p></o:p> > > > > }<o:p></o:p> > > > ><o:p></o:p> > > > > --<o:p></o:p> > > > > 2.16.2.windows.1<o:p></o:p> <o:p> </o:p> </div> </div> </body> </html> <div width="1" style="color:white;clear:both">_._,_._,_</div> <hr> Groups.io Links: You receive all messages sent to this group. <a target="_blank" href="https://edk2.groups.io/g/devel/message/68486">View/Reply Online (#68486)</a> | | <a target="_blank" href="https://groups.io/mt/78651896/1813853">Mute This Topic</a> | <a href="https://edk2.groups.io/g/devel/post">New Topic</a> <a href="https://edk2.groups.io/g/devel/editsub/1813853">Your Subscription</a> | <a href="mailto:devel+owner@edk2.groups.io">Contact Group Owner</a> | <a href="https://edk2.groups.io/g/devel/unsub">Unsubscribe</a> [edk2-devel-archive@redhat.com] <div width="1" style="color:white;clear:both">_._,_._,_</div>