Re: [PATCH 2/4] locking/rtmutex: Use cmp-cmpxchg

2015-06-15 Thread Jason Low
On Mon, Jun 15, 2015 at 12:37 PM, Davidlohr Bueso wrote: > On Mon, 2015-06-15 at 11:34 -0700, Jason Low wrote: >> The CCAS technique was typically used in the slow paths for those >> other locks, where the chance of the operation returning false is >> higher. > > That is true. Although I really wa

Re: [PATCH 2/4] locking/rtmutex: Use cmp-cmpxchg

2015-06-15 Thread Davidlohr Bueso
On Mon, 2015-06-15 at 11:34 -0700, Jason Low wrote: > The CCAS technique was typically used in the slow paths for those > other locks, where the chance of the operation returning false is > higher. That is true. Although I really want to use it in patch 4, I guess I could move the check in there,

Re: [PATCH 2/4] locking/rtmutex: Use cmp-cmpxchg

2015-06-15 Thread Jason Low
Hi David, On Sat, Jun 6, 2015 at 8:27 AM, Davidlohr Bueso wrote: > On Fri, 2015-06-05 at 14:38 +0200, Thomas Gleixner wrote: >> On Tue, 19 May 2015, Davidlohr Bueso wrote: >> >> > Avoid unnecessary cmpxchg calls, all of our other locks >> > use it as well. >> > >> > Signed-off-by: Davidlohr Bueso

Re: [PATCH 2/4] locking/rtmutex: Use cmp-cmpxchg

2015-06-06 Thread Davidlohr Bueso
On Fri, 2015-06-05 at 14:38 +0200, Thomas Gleixner wrote: > On Tue, 19 May 2015, Davidlohr Bueso wrote: > > > Avoid unnecessary cmpxchg calls, all of our other locks > > use it as well. > > > > Signed-off-by: Davidlohr Bueso > > --- > > kernel/locking/rtmutex.c | 4 +++- > > 1 file changed, 3 i

Re: [PATCH 2/4] locking/rtmutex: Use cmp-cmpxchg

2015-06-05 Thread Thomas Gleixner
On Tue, 19 May 2015, Davidlohr Bueso wrote: > Avoid unnecessary cmpxchg calls, all of our other locks > use it as well. > > Signed-off-by: Davidlohr Bueso > --- > kernel/locking/rtmutex.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/kernel/locking/rtmutex.c b/ker

[PATCH 2/4] locking/rtmutex: Use cmp-cmpxchg

2015-05-19 Thread Davidlohr Bueso
Avoid unnecessary cmpxchg calls, all of our other locks use it as well. Signed-off-by: Davidlohr Bueso --- kernel/locking/rtmutex.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/kernel/locking/rtmutex.c b/kernel/locking/rtmutex.c index 74188d8..1d5cc9d 100644 --- a/kerne