[PATCH 21/32] efi: arm: Drop exception code when running as an EFI app

[Tom Rini]

On Thu, Feb 06, 2025 at 02:27:13PM -0700, Simon Glass wrote:
> Hi Tom,
> 
> On Thu, 6 Feb 2025 at 10:08, Tom Rini <trini at konsulko.com> wrote:
> >
> > On Thu, Feb 06, 2025 at 05:33:06AM -0700, Simon Glass wrote:
> > > Hi Tom,
> > >
> > > On Mon, 3 Feb 2025 at 19:00, Tom Rini <trini at konsulko.com> wrote:
> > > >
> > > > On Mon, Feb 03, 2025 at 05:38:52PM -0700, Simon Glass wrote:
> > > > > Hi Tom,
> > > > >
> > > > > On Mon, 3 Feb 2025 at 13:09, Tom Rini <trini at konsulko.com> wrote:
> > > > > >
> > > > > > On Mon, Feb 03, 2025 at 10:42:14AM -0700, Simon Glass wrote:
> > > > > >
> > > > > > > This code cannot be used since EFI is in charge of managing exceptions.
> > > > > > > Drop it.
> > > > > > >
> > > > > > > Signed-off-by: Simon Glass <sjg at chromium.org>
> > > > > > > ---
> > > > > > >
> > > > > > >  arch/arm/cpu/armv8/Makefile | 2 +-
> > > > > > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > > > > >
> > > > > > Why is SPL on in this case?
> > > > >
> > > > > SPL is enabled by a specific Kconfig option, which seems to have
> > > > > exceptions but not exception levels. That seemed to match what the EFI
> > > > > app wants, for now.
> > > >
> > > > OK, so why aren't you turning off SPL? You're not using SPL at all,
> > > > right?
> > >
> > > Not using SPL, but using the same code as SPL happens to use.
> >
> > But is CONFIG_SPL enabled, or not? It shouldn't be if I follow you.
> 
> No, it isn't enabled. But we do want that exceptions.o both for the
> EFI app and for whatever boards use SPL.

Um. Isn't that the opposite of what your commit message says? And the
Makefile snippet today is:
ifdef CONFIG_XPL_BUILD
obj-$(CONFIG_ARMV8_SPL_EXCEPTION_VECTORS) += exceptions.o
else
obj-y   += exceptions.o
obj-y   += exception_level.o
endif

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20250206/9f7298a3/attachment.sig>