lkml.org 
[lkml]   [2004]   [Nov]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 2.6.10-rc1 2/5] driver-model: bus_recan_devices() locking fix
 Hello, Dmitry.

On Mon, Nov 08, 2004 at 08:43:21PM -0500, Dmitry Torokhov wrote:
> On Monday 08 November 2004 08:27 pm, Tejun Heo wrote:
> > Are you suggesting splitting every bus implementation into two
> > separate modules? Otherwise, we will have to move kobj field of every
> > bus-specific device structure to the head (to use the common release
> > function). Either way, it can be done, but it just seems another
> > twist added to the already twisted refcounting. Moreoever, not only
> > the devices are problematic, refcounting in class devices and block
> > device hierarchy seems broken too.
> >
>
> It is already defacto implementation standard - every subsystem has a core
> modules (pci, usbcore, serio) plus modules (drivers) actually talking to
> hardware and registering devices (ports, whatever) with the corresponding
> core. It is just a matter of splitting cleanup functions into core and
> device-specific parts.

AFAIK, PCI isn't compiled as modules and USB has ohci/uhci/ehci stuff
and serio has ports stuff. But still, I cannot agree with you.

1. It's too twisted.
a. to prevent premature unload, we depend on
- attr -> kobj -> module_ref to bus core module. By doing
this, we're making a chain of two different types of
reference counts. IMHO, it's just a bit too subtle.
b. to enable bus module unload, we depend on
- device creation/detach operations are done in a separate
module than device release.

2. too restrictive.
a. any bus implementation must be implemented as two separate
modules just to enable peaceful module unloading
b. all other places which make use of kobject and any type of
generic attr must use only the release routine proper to
the core module. No callbacks, no data dereferences....
(for example, scsi_host class uses scsi_host_template
structure which contain class device specific callbacks
when releaseing scsi_host class device. This is
currently broken and won't be fixed by your method.)
3. not intutive. Maybe this is the most important.
a. it's natural to expect the release code has access to
codes & data of whatever it's releasing.
b. children bypassing dependency hierarchy and depending
directly on indirect ancestor is just difficult to
track, memorize and do correctly.
IMO, above a and b are the primary reasons for why kobject/attr
refcountings are currently done incorrectly in many places.

> > But all attrs will be deflated and it just seems the right thing to
> > do(tm).
> >
>
> I think there are much more kobjects than attrs.

Yes, there are. But many objects which contain kobjs keep separate
owner field anyway, which can be removed after kobj can handle module
refcounting, and even without that, I think 4 or 8 bytes addition to
struct kobject is worth it if we can get the refcounting straight.

Thanks.

--
tejun

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-03-22 14:07    [from the cache]
©2003-2011 Jasper Spaans