From: Holger Macht <hmacht@suse.de>
To: Kristen Carlson Accardi <kristen.c.accardi@intel.com>
Cc: Matthew Garrett <mjg59@srcf.ucam.org>,
linux-acpi@vger.kernel.org, fseidel@suse.de
Subject: Re: Comments about commit 'PCI: docking station: remove dock uevents'?
Date: Mon, 13 Nov 2006 21:19:39 +0100 [thread overview]
Message-ID: <20061113201939.GB6465@homac2> (raw)
In-Reply-To: <20061113115423.f84adb22.kristen.c.accardi@intel.com>
On Mon 13. Nov - 11:54:23, Kristen Carlson Accardi wrote:
> On Mon, 13 Nov 2006 12:14:37 +0000
> Matthew Garrett <mjg59@srcf.ucam.org> wrote:
>
> > On Sun, Nov 12, 2006 at 03:07:37PM +0100, Holger Macht wrote:
> >
> > > I'm currently looking at docking station support for various
> > > laptops/vendors and what can be done to get them working. At the moment,
> > > userspace has to do some quirks to get support for them. For instance,
> > > when docking an X60 into a docking station where a SATA DVD drive is
> > > located, userland has to rescan the scsi host for the drive to show up.
> >
> > Does the docking not generate a bus rescan event on the SATA bus? If it
> > does, it's just a special case of ACPI drive hotplugging. You can't do
> > it reliably from userspace, but I think Kirsten has been looking at
> > doing it properly in the kernel.
>
> When you use the "bay" driver in addition to the "dock" driver (bay is
> in -mm right now) you would get a notification on dock/undock in the driver
> which would then cause the bay driver to notify user space about the
> drive event. I've not found any other devices that require user space
> notification.
Ok, thanks for the pointer, I'll have a look at it.
Regards,
Holger
>
> >
> > > Additionally, currently the dock driver automatically executes the _DCK
> > > method when it detects a dock/undock event. Wouldn't it be good to have
> > > common sysfs interface for docking so that userland can trigger the
> > > dock/undock? This way userland would also be able to do 'something before'
> > > docking/undocking, such as unregistering a device, showing a poput that it
> > > is now save to physically undock, etc.
> >
> > When the dock contains a drive, I think there's possibly an argument in
> > favour of this. Otherwise, the hardware should be able to deal with it -
> > every ACPI-based dock I've played with has had some indication to inform
> > the user that the device has disconnected.
>
> I think there is an arguement in favor of this, but there are more arguments
> in favor of not waiting for user space before executing the acpi _DCK to
> undock - mainly because not all dock stations actually lock the laptop
> in the dock, and so the user is going to just do it anyway. by undocking
> as soon as we get the hardware notification, we can at least clean up
> after this activity. Also, most people who have commented on my bay driver
> (which does assume user space handles the actual eject) have complained
> about the fact that I wait for user space. they general feedback I've gotten
> is that these activities should "just happen" and not require user intervention.
>
>
> >
> -
> To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
prev parent reply other threads:[~2006-11-13 20:20 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
2006-11-13 20:18 ` Kristen Carlson Accardi
2006-11-13 19:54 ` Kristen Carlson Accardi
2006-11-13 20:19 ` Holger Macht [this message]
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=20061113201939.GB6465@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