Re: Re: Re: Re: Re: Re: [PATCH v3] arm: Adding support for atomic half word exchange
On Fri, Oct 09, 2015 at 02:41:54AM +, Sarbojit Ganguly wrote: > Thank you Will, I will make the required corrections and mail it to > patchesAtarm.linux.co.uk and mark [1] as "superseded". Thanks. Also, you don't need to continually prepend "Re: " to your mail subject every time you reply and/or update the patch. Doing so breaks the email threading for everybody else. Will -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: Re: Re: Re: Re: Re: [PATCH v3] arm: Adding support for atomic half word exchange
On Fri, Oct 09, 2015 at 02:41:54AM +, Sarbojit Ganguly wrote: > Thank you Will, I will make the required corrections and mail it to > patchesAtarm.linux.co.uk and mark [1] as "superseded". Thanks. Also, you don't need to continually prepend "Re: " to your mail subject every time you reply and/or update the patch. Doing so breaks the email threading for everybody else. Will -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: Re: Re: Re: Re: Re: [PATCH v3] arm: Adding support for atomic half word exchange
Thank you Will, I will make the required corrections and mail it to patchesAtarm.linux.co.uk and mark [1] as "superseded". Sarbojit [1] http://www.arm.linux.org.uk/developer/patches/viewpatch.php?id=8442/1 --- Original Message --- Sender : Will Deacon Date : Oct 08, 2015 22:26 (GMT+05:30) Title : Re: Re: Re: Re: Re: [PATCH v3] arm: Adding support for atomic half word exchange On Wed, Oct 07, 2015 at 02:36:41PM +, Sarbojit Ganguly wrote: > Please have a look at this patch, please let me know if any modification > is required. > I have also submitted the same in your patch system. There are some problems with the version in the patch system[1]: (1) You still have the changelog in the commit message (I asked you to remove this last time). (2) The commit message isn't line-wrapper appropriately (3) Your Signed-off-by line is truncated The patch system does actually support git, so you can add: KernelVersion: 4.3-rc4 or whatever kernel you based your patch on somewhere after your SoB but before the diff and then mail it to patchesATarm.linux.org.uk. You probably also want to move the current patch to "superseded" so Russell doesn't end up with two copies. Will [1] http://www.arm.linux.org.uk/developer/patches/viewpatch.php?id=8442/1 ? ??? ?? ?? ?? --+ The Tao lies beyond Yin and Yang. It is silent and still as a pool of water. | It does not seek fame, therefore nobody knows its presence. | It does not seek fortune, for it is complete within itself. | It exists beyond space and time. | --+N‹§²æìr¸›yúèšØb²X¬¶Ç§vØ^–)Þº{.nÇ+‰·¥Š{±‘êçzX§¶›¡Ü¨}©ž²Æ zÚ:+v‰¨¾«‘êçzZ+€Ê+zf£¢·hšˆ§~††Ûiÿûàz¹®w¥¢¸?™¨èÚ&¢)ߢf”ù^jÇ«y§m…á@A«a¶Úÿ 0¶ìh®å’i
Re: Re: Re: Re: Re: [PATCH v3] arm: Adding support for atomic half word exchange
On Wed, Oct 07, 2015 at 02:36:41PM +, Sarbojit Ganguly wrote: > Please have a look at this patch, please let me know if any modification > is required. > I have also submitted the same in your patch system. There are some problems with the version in the patch system[1]: (1) You still have the changelog in the commit message (I asked you to remove this last time). (2) The commit message isn't line-wrapper appropriately (3) Your Signed-off-by line is truncated The patch system does actually support git, so you can add: KernelVersion: 4.3-rc4 or whatever kernel you based your patch on somewhere after your SoB but before the diff and then mail it to patchesATarm.linux.org.uk. You probably also want to move the current patch to "superseded" so Russell doesn't end up with two copies. Will [1] http://www.arm.linux.org.uk/developer/patches/viewpatch.php?id=8442/1 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: Re: Re: Re: Re: [PATCH v3] arm: Adding support for atomic half word exchange
On Wed, Oct 07, 2015 at 02:36:41PM +, Sarbojit Ganguly wrote: > Please have a look at this patch, please let me know if any modification > is required. > I have also submitted the same in your patch system. There are some problems with the version in the patch system[1]: (1) You still have the changelog in the commit message (I asked you to remove this last time). (2) The commit message isn't line-wrapper appropriately (3) Your Signed-off-by line is truncated The patch system does actually support git, so you can add: KernelVersion: 4.3-rc4 or whatever kernel you based your patch on somewhere after your SoB but before the diff and then mail it to patchesATarm.linux.org.uk. You probably also want to move the current patch to "superseded" so Russell doesn't end up with two copies. Will [1] http://www.arm.linux.org.uk/developer/patches/viewpatch.php?id=8442/1 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: Re: Re: Re: Re: Re: [PATCH v3] arm: Adding support for atomic half word exchange
Thank you Will, I will make the required corrections and mail it to patchesAtarm.linux.co.uk and mark [1] as "superseded". Sarbojit [1] http://www.arm.linux.org.uk/developer/patches/viewpatch.php?id=8442/1 --- Original Message --- Sender : Will Deacon<will.dea...@arm.com> Date : Oct 08, 2015 22:26 (GMT+05:30) Title : Re: Re: Re: Re: Re: [PATCH v3] arm: Adding support for atomic half word exchange On Wed, Oct 07, 2015 at 02:36:41PM +, Sarbojit Ganguly wrote: > Please have a look at this patch, please let me know if any modification > is required. > I have also submitted the same in your patch system. There are some problems with the version in the patch system[1]: (1) You still have the changelog in the commit message (I asked you to remove this last time). (2) The commit message isn't line-wrapper appropriately (3) Your Signed-off-by line is truncated The patch system does actually support git, so you can add: KernelVersion: 4.3-rc4 or whatever kernel you based your patch on somewhere after your SoB but before the diff and then mail it to patchesATarm.linux.org.uk. You probably also want to move the current patch to "superseded" so Russell doesn't end up with two copies. Will [1] http://www.arm.linux.org.uk/developer/patches/viewpatch.php?id=8442/1 ? ??? ?? ?? ?? --+ The Tao lies beyond Yin and Yang. It is silent and still as a pool of water. | It does not seek fame, therefore nobody knows its presence. | It does not seek fortune, for it is complete within itself. | It exists beyond space and time. | --+N‹§²æìr¸›yúèšØb²X¬¶Ç§vØ^–)Þº{.nÇ+‰·¥Š{±‘êçzX§¶›¡Ü¨}©ž²Æ zÚ:+v‰¨¾«‘êçzZ+€Ê+zf£¢·hšˆ§~††Ûiÿûàz¹®w¥¢¸?™¨èÚ&¢)ߢf”ù^jÇ«y§m…á@A«a¶Úÿ 0¶ìh®å’i
Re: Re: Re: Re: Re: [PATCH v3] arm: Adding support for atomic half word exchange
Hello Russell, Please have a look at this patch, please let me know if any modification is required. I have also submitted the same in your patch system. v2 -> v3: Removed the comment related to Qspinlock, changed !defined to #ifndef. v1 -> v2: Extended the guard code to cover the byte exchange case as well following opinion of Will Deacon. Checkpatch has been run and issues were taken care of. Since support for half-word atomic exchange was not there and Qspinlock on ARM requires it, modified __xchg() to add support for that as well. ARMv6 and lower does not support ldrex{b,h} so, added a guard code to prevent build breaks. Signed-off-by: Sarbojit Ganguly --- arch/arm/include/asm/cmpxchg.h | 12 1 file changed, 12 insertions(+) diff --git a/arch/arm/include/asm/cmpxchg.h b/arch/arm/include/asm/cmpxchg.h index 916a274..97882f9 100644 --- a/arch/arm/include/asm/cmpxchg.h +++ b/arch/arm/include/asm/cmpxchg.h @@ -39,6 +39,7 @@ static inline unsigned long __xchg(unsigned long x, volatile void *ptr, int size switch (size) { #if __LINUX_ARM_ARCH__ >= 6 +#ifndef CONFIG_CPU_V6 /* MIN ARCH >= V6K */ case 1: asm volatile("@ __xchg1\n" "1: ldrexb %0, [%3]\n" @@ -49,6 +50,17 @@ static inline unsigned long __xchg(unsigned long x, volatile void *ptr, int size : "r" (x), "r" (ptr) : "memory", "cc"); break; + case 2: + asm volatile("@ __xchg2\n" + "1: ldrexh %0, [%3]\n" + " strexh %1, %2, [%3]\n" + " teq %1, #0\n" + " bne 1b" + : "=" (ret), "=" (tmp) + : "r" (x), "r" (ptr) + : "memory", "cc"); + break; +#endif case 4: asm volatile("@ __xchg4\n" "1: ldrex %0, [%3]\n" -- 1.9.1 Regards, Sarbojit --- Original Message --- Sender : Will Deacon Date : Oct 06, 2015 20:24 (GMT+05:30) Title : Re: Re: Re: Re: [PATCH v3] arm: Adding support for atomic half word exchange On Tue, Oct 06, 2015 at 08:03:02AM +, Sarbojit Ganguly wrote: > Here is the version 3 of the patch correcting earlier issues. This looks good to me now: Acked-by: Will Deacon > v2 -> v3 : Removed the comment related to Qspinlock, changed !defined to > #ifndef. > v1 -> v2 : Extended the guard code to cover the byte exchange case as > well following opinion of Will Deacon. > Checkpatch has been run and issues were taken care of. The part of your text up until here doesn't belong in the commit message. You'll also need to send this to Russell's patch system. Will > Since support for half-word atomic exchange was not there and Qspinlock > on ARM requires it, modified __xchg() to add support for that as well. > ARMv6 and lower does not support ldrex{b,h} so, added a guard code > to prevent build breaks. > > Signed-off-by: Sarbojit Ganguly > --- > arch/arm/include/asm/cmpxchg.h | 12 > 1 file changed, 12 insertions(+) > > diff --git a/arch/arm/include/asm/cmpxchg.h b/arch/arm/include/asm/cmpxchg.h > index 916a274..c6436c1 100644 > --- a/arch/arm/include/asm/cmpxchg.h > +++ b/arch/arm/include/asm/cmpxchg.h > @@ -39,6 +39,7 @@ static inline unsigned long __xchg(unsigned long x, > volatile void *ptr, int size > > switch (size) { > #if __LINUX_ARM_ARCH__ >= 6 > +#ifndef CONFIG_CPU_V6 /* MIN ARCH >= V6K */ > case 1: > asm volatile("@ __xchg1\n" > "1: ldrexb %0, [%3]\n" > @@ -49,6 +50,17 @@ static inline unsigned long __xchg(unsigned long x, > volatile void *ptr, int size > : "r" (x), "r" (ptr) > : "memory", "cc"); > break; > + case 2: > + asm volatile("@ __xchg2\n" > + "1: ldrexh %0, [%3]\n" > + " strexh %1, %2, [%3]\n" > + " teq %1, #0\n" > + " bne 1b" > + : "=" (ret), "=" (tmp) > + : "r" (x), "r" (ptr) > + : "memory", "cc"); > + break; > +#endif > case 4: > asm volatile("@ __xchg4\n" > "1: ldrex %0, [%3]\n" > -- > 1.9.1
Re: Re: Re: Re: Re: [PATCH v3] arm: Adding support for atomic half word exchange
Hello Russell, Please have a look at this patch, please let me know if any modification is required. I have also submitted the same in your patch system. v2 -> v3: Removed the comment related to Qspinlock, changed !defined to #ifndef. v1 -> v2: Extended the guard code to cover the byte exchange case as well following opinion of Will Deacon. Checkpatch has been run and issues were taken care of. Since support for half-word atomic exchange was not there and Qspinlock on ARM requires it, modified __xchg() to add support for that as well. ARMv6 and lower does not support ldrex{b,h} so, added a guard code to prevent build breaks. Signed-off-by: Sarbojit Ganguly <gangul...@samsung.com> --- arch/arm/include/asm/cmpxchg.h | 12 1 file changed, 12 insertions(+) diff --git a/arch/arm/include/asm/cmpxchg.h b/arch/arm/include/asm/cmpxchg.h index 916a274..97882f9 100644 --- a/arch/arm/include/asm/cmpxchg.h +++ b/arch/arm/include/asm/cmpxchg.h @@ -39,6 +39,7 @@ static inline unsigned long __xchg(unsigned long x, volatile void *ptr, int size switch (size) { #if __LINUX_ARM_ARCH__ >= 6 +#ifndef CONFIG_CPU_V6 /* MIN ARCH >= V6K */ case 1: asm volatile("@ __xchg1\n" "1: ldrexb %0, [%3]\n" @@ -49,6 +50,17 @@ static inline unsigned long __xchg(unsigned long x, volatile void *ptr, int size : "r" (x), "r" (ptr) : "memory", "cc"); break; + case 2: + asm volatile("@ __xchg2\n" + "1: ldrexh %0, [%3]\n" + " strexh %1, %2, [%3]\n" + " teq %1, #0\n" + " bne 1b" + : "=" (ret), "=" (tmp) + : "r" (x), "r" (ptr) + : "memory", "cc"); + break; +#endif case 4: asm volatile("@ __xchg4\n" "1: ldrex %0, [%3]\n" -- 1.9.1 Regards, Sarbojit --- Original Message --- Sender : Will Deacon<will.dea...@arm.com> Date : Oct 06, 2015 20:24 (GMT+05:30) Title : Re: Re: Re: Re: [PATCH v3] arm: Adding support for atomic half word exchange On Tue, Oct 06, 2015 at 08:03:02AM +, Sarbojit Ganguly wrote: > Here is the version 3 of the patch correcting earlier issues. This looks good to me now: Acked-by: Will Deacon > v2 -> v3 : Removed the comment related to Qspinlock, changed !defined to > #ifndef. > v1 -> v2 : Extended the guard code to cover the byte exchange case as > well following opinion of Will Deacon. > Checkpatch has been run and issues were taken care of. The part of your text up until here doesn't belong in the commit message. You'll also need to send this to Russell's patch system. Will > Since support for half-word atomic exchange was not there and Qspinlock > on ARM requires it, modified __xchg() to add support for that as well. > ARMv6 and lower does not support ldrex{b,h} so, added a guard code > to prevent build breaks. > > Signed-off-by: Sarbojit Ganguly > --- > arch/arm/include/asm/cmpxchg.h | 12 > 1 file changed, 12 insertions(+) > > diff --git a/arch/arm/include/asm/cmpxchg.h b/arch/arm/include/asm/cmpxchg.h > index 916a274..c6436c1 100644 > --- a/arch/arm/include/asm/cmpxchg.h > +++ b/arch/arm/include/asm/cmpxchg.h > @@ -39,6 +39,7 @@ static inline unsigned long __xchg(unsigned long x, > volatile void *ptr, int size > > switch (size) { > #if __LINUX_ARM_ARCH__ >= 6 > +#ifndef CONFIG_CPU_V6 /* MIN ARCH >= V6K */ > case 1: > asm volatile("@ __xchg1\n" > "1: ldrexb %0, [%3]\n" > @@ -49,6 +50,17 @@ static inline unsigned long __xchg(unsigned long x, > volatile void *ptr, int size > : "r" (x), "r" (ptr) > : "memory", "cc"); > break; > + case 2: > + asm volatile("@ __xchg2\n" > + "1: ldrexh %0, [%3]\n" > + " strexh %1, %2, [%3]\n" > + " teq %1, #0\n" > + " bne 1b" > + : "=" (ret), "=" (tmp) > + : "r" (x), "r" (ptr) > + : "memory", "cc"); > + break; > +#endif > case 4: > asm volatile("@ __xchg4\n" > "1: ldrex %0, [%3]\n" > -- > 1.9.1