<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="ProgId" content="Word.Document"> <meta name="Generator" content="Microsoft Word 15"> <meta name="Originator" content="Microsoft Word 15"> <link rel="File-List" href="cid:filelist.xml@01D57843.4F51A3B0"><link rel="Edit-Time-Data" href="cid:editdata.mso"><style></style> </head> <body lang="EN-US" link="blue" vlink="purple" style="tab-interval:.5in"> <div class="WordSection1"> Hi Jason,<o:p></o:p> <o:p> </o:p> I believe the logic to check for uniqueness of FMP Descriptor is correct.<o:p></o:p> <o:p> </o:p> The UEFI Spec has 2 structs with HardwareInstance. One is FMP Descriptor and the other is UEFI Capsule for FMP.<o:p></o:p> <o:p> </o:p> The HardwareInstance in FMP must be unique and not 0 unless there is a guarantee there is only one instance.<o:p></o:p> <o:p> </o:p> ///<o:p></o:p> /// An optional number to identify the unique hardware instance within the system for<o:p></o:p> /// devices that may have multiple instances (Example: a plug in pci network card). This<o:p></o:p> /// number must be unique within the namespace of the ImageTypeId GUID and<o:p></o:p> /// ImageIndex. For FMP instances that have multiple descriptors for a single<o:p></o:p> /// hardware instance, all descriptors must have the same HardwareInstance value.<o:p></o:p> /// This number must be consistent between boots and should be based on some sort of<o:p></o:p> /// hardware identified unique id (serial number, etc) whenever possible. If a hardware<o:p></o:p> /// based number is not available the FMP provider may use some other characteristic<o:p></o:p> /// such as device path, bus/dev/function, slot num, etc for generating the<o:p></o:p> /// HardwareInstance. For implementations that will never have more than one<o:p></o:p> /// instance a zero can be used. A zero means the FMP provider is not able to determine a<o:p></o:p> /// unique hardware instance number or a hardware instance number is not needed. Only<o:p></o:p> /// present in version 3 or higher.<o:p></o:p> ///<o:p></o:p> UINT64 HardwareInstance;<o:p></o:p> } EFI_FIRMWARE_IMAGE_DESCRIPTOR;<o:p></o:p> <o:p> </o:p> The UEFI Capsule one can be 0 to specify match any.<o:p></o:p> <o:p> </o:p> ///<o:p></o:p> /// The HardwareInstance to target with this update. If value is zero it means match all<o:p></o:p> /// HardwareInstances. This field allows update software to target only a single device in<o:p></o:p> /// cases where there are more than one device with the same ImageTypeId GUID.<o:p></o:p> /// This header is outside the signed data of the Authentication Info structure and<o:p></o:p> /// therefore can be modified without changing the Auth data.<o:p></o:p> ///<o:p></o:p> UINT64 UpdateHardwareInstance;<o:p></o:p> } EFI_FIRMWARE_MANAGEMENT_CAPSULE_IMAGE_HEADER;<o:p></o:p> <o:p> </o:p> Best regards,<o:p></o:p> <o:p> </o:p> Mike<o:p></o:p> <o:p> </o:p> <o:p> </o:p> <div> <div style="border:none;border-top:solid #E1E1E1 1.0pt;padding:3.0pt 0in 0in 0in"> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Spottswood, Jason Sent: Monday, September 30, 2019 2:24 PM To: Rothman, Michael A <michael.a.rothman@intel.com>; devel@edk2.groups.io Subject: Re: [edk2-devel] Recent changes to EsrtFmp causing ASSERTs<o:p></o:p> </div> </div> <o:p> </o:p> Cut-n-paste problem. My apologies. <o:p></o:p> <o:p> </o:p> Copying again in plain text:<o:p></o:p> <o:p> </o:p> //<o:p></o:p> // Check to see of FmpImageInfoBuf GUID/HardwareInstance is unique<o:p></o:p> //<o:p></o:p> for (Index = 0; Index < *NumberOfDescriptors; Index++) {<o:p></o:p> if (CompareGuid (&HardwareInstances[Index].ImageTypeGuid, &FmpImageInfoBuf->ImageTypeId)) {<o:p></o:p> if (HardwareInstances[Index].HardwareInstance == FmpHardwareInstance) {<o:p></o:p> DEBUG ((DEBUG_ERROR, "EsrtFmpDxe: Duplicate firmware image descriptor with GUID %g HardwareInstance:0x%x\n", &FmpImageInfoBuf->ImageTypeId, FmpHardwareInstance));<o:p></o:p> ASSERT (<o:p></o:p> !CompareGuid (&HardwareInstances[Index].ImageTypeGuid, &FmpImageInfoBuf->ImageTypeId) ||<o:p></o:p> HardwareInstances[Index].HardwareInstance != FmpHardwareInstance<o:p></o:p> );<o:p></o:p> return EFI_UNSUPPORTED;<o:p></o:p> }<o:p></o:p> }<o:p></o:p> }<o:p></o:p> <o:p> </o:p> <div> -Jason<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: Rothman, Michael A <<a href="mailto:michael.a.rothman@intel.com">michael.a.rothman@intel.com</a>> Sent: Monday, September 30, 2019 3:53 PM To: <a href="mailto:devel@edk2.groups.io">devel@edk2.groups.io</a>; Spottswood, Jason <<a href="mailto:jason.spottswood@hpe.com">jason.spottswood@hpe.com</a>> Subject: Re: [edk2-devel] Recent changes to EsrtFmp causing ASSERTs<o:p></o:p> </div> </div> <o:p> </o:p> Jason, Agreed - though that image you sent was challenging for these old eyes. Black on dark grey? Ack!<o:p></o:p> <div id="AppleMailSignature"> <div> Thanks,<o:p></o:p> </div> <div> Michael A. Rothman<o:p></o:p> </div> <div> ---------------------------------------------------------------<o:p></o:p> </div> <div> Let no excuse be a barrier to your success.<o:p></o:p> </div> </div> <div> On Sep 30, 2019, at 1:48 PM, "<a href="mailto:jason.spottswood@hpe.com">jason.spottswood@hpe.com</a>" <<a href="mailto:jason.spottswood@hpe.com">jason.spottswood@hpe.com</a>> wrote:<o:p></o:p> </div> <blockquote style="margin-top:5.0pt;margin-bottom:5.0pt"> <div> In EsrtFmp.c, function CreateEsrtEntry, line 196, the code asserts if the FMP image hardware instance matches that of an existing instance. This is fine if the hardware instance is supported. The field is optional though. In the UEFI spec, "a zero hardware instance means the FMP provider is not able to determine a unique hardware instance number or a hardware instance number is not needed." The code below needs to also check if the hardware instance is supported (by comparing it to zero) before checking it against existing entries.<o:p></o:p> <div> <pre style="background:#293134"> //<o:p></o:p></pre> <pre style="background:#293134"> // Check to see of FmpImageInfoBuf GUID/HardwareInstance is unique<o:p></o:p></pre> <pre style="background:#293134"> //<o:p></o:p></pre> <pre style="background:#293134"> for (Index = 0; Index < *NumberOfDescriptors; Index++) {<o:p></o:p></pre> <pre style="background:#293134"> if (CompareGuid (&HardwareInstances[Index].ImageTypeGuid, &FmpImageInfoBuf->ImageTypeId)) {<o:p></o:p></pre> <pre style="background:#293134"> if (HardwareInstances[Index].HardwareInstance == FmpHardwareInstance) {<o:p></o:p></pre> <pre style="background:#293134"> DEBUG ((DEBUG_ERROR, "EsrtFmpDxe: Duplicate firmware image descriptor with GUID %g HardwareInstance:0x%x\n", &FmpImageInfoBuf->ImageTypeId, FmpHardwareInstance));<o:p></o:p></pre> <pre style="background:#293134"> ASSERT (<o:p></o:p></pre> <pre style="background:#293134"> !CompareGuid (&HardwareInstances[Index].ImageTypeGuid, &FmpImageInfoBuf->ImageTypeId) ||<o:p></o:p></pre> <pre style="background:#293134"> HardwareInstances[Index].HardwareInstance != FmpHardwareInstance<o:p></o:p></pre> <pre style="background:#293134"> );<o:p></o:p></pre> <pre style="background:#293134"> return EFI_UNSUPPORTED;<o:p></o:p></pre> <pre style="background:#293134"> }<o:p></o:p></pre> <pre style="background:#293134"> }<o:p></o:p></pre> <pre style="background:#293134"> }<o:p></o:p></pre> </div> </div> </blockquote> <div> </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/48352">View/Reply Online (#48352)</a> | | <a target="_blank" href="https://groups.io/mt/34350126/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>