public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Phillip Susi <psusi@cfl.rr.com>
To: Kyle Moffett <mrmacman_g4@mac.com>
Cc: Alan Stern <stern@rowland.harvard.edu>,
	Alon Bar-Lev <alon.barlev@gmail.com>,
	Kernel development list <linux-kernel@vger.kernel.org>
Subject: Re: Flames over -- Re: Which is simpler?
Date: Tue, 14 Feb 2006 16:13:16 -0500	[thread overview]
Message-ID: <43F247EC.3070503@cfl.rr.com> (raw)
In-Reply-To: <D2DF8AE9-51A0-42DF-8346-0EF4C264627D@mac.com>

Kyle Moffett wrote:
> How is swapping USB devices while suspended unreasonable?  Come to 
> think of it, I did it inadvertently about 15 minutes ago with my PowerBook

Because you can not go yanking devices out from under the kernel without 
it's knowledge or consent.  This is no more acceptable than ejecting a 
floppy without first unmounting it; the only difference is that the 
floppy drive doesn't erroneously inform the kernel that you have done 
this simply because you suspend. 

> (while booted into Mac OS).  I had a USB key that I was copying a file 
> to for someone.  After shutting the lid, I unplugged mouse, USB key, 
> and power block.  About 30 minutes later I had somebody else with a 
> key who wanted to give me a file.  I had the key plugged in before the 
> laptop was finished waking up from sleep.  Now, I don't know for 
> certain that neither key had a serial number, but the two I have here 
> in my hand certainly don't, and I could _easily_ see somebody swapping 
> USB keys not knowing that they're "not supposed to do that" and 
> getting massive data corruption when the filesystem reads and writes 
> pages from a completely different block device.
>

Then they shot themselves in the foot.  That is no different than 
switching mounted floppies while suspended, or removing a mounted IDE 
hard drive while suspended, so they get what they deserve. 

> Did you read what I wrote?  People don't generally expect to randomly 
> plug and unplug SCSI drives whenever they feel like it.  They _do_ 
> expect to randomly plug and unplug USB drives, mice, keyboards, 
> tablets, network adapters, etc, because _everything_ supports such 
> random plugging.
>

No, they don't.  Users do not expect ( or should not and are told not to 
by admins ) to be able to yank out their usb memory stick while it is 
mounted.  They are told to always unmount first.  If they fail to do so, 
then they get what they asked for.  It doesn't matter if the disk is 
SCSI or USB; you don't go yanking it out without unmounting it first, or 
you will loose data. 

> Creating an extremely odd and hard to predict failure mode (when you 
> reconnect USB devices while suspended on hardware that doesn't support 
> proper USB suspend) with a high probability of causing data corruption 
> or crashes is wrong.  Especially since you could easily teach users 
> that "You need to eject USB things before you sleep the computer _or_ 
> just fix the kernel to do it for you.  That's probably something we 
> should be doing for all network filesystems anyways.
>

Users are already told to eject/unmount the media before removing it.  
If they fail to do that, it doesn't matter if the system is suspended or 
not; they broke the drive when they yanked it out while mounted.  As for 
fixing the kernel to unmount it for you, that is not always possible; 
take the root on usb case. 

Maybe it does make sense to have hibernation scripts unmount removable 
media for the common silly user who can't remember to unmount disks 
before ejecting/unplugging them, but you should be able to suspend a 
system with its root on a usb disk and not have the kernel panic for no 
good reason when it resumes. 



  reply	other threads:[~2006-02-14 21:14 UTC|newest]

Thread overview: 78+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-02-12 16:57 Flames over -- Re: Which is simpler? Alan Stern
2006-02-13  0:51 ` Phillip Susi
2006-02-13  2:19   ` Alan Stern
2006-02-13  3:52     ` Phillip Susi
2006-02-13  5:43       ` Kyle Moffett
2006-02-13 16:40         ` Phillip Susi
2006-02-13 16:31       ` Alan Stern
2006-02-13 17:14         ` Phillip Susi
2006-02-13 20:04           ` Alan Stern
2006-02-13 20:38             ` Phillip Susi
2006-02-13 21:24               ` Alan Stern
2006-02-13 22:27                 ` Rafael J. Wysocki
2006-02-14 19:26                   ` Alan Stern
2006-02-14 20:41                     ` Rafael J. Wysocki
2006-02-14 21:08                       ` Lee Revell
2006-02-15 15:56                       ` Alan Stern
2006-02-13 22:51                 ` J. Bruce Fields
2006-02-13 23:47                 ` Phillip Susi
2006-02-14  0:50                   ` Kyle Moffett
2006-02-14  2:09                     ` Phillip Susi
2006-02-14  4:09                       ` Kyle Moffett
2006-02-14  4:28                         ` Alan Stern
2006-02-14  5:11                           ` Kyle Moffett
2006-02-14 15:33                             ` Alan Stern
2006-02-14  6:27                         ` Phillip Susi
2006-02-14 16:23                           ` Kyle Moffett
2006-02-14 18:39                             ` Phillip Susi
2006-02-14 19:55                               ` Kyle Moffett
2006-02-14 21:13                                 ` Phillip Susi [this message]
2006-02-14 23:32                                   ` Kyle Moffett
2006-02-15  3:08                                     ` Phillip Susi
2006-02-14 19:14                             ` Olivier Galibert
2006-02-14 19:37                               ` Phillip Susi
2006-02-17 21:04                   ` Pavel Machek
2006-02-18 16:34                     ` Phillip Susi
2006-02-18 17:29                       ` Pavel Machek
2006-02-19  5:52                         ` Phillip Susi
2006-02-19  9:02                           ` Pavel Machek
2006-02-19 16:35                             ` Phillip Susi
2006-02-19 16:41                               ` Alan Stern
2006-02-19 19:17                                 ` Phillip Susi
2006-02-19 19:43                                   ` Pavel Machek
2006-02-20  0:56                                     ` Olivier Galibert
2006-02-20  1:01                                       ` Pavel Machek
2006-02-20  1:26                                         ` Olivier Galibert
2006-02-20  4:04                                           ` Alan Stern
2006-02-19 20:16                                   ` Bernd Eckenfels
2006-02-18 21:04                     ` Alan Stern
2006-02-19  0:02                       ` Andrew Morton
2006-02-19  6:02                         ` Phillip Susi
2006-02-19  6:32                           ` Andrew Morton
2006-02-19 16:39                             ` Phillip Susi
2006-02-19 16:54                               ` Alan Stern
2006-02-19 20:02                                 ` Andrew Morton
2006-02-19 20:44                                   ` Oliver Neukum
2006-02-19 21:02                                     ` Andrew Morton
2006-02-20  6:55                                       ` Oliver Neukum
2006-02-20  7:29                                         ` Andrew Morton
2006-02-20  7:57                                           ` Andrew Morton
2006-02-14 14:15     ` hackmiester / Hunter Fuller
2006-02-15 23:51     ` Pavel Machek
2006-02-13  2:25   ` Kyle Moffett
  -- strict thread matches above, loose matches on Subject: below --
2006-02-13 19:16 David Brownell
2006-02-13 20:08 ` Phillip Susi
2006-02-14  3:10   ` David Brownell
2006-02-14  6:05     ` Phillip Susi
2006-02-14 17:04       ` David Brownell
2006-02-15 23:43   ` Pavel Machek
2006-02-18 20:51     ` David Brownell
2006-02-19  6:06       ` Phillip Susi
2006-02-20  5:50         ` David Brownell
2006-02-20 16:07           ` Phillip Susi
2006-02-20 16:51             ` Olivier Galibert
2006-02-20 18:20               ` Phillip Susi
2006-02-20 18:44                 ` Olivier Galibert
2006-02-20 21:45                   ` Phillip Susi
2006-02-21 16:19             ` David Brownell
2006-02-21 18:30               ` Phillip Susi

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=43F247EC.3070503@cfl.rr.com \
    --to=psusi@cfl.rr.com \
    --cc=alon.barlev@gmail.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mrmacman_g4@mac.com \
    --cc=stern@rowland.harvard.edu \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox