Re: [PATCH] block devices: validate block device capacity

From: Mikulas Patocka
Date: Thu Jan 30 2014 - 18:10:36 EST




On Thu, 30 Jan 2014, James Bottomley wrote:

> Why is this? the whole reason for CONFIG_LBDAF is supposed to be to
> allow 64 bit offsets for block devices on 32 bit. It sounds like
> there's somewhere not using sector_t ... or using it wrongly which needs
> fixing.

The page cache uses unsigned long as a page index. Therefore, if unsigned
long is 32-bit, the block device may have at most 2^32-1 pages.

> > On 32-bit architectures, we must limit block device size to
> > PAGE_SIZE*(2^32-1).
>
> So you're saying CONFIG_LBDAF can never work, why?
>
> James

CONFIG_LBDAF works, but it doesn't allow unlimited capacity: on x86,
without CONFIG_LBDAF, the limit is 2TiB. With CONFIG_LBDAF, the limit is
16TiB (4096*2^32).

Mikulas
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/