linux-ide.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
To: Andy Warner <andyw@pobox.com>
Cc: Jeff Garzik <jgarzik@pobox.com>,
	"linux-ide@vger.kernel.org" <linux-ide@vger.kernel.org>
Subject: Re: RFC: hotplug & libata.
Date: Fri, 15 Oct 2004 22:58:25 +0200	[thread overview]
Message-ID: <58cb370e04101513584f1bf314@mail.gmail.com> (raw)
In-Reply-To: <20041015151656.A26160@florence.linkmargin.com>

On Fri, 15 Oct 2004 15:16:56 -0500, Andy Warner <andyw@pobox.com> wrote:
> I've spent time over the last 10 days or so working
> through hotplug issues with libata and am coming round
> to the position that the current __init/probe mechanism
> may need some reorganisation for hotplug to work well.
> 
> I can currently (mostly) hotplug SATA drives that were
> present at __init/probe all day long; but things
> are less good for drives which were not present
> at __init/probe.
> 
> If it's not there at __init/probe time, the groundwork
> isn't in place to add devices later without some fairly
> nasty & repetitious code. I keep coming back to a model
> where the host adapter ports are initialised, then the
> initial drive scan & addition is effectively a hot-plug
> event (either automatic or forced, depending on hardware
> capabilities.)

Sounds fine.

> Unfortunately, I fear that any impending PATA/libata
> train-wreck may conflict with this, due to the behaviour
> of some (many) of the PATA chipsets out there (e.g.
> misbehaving seriously when no active devices are attached
> to the bus etc.)

misbehaving?

For the PATA I worry a bit about chipset tuning vs hotplug:
* single PCI config space byte holds settings for all devices etc.
* some hosts don't like when settings for the one port are
  changed while there is IO in flight for the other port

There should be some smart host wide locking for that.

Otherwise I see no major problems for PATA hotplug
(or rather "coldplug") support.

> Anyone have thoughts/advice to share on this concept? I think
> the re-org may be quite intrusive, and I don't want to waste
> cycles on something that "would never work with PATA, because
> of blah, blah, blah.."

Without seeing actual code it is hard to say but I'm an optimist. :)

      reply	other threads:[~2004-10-15 20:58 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-10-15 20:16 RFC: hotplug & libata Andy Warner
2004-10-15 20:58 ` Bartlomiej Zolnierkiewicz [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=58cb370e04101513584f1bf314@mail.gmail.com \
    --to=bzolnier@gmail.com \
    --cc=andyw@pobox.com \
    --cc=jgarzik@pobox.com \
    --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).