Re: [Xen-devel] [PATCH v2 1/4] asm-x86/monitor: Enable svm monitor events

[Tamas K Lengyel <tamas@xxxxxxxxxxxxx>]

On Fri, Feb 9, 2018 at 3:28 AM, George Dunlap <dunlapg@xxxxxxxxx> wrote:
> On Thu, Feb 8, 2018 at 3:25 PM, Alexandru Isaila
> <aisaila@xxxxxxxxxxxxxxx> 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.

Tamas

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel