Re: [PATCH RESEND 4/4] target/i386: Add notes for versioned CPU models

[Xiaoyao Li]

On 12/2/2019 2:32 PM, Tao Xu wrote:
Add which features are added or removed in this version. Remove the
changed model-id in versioned CPU models.
Signed-off-by: Tao Xu <tao3...@intel.com>
---
  target/i386/cpu.c | 50 +++++++++++++++++++++++------------------------
  1 file changed, 25 insertions(+), 25 deletions(-)

diff --git a/target/i386/cpu.c b/target/i386/cpu.c
index 7b3bd6d4db..c82fbfd02e 100644
--- a/target/i386/cpu.c
+++ b/target/i386/cpu.c

[...]

@@ -3141,6 +3133,7 @@ static X86CPUDefinition builtin_x86_defs[] = {
          .versions = (X86CPUVersionDefinition[]) {
              { .version = 1 },
              { .version = 2,
+              .note = "ARCH_CAPABILITIES",

Here ARCH_CAPABILITIES doesn't tell what bits in 
MSR_IA32_ARCH_CAPABILITIES this version has, which makes it meaningless.

Maybe
        .note = "ARCH_CAPABLITIES(rdctl-no, ibrs-all, skip-l1dfl-vmentry, 
mds-no)",

is better?

                .props = (PropValue[]) {
                    { "arch-capabilities", "on" },
                    { "rdctl-no", "on" },
@@ -3152,6 +3145,7 @@ static X86CPUDefinition builtin_x86_defs[] = {
              },
              { .version = 3,
                .alias = "Cascadelake-Server-noTSX",
+              .note = "no TSX",
                .props = (PropValue[]) {
                    { "hle", "off" },
                    { "rtm", "off" },