From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Paul E. McKenney" Subject: Re: [PATCH RFC memory-model 0/6] LKMM updates Date: Wed, 9 Jan 2019 20:20:27 -0800 Message-ID: <20190110042027.GG1215@linux.ibm.com> References: <20190109210706.GA27268@linux.ibm.com> <20190109231852.GA4494@andrea> <20190109234043.GD1215@linux.ibm.com> <20190110003952.GA4354@andrea> Reply-To: paulmck@linux.ibm.com Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20190110003952.GA4354@andrea> Sender: linux-kernel-owner@vger.kernel.org To: Andrea Parri Cc: linux-kernel@vger.kernel.org, linux-arch@vger.kernel.org, mingo@kernel.org, stern@rowland.harvard.edu, parri.andrea@gmail.com, will.deacon@arm.com, peterz@infradead.org, boqun.feng@gmail.com, npiggin@gmail.com, dhowells@redhat.com, j.alglave@ucl.ac.uk, luc.maranget@inria.fr, akiyks@gmail.com, willy@infradead.org List-Id: linux-arch.vger.kernel.org On Thu, Jan 10, 2019 at 01:39:52AM +0100, Andrea Parri wrote: > On Wed, Jan 09, 2019 at 03:40:43PM -0800, Paul E. McKenney wrote: > > On Thu, Jan 10, 2019 at 12:18:53AM +0100, Andrea Parri wrote: > > > On Wed, Jan 09, 2019 at 01:07:06PM -0800, Paul E. McKenney wrote: > > > > Hello! > > > > > > > > This series contains updates for the Linux-kernel memory model: > > > > > > > > 1-3. Add SRCU support, courtesy of Alan Stern. > > > > > > > > 4. Update README for adding of SRCU support. > > > > > > > > 5. Update memory-barriers.txt on enforcing heavy ordering for > > > > port-I/O accesses, courtesy of Will Deacon. This one needs > > > > an ack, preferably by someone from Intel. Matthew Wilcox > > > > posted some feedback from an Intel manual here, which might > > > > be considered to be a close substitute, but... ;-) > > > > > > > > http://lkml.kernel.org/r/20181127192234.GF10377@bombadil.infradead.org > > > > > > > > 6. Update Documentation/explanation.txt to include SRCU support, > > > > courtesy of Alan Stern. > > > > > > > > 7. Dynamically check SRCU lock-to-unlock matching, courtesy of > > > > Luc Maranget. This needs an ack. > > > > > > It seems that > > > > > > 1b52d0186177 ("tools/memory-model: Model smp_mb__after_unlock_lock()") > > > > > > from linux-rcu/dev got lost; this also needs an ack (probably yours! ;D, > > > considered that, IIRC, you introduced the primitive and RCU is currently > > > its only user.) > > > > That commit is in -tip: > > > > 4607abbcf464 ("tools/memory-model: Model smp_mb__after_unlock_lock()") > > > > So it has already left my -rcu tree. ;-) > > Oh, you're right: now I see the commit (e.g., with "git show"), but I > don't see the corresponding changes applied to the tree. > > https://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git/commit/?h=locking/core&id=4607abbcf464ea2be14da444215d05c73025cf6e > https://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git/tree/tools/memory-model/linux-kernel.bell?h=locking/core > > Is this expected? Are you asking why it is in -tip but not in mainline? I am not sure, but given that the merge window was over the holiday season and that the length of the merge window proved to be shorter than many people expected it to be, I am not too surprised. ;-) Thanx, Paul From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx0a-001b2d01.pphosted.com ([148.163.156.1]:39092 "EHLO mx0a-001b2d01.pphosted.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726877AbfAJEUh (ORCPT ); Wed, 9 Jan 2019 23:20:37 -0500 Received: from pps.filterd (m0098399.ppops.net [127.0.0.1]) by mx0a-001b2d01.pphosted.com (8.16.0.22/8.16.0.22) with SMTP id x0A4J0km093532 for ; Wed, 9 Jan 2019 23:20:36 -0500 Received: from e14.ny.us.ibm.com (e14.ny.us.ibm.com [129.33.205.204]) by mx0a-001b2d01.pphosted.com with ESMTP id 2pwv98wgpf-1 (version=TLSv1.2 cipher=AES256-GCM-SHA384 bits=256 verify=NOT) for ; Wed, 09 Jan 2019 23:20:36 -0500 Received: from localhost by e14.ny.us.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Thu, 10 Jan 2019 04:20:34 -0000 Date: Wed, 9 Jan 2019 20:20:27 -0800 From: "Paul E. McKenney" Subject: Re: [PATCH RFC memory-model 0/6] LKMM updates Reply-To: paulmck@linux.ibm.com References: <20190109210706.GA27268@linux.ibm.com> <20190109231852.GA4494@andrea> <20190109234043.GD1215@linux.ibm.com> <20190110003952.GA4354@andrea> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20190110003952.GA4354@andrea> Message-ID: <20190110042027.GG1215@linux.ibm.com> Sender: linux-arch-owner@vger.kernel.org List-ID: To: Andrea Parri Cc: linux-kernel@vger.kernel.org, linux-arch@vger.kernel.org, mingo@kernel.org, stern@rowland.harvard.edu, parri.andrea@gmail.com, will.deacon@arm.com, peterz@infradead.org, boqun.feng@gmail.com, npiggin@gmail.com, dhowells@redhat.com, j.alglave@ucl.ac.uk, luc.maranget@inria.fr, akiyks@gmail.com, willy@infradead.org Message-ID: <20190110042027.V9H1uLfuT7733aP6x3W29Oilsm09VaTQOX1kLha9FOc@z> On Thu, Jan 10, 2019 at 01:39:52AM +0100, Andrea Parri wrote: > On Wed, Jan 09, 2019 at 03:40:43PM -0800, Paul E. McKenney wrote: > > On Thu, Jan 10, 2019 at 12:18:53AM +0100, Andrea Parri wrote: > > > On Wed, Jan 09, 2019 at 01:07:06PM -0800, Paul E. McKenney wrote: > > > > Hello! > > > > > > > > This series contains updates for the Linux-kernel memory model: > > > > > > > > 1-3. Add SRCU support, courtesy of Alan Stern. > > > > > > > > 4. Update README for adding of SRCU support. > > > > > > > > 5. Update memory-barriers.txt on enforcing heavy ordering for > > > > port-I/O accesses, courtesy of Will Deacon. This one needs > > > > an ack, preferably by someone from Intel. Matthew Wilcox > > > > posted some feedback from an Intel manual here, which might > > > > be considered to be a close substitute, but... ;-) > > > > > > > > http://lkml.kernel.org/r/20181127192234.GF10377@bombadil.infradead.org > > > > > > > > 6. Update Documentation/explanation.txt to include SRCU support, > > > > courtesy of Alan Stern. > > > > > > > > 7. Dynamically check SRCU lock-to-unlock matching, courtesy of > > > > Luc Maranget. This needs an ack. > > > > > > It seems that > > > > > > 1b52d0186177 ("tools/memory-model: Model smp_mb__after_unlock_lock()") > > > > > > from linux-rcu/dev got lost; this also needs an ack (probably yours! ;D, > > > considered that, IIRC, you introduced the primitive and RCU is currently > > > its only user.) > > > > That commit is in -tip: > > > > 4607abbcf464 ("tools/memory-model: Model smp_mb__after_unlock_lock()") > > > > So it has already left my -rcu tree. ;-) > > Oh, you're right: now I see the commit (e.g., with "git show"), but I > don't see the corresponding changes applied to the tree. > > https://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git/commit/?h=locking/core&id=4607abbcf464ea2be14da444215d05c73025cf6e > https://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git/tree/tools/memory-model/linux-kernel.bell?h=locking/core > > Is this expected? Are you asking why it is in -tip but not in mainline? I am not sure, but given that the merge window was over the holiday season and that the length of the merge window proved to be shorter than many people expected it to be, I am not too surprised. ;-) Thanx, Paul