Re: [PATCH RFD] alternative kobject release wait mechanism

["Dmitry Torokhov" <dmitry.torokhov@gmail.com>]

Hi Tejun,

On 4/20/07, Tejun Heo <htejun@gmail.com> wrote:
> Hello, Dmitry.
>
> Dmitry Torokhov wrote:
> > On 4/19/07, Cornelia Huck <cornelia.huck@de.ibm.com> wrote:
> >> On Thu, 19 Apr 2007 09:13:43 -0400,
> >> "Dmitry Torokhov" <dmitry.torokhov@gmail.com> wrote:
> >>
> >> > Because they are managed by 2 different entities. the struct device
> >> > objects are managed by device core and driver-specific objects are
> >> > managed by their respective driver.
> >>
> >> Not sure if I understand you here. My view of this was always that the
> >> embedding object was kind of an extended device and that the relevant
> >> driver/subsystem managed it through the driver core infrastructure.
> >>
> >
> > I am not sure if I agree with this point of view. Driver (or
> > subsystem) provides an instance of struct device for the rest of the
> > system to iteract uniformly with (suspend/resume/tree
> > visualization/etc) i.e. struct device implement an interface for
> > subsystems. However most of the system use their own mechanisms to
> > manage their devices. They can rely on the driver core to a certain
> > degree but driver core is mostly a carries out helper functions, not
> > the meat.
>
> Many drivers (at least all the SCSI/IDE ones) consider struct device as
> the base class of the devices those drivers implement.  I don't think we
> can just consider those drivers to be wrong.

I am not saying they are wrong I am just saying that driver core is
not where most work is done. Every subsystem has its own locking rules
and lifetime rules and they are what is important. Whether subsystem
uses embedding or referencing of struct devices is implementation
detail.

What is the goal of driver core? I thought the main goal was to have
an uniform interface for device power management and presentation of
device tree to userspace. It has nothing to do with main purposes of
every individual subsystem - making some set of devices/services work.

-- 
Dmitry