Re: [RFC PATCH v2 1/5] futex: Add futex_set_timer() helper function

2016-09-22 Thread Waiman Long
On 09/22/2016 05:31 PM, Thomas Gleixner wrote: On Tue, 20 Sep 2016, Waiman Long wrote: Please be more careful of your subject lines. First thing I thought was that you add a helper which is used in later patches to find out that you actualy consolidate duplicated code. Something like:

Re: [RFC PATCH v2 1/5] futex: Add futex_set_timer() helper function

2016-09-22 Thread Waiman Long
On 09/22/2016 05:31 PM, Thomas Gleixner wrote: On Tue, 20 Sep 2016, Waiman Long wrote: Please be more careful of your subject lines. First thing I thought was that you add a helper which is used in later patches to find out that you actualy consolidate duplicated code. Something like:

Re: [RFC PATCH v2 1/5] futex: Add futex_set_timer() helper function

2016-09-22 Thread Thomas Gleixner
On Tue, 20 Sep 2016, Waiman Long wrote: Please be more careful of your subject lines. First thing I thought was that you add a helper which is used in later patches to find out that you actualy consolidate duplicated code. Something like: futex: Consolidate duplicated timer setup code would

Re: [RFC PATCH v2 1/5] futex: Add futex_set_timer() helper function

2016-09-22 Thread Thomas Gleixner
On Tue, 20 Sep 2016, Waiman Long wrote: Please be more careful of your subject lines. First thing I thought was that you add a helper which is used in later patches to find out that you actualy consolidate duplicated code. Something like: futex: Consolidate duplicated timer setup code would

[RFC PATCH v2 1/5] futex: Add futex_set_timer() helper function

2016-09-20 Thread Waiman Long
This patch adds a new futex_set_timer() function to consolidate all the sleeping hrtime setup code. Signed-off-by: Waiman Long --- kernel/futex.c | 51 --- 1 files changed, 24 insertions(+), 27 deletions(-) diff --git

[RFC PATCH v2 1/5] futex: Add futex_set_timer() helper function

2016-09-20 Thread Waiman Long
This patch adds a new futex_set_timer() function to consolidate all the sleeping hrtime setup code. Signed-off-by: Waiman Long --- kernel/futex.c | 51 --- 1 files changed, 24 insertions(+), 27 deletions(-) diff --git a/kernel/futex.c