]> git.proxmox.com Git - mirror_zfs-debian.git/commit - module/zfs/zfs_vnops.c
Fix "allocating allocated segment" panic
authorBrian Behlendorf <behlendorf1@llnl.gov>
Wed, 31 Oct 2012 17:06:34 +0000 (10:06 -0700)
committerBrian Behlendorf <behlendorf1@llnl.gov>
Sat, 10 Nov 2012 03:01:09 +0000 (19:01 -0800)
commit4c837f0d931546e656b832caf11b8d4c2063d905
treea5a19bfb82f907861f24bf2946e42917509a2a21
parente26ade5101ba1d8e8350ff1270bfca4258e1ffe3
Fix "allocating allocated segment" panic

Gunnar Beutner did all the hard work on this one by correctly
identifying that this issue is a race between dmu_sync() and
dbuf_dirty().

Now in all cases the caller is responsible for preventing this
race by making sure the zfs_range_lock() is held when dirtying
a buffer which may be referenced in a log record.  The mmap
case which relies on zfs_putpage() was not taking the range
lock.  This code was accidentally dropped when the function
was rewritten for the Linux VFS.

This patch adds the required range locking to zfs_putpage().

It also adds the missing ZFS_ENTER()/ZFS_EXIT() macros which
aren't strictly required due to the VFS holding a reference.
However, this makes the code more consistent with the upsteam
code and there's no harm in being extra careful here.

Original-patch-by: Gunnar Beutner <gunnar@beutner.name>
Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov>
Closes #541
module/zfs/zfs_vnops.c