On 9/28/2018 11:08 AM, Zeng, Star wrote:
Good idea. You prefer to introduce a new feature PCD for this with default value = TRUE?Could we group the emulator APIs to one structure like below? And add a Version field for potential further extension? typedef struct { UINTN Version; EDKII_PECOFF_IMAGE_EMULATOR_IS_IMAGE_SUPPORTED IsImageSupported; EDKII_PECOFF_IMAGE_EMULATOR_REGISTER_IMAGE RegisterImage; EDKII_PECOFF_IMAGE_EMULATOR_UNREGISTER_IMAGE UnregisterImage; } PECOFF_IMAGE_EMULATOR;
Star, What's the reason for the "Version" field? Can you explain the usage case? And why do we need the PCD to control? -- Thanks, Ray _______________________________________________ edk2-devel mailing list [email protected] https://lists.01.org/mailman/listinfo/edk2-devel
