From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1761365AbbBJDNm (ORCPT ); Mon, 9 Feb 2015 22:13:42 -0500 Received: from e38.co.us.ibm.com ([32.97.110.159]:55664 "EHLO e38.co.us.ibm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753888AbbBJDNl (ORCPT ); Mon, 9 Feb 2015 22:13:41 -0500 Date: Mon, 9 Feb 2015 19:13:36 -0800 From: "Paul E. McKenney" To: Rik van Riel Cc: Frederic Weisbecker , Will Deacon , "linux-kernel@vger.kernel.org" , Catalin Marinas , "kvm@vger.kernel.org" , "mtosatti@redhat.com" , "borntraeger@de.ibm.com" , "mingo@kernel.org" , "oleg@redhat.com" , "lcapitulino@redhat.com" , "pbonzini@redhat.com" Subject: Re: [PATCH -v3 0/6] rcu,nohz,kvm: use RCU extended quiescent state when running KVM guest Message-ID: <20150210031336.GO4166@linux.vnet.ibm.com> Reply-To: paulmck@linux.vnet.ibm.com References: <1423497884-21615-1-git-send-email-riel@redhat.com> <20150210011540.GA17955@arm.com> <54D95D73.3040805@redhat.com> <20150210014414.GB12802@lerouge> <20150210030146.GM4166@linux.vnet.ibm.com> <54D974EC.8080003@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <54D974EC.8080003@redhat.com> User-Agent: Mutt/1.5.21 (2010-09-15) X-TM-AS-MML: disable X-Content-Scanned: Fidelis XPS MAILER x-cbid: 15021003-0029-0000-0000-000007CDAB68 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Feb 09, 2015 at 10:03:08PM -0500, Rik van Riel wrote: > -----BEGIN PGP SIGNED MESSAGE----- > Hash: SHA1 > > On 02/09/2015 10:01 PM, Paul E. McKenney wrote: > > On Tue, Feb 10, 2015 at 02:44:17AM +0100, Frederic Weisbecker > > wrote: > >> On Mon, Feb 09, 2015 at 08:22:59PM -0500, Rik van Riel wrote: > >>> -----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 > >>> > >>> On 02/09/2015 08:15 PM, Will Deacon wrote: > >>>> Hi Rik, > >>>> > >>>> On Mon, Feb 09, 2015 at 04:04:38PM +0000, riel@redhat.com > >>>> wrote: > >>>>> Apologies to Catalin and Will for not fixing up ARM. I am > >>>>> not familiar with ARM assembly, and not sure how to pass a > >>>>> constant argument to a function from assembly code on ARM > >>>>> :) > >>>> > >>>> It's a bit of a faff getting enum values into asm -- we > >>>> actually have to duplicate the definitions using #defines to > >>>> get at the constants. Perhaps it would be cleaner to leave > >>>> context_tracking_user_{enter,exit} intact as C wrappers > >>>> around context_tracking_{enter,exit} passing the appropriate > >>>> constant? That way we don't actually need to change the arch > >>>> code at all. > >>> > >>> If Paul and Frederic have no objections, I would be happy to do > >>> that. > >>> > >>> Paul, Frederic? > >> > >> Sure, that's fine by me. > > > > And if it is fine by Frederic, it is fine by me! > > I'll send a new series tomorrow that addresses Will's concern, > as well as Paulo's latest suggestions. Very good, thank you! Thanx, Paul