All of lore.kernel.org
 help / color / mirror / Atom feed
From: David Brownell <david-b@pacbell.net>
To: linux-hotplug@vger.kernel.org
Subject: Re: /sbin/hotplug invocation for USB devices in 2.5.40
Date: Tue, 08 Oct 2002 18:14:22 +0000	[thread overview]
Message-ID: <marc-linux-hotplug-103410085706635@msgid-missing> (raw)
In-Reply-To: <marc-linux-hotplug-103406277101783@msgid-missing>

Patrick Mochel wrote:
>>>Now I know this overloads the existing DEVICE usage for USB devices, but
>>>I think we can determine what do properly (basically just test for
>>>DEVFS, and if it's not set, then we are looking at the driverfs entry
>>>for the device.)
>>
>>I'd rather see these changes:
>>
>>   - This new parameter to every (!!) hotplug event gets renamed to
>>     KFS_DEVICE (assuming that's the new driverfs name).
> 
> 
> What about DEVPATH, since that's what it is, right?

Well, it's different from usb_device->devpath ... devpath is just
one component of the KFS path.  One that's repeated because of
that requirement that names assigned by a bus be unique across
the whole system, making it easier to automate that symlinking. :)

It's not the only filesystem path used to access the device.
Would using KFS_DEVICE really be a problem?



>>By the way, what's the cost of a driverfs attribute on a 'struct device'?
>>Each USB device now has over a dozen of them.
> 
> 
> 32 bytes/attribute + size of the function. But, attributes are reused 
> across devices, so the total data cost is 32 x ~16 = 512. 

"Reused" meaning shared attributes like "name" and "power", I guess.

USB devices have at least one interface, and each will have six attributes
(including name+power).  So 32 x ~24 = 768 bytes minimum, on top of other
costs. These start to add up.

- Dave






-------------------------------------------------------
This sf.net email is sponsored by:ThinkGeek
Welcome to geek heaven.
http://thinkgeek.com/sf
_______________________________________________
Linux-hotplug-devel mailing list  http://linux-hotplug.sourceforge.net
Linux-hotplug-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-hotplug-devel

  parent reply	other threads:[~2002-10-08 18:14 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-10-08  7:32 /sbin/hotplug invocation for USB devices in 2.5.40 Greg KH
2002-10-08 17:02 ` David Brownell
2002-10-08 17:49 ` Patrick Mochel
2002-10-08 17:55 ` Oliver Neukum
2002-10-08 18:14 ` David Brownell [this message]
2002-10-08 18:18 ` Oliver Neukum
2002-10-08 18:18 ` Greg KH
2002-10-08 18:25 ` Greg KH
2002-10-08 21:00 ` David Brownell
2002-10-08 21:17 ` Greg KH
2002-10-08 21:18 ` David Brownell
2002-10-08 21:28 ` David Brownell
2002-10-08 21:39 ` Greg KH
2002-10-09 15:15 ` David Brownell
2002-10-10 20:35 ` Greg KH

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=marc-linux-hotplug-103410085706635@msgid-missing \
    --to=david-b@pacbell.net \
    --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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.