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
prev 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).