This patch changes hap parameter from boolean to integer. So users can disable and enable hap using "hap=0" and "hap=1". It also prints out nested paging message under SVM. Signed-off-by: Wei Huang <wei.huang2@amd.com> _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel
Huang2, Wei
2007-Sep-13 18:52 UTC
RE: [Xen-devel] Hardware Assisted Paging Param and Message
I just noticed that there is a similar change in start_svm() for printing out nested paging information. So ignore the change to svm.c. But opt_hap_enabled fix to paging.c is still needed. -Wei ________________________________ From: xen-devel-bounces@lists.xensource.com [mailto:xen-devel-bounces@lists.xensource.com] On Behalf Of Huang2, Wei Sent: Thursday, September 13, 2007 1:27 PM To: xen-devel@lists.xensource.com Subject: [Xen-devel] Hardware Assisted Paging Param and Message This patch changes hap parameter from boolean to integer. So users can disable and enable hap using "hap=0" and "hap=1". It also prints out nested paging message under SVM. Signed-off-by: Wei Huang <wei.huang2@amd.com> _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel
Keir Fraser
2007-Sep-14 06:43 UTC
Re: [Xen-devel] Hardware Assisted Paging Param and Message
On 13/9/07 19:27, "Huang2, Wei" <Wei.Huang2@amd.com> wrote:> This patch changes hap parameter from boolean to integer. So users can disable > and enable hap using "hap=0" and "hap=1".Why? You already have [no-]hap[={no,off,yes,on}]. So if your interest is in making hap default to on with manual negative override, we already have that ability (no-hap, hap=off, hap=no). None of this patch is required. -- Keir _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel
Ian Pratt
2007-Sep-14 13:32 UTC
RE: [Xen-devel] Hardware Assisted Paging Param and Message
> This patch changes hap parameter from boolean to > integer. So users can disable and enable hap using "hap=0" > and "hap=1". > > Why? You already have [no-]hap[={no,off,yes,on}]. So if your > interest is in making hap default to on with manual negative > override, we already have that ability (no-hap, hap=off, hap=no).Let''s just make ''0'' and ''1'' part of the bool set. Thanks, Ian _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel