From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755921AbbBJDB6 (ORCPT ); Mon, 9 Feb 2015 22:01:58 -0500 Received: from e33.co.us.ibm.com ([32.97.110.151]:38228 "EHLO e33.co.us.ibm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750808AbbBJDB4 (ORCPT ); Mon, 9 Feb 2015 22:01:56 -0500 Date: Mon, 9 Feb 2015 19:01:46 -0800 From: "Paul E. McKenney" To: Frederic Weisbecker Cc: Rik van Riel , 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: <20150210030146.GM4166@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> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20150210014414.GB12802@lerouge> User-Agent: Mutt/1.5.21 (2010-09-15) X-TM-AS-MML: disable X-Content-Scanned: Fidelis XPS MAILER x-cbid: 15021003-0009-0000-0000-000008A39FD9 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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! Thanx, Paul