public inbox for linux-acpi@vger.kernel.org
 help / color / mirror / Atom feed
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

      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