[bug] uboot 2022.07 hangs on rpi 2 with attached usb storage

[Jan Palus]

On 23.07.2022 16:19, Jan Palus wrote:
> On 22.07.2022 02:59, Simon Glass wrote:
> > Hi Jan,
> > 
> > On Mon, 18 Jul 2022 at 11:48, Jan Palus <jpalus at fastmail.com> wrote:
> > >
> > > u-boot 2022.07 boots fine without any USB devices attached to
> > > RaspberryPi 2 however it hangs early on if external USB drive is
> > > connected, right after:
> > >
> > >    Request Sense returned 02 04 01
> > >
> > > git bisect indicates first commit to cause regression is:
> > >
> > >   8c9812a5d557c4eacf164147d7380b3af1b222ec is the first bad commit
> > >   commit 8c9812a5d557c4eacf164147d7380b3af1b222ec
> > >   Author: AKASHI Takahiro <takahiro.akashi at linaro.org>
> > >   Date:   Tue Mar 8 20:36:40 2022 +0900
> > >
> > >       usb: storage: call device_probe() after scanning
> > >
> > >       Every time a usb bus/port is scanned and a new device is detected,
> > >       we want to call device_probe() as it will give us a chance to run
> > >       additional post-processings for some purposes.
> > >
> > >       In particular, support for creating partitions on a device will be added.
> > >
> > >       Signed-off-by: AKASHI Takahiro <takahiro.akashi at linaro.org>
> > >       Reviewed-by: Simon Glass <sjg at chromium.org>
> > >
> > > Reverting this commit fixes the issue.
> > >
> > > Note that USB drive is TOSHIBA MQ04UBD200 and it's not used for booting.
> > > Also note that without this change 0 storage devices are detected even
> > > when drive is attached.
> > 
> > I am not sure what is going on here. Can you provide the full console
> > trace of the boot? Any idea where it is hanging?
> 
...
> 
> Now the place where it hangs is:
> 
>   part_efi.c:
> 
>   static int part_test_efi(struct blk_desc *dev_desc)
>   {
>           ALLOC_CACHE_ALIGN_BUFFER_PAD(legacy_mbr, legacymbr, 1, dev_desc->blksz);
> 
> where dev_desc->blksz is 3782209548.
> 

So it appears block size is read incorrectly? Should be 512 but not sure
where this value 3782209548 is coming from, it's not block capacity
either. After plugging in Linux reports:

  sd 0:0:0:0: [sda] 3907029168 512-byte logical blocks: (2.00 TB/1.82 TiB)