Re: [patch] turning devctl into a "multiple openable" device

2011-12-02 Thread Kostik Belousov
On Fri, Dec 02, 2011 at 12:17:21AM +0100, Baptiste Daroussin wrote: > On Wed, Nov 30, 2011 at 05:20:17PM +0100, Olivier Houchard wrote: > > On Wed, Nov 30, 2011 at 06:04:50PM +0200, Kostik Belousov wrote: > > > > > I wonder why the waiting_threads stuff is needed at all. The cv could > > > > > be w

Re: [patch] turning devctl into a "multiple openable" device

2011-12-01 Thread Baptiste Daroussin
On Wed, Nov 30, 2011 at 05:20:17PM +0100, Olivier Houchard wrote: > On Wed, Nov 30, 2011 at 06:04:50PM +0200, Kostik Belousov wrote: > > > > I wonder why the waiting_threads stuff is needed at all. The cv could > > > > be woken up unconditionally everytime. What is the reason for the > > > > cv_wa

Re: [patch] turning devctl into a "multiple openable" device

2011-12-01 Thread John Baldwin
On Wednesday, November 30, 2011 11:41:25 am Hans Petter Selasky wrote: > On Wednesday 30 November 2011 13:43:20 Baptiste Daroussin wrote: > > Hi all, > > > > With the help of cognet, I wrote a patch to turn devctl into a multiple > > openable device, that mean that it will allow to open /dev/devct

Re: [patch] turning devctl into a "multiple openable" device

2011-11-30 Thread Olivier Houchard
On Wed, Nov 30, 2011 at 06:04:50PM +0200, Kostik Belousov wrote: > > > I wonder why the waiting_threads stuff is needed at all. The cv could > > > be woken up unconditionally everytime. What is the reason for the cv_wait > > > call in cdevpriv data destructor ? You cannot have a thread doing e.g. >

Re: [patch] turning devctl into a "multiple openable" device

2011-11-30 Thread Hans Petter Selasky
On Wednesday 30 November 2011 13:43:20 Baptiste Daroussin wrote: > Hi all, > > With the help of cognet, I wrote a patch to turn devctl into a multiple > openable device, that mean that it will allow to open /dev/devctl in > multiple programs, for example hald and everythings that want to receive >

Re: [patch] turning devctl into a "multiple openable" device

2011-11-30 Thread Olivier Houchard
On Wed, Nov 30, 2011 at 05:46:36PM +0200, Kostik Belousov wrote: > On Wed, Nov 30, 2011 at 10:05:11AM -0500, John Baldwin wrote: > > On Wednesday, November 30, 2011 7:43:20 am Baptiste Daroussin wrote: > > > Hi all, > > > > > > With the help of cognet, I wrote a patch to turn devctl into a multipl

Re: [patch] turning devctl into a "multiple openable" device

2011-11-30 Thread Kostik Belousov
On Wed, Nov 30, 2011 at 04:55:21PM +0100, Olivier Houchard wrote: > On Wed, Nov 30, 2011 at 05:46:36PM +0200, Kostik Belousov wrote: > > On Wed, Nov 30, 2011 at 10:05:11AM -0500, John Baldwin wrote: > > > On Wednesday, November 30, 2011 7:43:20 am Baptiste Daroussin wrote: > > > > Hi all, > > > >

Re: [patch] turning devctl into a "multiple openable" device

2011-11-30 Thread Baptiste Daroussin
On Wed, Nov 30, 2011 at 05:46:36PM +0200, Kostik Belousov wrote: > On Wed, Nov 30, 2011 at 10:05:11AM -0500, John Baldwin wrote: > > On Wednesday, November 30, 2011 7:43:20 am Baptiste Daroussin wrote: > > > Hi all, > > > > > > With the help of cognet, I wrote a patch to turn devctl into a multipl

Re: [patch] turning devctl into a "multiple openable" device

2011-11-30 Thread Kostik Belousov
On Wed, Nov 30, 2011 at 10:05:11AM -0500, John Baldwin wrote: > On Wednesday, November 30, 2011 7:43:20 am Baptiste Daroussin wrote: > > Hi all, > > > > With the help of cognet, I wrote a patch to turn devctl into a multiple > > openable > > device, that mean that it will allow to open /dev/devct

Re: [patch] turning devctl into a "multiple openable" device

2011-11-30 Thread Baptiste Daroussin
On Wed, Nov 30, 2011 at 10:05:11AM -0500, John Baldwin wrote: > On Wednesday, November 30, 2011 7:43:20 am Baptiste Daroussin wrote: > > Hi all, > > > > With the help of cognet, I wrote a patch to turn devctl into a multiple > > openable > > device, that mean that it will allow to open /dev/devct

Re: [patch] turning devctl into a "multiple openable" device

2011-11-30 Thread John Baldwin
On Wednesday, November 30, 2011 7:43:20 am Baptiste Daroussin wrote: > Hi all, > > With the help of cognet, I wrote a patch to turn devctl into a multiple > openable > device, that mean that it will allow to open /dev/devctl in multiple programs, > for example hald and everythings that want to re

[patch] turning devctl into a "multiple openable" device

2011-11-30 Thread Baptiste Daroussin
Hi all, With the help of cognet, I wrote a patch to turn devctl into a multiple openable device, that mean that it will allow to open /dev/devctl in multiple programs, for example hald and everythings that want to receive notification from the device won't need to depend on haveing devd running.