linux-ide.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Holger Macht <hmacht@suse.de>
To: Tejun Heo <htejun@gmail.com>
Cc: Kristen Carlson Accardi <kristen.c.accardi@intel.com>,
	Henrique de Moraes Holschuh <hmh@hmh.eng.br>,
	Len Brown <lenb@kernel.org>,
	linux-acpi@vger.kernel.org, ibm-acpi-devel@lists.sourceforge.net,
	linux-ide@vger.kernel.org
Subject: Re: CONFIG_IBM_BAY
Date: Sun, 18 Mar 2007 09:27:51 +0100	[thread overview]
Message-ID: <20070318082751.GA28591@homac2> (raw)
In-Reply-To: <45FCD062.3050001@gmail.com>

On Sun 18. Mar - 14:38:42, Tejun Heo wrote:
> Hello,
> 
> Kristen Carlson Accardi wrote:
> >> 1. It will handle all device types (ATAPI, PATA, SATA, batteries);
> >>
> >> 2. It will do the right thing on plug and unplug.  This means telling the
> >> rest of the kernel to disable the device in the bay, for example.  Right now
> >> we shutdown one end of the PATA/SATA link on ThinkPads eletrically, and
> >> leave libata to scream blood murder until it disables its end due to too
> >> many retries, for example;
> 
> :-)
> 
> > Personally - I think tighter integration to libata here would be beneficial.
> > Once libata acpi support is straightened out, if we can store acpi handles
> > associated with libata devices, we can perhaps have a mechanism of obtaining
> > ata_device structs so that we can have a nice way of telling libata to 
> > delete devices etc.  I am hoping libata-acpi will be straightened out for
> > 2.6.22.
> 
> I dunno what the thread is really about but can't this be dealt within
> acpid?  Finding out the correct scsi host node can be tricky but I think
> it can be done reliably by jumping through some sysfs whoops.  Once
> you're there, telling libata to kill or probe is really easy.

That's actually what I've created dockutils [1] for. What it basically
does is 'echo 1 > /sys/devices/.../delete' on undock, and 'echo "- - -" >
/sys/class/scsi_host/*/scan' on dock to get the bay device back to life on
those ThinkPads where it is needed. Afterwards it does the corresponding
dock/undock request on ibm_acpi. And this works reliably good what I can
see from the feedback I already got. But for this to work, userspace would
actually need an event on docking and preferably would need to do the ACPI
undock itself. Both things are not possible with the generic dock driver
currently. And I don't think it matters much if libata's hotplug
capabilities are improved in this case, userspace just needs some time to
interact with the user in case there is more to do like unmounting
filesystems inside the bay etc...

I don't prefer any solution, whether doing it inside the kernel, or doing
it in userspace. What would be good would be to know what's the 'right'
way to go, or at least what both kernel people and userspace people can
agree on so that we can find a solution across distributions, whatever.
I'm currently just looking how to integrate the generic dock and bay
driver into the openSUSE distribution, and this seems to be quite hard,
especially because of the above mentioned already working solution ;-)

Just my 2 cents,
     Holger

[1] http://en.opensuse.org/Dockutils

  reply	other threads:[~2007-03-18  8:27 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <11736588493609-git-send-email-hmh@hmh.eng.br>
     [not found] ` <20070315031015.GA6292@khazad-dum.debian.net>
     [not found]   ` <20070315031251.GB6292@khazad-dum.debian.net>
     [not found]     ` <200703150206.05811.lenb@kernel.org>
     [not found]       ` <20070315133900.GC26862@khazad-dum.debian.net>
     [not found]         ` <20070315080644.c73db24a.kristen.c.accardi@intel.com>
     [not found]           ` <20070315175505.GB6596@khazad-dum.debian.net>
     [not found]             ` <20070315110156.09e80b99.kristen.c.accardi@intel.com>
     [not found]               ` <20070315193123.GB14394@khazad-dum.debian.net>
2007-03-15 19:50                 ` CONFIG_IBM_BAY Kristen Carlson Accardi
2007-03-18  5:38                   ` CONFIG_IBM_BAY Tejun Heo
2007-03-18  8:27                     ` Holger Macht [this message]
2007-03-18 18:36                       ` CONFIG_IBM_BAY Henrique de Moraes Holschuh
2007-03-18 18:55                         ` CONFIG_IBM_BAY Holger Macht
2007-03-18 19:00                           ` CONFIG_IBM_BAY Henrique de Moraes Holschuh
2007-03-19 18:04                           ` CONFIG_IBM_BAY Kristen Carlson Accardi
2007-03-20 15:53                             ` CONFIG_IBM_BAY Holger Macht
2007-03-20 16:19                               ` CONFIG_IBM_BAY Kristen Carlson Accardi
     [not found]                                 ` <20070320091932.f61bbdcc.kristen.c.accardi-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2007-03-20 16:34                                   ` CONFIG_IBM_BAY Holger Macht
2007-03-19 18:17                         ` CONFIG_IBM_BAY Kristen Carlson Accardi
2007-03-19 13:22                       ` CONFIG_IBM_BAY Matthew Garrett
     [not found]                         ` <20070319132243.GA2869-1xO5oi07KQx4cg9Nei1l7Q@public.gmane.org>
2007-03-19 13:48                           ` CONFIG_IBM_BAY Holger Macht
2007-03-19 14:04                             ` CONFIG_IBM_BAY Matthew Garrett
2007-03-19 14:37                               ` CONFIG_IBM_BAY Holger Macht
     [not found]                                 ` <20070319143746.GC4885-tB0uoaAy/Z1bpigZmTR7Iw@public.gmane.org>
2007-03-19 18:11                                   ` CONFIG_IBM_BAY Kristen Carlson Accardi
2007-03-20 16:07                                     ` CONFIG_IBM_BAY Holger Macht
     [not found]                                       ` <20070320160707.GB6079-tB0uoaAy/Z1bpigZmTR7Iw@public.gmane.org>
2007-03-20 16:16                                         ` CONFIG_IBM_BAY Holger Macht
     [not found]                               ` <20070319140403.GA3697-1xO5oi07KQx4cg9Nei1l7Q@public.gmane.org>
2007-03-19 15:36                                 ` CONFIG_IBM_BAY Henrique de Moraes Holschuh
2007-03-19 15:47                                   ` CONFIG_IBM_BAY Holger Macht
     [not found]                                   ` <20070319153659.GB15942-ZGHd14iZgfaRjzvQDGKj+xxZW9W5cXbT@public.gmane.org>
2007-03-19 16:41                                     ` CONFIG_IBM_BAY Shem Multinymous
     [not found]                                       ` <41840b750703190941p77fc4a64k49361b678dd6aa90-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2007-03-19 23:04                                         ` CONFIG_IBM_BAY Henrique de Moraes Holschuh
2007-03-20 14:18                                           ` [ibm-acpi-devel] CONFIG_IBM_BAY Shem Multinymous
2007-03-20 15:12                                             ` Henrique de Moraes Holschuh
2007-03-19 18:00                         ` CONFIG_IBM_BAY Kristen Carlson Accardi
     [not found]                     ` <45FCD062.3050001-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2007-03-18 18:26                       ` CONFIG_IBM_BAY Henrique de Moraes Holschuh

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=20070318082751.GA28591@homac2 \
    --to=hmacht@suse.de \
    --cc=hmh@hmh.eng.br \
    --cc=htejun@gmail.com \
    --cc=ibm-acpi-devel@lists.sourceforge.net \
    --cc=kristen.c.accardi@intel.com \
    --cc=lenb@kernel.org \
    --cc=linux-acpi@vger.kernel.org \
    --cc=linux-ide@vger.kernel.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;
as well as URLs for NNTP newsgroup(s).