[kernel] r19237 - in dists/sid/linux/debian: . patches patches/bugfix/all
Ben Hutchings
benh at alioth.debian.org
Tue Jul 3 05:08:32 UTC 2012
Author: benh
Date: Tue Jul 3 05:08:29 2012
New Revision: 19237
Log:
scsi: Silence unnecessary warnings about ioctl to partition (Closes: #656899)
Added:
dists/sid/linux/debian/patches/bugfix/all/scsi-Silence-unnecessary-warnings-about-ioctl-to-par.patch
Modified:
dists/sid/linux/debian/changelog
dists/sid/linux/debian/patches/series
Modified: dists/sid/linux/debian/changelog
==============================================================================
--- dists/sid/linux/debian/changelog Tue Jul 3 05:03:29 2012 (r19236)
+++ dists/sid/linux/debian/changelog Tue Jul 3 05:08:29 2012 (r19237)
@@ -3,6 +3,8 @@
* linux-libc-dev: Fix redundant 'GNU glibc' in description (Closes: #631228)
* README.source: Correct name of main patch series file
* [sh] Fix up store queue code for subsys_interface changes (Closes: #680025)
+ * scsi: Silence unnecessary warnings about ioctl to partition
+ (Closes: #656899)
-- Ben Hutchings <ben at decadent.org.uk> Fri, 29 Jun 2012 15:01:22 +0100
Added: dists/sid/linux/debian/patches/bugfix/all/scsi-Silence-unnecessary-warnings-about-ioctl-to-par.patch
==============================================================================
--- /dev/null 00:00:00 1970 (empty, because file is newly added)
+++ dists/sid/linux/debian/patches/bugfix/all/scsi-Silence-unnecessary-warnings-about-ioctl-to-par.patch Tue Jul 3 05:08:29 2012 (r19237)
@@ -0,0 +1,46 @@
+From: Jan Kara <jack at suse.cz>
+Date: Fri, 15 Jun 2012 12:52:46 +0200
+Subject: scsi: Silence unnecessary warnings about ioctl to partition
+
+commit 6d9359280753d2955f86d6411047516a9431eb51 upstream.
+
+Sometimes, warnings about ioctls to partition happen often enough that they
+form majority of the warnings in the kernel log and users complain. In some
+cases warnings are about ioctls such as SG_IO so it's not good to get rid of
+the warnings completely as they can ease debugging of userspace problems
+when ioctl is refused.
+
+Since I have seen warnings from lots of commands, including some proprietary
+userspace applications, I don't think disallowing the ioctls for processes
+with CAP_SYS_RAWIO will happen in the near future if ever. So lets just
+stop warning for processes with CAP_SYS_RAWIO for which ioctl is allowed.
+
+CC: Paolo Bonzini <pbonzini at redhat.com>
+CC: James Bottomley <JBottomley at parallels.com>
+CC: linux-scsi at vger.kernel.org
+Acked-by: Paolo Bonzini <pbonzini at redhat.com>
+Signed-off-by: Jan Kara <jack at suse.cz>
+Signed-off-by: Jens Axboe <axboe at kernel.dk>
+[bwh: Backported to 3.2: use ENOTTY, not ENOIOCTLCMD]
+---
+ block/scsi_ioctl.c | 5 ++++-
+ 1 file changed, 4 insertions(+), 1 deletion(-)
+
+--- a/block/scsi_ioctl.c
++++ b/block/scsi_ioctl.c
+@@ -721,11 +721,14 @@ int scsi_verify_blk_ioctl(struct block_device *bd, unsigned int cmd)
+ break;
+ }
+
++ if (capable(CAP_SYS_RAWIO))
++ return 0;
++
+ /* In particular, rule out all resets and host-specific ioctls. */
+ printk_ratelimited(KERN_WARNING
+ "%s: sending ioctl %x to a partition!\n", current->comm, cmd);
+
+- return capable(CAP_SYS_RAWIO) ? 0 : -ENOTTY;
++ return -ENOTTY;
+ }
+ EXPORT_SYMBOL(scsi_verify_blk_ioctl);
+
Modified: dists/sid/linux/debian/patches/series
==============================================================================
--- dists/sid/linux/debian/patches/series Tue Jul 3 05:03:29 2012 (r19236)
+++ dists/sid/linux/debian/patches/series Tue Jul 3 05:08:29 2012 (r19237)
@@ -373,3 +373,5 @@
# Until next ABI bump
debian/driver-core-avoid-ABI-change-for-removal-of-__must_check.patch
+
+bugfix/all/scsi-Silence-unnecessary-warnings-about-ioctl-to-par.patch
More information about the Kernel-svn-changes
mailing list