Re: [PATCH] sh: intc: sort the prio and sense lists after filling them

[John Paul Adrian Glaubitz]

Hi Geert,

On Mon, 2026-09-28 at 19:00 +0200, Geert Uytterhoeven wrote:
> Hi Adrian,
> 
> On Mon, 28 Sept 2026 at 18:46, John Paul Adrian Glaubitz
> <glaubitz@xxxxxxxxxxxxxxxxxxx> wrote:
> > On Mon, 2026-09-28 at 18:03 +0200, Geert Uytterhoeven wrote:
> > > On Mon, 28 Sept 2026 at 17:10, John Paul Adrian Glaubitz
> > > <glaubitz@xxxxxxxxxxxxxxxxxxx> wrote:
> > > > => usb reset; fatload usb 0:1 0x89000000 uImage-7.3.0.gz ; pmb ; bootm
> > > > (Re)start USB...
> > > > USB0:   scanning bus 0 for devices... 2 USB Device(s) found
> > > >        scanning usb for storage devices... 1 Storage Device(s) found
> > > > reading uImage-7.3.0.gz
> > > > 3649306 bytes read in 2074 ms (1.7 MiB/s)
> > > > ## Booting kernel from Legacy Image at 89000000 ...
> > > >    Image Name:   Linux-7.3.0-rc5-00001-g8cd915e93
> > > >    Image Type:   SuperH Linux Kernel Image (gzip compressed)
> > > >    Data Size:    3649242 Bytes = 3.5 MiB
> > > >    Load Address: 8c010000
> > > >    Entry Point:  8c011000
> > > >    Verifying Checksum ... OK
> > > >    Uncompressing Kernel Image ... OK
> > > > 
> > > > I also never understood why the load address I used at the u-boot prompt
> > > > differed from the one that the uImage build process showed at the end of
> > > > the kernel build:
> > > > 
> > > >   GZIP    arch/sh/boot/vmlinux.bin.gz
> > > >   UIMAGE  arch/sh/boot/uImage.gz
> > > > Image Name:   Linux-7.3.0-rc5-00002-g02d53450e
> > > > Created:      Mon Sep 28 14:56:15 2026
> > > > Image Type:   SuperH Linux Kernel Image (gzip compressed)
> > > > Data Size:    3649249 Bytes = 3563.72 KiB = 3.48 MiB
> > > > Load Address: 8c010000
> > > > Entry Point:  8c011000
> > > >   Image arch/sh/boot/uImage is ready
> > > > 
> > > > Any idea?
> > > 
> > > 0x89000000 is the address where you load the uImage.
> > > 0x8c010000 is the address where the unpacked kernel image will be
> > > stored.
> > > 
> > > The latter comes from your kernel config:
> > > 
> > >     arch/sh/boot/Makefile:UIMAGE_LOADADDR = $(KERNEL_LOAD)
> > >     arch/sh/boot/Makefile:UIMAGE_ENTRYADDR = $(KERNEL_ENTRY)
> > > 
> > > It's similar on ARM platforms that (still) use uImage.
> > 
> > The thing is that it stopped working after 6.5.0 and I never figured
> > out why. I did test all kinds of changes and it just never worked
> > again while pre 6.5.0 kernels work just fine.
> 
> Have you tried to bisect it?

Yes, I did. I did not manage to find the commit that caused it.

Maybe Karl has an idea. This image is known to work [1]:

glaubitz@suse-laptop:~> file uImage-git.gz
uImage-git.gz: u-boot legacy uImage, Linux-6.5.0-rc2, Linux/SuperH, OS Kernel Image (gzip), 4416616 bytes, Mon Jul 17 14:17:35 2023, Load Address: 0X80001000, Entry Point: 0X80002000, Header CRC:
0XBA69203A, Data CRC: 0XD097D603
glaubitz@suse-laptop:~>

It boots fine with:

usb reset ; fatload usb 0:1 0x89000000 uImage-git.gz ; pmb ; bootm

Adrian

> [1] https://people.debian.org/~glaubitz/sh7785lcr/uImage-git.gz

-- 
 .''`.  John Paul Adrian Glaubitz
: :' :  Debian Developer
`. `'   Physicist
  `-    GPG: 62FF 8A75 84E0 2956 9546  0006 7426 3B37 F5B5 F913