linuxppc-dev.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
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   |_______/


  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).