Re: [patch] kernel sysfs events layer

Robert Love
2004-09-05T02:58:15Z

On Sat, 2004-09-04 at 02:54 +0200, Greg KH wrote:

> So, we're back to the original issue. Why is this kernel event system
> different from the hotplug system? I would argue there isn't one,
> becides the transport, as you seem to want everything that we currently
> provide in the current kobject_hotplug() call.
>
> But transports are important, I agree.
>
> How about you just add the ability to send hotplug calls across netlink?
> Make it so the kobject_hotplug() function does both the exec() call, and
> a netlink call (based on a config option for those people who like to
> configure such stuff.)

This smells.

Look, I agree that unifying the two ideas and transports as much as
possible is the right way to proceed. But the fact is, as you said,
transports _are_ important. And simply always sending out a hotplug
event _and_ a netlink event is silly and superfluous. We need to make
up our minds.

I don't think anyone argues that netlink makes sense for these low
priority asynchronous events.

I'd prefer to integrate the two approaches as much as possible, but keep
the two transports separate. Use hotplug for hotplug events as we do
now and use kevent, which is over netlink, for the new events we want to
add.

Maybe always do the kevent from the hotplug, but definitely do not do
the hotplug from all kevents. It is redundant and extra overhead.

Doing both simultaneous begs the question of why have both. Picking the
right tool for the job is, well, the right tool for the job.

Robert Love