On 9/10/24 1:40 PM, Jesper Dangaard Brouer wrote:

On 10/09/2024 21.21, Jens Axboe wrote:

quoted

On 9/10/24 1:19 PM, Jesper Dangaard Brouer wrote:

quoted

On 10/09/2024 20.38, Jens Axboe wrote:

quoted

On 9/10/24 11:53 AM, Jesper Dangaard Brouer wrote:

quoted

Hi Hellwig,

I bisected my boot problem down to this commit:

$ git bisect good
af2814149883e2c1851866ea2afcd8eadc040f79 is the first bad commit
commit af2814149883e2c1851866ea2afcd8eadc040f79
Author: Christoph Hellwig [off-list ref]
Date:   Mon Jun 17 08:04:38 2024 +0200

      block: freeze the queue in queue_attr_store

      queue_attr_store updates attributes used to control generating I/O, and
      can cause malformed bios if changed with I/O in flight.  Freeze the queue
      in common code instead of adding it to almost every attribute.

      Signed-off-by: Christoph Hellwig [off-list ref]
      Reviewed-by: Bart Van Assche [off-list ref]
      Reviewed-by: Damien Le Moal [off-list ref]
      Reviewed-by: Hannes Reinecke [off-list ref]
      Reviewed-by: Chaitanya Kulkarni [off-list ref]
      Link: https://lore.kernel.org/r/20240617060532.127975-12-hch@lst.de (local)
      Signed-off-by: Jens Axboe [off-list ref]

   block/blk-mq.c    | 5 +++--
   block/blk-sysfs.c | 9 ++-------
   2 files changed, 5 insertions(+), 9 deletions(-)

git describe --contains af2814149883e2c1851866ea2afcd8eadc040f79
v6.11-rc1~80^2~66^2~15

Curious, does your init scripts attempt to load a modular scheduler
for your root drive?

I have no idea, this is just a standard Fedora 40.

quoted

Reference: https://git.kernel.dk/cgit/linux/commit/?h=for-6.12/block&id=3c031b721c0ee1d6237719a6a9d7487ef757487b

[1] https://git.kernel.dk/cgit/linux/commit/?h=for-6.12/block&id=3c031b721c0ee1d6237719a6a9d7487ef757487b

quoted

quoted

The commit doesn't apply cleanly on top of af2814149883e2c185.

$ patch --dry-run -p1 < ../block-jens/block-jens-bootfix.patch
checking file block/blk-sysfs.c
Hunk #1 FAILED at 23.
Hunk #2 succeeded at 469 (offset 56 lines).
Hunk #3 succeeded at 484 (offset 56 lines).
Hunk #4 succeeded at 723 with fuzz 1 (offset 45 lines).
1 out of 4 hunks FAILED
checking file block/elevator.c
Hunk #1 FAILED at 698.
1 out of 1 hunk FAILED
checking file block/elevator.h
Hunk #1 FAILED at 148.
1 out of 1 hunk FAILED

I will try to apply and adjust manually.

Just apply it on top of current -git, doesn't have to be your bisection
point.

I applied it manually and now my testlab server boots :-)

Excellent! I'll get it staged for 6.11 instead. Thank for reporting and
testing.

Just with the patch[1] on top of bisection point
... as it was faster to recompile this way ;-)

That's a pathetic excuse for a test box then ;-)

-- 
Jens Axboe