From: Douglas Gilbert <dgilbert@interlog.com>
To: Greg KH <gregkh@suse.de>
Cc: Nao Nishijima <nao.nishijima.xt@hitachi.com>,
Tejun Heo <tj@kernel.org>, Jens Axboe <axboe@kernel.dk>,
"James E.J. Bottomley" <James.Bottomley@HansenPartnership.com>,
Kay Sievers <kay.sievers@vrfy.org>,
Alan Cox <alan@linux.intel.com>,
Al Viro <viro@zeniv.linux.org.uk>,
linux-kernel@vger.kernel.org, linux-scsi@vger.kernel.org
Subject: Re: [PATCH] block: Revert "[SCSI] genhd: add a new attribute "alias" in gendisk"
Date: Fri, 11 Nov 2011 14:09:29 -0500 [thread overview]
Message-ID: <4EBD72E9.4020805@interlog.com> (raw)
In-Reply-To: <20111111173633.GA24351@suse.de>
On 11-11-11 12:36 PM, Greg KH wrote:
> On Fri, Nov 11, 2011 at 11:50:06PM +0900, Nao Nishijima wrote:
>> Hi,
>>
>> Finally, I understood why this patch is not acceptable and would like to
>> solve the problem of the device name mismatch in *user space* using
>> udev. So, could you please give me your comment about it?
>>
>> I aim to unify the device name because the name shown in command output
>> is different from it that users use for command execution.
>>
>> So, I'd like to suggest that two new API should be added to udev.
>> - The API that returns any symlinks such as by-uuid, by id that users
>> selected before if a device name is given as arguments
>> If the commands directly access to proc filesystems, they cannot get
>> the symlinks. These commands should get the device name via this API.
>> - The API that replaces device name to symlink in text messages
>> The dmesg and syslog messages include the raw device names such as
>> sdX. They should be replaced with symlinks to unify the device names.
>> - udev support alias
>> Symlinks (by-uuid, by-id …) is too long, compared to before (e.g.
>> sdX). I guess most of users need more friendly and short names.
>>
>> Could you give me some advice or comments?
>> (I will post this idea udev mailing list with prototype code)
>
> Patches are always best to work off of, please post your changes to udev
> to the linux-hotplug list and we can take it from there.
... and getting examples of acceptable code and APIs
for that purpose might help the requester (and others)
produce those patches.
Doug Gilbert
next prev parent reply other threads:[~2011-11-11 19:09 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-11-09 16:25 [PATCH] block: Revert "[SCSI] genhd: add a new attribute "alias" in gendisk" Tejun Heo
2011-11-09 16:53 ` Greg KH
2011-11-09 17:19 ` Kay Sievers
2011-11-09 17:30 ` James Bottomley
2011-11-09 17:36 ` Greg KH
2011-11-09 17:38 ` Kay Sievers
2011-11-09 19:51 ` Karel Zak
2011-11-09 18:54 ` Jens Axboe
2011-11-10 21:21 ` Jeff Garzik
2011-11-11 14:50 ` Nao Nishijima
2011-11-11 17:36 ` Greg KH
2011-11-11 19:09 ` Douglas Gilbert [this message]
2011-11-24 12:21 ` Bernd Schubert
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=4EBD72E9.4020805@interlog.com \
--to=dgilbert@interlog.com \
--cc=James.Bottomley@HansenPartnership.com \
--cc=alan@linux.intel.com \
--cc=axboe@kernel.dk \
--cc=gregkh@suse.de \
--cc=kay.sievers@vrfy.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-scsi@vger.kernel.org \
--cc=nao.nishijima.xt@hitachi.com \
--cc=tj@kernel.org \
--cc=viro@zeniv.linux.org.uk \
/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