From: "Dr. David Alan Gilbert" <linux@treblig.org>
To: Stan Johnson <userm57@yahoo.com>
Cc: mpe@ellerman.id.au, npiggin@gmail.com,
christophe.leroy@csgroup.eu, sam@ravnborg.org,
benh@kernel.crashing.org, linuxppc-dev@lists.ozlabs.org,
linux-kernel@vger.kernel.org, rdunlap@infradead.org,
Cedar Maxwell <cedarmaxwell@mac.com>,
Finn Thain <fthain@linux-m68k.org>
Subject: Re: [PATCH v4] powerpc: Use shared font data
Date: Tue, 4 Nov 2025 21:59:49 +0000 [thread overview]
Message-ID: <aQp3Ve1C-aZ8BhXa@gallifrey> (raw)
In-Reply-To: <3017c67c-87e9-c06f-3146-ee461cd6603f@yahoo.com>
* Stan Johnson (userm57@yahoo.com) wrote:
> Please see the attached config file that Finn sent to me along with his
> patch. The relevant line is "CONFIG_BOOTX_TEXT=y"; without that, the kernel
> boots on the Wallstreet.
Thanks - I'll have a play.
Dave
> On 11/4/25 10:59 AM, Dr. David Alan Gilbert wrote:
> > * Finn Thain (fthain@linux-m68k.org) wrote:
> > >
> > > On Mon, 3 Nov 2025, Dr. David Alan Gilbert wrote:
> > >
> > > >
> > > > > Anyway, I imagine that the problem with your patch was that it relies
> > > > > on font data from a different (read only) section, which is
> > > > > unavailable for some reason (MMU not fully configured yet?)
> > > > >
> > > > > So I've asked Stan to test a patch that simply removes the relevant
> > > > > 'const' keywords. It's not a solution, but might narrow-down the
> > > > > search.
> > > >
> > >
> > > Stan tested my patch to remove 'const' from the font_desc and font_data
> > > structs but it did not help. (There goes that theory.)
> > >
> > > > I wonder if this is a compiler-flag-ism; I see
> > > > arch/powerpc/kernel/Makefile has a pile of special flags, and for
> > > > btext.o it has a -fPIC (as well as turning off some other flags). I
> > > > wonder if bodging those in lib/fonts/Makefile for
> > > > lib/fonts/font_sun8x16.c fixes it? But... this is data - there's no code
> > > > is there - are any of those flags relevant for data only?
> > > >
> > >
> > > I don't know. But I'm sure Stan would be willing to test a patch for you.
> >
> > Can one of you point me at the config you're testing with so I can
> > try some builds with it.
> >
> > Dave
> >
> > > > >
> > > > > The BootX bootloader doesn't work on New World systems, which is
> > > > > probably why we don't see this regression on anything newer than a
> > > > > Wallstreet.
> > > > >
> > > > > It's likely that other Old World systems are also affected, if they
> > > > > are using BootX. We don't yet know whether the regression also affects
> > > > > Old World systems using the iQUIK bootloader instead of BootX.
> > > >
> > > > OK, remember I don't think I've ever tried PPC via MacOS booting, so not
> > > > familiar with it.
> > > >
> > >
> > > I will try to set up a MacOS guest in QEMU, to see if the hang can be
> > > reproduced that way.
> > >
--
-----Open up your eyes, open up your mind, open up your code -------
/ Dr. David Alan Gilbert | Running GNU/Linux | Happy \
\ dave @ treblig.org | | In Hex /
\ _________________________|_____ http://www.treblig.org |_______/
next prev parent reply other threads:[~2025-11-04 22:00 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-08-25 14:27 [PATCH v4] powerpc: Use shared font data linux
2023-09-30 13:15 ` Dr. David Alan Gilbert
2023-10-01 12:08 ` Michael Ellerman
2023-10-02 12:21 ` Dr. David Alan Gilbert
2023-10-15 10:04 ` Michael Ellerman
2025-11-02 5:02 ` Stan Johnson
2025-11-02 11:20 ` Finn Thain
2025-11-02 17:09 ` Dr. David Alan Gilbert
2025-11-02 19:01 ` Stan Johnson
2025-11-03 1:33 ` Finn Thain
2025-11-03 1:48 ` Dr. David Alan Gilbert
2025-11-03 7:26 ` Finn Thain
2025-11-04 17:59 ` Dr. David Alan Gilbert
2025-11-04 19:49 ` Stan Johnson
2025-11-04 21:59 ` Dr. David Alan Gilbert [this message]
2025-11-05 4:13 ` QEMU limitations, was " Finn Thain
2025-11-05 14:58 ` Dr. David Alan Gilbert
2025-11-05 21:44 ` Christophe Leroy
2025-11-05 23:48 ` Finn Thain
2025-11-06 4:11 ` Finn Thain
2025-11-06 9:28 ` Christophe Leroy
2025-11-06 11:50 ` Finn Thain
2025-11-06 12:07 ` Dr. David Alan Gilbert
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=aQp3Ve1C-aZ8BhXa@gallifrey \
--to=linux@treblig.org \
--cc=benh@kernel.crashing.org \
--cc=cedarmaxwell@mac.com \
--cc=christophe.leroy@csgroup.eu \
--cc=fthain@linux-m68k.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linuxppc-dev@lists.ozlabs.org \
--cc=mpe@ellerman.id.au \
--cc=npiggin@gmail.com \
--cc=rdunlap@infradead.org \
--cc=sam@ravnborg.org \
--cc=userm57@yahoo.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).