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

Re: [Xen-devel] [PATCH] Simplify paging_invlpg when flush is not required.


  • To: "Tian, Kevin" <kevin.tian@xxxxxxxxx>, <xen-devel@xxxxxxxxxxxxxxxxxxx>
  • From: Keir Fraser <Keir.Fraser@xxxxxxxxxxxx>
  • Date: Sun, 03 Feb 2008 08:55:39 +0000
  • Delivery-date: Sun, 03 Feb 2008 00:55:35 -0800
  • List-id: Xen developer discussion <xen-devel.lists.xensource.com>
  • Thread-index: AchmKe/O98M9ImhQROaw3KDvTG8qogAGJyio
  • Thread-topic: [Xen-devel] [PATCH] Simplify paging_invlpg when flush is not required.

Is there a significant advantage to doing this? One other comment is that I
don't like extra boolean 0/1 arguments to functions. I'd rather have
something like paging_invlpg() and paging_invlpg_noflush() and only have the
boolean argument to paging.mode->invlpg().

 -- Keir

On 3/2/08 05:59, "Tian, Kevin" <kevin.tian@xxxxxxxxx> wrote:

> Simplify paging_invlpg when flush is not required.
> 
> New 'flush' parameter is added to paging_invlpg, to allow
> caller assigning whether flush check is required. It's
> wasteful to always validate shadow linear mapping if caller
> doesn't check return value at all.
> 
> Signed-off-by Kevin Tian <kevin.tian@xxxxxxxxx>
> 
> Thanks,
> Kevin
> _______________________________________________
> 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®.