Jordan: For now, this feature will break older OSes. The platform requires to add feature flag to enable/disable it. So, DSC BuildOptions extension is added to configure it. In future, if more and more platforms enable it, we could figure out the solution to extent tools_def.txt to configure those settings as the default one.
Thanks Liming -----Original Message----- From: Jordan Justen [mailto:[email protected]] Sent: Thursday, July 02, 2015 3:04 PM To: Andrew Fish; [email protected] Subject: Re: [edk2] [PATCH] BaseTools: fix a syntax error in 4 KB aligned GNU ld linker script On 2015-07-01 23:22:37, Andrew Fish wrote: > > On Jul 1, 2015, at 11:08 PM, Jordan Justen <[email protected]> > > wrote: > > > >> On 2015-07-01 22:35:12, Gao, Liming wrote: > >> Ard: > >> Good to know --script option can be appended and work fine. > > > > So, --script will be specified twice to ld. This seems like a very > > fragile implementation of a new UEFI feature. > > > > I'm not sure about specifics, but in general build tools always honor > the last invocation of a flag, as that's required to support local > overrides. For example you can turn off optimization in an INF as the > 2nd compiler optimization flag has precedence. > > Do the include a file flags work differently? I don't think we should base the build process for an important UEFI feature on the way that tools work 'in general'. Is this an important feature for EDK II to support? Assuming it is important, do we really expect all platforms to add linker specific flags to their .dsc to support such a feature? It seems like these tool flags should be specified as needed in the Conf files. Doesn't it seem more appropriate for the platform to add something like UEFI_PROPERTIES_TABLE = TRUE to the defines of the .dsc? After that all the magic should live in BaseTools. Is this a feature that platforms need to be able to disable? I think runtime drivers should always be compressed, so it probably won't cost significant flash space. -Jordan ------------------------------------------------------------------------------ Don't Limit Your Business. Reach for the Cloud. GigeNET's Cloud Solutions provide you with the tools and support that you need to offload your IT needs and focus on growing your business. Configured For All Businesses. Start Your Cloud Today. https://www.gigenetcloud.com/ _______________________________________________ edk2-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/edk2-devel ------------------------------------------------------------------------------ Don't Limit Your Business. Reach for the Cloud. GigeNET's Cloud Solutions provide you with the tools and support that you need to offload your IT needs and focus on growing your business. Configured For All Businesses. Start Your Cloud Today. https://www.gigenetcloud.com/ _______________________________________________ edk2-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/edk2-devel
