Fix send/recv 'dataset is busy' errors
authorBrian Behlendorf <behlendorf1@llnl.gov>
Fri, 15 Jul 2011 23:15:27 +0000 (16:15 -0700)
committerBrian Behlendorf <behlendorf1@llnl.gov>
Fri, 15 Jul 2011 23:37:19 +0000 (16:37 -0700)
commit61f218b0900194edfb2e67739221e49c46fc5ee0
tree8a8537b67e91b7c0a4d2b46575fb743743ecb323
parent615ab66d18f74b3f4979c06192e1aa1082523dd3
Fix send/recv 'dataset is busy' errors

This commit fixes a regression which was accidentally introduced by
the Linux 2.6.39 compatibility chanages.  As part of these changes
instead of holding an active reference on the namepsace (which is
no longer posible) a reference is taken on the super block.  This
reference ensures the super block remains valid while it is in use.

To handle the unlikely race condition of the filesystem being
unmounted concurrently with the start of a 'zfs send/recv' the
code was updated to only take the super block reference when there
was an existing reference.  This indicates that the filesystem is
active and in use.

Unfortunately, in the 'zfs recv' case this is not the case.  The
newly created dataset will not have a super block without an
active reference which results in the 'dataset is busy' error.

The most straight forward fix for this is to simply update the
code to always take the reference even when it's zero.  This
may expose us to very very unlikely concurrent umount/send/recv
case but the consequences of that are minor.

Closes #319
module/zfs/zfs_ioctl.c