[linux-lvm] Re: LVM 2.2 snapshot bug
andrea at suse.de
Tue Nov 7 13:21:52 UTC 2000
On Tue, Nov 07, 2000 at 11:55:42AM +0100, Rik van Riel wrote:
> On snapshot creation, the snapshot block device (/dev/vg0/snap1)
> is NOT made a read-only device, so ext3 tries to do journal
It is made read only.
> (leading to all kinds of nasty oopses)
Could you show me the Oopses?
> It should be easy enough to do an set_device_ro() on the LVM
> snapshot, shouldn't it?
That shouldn't be necessary. The way LVM handle this looks correct
to me, but maybe it's never been tested in the ll_rw_block layer because the
open(O_RDWR) check always handled it with ext2. Maybe ext3 forces writes via
ll_rw_block also when the device is mounted read only (probably when doing log
reply?) and maybe it hits the ll_rw_block check for the first time.
Currently we choose if a device is readable or not using the
VM_WRITE bitflag in the lv->lv_access field. If the bitflag is set
the device is writeable. You'll find that the snapshots has that bitflag
unset (please verify via /proc that they don't have the W capability set).
If the snapshot is not writeable as expected, the lvm hook in ll_rw_block
should return -1 and ll_rw_block should goto sorry just like if we would be
using the ro_bits via set_device_ro. So it should not be necessary
to use the set_device_ro.
Or maybe the bug is in ext3 that doesn't handle real read only blockdevices?
> That would fix the oopses I've been seeing and would make
Hopefully the Oopses will tell us more about this ext3/snapshot collision.
Also please ensure you can reproduce with 2.2.18pre17aa1 or 2.4.0-test10 to
make sure we're looking at the same sources.
More information about the linux-lvm