+Mark, for my comments on the process On 02/20/19 03:21, Ni, Ray wrote: > Laszlo, > Thanks for catching this. > > GenPerformMemoryTest() in > MdeModulePkg\Universal\MemoryTest\GenericMemoryTestDxe\LightMemoryTest.c > uses the same technics as you suggested. > > I give up to propose another option: having pack(1) for the new status > structure.
I think that byte-packing EFI_RETURN_STATUS_EXTENDED_DATA in the PI-1.7 spec would have been viable, but we should have thought about that in advance. The PI and UEFI specs require natural alignment for structure members, unless stated otherwise. There *are* a number of examples of "stated otherwise" (the term that the specs use is frequently "byte-packed structure"). Again, we should have thought of that in advance, before PI-1.7 was released, so now we can only fix the way the object is populated. More generally, I think this demonstrates a flaw in the standardization process. PIWG and USWG should only standardize existing practice; that is, features that have at least one functional implementation. (Not necessarily open source, but the interfaces should be battle-hardened *first*.) The restriction where we can't even propose patches for edk2 before the standards are updated is very counter-productive. There's practically zero chance for getting an actual implementation peer-reviewed and peer-tested before proposing the API for the standard. Personally I have suggested in the past to implement some new features as edk2 extensions first, and once they work, to upstream them to the standards. This idea is usually rejected by maintainers, because maintainers worry about becoming stuck with more and more edk2 extensions to core code (i.e., they worry about the feature proponents not making good on their promise to standardize). Unfortunately, the alternative (= the current practice) is that we standardize speculative interfaces, which then prove suboptimal once we implement them. Thanks Laszlo _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel
