From mboxrd@z Thu Jan 1 00:00:00 1970 From: Will Deacon Subject: Re: [PATCH 16/18] x86: io: implement dummy relaxed accessor macros for writes Date: Tue, 22 Apr 2014 17:08:21 +0100 Message-ID: <20140422160821.GA31322@arm.com> References: <1397742261-15621-1-git-send-email-will.deacon@arm.com> <1397742261-15621-17-git-send-email-will.deacon@arm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from cam-admin0.cambridge.arm.com ([217.140.96.50]:51446 "EHLO cam-admin0.cambridge.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757270AbaDVQJF (ORCPT ); Tue, 22 Apr 2014 12:09:05 -0400 Content-Disposition: inline In-Reply-To: <1397742261-15621-17-git-send-email-will.deacon@arm.com> Sender: linux-arch-owner@vger.kernel.org List-ID: To: "linux-arch@vger.kernel.org" , "linux-kernel@vger.kernel.org" Cc: "arnd@arndb.de" , "monstr@monstr.eu" , "dhowells@redhat.com" , "broonie@linaro.org" , "benh@kernel.crashing.org" , "peterz@infradead.org" , "paulmck@linux.vnet.ibm.com" , Thomas Gleixner , Ingo Molnar , "H. Peter Anvin" On Thu, Apr 17, 2014 at 02:44:19PM +0100, Will Deacon wrote: > write{b,w,l,q}_relaxed are implemented by some architectures in order to > permit memory-mapped I/O accesses with weaker barrier semantics than the > non-relaxed variants. > > This patch adds dummy macros for the write accessors to x86, in the > same vein as the dummy definitions for the relaxed read accessors. > > Cc: Thomas Gleixner > Cc: Ingo Molnar > Cc: "H. Peter Anvin" > Signed-off-by: Will Deacon > --- > arch/x86/include/asm/io.h | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/arch/x86/include/asm/io.h b/arch/x86/include/asm/io.h > index b8237d8a1e0c..2ea07f5ec7b7 100644 > --- a/arch/x86/include/asm/io.h > +++ b/arch/x86/include/asm/io.h > @@ -74,6 +74,9 @@ build_mmio_write(__writel, "l", unsigned int, "r", ) > #define __raw_readw __readw > #define __raw_readl __readl > > +#define writeb_relaxed(v, a) __writeb(v, a) > +#define writew_relaxed(v, a) __writew(v, a) > +#define writel_relaxed(v, a) __writel(v, a) > #define __raw_writeb __writeb > #define __raw_writew __writew > #define __raw_writel __writel > @@ -86,6 +89,7 @@ build_mmio_read(readq, "q", unsigned long, "=r", :"memory") > build_mmio_write(writeq, "q", unsigned long, "r", :"memory") > > #define readq_relaxed(a) readq(a) > +#define writeq_relaxed(v, a) writeq(v, a) Actually, I should be using the regular (i.e. without the double underscore prefix) accessors for the relaxed variants, including the existing read flavours here. The proposed semantics are that the accessors are ordered with respect to each other, which necessitates a compiler barrier. Will