From mboxrd@z Thu Jan 1 00:00:00 1970 From: matt@console-pimps.org (Matt Fleming) Date: Wed, 7 Aug 2013 22:55:54 +0100 Subject: [PATCH 03/17] Add system pointer argument to shared EFI stub related functions so they no longer use global system table pointer as they did when part of eboot.c. In-Reply-To: References: <1375847113-24884-1-git-send-email-roy.franz@linaro.org> <1375847113-24884-4-git-send-email-roy.franz@linaro.org> <20130807130851.GC2515@console-pimps.org> Message-ID: <20130807215554.GI2515@console-pimps.org> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Wed, 07 Aug, at 10:10:54AM, Roy Franz wrote: > I went this way since the shared code is in a separate file - I really > didn't like using a global variable as part of the interface to > the shared code. This has the nice side benefit of allowing the ARM > stub to not use any global variables, so we don't have to do > any GOT fixups to relocate the code - it is position independent if we > don't use global variables. OK, that would be a win. Please include this rationale in the patch. -- Matt Fleming, Intel Open Source Technology Center