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@xxxxxxxxxx > >>>> 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 -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html