From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754280Ab2ILLYL (ORCPT ); Wed, 12 Sep 2012 07:24:11 -0400 Received: from mail-pz0-f46.google.com ([209.85.210.46]:33287 "EHLO mail-pz0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753531Ab2ILLYD (ORCPT ); Wed, 12 Sep 2012 07:24:03 -0400 From: Paolo Bonzini To: linux-kernel@vger.kernel.org Cc: Tejun Heo , James Bottomley , Jens Axboe , Ric Wheeler , linux-iscsi@vger.kernel.org Subject: [PATCH 1/3] block: add back queue-private command filter Date: Wed, 12 Sep 2012 13:23:42 +0200 Message-Id: <1347449024-26387-2-git-send-email-pbonzini@redhat.com> X-Mailer: git-send-email 1.7.12 In-Reply-To: <1347449024-26387-1-git-send-email-pbonzini@redhat.com> References: <1347449024-26387-1-git-send-email-pbonzini@redhat.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org The command filter used to be mutable via sysfs, but this was broken and backed out. Let's add it back. This patch adds the infrastructure for filtering, but unlike the old code this one just adds a pointer to request_queue, so as to make it cheaper in the majority of cases where no special filtering is desired. This is a partial (and massaged) revert of commit 018e044 (block: get rid of queue-private command filter, 2009-06-26). Cc: linux-iscsi@vger.kernel.org Signed-off-by: Paolo Bonzini --- block/blk-sysfs.c | 2 ++ block/bsg.c | 2 +- block/scsi_ioctl.c | 17 +++++------------ drivers/scsi/sg.c | 4 +++- include/linux/blkdev.h | 10 +++++++++- 5 files changed, 20 insertions(+), 15 deletions(-) diff --git a/block/blk-sysfs.c b/block/blk-sysfs.c index 9628b29..5a0de07 100644 --- a/block/blk-sysfs.c +++ b/block/blk-sysfs.c @@ -487,6 +487,8 @@ static void blk_release_queue(struct kobject *kobj) blkcg_exit_queue(q); + kfree(q->cmd_filter); + if (q->elevator) { spin_lock_irq(q->queue_lock); ioc_clear_queue(q); diff --git a/block/bsg.c b/block/bsg.c index ff64ae3..09956da 100644 --- a/block/bsg.c +++ b/block/bsg.c @@ -187,7 +187,7 @@ static int blk_fill_sgv4_hdr_rq(struct request_queue *q, struct request *rq, return -EFAULT; if (hdr->subprotocol == BSG_SUB_PROTOCOL_SCSI_CMD) { - if (blk_verify_command(rq->cmd, has_write_perm)) + if (blk_verify_command(q->cmd_filter, rq->cmd, has_write_perm)) return -EPERM; } else if (!capable(CAP_SYS_RAWIO)) return -EPERM; diff --git a/block/scsi_ioctl.c b/block/scsi_ioctl.c index 9a87daa..c8862e9 100644 --- a/block/scsi_ioctl.c +++ b/block/scsi_ioctl.c @@ -33,11 +33,6 @@ #include #include -struct blk_cmd_filter { - unsigned long read_ok[BLK_SCSI_CMD_PER_LONG]; - unsigned long write_ok[BLK_SCSI_CMD_PER_LONG]; -}; - static struct blk_cmd_filter blk_default_cmd_filter; /* Command group 3 is reserved and should never be used. */ @@ -196,17 +191,15 @@ static void blk_set_cmd_filter_defaults(struct blk_cmd_filter *filter) __set_bit(GPCMD_SET_READ_AHEAD, filter->write_ok); } -int blk_verify_command(unsigned char *cmd, fmode_t has_write_perm) +int blk_verify_command(struct blk_cmd_filter *filter, + unsigned char *cmd, fmode_t has_write_perm) { - struct blk_cmd_filter *filter = &blk_default_cmd_filter; - /* root can do any command. */ if (capable(CAP_SYS_RAWIO)) return 0; - /* if there's no filter set, assume we're filtering everything out */ if (!filter) - return -EPERM; + filter = &blk_default_cmd_filter; /* Anybody who can open the device can do a read-safe command */ if (test_bit(cmd[0], filter->read_ok)) @@ -225,7 +218,7 @@ static int blk_fill_sghdr_rq(struct request_queue *q, struct request *rq, { if (copy_from_user(rq->cmd, hdr->cmdp, hdr->cmd_len)) return -EFAULT; - if (blk_verify_command(rq->cmd, mode & FMODE_WRITE)) + if (blk_verify_command(q->cmd_filter, rq->cmd, mode & FMODE_WRITE)) return -EPERM; /* @@ -472,7 +465,7 @@ int sg_scsi_ioctl(struct request_queue *q, struct gendisk *disk, fmode_t mode, if (in_len && copy_from_user(buffer, sic->data + cmdlen, in_len)) goto error; - err = blk_verify_command(rq->cmd, mode & FMODE_WRITE); + err = blk_verify_command(q->cmd_filter, rq->cmd, mode & FMODE_WRITE); if (err) goto error; diff --git a/drivers/scsi/sg.c b/drivers/scsi/sg.c index 9c5c5f2..2ba7c82 100644 --- a/drivers/scsi/sg.c +++ b/drivers/scsi/sg.c @@ -217,11 +217,13 @@ static void sg_put_dev(Sg_device *sdp); static int sg_allow_access(struct file *filp, unsigned char *cmd) { struct sg_fd *sfp = filp->private_data; + struct request_queue *q = sfp->parentdp->device->request_queue; if (sfp->parentdp->device->type == TYPE_SCANNER) return 0; - return blk_verify_command(cmd, filp->f_mode & FMODE_WRITE); + return blk_verify_command(q->cmd_filter, + cmd, filp->f_mode & FMODE_WRITE); } static int get_exclude(Sg_device *sdp) diff --git a/include/linux/blkdev.h b/include/linux/blkdev.h index 4a2ab7c..b5c5f8a 100644 --- a/include/linux/blkdev.h +++ b/include/linux/blkdev.h @@ -258,6 +258,11 @@ struct blk_queue_tag { #define BLK_SCSI_MAX_CMDS (256) #define BLK_SCSI_CMD_PER_LONG (BLK_SCSI_MAX_CMDS / (sizeof(long) * 8)) +struct blk_cmd_filter { + unsigned long read_ok[BLK_SCSI_CMD_PER_LONG]; + unsigned long write_ok[BLK_SCSI_CMD_PER_LONG]; +}; + struct queue_limits { unsigned long bounce_pfn; unsigned long seg_boundary_mask; @@ -423,6 +428,8 @@ struct request_queue { struct bsg_class_device bsg_dev; #endif + struct blk_cmd_filter *cmd_filter; + #ifdef CONFIG_BLK_CGROUP struct list_head all_q_node; #endif @@ -1005,7 +1012,8 @@ static inline int sb_issue_zeroout(struct super_block *sb, sector_t block, gfp_mask); } -extern int blk_verify_command(unsigned char *cmd, fmode_t has_write_perm); +extern int blk_verify_command(struct blk_cmd_filter *filter, + unsigned char *cmd, fmode_t has_write_perm); enum blk_default_limits { BLK_MAX_SEGMENTS = 128, -- 1.7.1