[PATCH] Revert "drivers: scsi: fix inaccurate block count reporting in scsi operations"

[Tony Dinh]

Hi Balaji,

On Fri, Dec 19, 2025 at 1:17 PM Tony Dinh <mibodhi at gmail.com> wrote:
>
> Hi Balaji,
>
> On Fri, Dec 19, 2025 at 8:29 AM Tom Rini <trini at konsulko.com> wrote:
> >
> > On Fri, Dec 19, 2025 at 11:04:07AM +0530, Balaji Selvanathan wrote:
> > > Hi Tom,
> > >
> > > Apologies for late reply. Somehow missed the mail.
> > >
> > > Upon analysis, it seems the most likely root cause is Tony's 238GB disk
> > > triggers the CONFIG_SYS_64BIT_LBA code path in scsi_read function. Any read
> > > beyond block 268,435,455 blocks enters this path.
> > >
> > > 'blks' is decremented both inside the 64-bit LBA block AND after successful
> > > scsi_exec(), causing incorrect block count tracking and (most likely) the
> > > data abort observed.
> > >
> > > The fix is to remove 'blks -= blocks;' from inside the CONFIG_SYS_64BIT_LBA
> > > block in scsi_read() function.
> > >
> > > Hi Tony, can you please check with this change?

Confirmed that the above fix works fine! I was able to boot the Linux
kernel from the 238GB HDD.

All the best,
Tony

> > >
> > > Tom, shall I raise a new patch for this now?
> >
> > Thanks for looking in to this now. Yes, please post a proper patch for
> > it so Tony can test and add a Tested-by if it works, that'll make
> > pulling to master before release easiest.
>
> FYI, I will be traveling a week from now. So it would be best if you
> can send the patch in the next couple days. Otherwise, someone else
> needs to do the test in time for the Jan 05 release.
>
> All the best,
> Tony
>
> >
> > --
> > Tom