Bug 130126 - "hotplug mounting without sync" should also mention umount
Summary: "hotplug mounting without sync" should also mention umount
Status: VERIFIED FIXED
Alias: None
Product: SUSE LINUX 10.0
Classification: openSUSE
Component: Release Notes (show other bugs)
Version: Final
Hardware: Other Other
: P5 - None : Normal
Target Milestone: ---
Assignee: Anja
QA Contact: Andreas Jaeger
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 85413
  Show dependency treegraph
 
Reported: 2005-10-21 22:58 UTC by Christian Boltz
Modified: 2005-11-17 13:32 UTC (History)
2 users (show)

See Also:
Found By: Other
Services Priority:
Business Priority:
Blocker: ---
Marketing QA Status: ---
IT Deployment: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Christian Boltz 2005-10-21 22:58:35 UTC
The 10.0 release notes contain a section 'Mounting Hotplugged Devices without 
"sync"'. 
 
There's nothing wrong with that (I guess, I did't test it ;-)  - but if you 
explain users how to mount removeable devices without sync option, you should 
also tell them to call   umount   before unplugging the device. 
The result of not umount'ing can be damaged filesystems or even data loss 
which is not very nice.
Comment 1 Andreas Jaeger 2005-10-25 06:55:28 UTC
Hannes, Stefan, what do you think?
Comment 2 Hannes Reinecke 2005-10-25 07:00:36 UTC
Yes, definitely.
Comment 3 Andreas Jaeger 2005-10-25 09:42:24 UTC
How should the text get changed?  Karl, please work with Hannes.
Comment 4 Karl Eichwalder 2005-10-25 11:43:00 UTC
Current text:

Mounting Hotplugged Devices without "sync"

   If writing to USB storage devices, such as USB sticks or USB hard drives, is rather
   slow or if you do not want synchronized writing to these devices, disable the sync
   mount option.
    1. Create a file with the name 90-storage-nosync.fdi and the following contents:
       <?xml version="1.0" encoding="ISO-8859-1"?>
       <deviceinfo version="0.2">
       <device>
       <!-- disable sync for mount -->
       <match key="block.is_volume" bool="true">
       <match key="volume.fsusage" string="filesystem">
       <match key="volume.uuid" string="UUID">
       <merge key="volume.policy.mount_option.sync" type="bool">false</merge>  
       </match>
       </match>
       </match>
       </device>
       </deviceinfo>
    2. Adjust the line <match key="volume.uuid" string="==UUID=="> according to your
       hardware. Run lshal to retrieve the volume.uuid and enter that ID instead of
       ==UUID==
    3. Copy 90-storage-nosync.fdi into this directory:
       cp 90-storage-nosync.fdi /usr/share/hal/fdi/policy/20thirdparty
    4. Restart the HAL service with rchal restart.

Is it enough to ask the user to enter "sync;sync;sync"?  In this case I'd propose to write something as follows:

To ensure data integrity, it is strongly recommended that you enter sync;sync;sync at the command line, before unplugging a device mounted without the sync option.
Comment 5 Danny Al-Gaaf 2005-10-26 11:20:28 UTC
IMO this is normaly not needed with subfs. If nobody writes to the device and the device is not in use the submountd try after 1 or 2 seconds to umount the device.

You must only be sure that the device is not in use and wait some seconds to remove the device. 
Comment 6 Karl Eichwalder 2005-10-28 05:20:13 UTC
Okay, I propose the following text now:

If you mount the device without "sync" you are responsible for data integrity on your own.  Before unplugging the medium, check that nobody writes to the device.  After any write operation watch the status LEDs (if available) and when the lights do not signal activity, wait some seconds and then remove the device.  If unsure, call "sync" at the command line.
Comment 7 Danny Al-Gaaf 2005-10-28 11:18:06 UTC
Sounds good for me.
Comment 8 Karl Eichwalder 2005-11-02 12:56:27 UTC
Thanks - I'm going to add it.
Comment 9 Karl Eichwalder 2005-11-07 12:54:48 UTC
Anja, I need a SWAMP ID for release-notes 10.0.16.  A nice one, please ;)
This update will also cover bug 119406 and bug 116930.
Comment 10 Karl Eichwalder 2005-11-07 13:12:08 UTC
release-notes 10.0.16 submitted to /work.
Comment 11 Anja Stock 2005-11-07 13:31:33 UTC
Karl, especially for you: SWAMPID 2754 
Comment 12 Karl Eichwalder 2005-11-07 14:02:44 UTC
Not too bad 2+7 = 9 and 5+4 = 9 :-)
Comment 13 Karl Eichwalder 2005-11-07 15:24:45 UTC
/work/src/done/PATCHINFO/release-notes.patch.box, created with "edit_patchinfo -b".  Must I add additional Distributions like 10.0-ppc and 10.0-x86_64?  Otherwise please finalize the YOU process.
Comment 14 Anja Stock 2005-11-14 12:33:35 UTC
you do not have to add a distribution string - thanks for submitting
Comment 15 Anja Stock 2005-11-17 13:32:11 UTC
released