Fix for access beyond end of device error
authorNed Bass <bass6@llnl.gov>
Wed, 10 Nov 2010 21:36:18 +0000 (13:36 -0800)
committerBrian Behlendorf <behlendorf1@llnl.gov>
Thu, 11 Nov 2010 05:29:07 +0000 (21:29 -0800)
commite06be586410cdad14d2dce76af4f2d43eebe7c83
tree62b9627e6cfdaec9fe187f062c3b4bdb8a839a23
parent1f30b9d432dd48826cd562757d19c2771b0973e8
Fix for access beyond end of device error

This commit fixes a sign extension bug affecting l2arc devices.  Extremely
large offsets may be passed down to the low level block device driver on
reads, generating errors similar to

    attempt to access beyond end of device
    sdbi1: rw=14, want=36028797014862705, limit=125026959

The unwanted sign extension occurrs because the function arc_read_nolock()
stores the offset as a daddr_t, a 32-bit signed int type in the Linux kernel.
This offset is then passed to zio_read_phys() as a uint64_t argument, causing
sign extension for values of 0x80000000 or greater.  To avoid this, we store
the offset in a uint64_t.

This change also changes a few daddr_t struct members to uint64_t in the libspl
headers to avoid similar bugs cropping up in the future.  We also add an ASSERT
to __vdev_disk_physio() to check for invalid offsets.

Closes #66
Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov>
lib/libspl/include/sys/dkio.h
lib/libspl/include/sys/dklabel.h
lib/libspl/include/sys/vtoc.h
module/zfs/arc.c
module/zfs/vdev_disk.c