linux-hotplug.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Robert Milasan <rmilasan@suse.cz>
To: linux-hotplug@vger.kernel.org
Subject: Re: Re-enable by-path links for ata device
Date: Fri, 03 Aug 2012 09:31:45 +0000	[thread overview]
Message-ID: <20120803113145.4546c5af@coolcat.suse.cz> (raw)
In-Reply-To: <20120802203126.71a8f2af@coolcat.suse.cz>

On Fri, 3 Aug 2012 10:44:11 +0200
Kay Sievers <kay@vrfy.org> wrote:

> On Thu, Aug 2, 2012 at 8:31 PM, Robert Milasan <rmilasan@suse.cz>
> wrote:
> > Hello, just created a patch for openSUSE to re-enable by-path links
> > for ata devices. The patch is based on handle_scsi_default
> > function, but instead of using "host{NUMBER}", but "ata{NUMBER}"
> > and it works almost the same as for scsi devices.
> >
> > The patch is not the best or the cleanest, but it works.
> 
> We can not play these games for any new stuff, like SCSI path_id is
> doing here, it's a legacy from a time we had we no good idea how
> things should work. What SCSI path_id is doing here is "I hope it
> works, let's try ..." software, which is a model that we don't support
> anymore. :)
> 
> We need the ATA transport class to export the local port number of the
> host adapter, and use that.
> 
> Rebasing in userspace by making assumptions about a global in-kernel
> instance counter is racy, unreliable and can only work if all things
> would always happen in strict order, and they don't. Manual driver
> unbind/bind, parallel hotplug gets into the way of this too simple
> logic.
> 
> The proper fix is to teach the kernel to export the numbers directly,
> or get the numbers in some way, but not by doing readdir() and
> starting to count and make a guess.
> 
> Hope you understand that we can not do this for any new stuff, it just
> does not meet the requirements we have today.
> 
> Thanks,
> Kay

It's fine with me, just wanted to make sure people know, maybe someone
needs it or wants it.

Thanks,
Robert

      parent reply	other threads:[~2012-08-03  9:31 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-08-02 18:31 Re-enable by-path links for ata device Robert Milasan
2012-08-03  8:44 ` Kay Sievers
2012-08-03  9:31 ` Robert Milasan [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=20120803113145.4546c5af@coolcat.suse.cz \
    --to=rmilasan@suse.cz \
    --cc=linux-hotplug@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).