From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751919AbdIUOxG (ORCPT ); Thu, 21 Sep 2017 10:53:06 -0400 Received: from bombadil.infradead.org ([65.50.211.133]:58789 "EHLO bombadil.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751687AbdIUOxD (ORCPT ); Thu, 21 Sep 2017 10:53:03 -0400 Date: Thu, 21 Sep 2017 07:53:00 -0700 From: Christoph Hellwig To: Paolo Bonzini Cc: linux-kernel@vger.kernel.org, kvm@vger.kernel.org, axboe@kernel.de, linux-block@vger.kernel.org Subject: Re: [PATCH] block: drop "sending ioctl to a partition" message Message-ID: <20170921145300.GA1617@infradead.org> References: <1506005362-13429-1-git-send-email-pbonzini@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1506005362-13429-1-git-send-email-pbonzini@redhat.com> User-Agent: Mutt/1.8.3 (2017-05-23) X-SRS-Rewrite: SMTP reverse-path rewritten from by bombadil.infradead.org. See http://www.infradead.org/rpr.html Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org This looks ok to me, but do we even need to keep the special cases above? Is there anything relying on the safe but not very useful ioctls? Condensing the thing down to: int scsi_verify_blk_ioctl(struct block_device *bd, unsigned int cmd) { if (bd && bd == bd->bd_contains) return 0; if (capable(CAP_SYS_RAWIO)) return 0; return -ENOIOCTLCMD; } would certainly be nice.