From: Frederic Weisbecker <fweisbec@gmail.com>
To: Rik van Riel <riel@redhat.com>
Cc: Will Deacon <will.deacon@arm.com>,
"paulmck@linux.vnet.ibm.com" <paulmck@linux.vnet.ibm.com>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
Catalin Marinas <Catalin.Marinas@arm.com>,
"kvm@vger.kernel.org" <kvm@vger.kernel.org>,
"mtosatti@redhat.com" <mtosatti@redhat.com>,
"borntraeger@de.ibm.com" <borntraeger@de.ibm.com>,
"mingo@kernel.org" <mingo@kernel.org>,
"oleg@redhat.com" <oleg@redhat.com>,
"lcapitulino@redhat.com" <lcapitulino@redhat.com>,
"pbonzini@redhat.com" <pbonzini@redhat.com>
Subject: Re: [PATCH -v3 0/6] rcu,nohz,kvm: use RCU extended quiescent state when running KVM guest
Date: Tue, 10 Feb 2015 02:44:17 +0100 [thread overview]
Message-ID: <20150210014414.GB12802@lerouge> (raw)
In-Reply-To: <54D95D73.3040805@redhat.com>
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.
Thanks.
next prev parent reply other threads:[~2015-02-10 1:44 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-02-09 16:04 [PATCH -v3 0/6] rcu,nohz,kvm: use RCU extended quiescent state when running KVM guest riel
2015-02-09 16:04 ` [PATCH 1/6] rcu,nohz: add state parameter to context_tracking_user_enter/exit riel
2015-02-09 16:04 ` [PATCH 2/6] rcu,nohz: rename context_tracking_enter & _exit riel
2015-02-09 16:04 ` [PATCH 3/6] rcu,nohz: run vtime_user_enter/exit only when state == IN_USER riel
2015-02-09 16:04 ` [PATCH 4/6] nohz,kvm: export context_tracking_user_enter/exit riel
2015-02-09 17:05 ` Paolo Bonzini
2015-02-09 20:02 ` Rik van Riel
2015-02-09 16:04 ` [PATCH 5/6] kvm,rcu,nohz: use RCU extended quiescent state when running KVM guest riel
2015-02-09 16:04 ` [PATCH 6/6] nohz: add stub context_tracking_is_enabled riel
2015-02-09 17:07 ` Paolo Bonzini
2015-02-10 1:15 ` [PATCH -v3 0/6] rcu,nohz,kvm: use RCU extended quiescent state when running KVM guest Will Deacon
2015-02-10 1:22 ` Rik van Riel
2015-02-10 1:44 ` Frederic Weisbecker [this message]
2015-02-10 3:01 ` Paul E. McKenney
2015-02-10 3:03 ` Rik van Riel
2015-02-10 3:13 ` Paul E. McKenney
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20150210014414.GB12802@lerouge \
--to=fweisbec@gmail.com \
--cc=Catalin.Marinas@arm.com \
--cc=borntraeger@de.ibm.com \
--cc=kvm@vger.kernel.org \
--cc=lcapitulino@redhat.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@kernel.org \
--cc=mtosatti@redhat.com \
--cc=oleg@redhat.com \
--cc=paulmck@linux.vnet.ibm.com \
--cc=pbonzini@redhat.com \
--cc=riel@redhat.com \
--cc=will.deacon@arm.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox