From: <vincentc@andestech.com> To: <arnd@arndb.de> Cc: <greentime@andestech.com>, <linux-kernel@vger.kernel.org>, <linux-arch@vger.kernel.org>, <tglx@linutronix.de>, <jason@lakedaemon.net>, <marc.zyngier@arm.com>, <robh+dt@kernel.org>, <netdev@vger.kernel.org>, <palmer@dabbelt.com> Subject: RE: [PATCH 11/31] nds32: Atomic operations Date: Wed, 8 Nov 2017 09:32:36 +0000 [thread overview] Message-ID: <E26F4CF8B7DDDB4383A6C2D78D5C3CD56B497064@ATCPCS16.andestech.com> (raw) In-Reply-To: <CAK8P3a3w4gfO-VMM+L5J5+kSbBD_Kr0iv3BndF_3tt2QTuEFRw@mail.gmail.com> > -----Original Message----- > From: arndbergmann@gmail.com [mailto:arndbergmann@gmail.com] On > Behalf Of Arnd Bergmann > Sent: Wednesday, November 08, 2017 4:54 PM > To: Greentime Hu > Cc: Greentime Ying-Han Hu(胡英漢); Linux Kernel Mailing List; linux-arch; > Thomas Gleixner; Jason Cooper; Marc Zyngier; Rob Herring; Networking; > Vincent Ren-Wei Chen(陳人維); Palmer Dabbelt > Subject: Re: [PATCH 11/31] nds32: Atomic operations > > On Wed, Nov 8, 2017 at 6:54 AM, Greentime Hu <green.hu@gmail.com> > wrote: > > From: Greentime Hu <greentime@andestech.com> > > > > Signed-off-by: Vincent Chen <vincentc@andestech.com> > > Signed-off-by: Greentime Hu <greentime@andestech.com> > > --- > > arch/nds32/include/asm/futex.h | 116 ++++++++++++++++++++++++ > > arch/nds32/include/asm/spinlock.h | 178 > > +++++++++++++++++++++++++++++++++++++ > > 2 files changed, 294 insertions(+) > > create mode 100644 arch/nds32/include/asm/futex.h create mode > 100644 > > arch/nds32/include/asm/spinlock.h > > > > diff --git a/arch/nds32/include/asm/spinlock.h > > b/arch/nds32/include/asm/spinlock.h > > new file mode 100644 > > index 0000000..dd5fc71 > > --- /dev/null > > +++ b/arch/nds32/include/asm/spinlock.h > > @@ -0,0 +1,178 @@ > > + > > +#define arch_spin_unlock_wait(lock) \ > > + do { while (arch_spin_is_locked(lock)) cpu_relax(); } while > > +(0) > > This was removed from the other architectures in commit > 952111d7db02 ("arch: Remove spin_unlock_wait() arch-specific definitions") > > Please remove this as well. > Dear Arnd: I will remove them in the next version patch. > Palmer, I see riscv has the same thing, please also add a patch to your tree to > remove it. > > > +#define arch_spin_lock_flags(lock, flags) arch_spin_lock(lock) > > + > > +static inline void arch_spin_lock(arch_spinlock_t * lock) { > > + unsigned long tmp; > > + > > + __asm__ __volatile__("1:\n" > > + "\tllw\t%0, [%1]\n" > > + "\tbnez\t%0, 1b\n" > > + "\tmovi\t%0, #0x1\n" > > + "\tscw\t%0, [%1]\n" > > + "\tbeqz\t%0, 1b\n" > > + :"=&r"(tmp) > > + :"r"(&lock->lock) > > + :"memory"); > > The coding style seems inconsistent here, the other inline asm uses real tabs > instead of \t, and 'asm volatile' is generally preferred over '__asm__ > __volatile__'. > > Arnd OK, I will modify it in the next version patch. Thanks Best regards Vincent CONFIDENTIALITY NOTICE: This e-mail (and its attachments) may contain confidential and legally privileged information or information protected from disclosure. If you are not the intended recipient, you are hereby notified that any disclosure, copying, distribution, or use of the information contained herein is strictly prohibited. In this case, please immediately notify the sender by return e-mail, delete the message (and any accompanying documents) and destroy all printed hard copies. Thank you for your cooperation. Copyright ANDES TECHNOLOGY CORPORATION - All Rights Reserved.
WARNING: multiple messages have this Message-ID (diff)
From: <vincentc@andestech.com> To: arnd@arndb.de Cc: greentime@andestech.com, linux-kernel@vger.kernel.org, linux-arch@vger.kernel.org, tglx@linutronix.de, jason@lakedaemon.net, marc.zyngier@arm.com, robh+dt@kernel.org, netdev@vger.kernel.org, palmer@dabbelt.com Subject: RE: [PATCH 11/31] nds32: Atomic operations Date: Wed, 8 Nov 2017 09:32:36 +0000 [thread overview] Message-ID: <E26F4CF8B7DDDB4383A6C2D78D5C3CD56B497064@ATCPCS16.andestech.com> (raw) In-Reply-To: <CAK8P3a3w4gfO-VMM+L5J5+kSbBD_Kr0iv3BndF_3tt2QTuEFRw@mail.gmail.com> > -----Original Message----- > From: arndbergmann@gmail.com [mailto:arndbergmann@gmail.com] On > Behalf Of Arnd Bergmann > Sent: Wednesday, November 08, 2017 4:54 PM > To: Greentime Hu > Cc: Greentime Ying-Han Hu(胡英漢); Linux Kernel Mailing List; linux-arch; > Thomas Gleixner; Jason Cooper; Marc Zyngier; Rob Herring; Networking; > Vincent Ren-Wei Chen(陳人維); Palmer Dabbelt > Subject: Re: [PATCH 11/31] nds32: Atomic operations > > On Wed, Nov 8, 2017 at 6:54 AM, Greentime Hu <green.hu@gmail.com> > wrote: > > From: Greentime Hu <greentime@andestech.com> > > > > Signed-off-by: Vincent Chen <vincentc@andestech.com> > > Signed-off-by: Greentime Hu <greentime@andestech.com> > > --- > > arch/nds32/include/asm/futex.h | 116 ++++++++++++++++++++++++ > > arch/nds32/include/asm/spinlock.h | 178 > > +++++++++++++++++++++++++++++++++++++ > > 2 files changed, 294 insertions(+) > > create mode 100644 arch/nds32/include/asm/futex.h create mode > 100644 > > arch/nds32/include/asm/spinlock.h > > > > diff --git a/arch/nds32/include/asm/spinlock.h > > b/arch/nds32/include/asm/spinlock.h > > new file mode 100644 > > index 0000000..dd5fc71 > > --- /dev/null > > +++ b/arch/nds32/include/asm/spinlock.h > > @@ -0,0 +1,178 @@ > > + > > +#define arch_spin_unlock_wait(lock) \ > > + do { while (arch_spin_is_locked(lock)) cpu_relax(); } while > > +(0) > > This was removed from the other architectures in commit > 952111d7db02 ("arch: Remove spin_unlock_wait() arch-specific definitions") > > Please remove this as well. > Dear Arnd: I will remove them in the next version patch. > Palmer, I see riscv has the same thing, please also add a patch to your tree to > remove it. > > > +#define arch_spin_lock_flags(lock, flags) arch_spin_lock(lock) > > + > > +static inline void arch_spin_lock(arch_spinlock_t * lock) { > > + unsigned long tmp; > > + > > + __asm__ __volatile__("1:\n" > > + "\tllw\t%0, [%1]\n" > > + "\tbnez\t%0, 1b\n" > > + "\tmovi\t%0, #0x1\n" > > + "\tscw\t%0, [%1]\n" > > + "\tbeqz\t%0, 1b\n" > > + :"=&r"(tmp) > > + :"r"(&lock->lock) > > + :"memory"); > > The coding style seems inconsistent here, the other inline asm uses real tabs > instead of \t, and 'asm volatile' is generally preferred over '__asm__ > __volatile__'. > > Arnd OK, I will modify it in the next version patch. Thanks Best regards Vincent CONFIDENTIALITY NOTICE: This e-mail (and its attachments) may contain confidential and legally privileged information or information protected from disclosure. If you are not the intended recipient, you are hereby notified that any disclosure, copying, distribution, or use of the information contained herein is strictly prohibited. In this case, please immediately notify the sender by return e-mail, delete the message (and any accompanying documents) and destroy all printed hard copies. Thank you for your cooperation. Copyright ANDES TECHNOLOGY CORPORATION - All Rights Reserved.
next prev parent reply other threads:[~2017-11-08 10:08 UTC|newest] Thread overview: 116+ messages / expand[flat|nested] mbox.gz Atom feed top 2017-11-08 5:54 [PATCH 00/31] Andes(nds32) Linux Kernel Port Greentime Hu 2017-11-08 5:54 ` [PATCH 01/31] nds32: Assembly macros and definitions Greentime Hu 2017-11-08 5:54 ` [PATCH 02/31] nds32: Kernel booting and initialization Greentime Hu 2017-11-08 13:38 ` Rob Herring 2017-11-09 9:49 ` Greentime Hu 2017-11-08 5:54 ` [PATCH 03/31] nds32: Support early_printk Greentime Hu 2017-11-08 9:47 ` Tobias Klauser 2017-11-09 7:19 ` Greentime Hu 2017-11-08 5:54 ` [PATCH 04/31] nds32: Exception handling Greentime Hu 2017-11-08 8:23 ` Arnd Bergmann [not found] ` <E26F4CF8B7DDDB4383A6C2D78D5C3CD56B4974CE@ATCPCS16.andestech.com> 2017-11-13 10:54 ` Fwd: FW: " Vincent Chen 2017-11-08 5:54 ` [PATCH 05/31] nds32: MMU definitions Greentime Hu 2017-11-08 8:36 ` Arnd Bergmann 2017-11-08 8:46 ` Greentime Hu 2017-11-08 5:54 ` [PATCH 06/31] nds32: MMU initialization Greentime Hu 2017-11-08 5:54 ` [PATCH 07/31] nds32: MMU fault handling and page table management Greentime Hu 2017-11-08 5:54 ` [PATCH 08/31] nds32: Cache and TLB routines Greentime Hu 2017-11-08 8:45 ` Arnd Bergmann 2017-11-08 9:01 ` Greentime Hu 2017-11-08 5:54 ` [PATCH 09/31] nds32: Process management Greentime Hu 2017-11-08 5:54 ` [PATCH 10/31] nds32: IRQ handling Greentime Hu 2017-11-08 8:49 ` Arnd Bergmann 2017-11-08 9:06 ` Greentime Hu 2017-11-08 5:54 ` [PATCH 11/31] nds32: Atomic operations Greentime Hu 2017-11-08 8:54 ` Arnd Bergmann 2017-11-08 9:32 ` vincentc [this message] 2017-11-08 9:32 ` vincentc 2017-11-20 14:29 ` Will Deacon 2017-11-22 3:02 ` Vincent Chen 2017-11-08 5:55 ` [PATCH 12/31] nds32: Device specific operations Greentime Hu 2017-11-08 9:04 ` Arnd Bergmann 2017-11-09 7:04 ` Greentime Hu 2017-11-10 16:07 ` Greentime Hu 2017-11-10 16:14 ` Arnd Bergmann 2017-11-22 10:02 ` Greentime Hu 2017-11-08 5:55 ` [PATCH 13/31] nds32: DMA mapping API Greentime Hu 2017-11-08 9:09 ` Arnd Bergmann 2017-11-09 7:12 ` Greentime Hu 2017-11-09 10:14 ` Arnd Bergmann 2017-11-10 8:13 ` Greentime Hu 2017-11-08 5:55 ` [PATCH 14/31] nds32: ELF definitions Greentime Hu 2017-11-08 5:55 ` [PATCH 15/31] nds32: System calls handling Greentime Hu 2017-11-08 9:30 ` Arnd Bergmann [not found] ` <E26F4CF8B7DDDB4383A6C2D78D5C3CD56B497241@ATCPCS16.andestech.com> 2017-11-13 2:51 ` Fwd: FW: " Vincent Chen 2017-11-13 11:42 ` Arnd Bergmann 2017-11-22 3:13 ` Vincent Chen 2017-11-08 5:55 ` [PATCH 16/31] nds32: VDSO support Greentime Hu 2017-11-08 9:37 ` Arnd Bergmann 2017-11-08 20:00 ` Deepa Dinamani 2017-11-08 20:06 ` Arnd Bergmann 2017-11-08 20:14 ` Deepa Dinamani 2017-11-08 5:55 ` [PATCH 17/31] nds32: Signal handling support Greentime Hu 2017-11-09 1:26 ` Al Viro [not found] ` <E26F4CF8B7DDDB4383A6C2D78D5C3CD56B497460@ATCPCS16.andestech.com> 2017-11-13 2:34 ` Fwd: FW: " Vincent Chen 2017-11-08 5:55 ` [PATCH 18/31] nds32: Library functions Greentime Hu 2017-11-08 9:45 ` Arnd Bergmann 2017-11-09 0:40 ` Al Viro [not found] ` <E26F4CF8B7DDDB4383A6C2D78D5C3CD56B497559@ATCPCS16.andestech.com> 2017-11-14 4:47 ` Fwd: FW: " Vincent Chen 2017-11-18 2:44 ` Al Viro 2017-11-08 5:55 ` [PATCH 19/31] nds32: Debugging support Greentime Hu 2017-11-08 5:55 ` [PATCH 20/31] nds32: L2 cache support Greentime Hu 2017-11-08 9:48 ` Arnd Bergmann 2017-11-09 7:24 ` Greentime Hu 2017-11-08 5:55 ` [PATCH 21/31] nds32: Loadable modules Greentime Hu 2017-11-08 5:55 ` [PATCH 22/31] nds32: Generic timers support Greentime Hu 2017-11-08 5:55 ` [PATCH 23/31] nds32: Device tree support Greentime Hu 2017-11-08 9:53 ` Arnd Bergmann 2017-11-09 7:48 ` Greentime Hu 2017-11-08 5:55 ` [PATCH 24/31] nds32: Miscellaneous header files Greentime Hu 2017-11-08 9:57 ` Arnd Bergmann 2017-11-08 5:55 ` [PATCH 25/31] nds32: defconfig Greentime Hu 2017-11-08 10:03 ` Arnd Bergmann 2017-11-09 8:00 ` Greentime Hu 2017-11-09 10:20 ` Arnd Bergmann 2017-11-10 8:16 ` Greentime Hu 2017-11-08 5:55 ` [PATCH 26/31] nds32: Build infrastructure Greentime Hu 2017-11-08 10:16 ` Arnd Bergmann 2017-11-09 9:02 ` Greentime Hu 2017-11-09 10:33 ` Arnd Bergmann 2017-11-10 8:26 ` Greentime Hu 2017-11-17 12:39 ` Greentime Hu 2017-11-17 12:50 ` Arnd Bergmann 2017-11-17 13:50 ` Greentime Hu 2017-11-13 10:45 ` Geert Uytterhoeven 2017-11-13 10:45 ` Geert Uytterhoeven 2017-11-16 10:03 ` Greentime Hu 2017-11-16 10:25 ` Arnd Bergmann 2017-11-17 13:53 ` Greentime Hu 2017-11-08 5:55 ` [PATCH 27/31] dt-bindings: interrupt-controller: Andestech Internal Vector Interrupt Controller Greentime Hu 2017-11-08 13:25 ` Rob Herring 2017-11-09 9:43 ` Greentime Hu 2017-11-08 5:55 ` [PATCH 28/31] irqchip: Andestech Internal Vector Interrupt Controller driver Greentime Hu 2017-11-08 14:24 ` Marc Zyngier 2017-11-09 10:10 ` Greentime Hu 2017-11-08 5:55 ` [PATCH 29/31] MAINTAINERS: Add nds32 Greentime Hu 2017-11-08 13:31 ` Rob Herring 2017-11-09 9:46 ` Greentime Hu 2017-11-09 10:36 ` Arnd Bergmann 2017-11-14 15:39 ` Joe Perches 2017-11-16 12:22 ` Greentime Hu 2017-11-08 5:55 ` [PATCH 30/31] dt-bindings: nds32 CPU Bindings Greentime Hu 2017-11-08 13:18 ` Rob Herring 2017-11-09 9:39 ` Greentime Hu 2017-11-09 13:57 ` Rob Herring 2017-11-09 13:57 ` Rob Herring 2017-11-10 6:22 ` Greentime Hu 2017-11-10 8:25 ` Arnd Bergmann 2017-11-10 8:43 ` Greentime Hu 2017-11-10 8:43 ` Greentime Hu 2017-11-08 5:55 ` [PATCH 31/31] net: faraday add nds32 support Greentime Hu 2017-11-08 8:32 ` [PATCH 00/31] Andes(nds32) Linux Kernel Port David Howells 2017-11-08 8:41 ` Greentime Hu 2017-11-08 10:18 ` Arnd Bergmann 2017-11-09 9:26 ` Greentime Hu 2017-11-08 10:26 ` Arnd Bergmann 2017-11-09 9:33 ` Greentime Hu
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=E26F4CF8B7DDDB4383A6C2D78D5C3CD56B497064@ATCPCS16.andestech.com \ --to=vincentc@andestech.com \ --cc=arnd@arndb.de \ --cc=greentime@andestech.com \ --cc=jason@lakedaemon.net \ --cc=linux-arch@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=marc.zyngier@arm.com \ --cc=netdev@vger.kernel.org \ --cc=palmer@dabbelt.com \ --cc=robh+dt@kernel.org \ --cc=tglx@linutronix.de \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.