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 woken up

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/devctl in

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_wait call

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 receive

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/devctl in

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/devctl in

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 multiple

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, With the

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 multiple

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 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. read