lkml.org 
[lkml]   [2007]   [Nov]   [5]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: device struct bloat
From
Date
On Sun, 2007-11-04 at 19:58 -0800, Greg KH wrote:
> On Sun, Nov 04, 2007 at 09:29:18PM +0100, Peter Zijlstra wrote:
> >
> > On Sat, 2007-11-03 at 12:48 -0700, Stephen Hemminger wrote:
> > > The sizeof(struct device) is way too big, especially in the network device case.
> > > We want to support 1000's of device's and the change from class_device to
> > > net_device has caused needless bloat.
> > >
> > > sizeof(struct device) = 272
> > > sizeof(struct class_device) = 92
> > > * not the class_id in class_device could also be removed or changed to
> > > a ptr, since it is redundant for net_devices.
> >
> > The thing that surprised me most was that it contains a struct
> > semaphore, Greg, is that really needed?
>
> Yes, it serializes bind and unbind stuff for the device. There are
> comments about it in drivers/base/dd.c if you want to look into it.

Ah, that is not what I meant. My question was, why a semaphore, not a
mutex? Semaphores should not be used if at all possible - for one you
don't get lockdep checking your code.

/me converts the stuff over, and boots

Ah, that probably is the reason. This device tree is hard to annotate.
I'll try and think of something..

Anyway, Andy, could you make checkpatch whine on something like:
^\+[[::space:]]*struct semaphore



[unhandled content-type:application/pgp-signature]
\
 
 \ /
  Last update: 2007-11-05 11:49    [W:0.061 / U:0.980 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site