public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: sullivan <sullivan@austin.ibm.com>
To: Patrick Mochel <mochel@osdl.org>
Cc: Martin Dalecki <dalecki@evision-ventures.com>,
	Linus Torvalds <torvalds@transmeta.com>,
	Kurt Garloff <garloff@suse.de>,
	Linux kernel list <linux-kernel@vger.kernel.org>,
	Linux SCSI list <linux-scsi@vger.kernel.org>
Subject: Re: [PATCH] /proc/scsi/map
Date: Fri, 21 Jun 2002 09:29:43 -0500	[thread overview]
Message-ID: <20020621092943.D1243@austin.ibm.com> (raw)
In-Reply-To: <Pine.LNX.4.33.0206201230190.654-100000@geena.pdx.osdl.net>; from mochel@osdl.org on Thu, Jun 20, 2002 at 01:12:08PM -0700

On Thu, Jun 20, 2002 at 01:12:08PM -0700, Patrick Mochel wrote:
<snip>
> 
> Sure. Once device class supports materializes, classes will register and
> can be assigned a dynamic major number even (if they don't already have
> one). As devices (and partitions) are discovered, we can assign minor
> numbers (dynamically!), and call /sbin/hotplug to notify userspace of the
> discovery. It can use that information to create device nodes based on 
> user-defined policy. 
> 
<snip>

The driverfs patch for SCSI that was recently posted was the kernel portion of 
a device naming project that is intended to support all devices, at least the 
ones that implement to driverfs in a standard way. There are three items that
IMHO should be considered as part of the standard set that driverfs requires:

1. device type - It appears that Pat is heading down this path with the class
	type support so maybe this is a no brainer. Currently the scsi
	driverfs provides a "type" file to contain this info. The current
	strings used are taken from the scsi_device_types[] but should be
	replaced with the system wide device types that driverfs will provide.

2. uid - Since topology and discovery order of hardware can change, the
	driverfs path names to a device are also subject to change. To
	easily identify a device I think it's important that the driverfs
	bus implementations be responsible for create a unique identifier.

	Since each bus and the devices attached to it will have varying
	capabilities for identifying themselves the contents for this file
	should probably be a variable length string.

	Even for older devices that can't do a great job of providing info to
	uniquely identify themselves, the driverfs tree provides the nice
	topological context to fall back upon that allows at least as
	good of a job to be done as we do today.

	The scsi patch currently creates uid info from the INQUIRY evpd pages
	and makes it available in the name file. I would prefer to see a
	new standard uid file and let the name file contain a descriptive 
	(non-unique) name.

3. kdev - To create/manage/interface with the device node we need to know the
 	kdev.

Because of coldplugging this information should be available in each driverfs
device directory. Also, adding the driverfs path name on /sbin/hotplug
events and allowing the consumer to retrieve the info from the filesystem might
help simplify some of these implementations too.

The devnaming utility that is based on this strategy is available at
http://www-124.ibm.com/devreg/ 

I'd welcome any thoughts or suggestions.	

- Mike Sullivan

  parent reply	other threads:[~2002-06-21 13:44 UTC|newest]

Thread overview: 65+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-06-20  0:44 [PATCH] /proc/scsi/map Kurt Garloff
2002-06-20  5:03 ` Linus Torvalds
2002-06-20  7:09   ` Martin Schwenke
2002-06-20 15:13     ` Linus Torvalds
2002-06-20 15:36       ` Dave Jones
2002-06-20 17:01         ` Linus Torvalds
2002-06-20 16:55       ` Andries Brouwer
2002-06-20 17:52         ` Patrick Mansfield
2002-06-20 18:36           ` Linus Torvalds
2002-06-20 18:52             ` James Bottomley
2002-06-20 19:15               ` Linus Torvalds
2002-06-20 16:28                 ` Benjamin Herrenschmidt
2002-06-21  0:46                   ` Linus Torvalds
2002-06-20 16:49                     ` Benjamin Herrenschmidt
2002-06-20 20:06                 ` Oliver Xymoron
2002-06-22 18:27             ` Pavel Machek
2002-06-20 18:11         ` Linus Torvalds
2002-06-20 22:59           ` Martin Schwenke
2002-06-20 23:13             ` Linus Torvalds
2002-06-22 18:25               ` Pavel Machek
2002-06-26 16:03           ` Ihno Krumreich
2002-07-01 17:33             ` Patrick Mochel
2002-06-20 19:55         ` Greg KH
2002-06-20 19:18       ` Patrick Mochel
2002-06-21  6:28       ` Mike Touloumtzis
2002-06-20 11:25   ` Kurt Garloff
2002-06-20 15:34     ` Linus Torvalds
2002-06-20 16:30       ` Martin Dalecki
2002-06-20 16:58         ` James Bottomley
2002-06-20 18:27           ` Linus Torvalds
2002-06-20 20:55             ` Martin Dalecki
2002-06-20 21:04               ` Linus Torvalds
2002-06-20 21:36                 ` Martin Dalecki
2002-06-20 20:12         ` Patrick Mochel
2002-06-20 22:29           ` Martin Dalecki
2002-06-22 18:42             ` Pavel Machek
2002-06-21 14:29           ` sullivan [this message]
2002-06-21 16:17             ` Patrick Mochel
2002-06-21 21:33           ` Oliver Xymoron
2002-06-22  4:38             ` Nick Bellinger
2002-06-22 19:41               ` Douglas Gilbert
2002-06-22 19:11                 ` Nick Bellinger
2002-06-25 18:13                 ` Patrick Mochel
2002-06-25 16:05             ` Patrick Mochel
2002-06-25 16:57               ` Oliver Xymoron
2002-06-25 18:58                 ` Patrick Mochel
2002-07-03  1:01                   ` Pavel Machek
2002-06-20 18:32       ` Kurt Garloff
2002-06-20 18:53         ` Linus Torvalds
2002-06-21  9:07         ` Kurt Garloff
  -- strict thread matches above, loose matches on Subject: below --
2002-06-20 23:59 Grover, Andrew
2002-06-22  6:29 Adam J. Richter
2002-06-25 16:17 ` Patrick Mochel
2002-06-22 17:24 David Brownell
2002-06-22 17:48 ` Roman Zippel
2002-06-22 20:11   ` Douglas Gilbert
2002-06-22 20:57     ` Roman Zippel
2002-06-22 18:18 ` Nick Bellinger
2002-06-24  1:50   ` David Brownell
2002-06-25 16:46   ` Patrick Mochel
2002-06-25 16:33 ` Patrick Mochel
2002-06-25 17:49   ` David Brownell
2002-06-26 23:39     ` Nick Bellinger
2002-07-01 17:45       ` Patrick Mochel
2002-07-03  0:59     ` Pavel Machek

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=20020621092943.D1243@austin.ibm.com \
    --to=sullivan@austin.ibm.com \
    --cc=dalecki@evision-ventures.com \
    --cc=garloff@suse.de \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-scsi@vger.kernel.org \
    --cc=mochel@osdl.org \
    --cc=torvalds@transmeta.com \
    /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