From: Holger Macht <hmacht@suse.de>
To: tom@dbservice.com
Cc: linux-kernel@vger.kernel.org, Tejun Heo <htejun@gmail.com>,
linux-ide@vger.kernel.org, Jeff Garzik <jeff@garzik.org>,
linux-acpi@vger.kernel.org, Matthew Garrett <mjg59@srcf.ucam.org>,
akpm@linux-foundation.org
Subject: Re: [PATCH] libata: Handle bay devices in dock stations
Date: Sun, 1 Jun 2008 18:06:31 +0200 [thread overview]
Message-ID: <20080601160631.GB4599@homac> (raw)
In-Reply-To: <20080530130739.25bz9mdjwwo080cw@dbservice.com>
On Fri 30. May - 13:07:39, tom@dbservice.com wrote:
> Quoting Holger Macht <hmacht@suse.de>:
>
>> * Differentiate between bay devices in dock stations and others:
>>
>> - When an ACPI_NOTIFY_EJECT_REQUEST appears, just signal uevent to
>> userspace (that is when the optional eject button on a bay device is
>> pressed/pulled) giving the possibility to unmount file systems and to
>> clean up. Also, only send uevent in case we get an EJECT_REQUEST
>> without doing anything else. In other cases, you'll get an add/remove
>> event because libata attaches/detaches the device.
>>
>> - In case of a dock event, which in turn signals an
>> ACPI_NOTIFY_EJECT_REQUEST, immediately detach the device, because it
>> may already have been gone
>>
>> * In case of an ACPI_NOTIFY_DEVICE/BUS_CHECK, evaluate _STA to check if
>> the device has been plugged or unplugged. If plugged, hotplug it, if
>> unplugged, just signal event to userspace
>> (initial patch by Matthew Garrett <mjg59@srcf.ucam.org>)
>>
>> * Call ACPI _EJ0 for detached devices
>
> Will this patch or the other bay/dock related patches you send in the past
> days allow me to undock my laptop and still be able to suspend/resume
> without locking the laptop up? And without having to run any userspace
> scripts?
>
> My dock is not a simple port replicator, it has an USB hub and an ATA bay
> in it. My tests showed that I need to 'echo 1 > /sys/.../scsi/.../eject' or
> something like that before I can take the laptop out of the docking
> station. If I don't do that and try to access the cdrom in the bay (or even
> rescaning the scsi bus) after I have taken the laptop out of the dock it
> results in a hard lockup. That in itself would not be a problem, because
> it's just a simple command that I can do in an acpid script. But far worse
> is that even if I do that, the computer locks up when I resume from
> suspend. I run 2.6.24.7 which locks up every time I resume. In recent
> versions from git it has somehow improved, there are situations where it
> doesn't lock up, but there are still a few left where it does (I don't
> remember the exact actions/commands I have to take).
Which kernel versions did you already try? But yes, those patches will
most likely help. But not sure about the resume issues...
Regards,
Holger
next prev parent reply other threads:[~2008-06-01 16:03 UTC|newest]
Thread overview: 32+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-05-28 14:38 [PATCH] libata: Handle bay devices in dock stations Holger Macht
2008-05-28 14:39 ` Tejun Heo
2008-05-29 3:02 ` Andrew Morton
2008-05-29 3:08 ` Tejun Heo
2008-05-29 13:22 ` Matthew Garrett
2008-05-29 13:33 ` Holger Macht
2008-05-29 13:32 ` Matthew Garrett
2008-05-29 13:39 ` Holger Macht
2008-05-29 13:40 ` Matthew Garrett
2008-05-29 13:44 ` Tejun Heo
2008-05-29 14:02 ` Matthew Garrett
2008-05-29 14:14 ` Holger Macht
2008-05-29 14:35 ` Matthew Garrett
2008-05-29 14:37 ` Tejun Heo
2008-05-29 14:49 ` Matthew Garrett
2008-05-29 16:32 ` Holger Macht
2008-05-29 16:40 ` Tejun Heo
2008-06-01 16:05 ` Holger Macht
2008-05-29 16:46 ` Holger Macht
2008-05-29 17:51 ` Henrique de Moraes Holschuh
2008-05-30 11:07 ` tom
2008-06-01 16:06 ` Holger Macht [this message]
2008-06-03 18:07 ` Jeff Garzik
2008-06-03 18:13 ` Matthew Garrett
2008-06-03 18:23 ` Holger Macht
2008-06-09 1:44 ` Tejun Heo
2008-06-09 1:48 ` Matthew Garrett
2008-06-09 4:56 ` Tejun Heo
2008-06-03 18:27 ` Holger Macht
2008-06-03 18:29 ` Matthew Garrett
2008-06-03 18:54 ` Jeff Garzik
2008-06-04 10:29 ` Jeff Garzik
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=20080601160631.GB4599@homac \
--to=hmacht@suse.de \
--cc=akpm@linux-foundation.org \
--cc=htejun@gmail.com \
--cc=jeff@garzik.org \
--cc=linux-acpi@vger.kernel.org \
--cc=linux-ide@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mjg59@srcf.ucam.org \
--cc=tom@dbservice.com \
/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;
as well as URLs for NNTP newsgroup(s).