On 4/22/2010 1:21 PM, I wrote:

raid1.c's read_balance() function has:

        const unsigned long this_sector = r1_bio->sector;

Doesn't that need to be:

        const sector_t this_sector = r1_bio->sector;

for compatibility with LBD on 32-bit platforms?

Any comments on this?

At very least it appears the truncation to 32 bits will break the "disk
whose head is closest" logic.

-- 
Jordan Russell