From: Henrique de Moraes Holschuh <hmh-N3TV7GIv+o9fyO9Q7EP/yw@public.gmane.org>
To: Tejun Heo <htejun-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
Cc: linux-ide-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
linux-acpi-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
ibm-acpi-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org,
Kristen Carlson Accardi
<kristen.c.accardi-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>,
Len Brown <lenb-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
Subject: Re: CONFIG_IBM_BAY
Date: Sun, 18 Mar 2007 15:26:24 -0300 [thread overview]
Message-ID: <20070318182624.GA29422@khazad-dum.debian.net> (raw)
In-Reply-To: <45FCD062.3050001-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
On Sun, 18 Mar 2007, Tejun Heo wrote:
> 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;
>
> :-)
Yeah :) Try it with old ide, and the results are not... pretty. libata new
EH *rules*.
> > 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 could, yes.
But to me the kernel looks like the proper place for this one. Not to
mention that we *really* should know the ACPI namespace -> kernel device
mapping in kernel, instead of doing half-assed things in userspace to call
back into kernel land later... it may come in handy for other things (proper
ACPI/driver exclusion of access to devices, for one), not just libata
matters and dock/bay ejects.
It is not like it is a policy thing. The only possible correct path to
follow when doing an eject is to disable the devices and buses in the "it
will be powered off" side of the ejection hardware. And depending on
userspace for this is just (IMHO) ugly and error-prone. The kernel really
should know how to do it, and doing it is *easy* as long as you know the
mapping (which you should know for other reasons).
--
"One disk to rule them all, One disk to find them. One disk to bring
them all and in the darkness grind them. In the Land of Redmond
where the shadows lie." -- The Silicon Valley Tarot
Henrique Holschuh
-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
next prev parent reply other threads:[~2007-03-18 18:26 UTC|newest]
Thread overview: 60+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-03-12 0:20 [GIT PULL] ibm-acpi changes acpi-test/2.6.22 Henrique de Moraes Holschuh
2007-03-12 0:20 ` [PATCH 1/6] ACPI: ibm-acpi: kill trailing whitespace Henrique de Moraes Holschuh
[not found] ` <11736588493322-git-send-email-hmh-N3TV7GIv+o9fyO9Q7EP/yw@public.gmane.org>
2007-03-12 0:20 ` [PATCH 2/6] ACPI: ibm-acpi: rename some identifiers Henrique de Moraes Holschuh
[not found] ` <11736588491476-git-send-email-hmh-N3TV7GIv+o9fyO9Q7EP/yw@public.gmane.org>
2007-03-12 0:20 ` [PATCH 3/6] ACPI: ibm-acpi: add header file Henrique de Moraes Holschuh
[not found] ` <11736588493502-git-send-email-hmh-N3TV7GIv+o9fyO9Q7EP/yw@public.gmane.org>
2007-03-12 0:20 ` [PATCH 4/6] ACPI: ibm-acpi: organize code Henrique de Moraes Holschuh
2007-03-12 0:20 ` [PATCH 5/6] ACPI: ibm-acpi: update copyright notice Henrique de Moraes Holschuh
2007-03-12 0:20 ` [PATCH 6/6] ACPI: ibm-acpi: update documentation Henrique de Moraes Holschuh
2007-03-12 0:54 ` [GIT PULL] ibm-acpi changes acpi-test/2.6.22 Len Brown
2007-03-12 1:27 ` Henrique de Moraes Holschuh
2007-03-15 3:10 ` [GIT PULL] ibm-acpi move to drivers/misc Henrique de Moraes Holschuh
2007-03-15 3:12 ` [PATCH] ACPI: ibm-acpi: move driver to drivers/misc hierarchy Henrique de Moraes Holschuh
2007-03-15 6:06 ` CONFIG_IBM_BAY Len Brown
2007-03-15 13:39 ` CONFIG_IBM_BAY Henrique de Moraes Holschuh
2007-03-15 15:06 ` CONFIG_IBM_BAY Kristen Carlson Accardi
2007-03-15 17:55 ` CONFIG_IBM_BAY Henrique de Moraes Holschuh
2007-03-15 18:01 ` CONFIG_IBM_BAY Kristen Carlson Accardi
[not found] ` <20070315110156.09e80b99.kristen.c.accardi-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2007-03-15 19:31 ` CONFIG_IBM_BAY Henrique de Moraes Holschuh
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 ` CONFIG_IBM_BAY Holger Macht
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-20 0:07 ` CONFIG_IBM_BAY Henrique de Moraes Holschuh
[not found] ` <20070320000706.GE5932-ZGHd14iZgfaRjzvQDGKj+xxZW9W5cXbT@public.gmane.org>
2007-03-20 16:14 ` 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-19 23:11 ` [ibm-acpi-devel] CONFIG_IBM_BAY Henrique de Moraes Holschuh
2007-03-19 23:56 ` Kristen Carlson Accardi
2007-03-20 0:12 ` Henrique de Moraes Holschuh
2007-03-20 16:11 ` 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 ` Henrique de Moraes Holschuh [this message]
2007-03-15 20:31 ` [ibm-acpi-devel] CONFIG_IBM_BAY Shem Multinymous
2007-03-15 20:49 ` Henrique de Moraes Holschuh
2007-03-19 12:04 ` Theodore Tso
2007-03-19 23:24 ` Henrique de Moraes Holschuh
2007-03-20 14:57 ` Theodore Tso
2007-03-20 15:14 ` Henrique de Moraes Holschuh
2007-03-20 18:45 ` Theodore Tso
2007-03-20 20:25 ` Shem Multinymous
[not found] ` <20070320184541.GB29493-AKGzg7BKzIDYtjvyW6yDsg@public.gmane.org>
2007-03-20 20:44 ` CONFIG_IBM_BAY Henrique de Moraes Holschuh
2007-03-15 18:22 ` CONFIG_IBM_BAY Kristen Carlson Accardi
2007-03-15 19:33 ` 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=20070318182624.GA29422@khazad-dum.debian.net \
--to=hmh-n3tv7giv+o9fyo9q7ep/yw@public.gmane.org \
--cc=htejun-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
--cc=ibm-acpi-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org \
--cc=kristen.c.accardi-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org \
--cc=lenb-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \
--cc=linux-acpi-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=linux-ide-u79uwXL29TY76Z2rM5mHXA@public.gmane.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