[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" <xen-devel@xxxxxxxxxxxxxxxxxxx>
  • From: Keir Fraser <keir.xen@xxxxxxxxx>
  • Date: Wed, 13 Apr 2011 14:06:43 +0100
  • Cc:
  • Delivery-date: Wed, 13 Apr 2011 06:09:01 -0700
  • Domainkey-signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=user-agent:date:subject:from:to:message-id:thread-topic :thread-index:in-reply-to:mime-version:content-type :content-transfer-encoding; b=oF2DpdlSO5bQ+HPXGZLxF2HfJ2fS+OQDPUlWo0jZFUbjC5JBKvmOCODzcXJt4oNFfp ZQaJ2twCILPR/3hgYNm4wlXfpsYQuIlWuW8Y5lbzFQnmyPhnRuqhr5NEPHL4lLnjAoU8 rcjbA51/nTtCPvx9tXLvjBzlSW59PY9QJPR68=
  • List-id: Xen developer discussion <xen-devel.lists.xensource.com>
  • Thread-index: Acv526KFHK77Cf/cRE+0E33kbu9p2A==
  • Thread-topic: [Xen-devel] [PATCH] nestedhvm: ASID emulation (cleanup)

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.

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


 


Rackspace

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