[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CAMuHMdVYqojGdKaCkY63mOhxupZ3Ag5+Y-haZZf43wyuCsdKkg@mail.gmail.com>
Date: Tue, 9 Jan 2024 15:27:05 +0100
From: Geert Uytterhoeven <geert@...ux-m68k.org>
To: Daniel Palmer <daniel@...f.com>
Cc: gerg@...ux-m68k.org, fthain@...ux-m68k.org,
linux-m68k@...ts.linux-m68k.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2 2/2] m68k: Fix interrupt stack frames for 68000
Hi Daniel,
On Tue, Jan 9, 2024 at 3:10 PM Daniel Palmer <daniel@...f.com> wrote:
> On Mon, 8 Jan 2024 at 18:56, Geert Uytterhoeven <geert@...ux-m68k.org> wrote:
> > I think it would be better to use the classic m68k stack frame.
> > That would pave the way for building a single nommu kernel for
> > MC680[012346]0 that runs on e.g. any Amiga.
> > MC68000 and Coldfire are incompatible anyway.
>
> While looking at how to do this I realised that the addql #2,%sp in
> RESTORE_ALL in entry.h will now break the stack frames for those fancy
> 68010+ users.
> So that needs to be #ifdef'd to make it only compile for 68000. I saw
> an error email from the next build stuff so I guess the change has
> been queued somewhere? If so I should send a fix..
AFAIK it hasn't been applied yet. These days the bots also test
patches from mailing lists...
> I'm not sure how to actually make that generic without patching the
> code at runtime (remove the 68000 specific bit, reserve enough extra
> space to rewrite the code..) but it's a macro so not so simple.
Or use different entry points depending on CPU type?
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@...ux-m68korg
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
Powered by blists - more mailing lists