Re: Delayed hotplug events

[Hannes Reinecke <hare@suse.de>]

Oliver Neukum wrote:
> Am Freitag, 18. Juni 2004 10:12 schrieb Hannes Reinecke:
> 
>>>Oliver Neukum wrote:
>>>
[ .. ]
>>>>If you are willing to use a model of a blocking counter you can confine
>>>>the changes to just the actual hotplug generation. The change is well
>>>>encapsuled. Drivers not caring about the block/unblock affair simply get
>>>>the old behavior.
>>>>It's less elegant than waiting for the specific files, but workable.
>>>>
>>>
>>>You mean add an additional sysfs attribute which serves as an checkpoint 
>>>(i.e. if this attribute exists, the initialisation is done)?
> 
> 
> No, I would consider that redundant. If you let events queue up until
> a counter reaches zero, you can guarantee that there's time to generate
> all files being refered to in the event. There's no need to export it.
> 
Hmm. I see. But isn't this approach conceptually identical to my patch? 
I mean, you're blocking events until a semaphore is set, whereas I'm 
dropping events during the critical section and generate them again. No 
big change.

And it doesn't change the main issue, which is that the behaviour of the 
hotplug subsystem is changed (i.e. the time when events are sent).
Adding just another attribute doesn't change it, as it's up to the 
drivers to generate sysfs attributes.

Cheers,

hannes
-- 
Dr. Hannes Reinecke			hare@suse.de
SuSE Linux AG				S390 & zSeries
Maxfeldstraße 5				+49 911 74053 688
90409 Nürnberg				http://www.suse.de


-------------------------------------------------------
This SF.Net email is sponsored by The 2004 JavaOne(SM) Conference
Learn from the experts at JavaOne(SM), Sun's Worldwide Java Developer
Conference, June 28 - July 1 at the Moscone Center in San Francisco, CA
REGISTER AND SAVE! http://java.sun.com/javaone/sf Priority Code NWMGYKND
_______________________________________________
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