Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932093Ab0KAQRK (ORCPT ); Mon, 1 Nov 2010 12:17:10 -0400 Received: from hera.kernel.org ([140.211.167.34]:46857 "EHLO hera.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1758533Ab0KAQRB (ORCPT ); Mon, 1 Nov 2010 12:17:01 -0400 From: Tejun Heo To: axboe@kernel.dk, hch@infradead.org, linux-kernel@vger.kernel.org, petero2@telia.com, schwidefsky@de.ibm.com, heiko.carstens@de.ibm.com, jack@suse.cz, akpm@linux-foundation.org, adilger.kernel@dilger.ca, tytso@mit.edu, mfasheh@suse.com, joel.becker@oracle.com, aelder@sgi.com, dm-devel@redhat.com, drbd-dev@lists.linbit.com, neilb@suse.de, leochen@broadcom.com, sbranden@broadcom.com, chris.mason@oracle.com, swhiteho@redhat.com, shaggy@linux.vnet.ibm.com, joern@logfs.org, konishi.ryusuke@lab.ntt.co.jp, reiserfs-devel@vger.kernel.org, viro@zeniv.linux.org.uk Subject: [PATCHSET] block: clean up bdev claim/release handling Date: Mon, 1 Nov 2010 17:15:24 +0100 Message-Id: <1288628129-12811-1-git-send-email-tj@kernel.org> X-Mailer: git-send-email 1.7.1 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.2.3 (hera.kernel.org [127.0.0.1]); Mon, 01 Nov 2010 16:15:37 +0000 (UTC) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3246 Lines: 78 bdev exclusive access handling is somewhat clumsy. The primary interface is bd_claim() and bd_release() but there also are open_bdev_exclusive(), close_bdev_exclusive() and bd_claim_by_disk() and bd_release_from_disk(). Gaining exclusive access as a separate step from open itself also has the inherent problem that the block layer doesn't know whether the current open attempt is for an exclusive access or not. The action of open itself may affect the existing exclusive access but there's no way to prevent it. This patchset simplifies the claim/release handling by folding it into blkdev_get/put() and making others simple wrappers around them. This simplifies code and removes strange corner cases like the non-atomicity of exclusive open described above. There are five patches in this series. 0001-mtd-fix-bdev-exclusive-open-bugs-in-block2mtd-add_de.patch 0002-btrfs-close_bdev_exclusive-should-use-the-same-flags.patch 0003-block-simplify-holder-symlink-handling.patch 0004-block-make-blkdev_get-put-handle-exclusive-access.patch 0005-block-reorganize-claim-release-implementation.patch 0001-0002 are fixes regarding exclusive bdev opens. 0003 simplifies holder/slave symlink handling. 0004 folds bdev claim/release into blkdev_get/put(). 0005 reorganizes implementation a bit. Cleaning up claim/release handling enables implementation of in-kernel disk event polling which be posted soon. Although this patchset touches various files including drivers and filesystems, the changes there are mostly trivial and I think it would be best to route these through the block tree. This patchset is on top of 2.6.37-rc1 and available in the following git tree. git://git.kernel.org/pub/scm/linux/kernel/git/tj/misc.git cleanup-bd_claim This patchset reduces about 400 lines of code. Thanks. block/ioctl.c | 5 drivers/block/drbd/drbd_int.h | 2 drivers/block/drbd/drbd_main.c | 7 drivers/block/drbd/drbd_nl.c | 103 ++----- drivers/block/pktcdvd.c | 22 - drivers/char/raw.c | 14 drivers/md/dm-table.c | 20 - drivers/md/md.c | 16 - drivers/mtd/devices/block2mtd.c | 10 drivers/s390/block/dasd_genhd.c | 2 fs/block_dev.c | 576 +++++++++------------------------------- fs/btrfs/volumes.c | 14 fs/ext3/super.c | 12 fs/ext4/super.c | 12 fs/gfs2/ops_fstype.c | 4 fs/jfs/jfs_logmgr.c | 17 - fs/logfs/dev_bdev.c | 4 fs/nilfs2/super.c | 4 fs/ocfs2/cluster/heartbeat.c | 2 fs/partitions/check.c | 2 fs/reiserfs/journal.c | 17 - fs/super.c | 14 fs/xfs/linux-2.6/xfs_super.c | 2 include/linux/fs.h | 22 - kernel/power/swap.c | 5 mm/swapfile.c | 7 26 files changed, 260 insertions(+), 655 deletions(-) -- tejun -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/