|| ||Linus Torvalds <torvalds-AT-linux-foundation.org> |
|| ||Paolo Bonzini <pbonzini-AT-redhat.com> |
|| ||Re: [PATCH 2/3] block: fail SCSI passthrough ioctls on partition devices |
|| ||Fri, 23 Dec 2011 14:46:11 -0800|
|| ||Willy Tarreau <w-AT-1wt.eu>, linux-kernel-AT-vger.kernel.org,
security-AT-kernel.org, pmatouse-AT-redhat.com, agk-AT-redhat.com,
jbottomley-AT-parallels.com, mchristi-AT-redhat.com, msnitzer-AT-redhat.com|
|| ||Article, Thread
On Fri, Dec 23, 2011 at 6:15 AM, Paolo Bonzini <email@example.com> wrote:
> But does it actually do anything? For me, "eject /dev/sdf1" does nothing
> beyond unmounting the disk. Yes, it sends a CDROMEJECT ioctl which becomes
> a start/stop unit SCSI command, but it has no effect on the USB stick I
Guys, you are TOTALLY MISSING THE POINT.
The point is that the patch-series is damned dangerous. You have no
clue at all what it will do on various different distributions and
with various different hardware.
What's so hard to understand about this? Applying it at this point in
the release cycle with basically *zero* testing would be crazy. I gave
you just one example of where real people do ioctl's on a partition,
and where the behavior of the kernel clearly changes as a result.
The fact is, partitions are what most user interactions see. Suddenly
totally changing things and saying "you can't do that on a partition"
when clearly people *have* been doing that on partitions isn't
something we can do without serious testing.
No amount of "it didn't change in the one situation I tested" is going
to change that. What about somebody running their own distro (there's
this Russian distro maker who regularly finds regressions that nobody
else has ever seen)? What about somebody upgrading kernels on a
five-year-old user space (== pretty much any of the "enterprise"
distros)? What about other random hardware than just the USB disk
example that I gave?
It sounds like people didn't even *think* of the potential issues this
patch can bring. I'd absolutely be insane to apply them for -rc7.
to post comments)