Talk:How to hotswap Ultrabay devices - Revision history

Kevinoid: /* hdparm -Y removed from script for Ultrabay eject */ new section

2024-01-11T01:12:41Z

‎hdparm -Y removed from script for Ultrabay eject: new section

Kevinoid: Add warning about docked sysfs attr

2017-03-14T19:45:17Z

Add warning about docked sysfs attr

Jicama: /* HAL script */

2010-11-07T01:58:27Z

‎HAL script

Hackepeter: /* changed regex in DISPO_USER to catch tabs in lastlog output */

2010-01-19T09:23:15Z

‎changed regex in DISPO_USER to catch tabs in lastlog output

Roo: /* get notify-send working */ replaced w with lastlog as w has an username length limit

2010-01-08T16:47:36Z

‎get notify-send working: replaced w with lastlog as w has an username length limit

Roo: /* get notify-send working */ new section

2010-01-07T22:19:52Z

‎get notify-send working: new section

Tonko: /* Recent Ubuntu versions */

2009-08-11T09:07:34Z

‎Recent Ubuntu versions

Bombenbach: /* Recent Ubuntu versions */

2009-08-11T07:52:25Z

‎Recent Ubuntu versions

Tonko: /* Recent Ubuntu versions */

2009-08-10T09:31:51Z

‎Recent Ubuntu versions

Bombenbach at 12:01, 9 August 2009

2009-08-09T12:01:05Z

← Older revision		Revision as of 01:12, 11 January 2024
Line 477:		Line 477:

	--[[User:Kevinoid\|Kevinoid]] ([[User talk:Kevinoid\|talk]]) 20:45, 14 March 2017 (CET)		--[[User:Kevinoid\|Kevinoid]] ([[User talk:Kevinoid\|talk]]) 20:45, 14 March 2017 (CET)
		+
		+	== hdparm -Y removed from script for Ultrabay eject ==
		+
		+	Previously the [[How_to_hotswap_Ultrabay_devices#Script_for_Ultrabay_eject\|Script for Ultrabay eject]] invoked <code>hdparm -Y</code> before <code>echo 1 > $ULTRABAY_SYSDIR/delete</code>. From the comment, it appears that the intent was to spin down the drive before deleting the ATA device. Unfortunately, this is both unnecessary, [https://lore.kernel.org/linux-scsi/b6da4b95-f6e4-4cf7-9cdc-239d91655b6d@kernel.org/ since it is done automatically by the kernel], and counterproductive, since [https://lore.kernel.org/linux-scsi/ZZ8CzOaXBkxyKxNw@x1-carbon/ the link must be reset to wake the device then enumerate/validate it before removing it]. Several kernel developers believe that [https://lore.kernel.org/linux-scsi/ZZ8HjQXC3vp2lCuv@x1-carbon/ mixing SCSI pass-through commands (from hdparm) and commands submitted by the sd driver (from sysfs) is unsupported/undefined behavior].
		+
		+	--[[User:Kevinoid\|Kevinoid]] ([[User talk:Kevinoid\|talk]]) 02:12, 11 January 2024 (CET)

← Older revision		Revision as of 19:45, 14 March 2017
Line 471:		Line 471:

	i hope this is useful to other users as well --[[User:Roo\|Roo]] 22:19, 7 January 2010 (UTC)		i hope this is useful to other users as well --[[User:Roo\|Roo]] 22:19, 7 January 2010 (UTC)
		+
		+	= A warning about sysfs docked =
		+
		+	A note to script writers/users: [https://marc.info/?l=linux-acpi&m=148952016304094&w=2 The docked sysfs attribute can be unreliable.] Be sure the script fails safely if it reports 0 when there are docked devices.
		+
		+	--[[User:Kevinoid\|Kevinoid]] ([[User talk:Kevinoid\|talk]]) 20:45, 14 March 2017 (CET)

@@ Line 110: / Line 110: @@
 Mandriva One 2009.1,
 haldaemon 0.5.12
 Thanks, it is possible that you have AHCI mode disabled in the BIOS for your HDD?
 --[[User:Tonko|Tonko]] 11:00, 31 July 2009 (UTC)
 = bash script for hotswapping UltraBay =

@@ Line 342: / Line 342: @@
 export DISPLAY=:0.0 # required for notify-send
-DISP0_USER=`lastlog | grep " :0" | awk '{ print $1 }'` # the space before the :0 is needed to exclude ipv6 adresses
+DISP0_USER=`lastlog | grep "[[:space:]]:0" | awk '{ print $1 }'` # the [[:space:]] before the :0 is needed to exclude ipv6 adresses
 DISP0_USER_HOME=`getent passwd ${DISP0_USER} | cut -d':' -f6`
 export XAUTHORITY=${DISP0_USER_HOME}/.Xauthority # also required for notify-send
@@ Line 447: / Line 447: @@
   export DISPLAY=:0.0 # required for notify-send
++
-+DISP0_USER=`lastlog | grep " :0" | awk '{ print $1 }'` # the space before the :0 is needed to exclude ipv6 adresses
++DISP0_USER=`lastlog | grep "[[:space:]]:0" | awk '{ print $1 }'` # the [[:space:]] before the :0 is needed to exclude ipv6 adresses
 +DISP0_USER_HOME=`getent passwd ${DISP0_USER} | cut -d':' -f6`
 +export XAUTHORITY=${DISP0_USER_HOME}/.Xauthority # also required for notify-send

@@ Line 342: / Line 342: @@
 export DISPLAY=:0.0 # required for notify-send
-DISP0_USER=`w | grep ":0 " | awk '{ print $1 }'`
+DISP0_USER=`lastlog | grep " :0" | awk '{ print $1 }'` # the space before the :0 is needed to exclude ipv6 adresses
 DISP0_USER_HOME=`getent passwd ${DISP0_USER} | cut -d':' -f6`
 export XAUTHORITY=${DISP0_USER_HOME}/.Xauthority # also required for notify-send
@@ Line 447: / Line 447: @@
   export DISPLAY=:0.0 # required for notify-send
++
-+DISP0_USER=`w | grep ":0 " | awk '{ print $1 }'`
++DISP0_USER=`lastlog | grep " :0" | awk '{ print $1 }'` # the space before the :0 is needed to exclude ipv6 adresses
 +DISP0_USER_HOME=`getent passwd ${DISP0_USER} | cut -d':' -f6`
 +export XAUTHORITY=${DISP0_USER_HOME}/.Xauthority # also required for notify-send

← Older revision		Revision as of 22:19, 7 January 2010
Line 319:		Line 319:
	Or perhaps we should run it ourselves automatically and just tell the user what is the cause of the busy filesystem.<br>		Or perhaps we should run it ourselves automatically and just tell the user what is the cause of the busy filesystem.<br>
	--[[User:Tonko\|Tonko]] 21:21, 8 June 2009 (UTC)		--[[User:Tonko\|Tonko]] 21:21, 8 June 2009 (UTC)
		+
		+	== get notify-send working ==
		+
		+	Hi there,
		+
		+	I've modified your ultrabay_eject script so that the notify-send is working for the user which is logged onto screen :0
		+	for an unified diff see below, here is the whole modified script
		+
		+	* whole script
		+	<pre>
		+	#!/bin/bash
		+
		+	# Change the following DEVPATH= to match your system, if you want to run this directly instead of having it called by the udev eject script
		+	# To find the right value, insert the UltraBay optical drive and run:
		+	# udevadm info --query=path --name=/dev/sr0 \| perl -pe 's!/block/...$!!'
		+	if [ "$DEVPATH" = "" ]
		+	then
		+	DEVPATH="/devices/pci0000:00/0000:00:1f.2/host1/target1:0:0/1:0:0:0"
		+	fi
		+
		+	shopt -s nullglob
		+	export DISPLAY=:0.0 # required for notify-send
		+
		+	DISP0_USER=`w \| grep ":0 " \| awk '{ print $1 }'`
		+	DISP0_USER_HOME=`getent passwd ${DISP0_USER} \| cut -d':' -f6`
		+	export XAUTHORITY=${DISP0_USER_HOME}/.Xauthority # also required for notify-send
		+
		+	ULTRABAY_SYSDIR=/sys$DEVPATH
		+
		+	# Find generic dock interface for UltraBay
		+	dock=$( /bin/grep -l ata_bay /sys/devices/platform/dock.?/type )
		+	dock=${dock%%/type}
		+	if [ -n "$dock" -a -d "$dock" ]; then
		+	logger ultrabay_eject starting eject of $dock
		+	else
		+	logger ultrabay_eject cannot locate bay dock device
		+	notify-send -u critical -t 100000 "ThinkPad Ultrabay eject failed" "Cannot locate bay dock device"
		+	fi
		+
		+	# Umount the filesystem(s) backed by the given major:minor device(s)
		+	unmount_rdev() { perl - "$@" <<'EOPERL' # let's do it in Perl
		+	for $major_minor (@ARGV) {
		+	$major_minor =~ m/^(\d+):(\d+)$/ or die;
		+	push(@tgt_rdevs, ($1<<8)\|$2);
		+	}
		+	# Sort by reverse length of mount point, to unmount sub-directories first
		+	open MOUNTS,"</proc/mounts" or die "$!";
		+	@mounts=sort { length($b->[1]) <=> length($a->[1]) } map { [ split ] } <MOUNTS>;
		+	close MOUNTS;
		+	foreach $m (@mounts) {
		+	($dev,$dir)=@$m;
		+	next unless -b $dev; $rdev=(stat($dev))[6];
		+	next unless grep($_==$rdev, @tgt_rdevs);
		+	system("umount","-v","$dir")==0 or $bad=1;
		+	if ($bad == 1) {
		+	system("logger","ultrabay_eject","ERROR unmounting",$dev,$dir);
		+	system("notify-send -u critical -t 100000 \"Error unmounting $dir\" \"Unmounting of $dir on $dev failed!\"");
		+	} else {
		+	system("logger","ultrabay_eject","unmounted",$dev,$dir);
		+	system("notify-send -u normal -t 5000 \"Unmounted $dir\"");
		+	};
		+	}
		+	exit 1 if $bad;
		+	EOPERL
		+	}
		+
		+	# Get the UltraBay's /dev/foo block device node
		+	ultrabay_dev_node() {
		+	UDEV_PATH="`readlink -e "$ULTRABAY_SYSDIR/block/"*`" \|\| return 1
		+	UDEV_NAME="`udevadm info --query=name --path=$UDEV_PATH`" \|\| return 1
		+	echo /dev/$UDEV_NAME
		+	}
		+
		+	if [ $( cat $dock/docked ) == 0 ]; then
		+	logger ultrabay_eject dock reports empty
		+	else
		+	if [ -d $ULTRABAY_SYSDIR ]; then
		+	logger ultrabay_eject dock occupied, shutting down storage device $DEVPATH
		+	sync
		+	# Unmount filesystems backed by this device
		+	## This seems to be very inelegant and prone to failure
		+	unmount_rdev `cat $ULTRABAY_SYSDIR/block/*/dev \
		+	$ULTRABAY_SYSDIR/block///dev` \
		+	\|\| {
		+	logger ultrabay_eject umounting failed
		+	echo 2 > /proc/acpi/ibm/beep # triple error tone
		+	notify-send -u critical -t 100000 "ThinkPad Ultrabay eject failed" "Please do not pull the device, doing so could cause file corruption and possibly hang the system. Unmounting of the filesystem on the ThinkPad Ultrabay device failed. Please put the eject leaver back in place, and try to unmount the filesystem manually. If this succeeds you can try the eject again"
		+	exit 1;
		+	}
		+	sync
		+	# Nicely power off the device
		+	DEVNODE=`ultrabay_dev_node` && hdparm -Y $DEVNODE
		+	# Let HAL+KDE notice the unmount and let the disk spin down
		+	sleep 0.5
		+	# Unregister this SCSI device:
		+	sync
		+	echo 1 > $ULTRABAY_SYSDIR/delete
		+	else
		+	logger ultrabay_eject bay occupied but incorrect device path $DEVPATH
		+	notify-send -u critical -t 100000 "ThinkPad Ultrabay eject failed" "Bay occupied but incorrect device path"
		+	echo 2 > /proc/acpi/ibm/beep # triple error tone
		+	exit 1
		+	fi
		+	fi
		+
		+	# We need sleep here so someone can disconnect the bay and the drive
		+	sleep 1
		+
		+	# Turn off power to the UltraBay
		+	logger ultrabay_eject undocking $dock
		+	echo 1 > $dock/undock
		+
		+	# Tell the user we're OK
		+	logger ultrabay_eject done
		+	echo 12 > /proc/acpi/ibm/beep
		+	notify-send -u normal -t 10000 "Safe to remove device" "The ThinkPad Ultrabay device can now safely be removed"
		+
		+	</pre>
		+
		+	*unified diff
		+	<pre>
		+	--- ultrabay_eject.orig 2010-01-07 22:51:40.000000000 +0100
		+	+++ ultrabay_eject 2010-01-07 22:54:55.000000000 +0100
		+	@@ -10,6 +10,11 @@
		+
		+	shopt -s nullglob
		+	export DISPLAY=:0.0 # required for notify-send
		+	+
		+	+DISP0_USER=`w \| grep ":0 " \| awk '{ print $1 }'`
		+	+DISP0_USER_HOME=`getent passwd ${DISP0_USER} \| cut -d':' -f6`
		+	+export XAUTHORITY=${DISP0_USER_HOME}/.Xauthority # also required for notify-send
		+	+
		+	ULTRABAY_SYSDIR=/sys$DEVPATH
		+
		+	# Find generic dock interface for UltraBay
		+	</pre>
		+
		+	i hope this is useful to other users as well --[[User:Roo\|Roo]] 22:19, 7 January 2010 (UTC)

← Older revision		Revision as of 09:07, 11 August 2009
Line 18:		Line 18:

	--[[User:Bombenbach\|Bombenbach]] 07:52, 11 August 2009 (UTC)		--[[User:Bombenbach\|Bombenbach]] 07:52, 11 August 2009 (UTC)
		+
		+	By default on most modern distributions based on a recent kernel with libata, the libata driver gets to cleanup the mess after the fact, if you pull out an ultrabay device without first
		+	* unmounting the filesystem
		+	* powering down the device
		+	You may be lucky, or you might end up with a hung system, corrupted filesystem, etc. Your choice.
		+
		+	There is work going on upstream to resolve this with DeviceKit-disks, but that depends on a patch to the Linux kernel, which has not made it in yet, since right now there is no correlation between the udev event and the actual Bay device, and since you can have multiple Bay devices (in fact your ThinkPad likely has 3) this correlation is needed, otherwise DeviceKit-disks would need to know about every single Laptop out there and the mapping between the Bay numbering and the actual devices (something the kernel can easily just provide instead).
		+
		+	Here is a Feature request against DeviceKit-disks that provides some status information
		+	http://bugs.freedesktop.org/show_bug.cgi?id=23110
		+
		+	--[[User:Tonko\|Tonko]] 09:07, 11 August 2009 (UTC)

	=HAL script=		=HAL script=

← Older revision		Revision as of 07:52, 11 August 2009
Line 13:		Line 13:

	--[[User:Tonko\|Tonko]] 09:31, 10 August 2009 (UTC)		--[[User:Tonko\|Tonko]] 09:31, 10 August 2009 (UTC)
		+
		+	Well, actually I couldn't imagine that someone would like to pull out an Ultrabay device with mounted file system.
		+	As far a usb flash drives are concerned, you won't do that either, right? My point was, that if the file system is '''properly unmounted''', I can simply pull my Ultrabay DVD-RW out without any freezes or crashes. And when I put it back again, the drive is automatically recognized by Ubuntu so that I can use it. I don't have an Ultrabay HDD, but I assume that if it works with DVD-RW it also should work with HDD. Correct me if I'm wrong. If the sole purpose of eject script is hotswapping Ultrabay devices '''with mounted filesystem''' then I still think that it would be nice to point it out in the article.
		+
		+	--[[User:Bombenbach\|Bombenbach]] 07:52, 11 August 2009 (UTC)

	=HAL script=		=HAL script=

← Older revision		Revision as of 09:31, 10 August 2009
Line 8:		Line 8:

	--[[User:Bombenbach\|Bombenbach]] 12:01, 9 August 2009 (UTC)		--[[User:Bombenbach\|Bombenbach]] 12:01, 9 August 2009 (UTC)
		+
		+	Really? Did you try to swap out a Ultrabay device with mounted filesystem? Good luck!
		+	The udev handler and eject script are there for a reason. By default no action is taken when you pull out a device, which can cause real problems if not handled properly.
		+
		+	--[[User:Tonko\|Tonko]] 09:31, 10 August 2009 (UTC)
		+
	=HAL script=		=HAL script=

← Older revision		Revision as of 12:01, 9 August 2009
Line 1:		Line 1:
		+	=Recent Ubuntu versions=
		+	I think, it must be mentioned that hotswapping Ultrabay devices using recent distros
		+	is by far not that hard as before. I'm using Ubuntu 9.04 on Z61m and in my case hotswapping
		+	Ultrabay DVD-RW works out-of-the-box without any scripts or whatsoever. The article
		+	however suggests that one always needs some ejecting scripts which might confuse certain
		+	users. So it would be nice to add something like: "If you are using Ubuntu .* everything should
		+	work out-of-the-box"
		+
		+	--[[User:Bombenbach\|Bombenbach]] 12:01, 9 August 2009 (UTC)
	=HAL script=		=HAL script=