On Mon, 11 Jun 2007 15:59:30 -0500 "Jose R. Santos" <jrs@xxxxxxxxxx> wrote: > On Mon, 11 Jun 2007 18:42:28 +0200 > Valerie Clement <valerie.clement@xxxxxxxx> wrote: > > > This patch converts some int or unsigned int block number definitions > > to blk_t type. Most conversions are necessary for 64-bit porting. > > What about unsigned long? There are various places where block is > represented by using unsigned long which is 32-bit when running 32 bit > binaries. Most PPC64 distributions compile their user-space tools in > 32-bit binaries. > > I wonder how badly it will break. I found a PPC64 box to test this out and it seems to break pretty badly when compiled in the default 32bit mode. Should ext2_get_device_size() fail if the number of block is greater than 2**32 and the size of unsigned long is 4? At least until it can be verified that this will work under 32bit binaries since it could corrupt a filesystem to do an fsck. mke2fs 1.39-tyt3 (29-Apr-2007) Warning:Filesystem greater than 2**32 blocks, 64-bit incompatible feature forced. Filesystem label= OS type: Linux Block size=1024 (log=0) Fragment size=1024 (log=0) 4194304 inodes, 5 blocks 0 blocks (0.00%) reserved for the super user First data block=1 524288 block groups 8192 blocks per group, 8192 fragments per group 8 inodes per group Superblock backups stored on blocks: 267387660, 267387660, 267387660, 267387660, 267387660, 267387660, 267387660, 267387660, 267387660, 267387660, 267387660, 267387660, 267387660, 267387660, 267387660, 267387660, 267387660, 267387660, 267387660, 267387660, 267387660, 267387660, 267387660, 267387660, 267387660, 267387660 mke2fs: Could not allocate block in ext2 filesystem while trying to allocate filesystem tables -JRS - To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html