Re: [PATCH 1/1] use documented cmpxchg api

2014-06-06 Thread Pranith Kumar
On 06/06/2014 06:55 AM, Peter Hurley wrote: > > This won't work either, and should generate a compile or link error > on CONFIG_X86_32. > > The auto-sizing done by __cmpxchg() does not substitute the cmpxchg8b > instruction for the cmpxchg instruction. __X86_CASE_Q is defined as -1 for > CONFIG_X

Re: [PATCH 1/1] use documented cmpxchg api

2014-06-06 Thread Peter Hurley
On 06/05/2014 09:03 PM, Pranith Kumar wrote: On 06/05/2014 08:34 PM, Peter Hurley wrote: On 06/05/2014 06:19 PM, Pranith Kumar wrote: use the documented atomic_cmpxchg instead of __cmpxchg64 This kills the last user of said API in drivers code. Signed-off-by: Pranith Kumar --- drivers/io

Re: [PATCH 1/1] use documented cmpxchg api

2014-06-05 Thread Pranith Kumar
On 06/05/2014 08:34 PM, Peter Hurley wrote: > On 06/05/2014 06:19 PM, Pranith Kumar wrote: >> use the documented atomic_cmpxchg instead of __cmpxchg64 >> >> This kills the last user of said API in drivers code. >> >> >> Signed-off-by: Pranith Kumar >> --- >> drivers/iommu/intel-iommu.c |2 +-

Re: [PATCH 1/1] use documented cmpxchg api

2014-06-05 Thread Peter Hurley
On 06/05/2014 06:19 PM, Pranith Kumar wrote: use the documented atomic_cmpxchg instead of __cmpxchg64 This kills the last user of said API in drivers code. Signed-off-by: Pranith Kumar --- drivers/iommu/intel-iommu.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dr

[PATCH 1/1] use documented cmpxchg api

2014-06-05 Thread Pranith Kumar
use the documented atomic_cmpxchg instead of __cmpxchg64 This kills the last user of said API in drivers code. Signed-off-by: Pranith Kumar --- drivers/iommu/intel-iommu.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/iommu/intel-iommu.c b/drivers/iommu/intel-i