On Fri, Feb 9, 2018 at 3:28 AM, George Dunlap <dunl...@umich.edu> wrote:
> On Thu, Feb 8, 2018 at 3:25 PM, Alexandru Isaila
> <aisa...@bitdefender.com> wrote:
>> This commit separates the svm caps from the vmx caps.
> I can see how the patch relates to the description here, but it's not
> immediately clear how it relates to the title.
> A good "template" to start with for any commit message is:
> 1. What the current situation is
> 2. Why that's a problem
> 3. How this patch fixes it
> The only time you should omit any of these is if it's completely obvious.
> In this case, it looks like the answers would look like:
> 1. Only a subset of the monitor features are available on AMD, but all
> capabilities are passed regardless of the processor architecture.
> 2. This means that the majority of functionality advertized in
> 'capabilities' is actually broken when running under AMD.
It is not broken under AMD. What is being reported there is that it is
no monitor option is supported - ie. capabilities = 0. The whole thing
is gated on cpu_has_vmx.
Xen-devel mailing list