From: Holger Macht <hmacht@suse.de>
To: Matthew Garrett <mjg59@srcf.ucam.org>
Cc: linux-acpi@vger.kernel.org, kristen.c.accardi@intel.com, fseidel@suse.de
Subject: Re: Comments about commit 'PCI: docking station: remove dock uevents'?
Date: Mon, 13 Nov 2006 20:57:40 +0100 [thread overview]
Message-ID: <20061113195740.GA6465@homac2> (raw)
In-Reply-To: <20061113195025.GA23091@srcf.ucam.org>
On Mon 13. Nov - 19:50:25, Matthew Garrett wrote:
> On Mon, Nov 13, 2006 at 08:43:27PM +0100, Holger Macht wrote:
>
> > What I'm doing at the moment (for about a week ;-) is to echo "- - -" to
> > /sys/class/scsi_host/host*/scan for all unoccupied hosts on a dock
> > event. And this works pretty good. But it's of course only a workaround if
> > the scsi bus generates an event.
>
> That's just about workable for adding a device, but it loses badly for
> removing one.
On before physically removing the laptop I did an echo 1 >
/sys/.../device/delete . That's actually the reason why I need a undock
event in userspace for now.
> > With recent kernel I have to unregister from the drive from userspace to
> > prevent confusion about if the drive is still there or not.
>
> Right. And that doesn't help in the case where the user undocks without
> waiting for everything to stop flashing first. Of course, in that case
> it's probably reasonable to tell the user that they lose, but we can do
> better than leaving a random drive lying around and a partially wedged
> sata bus...
User presses lever on dock station or presses Fn+F9 on a thinkpad
generates a undock acpi event, then I'm unregistering the device, then I
trigger the undock from userspace and at this point the indicator on the
dock station turns green so the user knows that he can remove the laptop.
Regards,
Holger
next prev parent reply other threads:[~2006-11-13 19:58 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-11-12 14:07 Comments about commit 'PCI: docking station: remove dock uevents'? Holger Macht
2006-11-13 12:14 ` Matthew Garrett
2006-11-13 19:43 ` Holger Macht
2006-11-13 19:50 ` Matthew Garrett
2006-11-13 19:57 ` Holger Macht [this message]
2006-11-13 20:18 ` Kristen Carlson Accardi
2006-11-13 19:54 ` Kristen Carlson Accardi
2006-11-13 20:19 ` Holger Macht
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=20061113195740.GA6465@homac2 \
--to=hmacht@suse.de \
--cc=fseidel@suse.de \
--cc=kristen.c.accardi@intel.com \
--cc=linux-acpi@vger.kernel.org \
--cc=mjg59@srcf.ucam.org \
/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