From: Yury Norov <ynorov@caviumnetworks.com> To: Boqun Feng <boqun.feng@gmail.com> Cc: Will Deacon <will.deacon@arm.com>, Peter Zijlstra <peterz@infradead.org>, linux-kernel@vger.kernel.org, linux-arch@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Adam Wallis <awallis@codeaurora.org>, Andrew Pinski <Andrew.Pinski@cavium.com>, Arnd Bergmann <arnd@arndb.de>, Catalin Marinas <catalin.marinas@arm.com>, Ingo Molnar <mingo@redhat.com>, Jan Glauber <jglauber@cavium.com>, Mark Rutland <mark.rutland@arm.com>, Pan Xinhui <xinhui.pan@linux.vnet.ibm.com> Subject: Re: [PATCH 3/3] arm64/locking: qspinlocks and qrwlocks support Date: Tue, 9 May 2017 21:48:29 +0300 [thread overview] Message-ID: <20170509184829.scwjixgc2kxxxku7@yury-N73SV> (raw) In-Reply-To: <20170509044708.vvwgzhvdrqyljy6s@tardis> On Tue, May 09, 2017 at 12:47:08PM +0800, Boqun Feng wrote: > On Wed, May 03, 2017 at 05:51:41PM +0300, Yury Norov wrote: > > From: Jan Glauber <jglauber@cavium.com> > > > > Ported from x86_64 with paravirtualization support removed. > > > > Signed-off-by: Jan Glauber <jglauber@cavium.com> > > > > Note. This patch removes protection from direct inclusion of > > arch/arm64/include/asm/spinlock_types.h. It's done because > > kernel/locking/qrwlock.c file does it thru the header > > include/asm-generic/qrwlock_types.h. Until now the only user > > of qrwlock.c was x86, and there's no such protection too. > > > > I'm not happy to remove the protection, but if it's OK for x86, > > it should be also OK for arm64. If not, I think we'd fix it > > for x86, and add the protection there too. > > > > Yury > > > > Signed-off-by: Yury Norov <ynorov@caviumnetworks.com> [...] > > +#define queued_spin_unlock queued_spin_unlock > > +/** > > + * queued_spin_unlock - release a queued spinlock > > + * @lock : Pointer to queued spinlock structure > > + * > > + * A smp_store_release() on the least-significant byte. > > + */ > > +static inline void queued_spin_unlock(struct qspinlock *lock) > > +{ > > + smp_store_release((u8 *)lock, 0); > > I think this part will cause endian issues, maybe you want something > like what we do in queued_write_lock(). > > Have you tested this on an BE environment? No. I think I have to. Thanks for the pointing it. > > Regards, > Boqun I think it's just the issue of copying from x86, and there's no any specific need to cast to u8* type on arm64. So the correct version of it would be like this, I believe: smp_store_release(&lock->val). Yury
WARNING: multiple messages have this Message-ID (diff)
From: ynorov@caviumnetworks.com (Yury Norov) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH 3/3] arm64/locking: qspinlocks and qrwlocks support Date: Tue, 9 May 2017 21:48:29 +0300 [thread overview] Message-ID: <20170509184829.scwjixgc2kxxxku7@yury-N73SV> (raw) In-Reply-To: <20170509044708.vvwgzhvdrqyljy6s@tardis> On Tue, May 09, 2017 at 12:47:08PM +0800, Boqun Feng wrote: > On Wed, May 03, 2017 at 05:51:41PM +0300, Yury Norov wrote: > > From: Jan Glauber <jglauber@cavium.com> > > > > Ported from x86_64 with paravirtualization support removed. > > > > Signed-off-by: Jan Glauber <jglauber@cavium.com> > > > > Note. This patch removes protection from direct inclusion of > > arch/arm64/include/asm/spinlock_types.h. It's done because > > kernel/locking/qrwlock.c file does it thru the header > > include/asm-generic/qrwlock_types.h. Until now the only user > > of qrwlock.c was x86, and there's no such protection too. > > > > I'm not happy to remove the protection, but if it's OK for x86, > > it should be also OK for arm64. If not, I think we'd fix it > > for x86, and add the protection there too. > > > > Yury > > > > Signed-off-by: Yury Norov <ynorov@caviumnetworks.com> [...] > > +#define queued_spin_unlock queued_spin_unlock > > +/** > > + * queued_spin_unlock - release a queued spinlock > > + * @lock : Pointer to queued spinlock structure > > + * > > + * A smp_store_release() on the least-significant byte. > > + */ > > +static inline void queued_spin_unlock(struct qspinlock *lock) > > +{ > > + smp_store_release((u8 *)lock, 0); > > I think this part will cause endian issues, maybe you want something > like what we do in queued_write_lock(). > > Have you tested this on an BE environment? No. I think I have to. Thanks for the pointing it. > > Regards, > Boqun I think it's just the issue of copying from x86, and there's no any specific need to cast to u8* type on arm64. So the correct version of it would be like this, I believe: smp_store_release(&lock->val). Yury
next prev parent reply other threads:[~2017-05-09 18:48 UTC|newest] Thread overview: 57+ messages / expand[flat|nested] mbox.gz Atom feed top 2017-05-03 14:51 [PATCH 0/3] arm64: queued spinlocks and rw-locks Yury Norov 2017-05-03 14:51 ` Yury Norov 2017-05-03 14:51 ` [PATCH 1/3] kernel/locking: #include <asm/spinlock.h> in qrwlock.c Yury Norov 2017-05-03 14:51 ` Yury Norov 2017-05-03 15:05 ` Geert Uytterhoeven 2017-05-03 15:05 ` Geert Uytterhoeven 2017-05-03 15:05 ` Geert Uytterhoeven 2017-05-03 15:05 ` Geert Uytterhoeven 2017-05-03 20:32 ` Yury Norov 2017-05-03 20:32 ` Yury Norov 2017-05-03 20:32 ` Yury Norov 2017-05-03 14:51 ` [PATCH 2/3] asm-generic: don't #include <linux/atomic.h> in qspinlock_types.h Yury Norov 2017-05-03 14:51 ` Yury Norov 2017-05-04 8:01 ` Arnd Bergmann 2017-05-04 8:01 ` Arnd Bergmann 2017-05-03 14:51 ` [PATCH 3/3] arm64/locking: qspinlocks and qrwlocks support Yury Norov 2017-05-03 14:51 ` Yury Norov 2017-05-09 4:47 ` Boqun Feng 2017-05-09 4:47 ` Boqun Feng 2017-05-09 18:48 ` Yury Norov [this message] 2017-05-09 18:48 ` Yury Norov 2017-05-09 19:37 ` Yury Norov 2017-05-09 19:37 ` Yury Norov 2017-05-03 18:59 ` 答复: [PATCH 0/3] arm64: queued spinlocks and rw-locks pan xinhui 2017-05-03 18:59 ` pan xinhui 2017-05-04 20:28 ` Yury Norov 2017-05-04 20:28 ` Yury Norov 2017-05-04 20:28 ` Yury Norov 2017-05-05 11:37 ` 答复: " pan xinhui 2017-05-05 11:37 ` pan xinhui 2017-05-05 11:53 ` Peter Zijlstra 2017-05-05 11:53 ` Peter Zijlstra 2017-05-05 11:53 ` Peter Zijlstra 2017-05-05 11:53 ` Peter Zijlstra 2017-05-05 12:26 ` Will Deacon 2017-05-05 12:26 ` Will Deacon 2017-05-05 12:26 ` Will Deacon 2017-05-05 15:28 ` Yury Norov 2017-05-05 15:28 ` Yury Norov 2017-05-05 15:28 ` Yury Norov 2017-05-05 15:32 ` Will Deacon 2017-05-05 15:32 ` Will Deacon 2017-05-05 15:32 ` Will Deacon -- strict thread matches above, loose matches on Subject: below -- 2017-04-10 21:35 [RFC PATCH " Yury Norov 2017-04-10 21:35 ` [PATCH 3/3] arm64/locking: qspinlocks and qrwlocks support Yury Norov 2017-04-10 21:35 ` Yury Norov 2017-04-13 18:12 ` Peter Zijlstra 2017-04-13 18:12 ` Peter Zijlstra 2017-04-20 18:23 ` Yury Norov 2017-04-20 18:23 ` Yury Norov 2017-04-20 19:00 ` Mark Rutland 2017-04-20 19:00 ` Mark Rutland 2017-04-20 19:05 ` Peter Zijlstra 2017-04-20 19:05 ` Peter Zijlstra 2017-04-26 12:39 ` Yury Norov 2017-04-26 12:39 ` Yury Norov 2017-04-28 15:44 ` Will Deacon 2017-04-28 15:44 ` Will Deacon
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=20170509184829.scwjixgc2kxxxku7@yury-N73SV \ --to=ynorov@caviumnetworks.com \ --cc=Andrew.Pinski@cavium.com \ --cc=arnd@arndb.de \ --cc=awallis@codeaurora.org \ --cc=boqun.feng@gmail.com \ --cc=catalin.marinas@arm.com \ --cc=jglauber@cavium.com \ --cc=linux-arch@vger.kernel.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=mark.rutland@arm.com \ --cc=mingo@redhat.com \ --cc=peterz@infradead.org \ --cc=will.deacon@arm.com \ --cc=xinhui.pan@linux.vnet.ibm.com \ /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.