[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH] nestedhvm: ASID emulation (cleanup)


  • To: Christoph Egger <Christoph.Egger@xxxxxxx>, <xen-devel@xxxxxxxxxxxxxxxxxxx>
  • From: Keir Fraser <keir@xxxxxxx>
  • Date: Wed, 13 Apr 2011 14:28:40 +0100
  • Cc:
  • Delivery-date: Wed, 13 Apr 2011 06:29:52 -0700
  • Domainkey-signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=sender:user-agent:date:subject:from:to:message-id:thread-topic :thread-index:in-reply-to:mime-version:content-type :content-transfer-encoding; b=e1V77d7ds3dH12WJVvllMRG3tdY05GLmzdqXlPKIy1ZYuOxxDZm6TUxj+kHu+FfyRi z91UXH0G/32ygw6t2tfOi11u5V7HTu5Iw2ue+2VMwDzsxJejzAfSmy7awaxBFhzvIHZI s1NqjoKe3MCU7rps/aT4QjnEQMsB6WJm3P7Ms=
  • List-id: Xen developer discussion <xen-devel.lists.xensource.com>
  • Thread-index: Acv53rODQnpaPbaimUKACL/z43x7KA==
  • Thread-topic: [Xen-devel] [PATCH] nestedhvm: ASID emulation (cleanup)

On 13/04/2011 14:20, "Christoph Egger" <Christoph.Egger@xxxxxxx> wrote:

> On Wednesday 13 April 2011 15:06:43 Keir Fraser wrote:
>> On 13/04/2011 11:35, "Christoph Egger" <Christoph.Egger@xxxxxxx> wrote:
>>> Cleanup for ASID emulation:
>>> - Use C99 integer types for asid numbers
>>> 
>>> - asid.c: consistently use 'v' instead of 'curr'
>> 
>> On what planet is that an improvement??
>> 
>> We use 'curr' as a convenient idiom to represent cached current. Whereas
>> 'v' is supposed to represent an arbitrary vcpu.
> 
> Oh, that's the difference. I thought 'curr' is just another random name for an
> arbitrary vcpu.
> 
> I will resend the patch w/o that hunk.

I already applied the two cleanups that I didn't mind, as two separate
changesets. And I've nacked your main patch so you can go take a look at
that. :-)

 -- Keir

>> 
>> I know it costs three extra characters per use, but it does make code
>> easier to understand.
>> 
>>  -- Keir
>> 
>>> - Introduce svm_invlpga() used in ASID emulation patch
>>> 
>>> Signed-off-by: Christoph Egger <Christoph.Egger@xxxxxxx>
>> 
>> _______________________________________________
>> Xen-devel mailing list
>> Xen-devel@xxxxxxxxxxxxxxxxxxx
>> http://lists.xensource.com/xen-devel
> 
> 



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.