Re: [patch 6/13] signal/timer/event fds v8 - timerfd core ...

2007-03-31 Thread Davide Libenzi
On Fri, 30 Mar 2007, Andrew Morton wrote: > On Fri, 30 Mar 2007 17:47:28 -0700 (PDT) Davide Libenzi > wrote: > > > On Fri, 30 Mar 2007, Andrew Morton wrote: > > > > > > +struct timerfd_ctx { > > > > + struct hrtimer tmr; > > > > + ktime_t tintv; > > > > + spinlock_t lock; >

Re: [patch 6/13] signal/timer/event fds v8 - timerfd core ...

2007-03-31 Thread Davide Libenzi
On Fri, 30 Mar 2007, Andrew Morton wrote: On Fri, 30 Mar 2007 17:47:28 -0700 (PDT) Davide Libenzi davidel@xmailserver.org wrote: On Fri, 30 Mar 2007, Andrew Morton wrote: +struct timerfd_ctx { + struct hrtimer tmr; + ktime_t tintv; + spinlock_t lock;

Re: [patch 6/13] signal/timer/event fds v8 - timerfd core ...

2007-03-30 Thread Andrew Morton
On Fri, 30 Mar 2007 17:47:28 -0700 (PDT) Davide Libenzi wrote: > On Fri, 30 Mar 2007, Andrew Morton wrote: > > > > +struct timerfd_ctx { > > > + struct hrtimer tmr; > > > + ktime_t tintv; > > > + spinlock_t lock; > > > + wait_queue_head_t wqh; > > > + unsigned long ticks; > > > +}; > > > >

Re: [patch 6/13] signal/timer/event fds v8 - timerfd core ...

2007-03-30 Thread Davide Libenzi
On Fri, 30 Mar 2007, Andrew Morton wrote: > > +struct timerfd_ctx { > > + struct hrtimer tmr; > > + ktime_t tintv; > > + spinlock_t lock; > > + wait_queue_head_t wqh; > > + unsigned long ticks; > > +}; > > Did you consider using the (presently unused) lock inside wqh instead of >

Re: [patch 6/13] signal/timer/event fds v8 - timerfd core ...

2007-03-30 Thread Andrew Morton
On Tue, 20 Mar 2007 11:37:14 -0700 Davide Libenzi wrote: > This patch introduces a new system call for timers events delivered > though file descriptors. This allows timer event to be used with > standard POSIX poll(2), select(2) and read(2). As a consequence of > supporting the Linux f_op->poll

Re: [patch 6/13] signal/timer/event fds v8 - timerfd core ...

2007-03-30 Thread Andrew Morton
On Tue, 20 Mar 2007 11:37:14 -0700 Davide Libenzi davidel@xmailserver.org wrote: This patch introduces a new system call for timers events delivered though file descriptors. This allows timer event to be used with standard POSIX poll(2), select(2) and read(2). As a consequence of supporting

Re: [patch 6/13] signal/timer/event fds v8 - timerfd core ...

2007-03-30 Thread Davide Libenzi
On Fri, 30 Mar 2007, Andrew Morton wrote: +struct timerfd_ctx { + struct hrtimer tmr; + ktime_t tintv; + spinlock_t lock; + wait_queue_head_t wqh; + unsigned long ticks; +}; Did you consider using the (presently unused) lock inside wqh instead of adding a new one?

Re: [patch 6/13] signal/timer/event fds v8 - timerfd core ...

2007-03-30 Thread Andrew Morton
On Fri, 30 Mar 2007 17:47:28 -0700 (PDT) Davide Libenzi davidel@xmailserver.org wrote: On Fri, 30 Mar 2007, Andrew Morton wrote: +struct timerfd_ctx { + struct hrtimer tmr; + ktime_t tintv; + spinlock_t lock; + wait_queue_head_t wqh; + unsigned long ticks; +}; Did

[patch 6/13] signal/timer/event fds v8 - timerfd core ...

2007-03-20 Thread Davide Libenzi
This patch introduces a new system call for timers events delivered though file descriptors. This allows timer event to be used with standard POSIX poll(2), select(2) and read(2). As a consequence of supporting the Linux f_op->poll subsystem, they can be used with epoll(2) too. The system call is

[patch 6/13] signal/timer/event fds v8 - timerfd core ...

2007-03-20 Thread Davide Libenzi
This patch introduces a new system call for timers events delivered though file descriptors. This allows timer event to be used with standard POSIX poll(2), select(2) and read(2). As a consequence of supporting the Linux f_op-poll subsystem, they can be used with epoll(2) too. The system call is