From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752556AbdJDPYf (ORCPT ); Wed, 4 Oct 2017 11:24:35 -0400 Received: from bombadil.infradead.org ([65.50.211.133]:49790 "EHLO bombadil.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752403AbdJDPYb (ORCPT ); Wed, 4 Oct 2017 11:24:31 -0400 Date: Wed, 4 Oct 2017 17:24:23 +0200 From: Peter Zijlstra To: "Paul E. McKenney" Cc: Steven Rostedt , pmladek@suse.com, sergey.senozhatsky@gmail.com, linux-kernel@vger.kernel.org, mingo@kernel.org, tglx@linutronix.de Subject: Re: [PATCH 3/3] early_printk: Add simple serialization to early_vprintk() Message-ID: <20171004152423.pifhlr72y2uzq63r@hirez.programming.kicks-ass.net> References: <20170928121823.430053219@infradead.org> <20170928122513.431444176@infradead.org> <20171003182422.025d0a67@gandalf.local.home> <20171004090830.p5kwzu6ex2wimh4v@hirez.programming.kicks-ass.net> <20171004090401.3a5123a6@gandalf.local.home> <20171004141745.GH3521@linux.vnet.ibm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20171004141745.GH3521@linux.vnet.ibm.com> User-Agent: NeoMutt/20170609 (1.8.3) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Oct 04, 2017 at 07:17:45AM -0700, Paul E. McKenney wrote: > If you use READ_ONCE(), then all architectures I know of enforce > full ordering for accesses to a single variable. (If you don't use > READ_ONCE(), then in theory Itanium can reorder reads.) Me, I would > argue for WRITE_ONCE() as well to prevent store tearing. Note that the stores are either cmpxchg() or smp_store_release() both of which imply a WRITE_ONCE().