On 2 July 2015 at 09:21, Gao, Liming <liming....@intel.com> wrote: > 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. >
Building DXE_RUNTIME_DRIVER modules with 4 KB section alignment does not break older OSes, so it makes sense to at least have the plumbing implemented in BaseTools in an elegant way. What does break older OSes is actually splitting the PE/COFF image into separate memory regions. This is not such a concern for AARCH64 since it is still new, so I am pushing for it to be enabled when possible. (It does break on the current upstream Linux kernel, but I have some patches on the list that fix that) Regards, Ard. > From: Jordan Justen [mailto:jordan.l.jus...@intel.com] > Sent: Thursday, July 02, 2015 3:04 PM > To: Andrew Fish; edk2-devel@lists.sourceforge.net > 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 <jordan.l.jus...@intel.com> >> > 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 > edk2-devel@lists.sourceforge.net > 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 > edk2-devel@lists.sourceforge.net > 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 edk2-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/edk2-devel
