Re: [Xen-devel] [PATCH 03/17] vmx: nest: wrapper for control update

[Keir Fraser <keir.fraser@xxxxxxxxxxxxx>]

On 20/05/2010 10:46, "Qing He" <qing.he@xxxxxxxxx> wrote:

>> Shouldn't this use the new vmx_update_exception_bitmap()?
> 
> I left it unchanged because it's in vmcs.c. To me, vmx.c is on top of
> vmcs.c and I feel against inter-dependeny.

The pslit between vmx.c and vmcs.c is a bit arbitrary in some cases. But
it's cleaner and more maintainable to have everyone use a single interface
to the exception bitmap if possible.

 -- Keir

> Anyway this feeling is not strong. And I'm fine with using
> vmx_update_exception_bitmap here since inter-dependency is already
> the case.



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