Re: [PATCH v5] posix timers: allocate timer id per process

2012-10-23 Thread Ingo Molnar
* Eric Dumazet wrote: > On Wed, 2012-10-24 at 00:33 +0200, Thomas Gleixner wrote: > > On Tue, 23 Oct 2012, Eric Dumazet wrote: > > > > > On Tue, 2012-10-23 at 23:47 +0200, Thomas Gleixner wrote: > > > > > > > Not so good to me. > > > > > > > > > Signed-off-by: Eric Dumazet > > > > > > > >

Re: [PATCH v5] posix timers: allocate timer id per process

2012-10-23 Thread Eric Dumazet
On Wed, 2012-10-24 at 00:33 +0200, Thomas Gleixner wrote: > On Tue, 23 Oct 2012, Eric Dumazet wrote: > > > On Tue, 2012-10-23 at 23:47 +0200, Thomas Gleixner wrote: > > > > > Not so good to me. > > > > > > > Signed-off-by: Eric Dumazet > > > > > > And that should be either an Acked-by or a Re

Re: [PATCH v5] posix timers: allocate timer id per process

2012-10-23 Thread Thomas Gleixner
On Tue, 23 Oct 2012, Eric Dumazet wrote: > On Tue, 2012-10-23 at 23:47 +0200, Thomas Gleixner wrote: > > > Not so good to me. > > > > > Signed-off-by: Eric Dumazet > > > > And that should be either an Acked-by or a Reviewed-by. You can't sign > > off on patches which have not been submitted o

Re: [PATCH v5] posix timers: allocate timer id per process

2012-10-23 Thread Eric Dumazet
On Tue, 2012-10-23 at 23:47 +0200, Thomas Gleixner wrote: > Not so good to me. > > > Signed-off-by: Eric Dumazet > > And that should be either an Acked-by or a Reviewed-by. You can't sign > off on patches which have not been submitted or transported by you. I actually gave some input, provide

Re: [PATCH v5] posix timers: allocate timer id per process

2012-10-23 Thread Thomas Gleixner
On Tue, 23 Oct 2012, Eric Dumazet wrote: > On Tue, 2012-10-23 at 11:40 +0400, Stanislav Kinsbursky wrote: > > This patch is required CRIU project (www.criu.org). > > To migrate processes with posix timers we have to make sure, that we can > > restore posix timer with proper id. > > Currently, this

Re: [PATCH v5] posix timers: allocate timer id per process

2012-10-23 Thread Thomas Gleixner
B1;2601;0cOn Tue, 23 Oct 2012, Stanislav Kinsbursky wrote: > Patch replaces global idr with global hash table for posix timers and > makes timer ids unique not globally, but per process. Next free timer id is > type of integer and stored on signal struct (posix_timer_id). If free timer id > reaches

Re: [PATCH v5] posix timers: allocate timer id per process

2012-10-23 Thread Eric Dumazet
On Tue, 2012-10-23 at 11:40 +0400, Stanislav Kinsbursky wrote: > This patch is required CRIU project (www.criu.org). > To migrate processes with posix timers we have to make sure, that we can > restore posix timer with proper id. > Currently, this is not true, because timer ids are allocated global

[PATCH v5] posix timers: allocate timer id per process

2012-10-23 Thread Stanislav Kinsbursky
This patch is required CRIU project (www.criu.org). To migrate processes with posix timers we have to make sure, that we can restore posix timer with proper id. Currently, this is not true, because timer ids are allocated globally. So, this is precursor patch and it's purpose is make posix timer id