From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Michael S. Tsirkin" Subject: Re: [PATCH v2 1/3] checkpatch.pl: add missing memory barriers Date: Sun, 10 Jan 2016 21:29:06 +0200 Message-ID: <20160110212531-mutt-send-email-mst__38608.2071731382$1452454237$gmane$org@redhat.com> References: <1452427000-4520-1-git-send-email-mst@redhat.com> <1452427000-4520-2-git-send-email-mst@redhat.com> <1452438425.7773.21.camel@perches.com> <1452439051.7773.27.camel@perches.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail6.bemta3.messagelabs.com ([195.245.230.39]) by lists.xen.org with esmtp (Exim 4.72) (envelope-from ) id 1aILg1-0002c6-Mo for xen-devel@lists.xenproject.org; Sun, 10 Jan 2016 19:29:21 +0000 Content-Disposition: inline In-Reply-To: <1452439051.7773.27.camel@perches.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Joe Perches Cc: linux-mips@linux-mips.org, linux-ia64@vger.kernel.org, linux-sh@vger.kernel.org, Peter Zijlstra , virtualization@lists.linux-foundation.org, "H. Peter Anvin" , sparclinux@vger.kernel.org, Ingo Molnar , linux-arch@vger.kernel.org, linux-s390@vger.kernel.org, Russell King - ARM Linux , Arnd Bergmann , x86@kernel.org, Tony Lindgren , xen-devel@lists.xenproject.org, Ingo Molnar , linux-xtensa@linux-xtensa.org, user-mode-linux-devel@lists.sourceforge.net, Stefano Stabellini , Andrey Konovalov , adi-buildroot-devel@lists.sourceforge.net, Andy Whitcroft , Thomas Gleixner , linux-metag@vger.kernel.org, linux-arm-kernel@lists.infradead.orgAndrew Cooper List-Id: xen-devel@lists.xenproject.org On Sun, Jan 10, 2016 at 07:17:31AM -0800, Joe Perches wrote: > On Sun, 2016-01-10 at 07:07 -0800, Joe Perches wrote: > > On Sun, 2016-01-10 at 13:56 +0200, Michael S. Tsirkin wrote: > > > SMP-only barriers were missing in checkpatch.pl > > > > > > Refactor code slightly to make adding more variants easier. > > [] > > > diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl > [] > > If I use a variable called $smp_barriers, I'd expect > > it to actually be the smp_barriers, not to have to > > prefix it with smp_ before using it. > > > > my $smp_barriers = qr{ > > smp_store_release| > > smp_load_acquire| > > smp_store_mb| > > smp_read_barrier_depends > > That's missing (?:barriers) too. My version has it but need to add ?: to avoid a capture group. > btw: shouldn't this also have > smp_mb__(?:before|after)_atomic > ? Good catch, included in the next version. -- MST