Re: [PATCH v3 2/2] Mark Icelake-Client CPU models deprecated

[Robert Hoo <robert.hu@linux.intel.com>]

On Thu, 2020-09-17 at 14:01 -0400, Eduardo Habkost wrote:
> On Wed, Sep 16, 2020 at 04:37:14PM +0800, Robert Hoo wrote:
> > Going to obsolete Icelake-Client CPU models in the future.
> > 
> > Signed-off-by: Robert Hoo <robert.hu@linux.intel.com>
> > ---
> > Change log
> > v3:
> > Obsolete in v5.2 --> v5.3.
> > 
> >  target/i386/cpu.c | 10 +++++++++-
> >  1 file changed, 9 insertions(+), 1 deletion(-)
> > 
> > diff --git a/target/i386/cpu.c b/target/i386/cpu.c
> > index 9cb82b7..15c1c00 100644
> > --- a/target/i386/cpu.c
> > +++ b/target/i386/cpu.c
> > @@ -3467,7 +3467,12 @@ static X86CPUDefinition builtin_x86_defs[] =
> > {
> >          .xlevel = 0x80000008,
> >          .model_id = "Intel Core Processor (Icelake)",
> >          .versions = (X86CPUVersionDefinition[]) {
> > -            { .version = 1 },
> > +            {
> > +                .version = 1,
> > +                .deprecated = true,
> > +                .note = "Deprecated. Will be obsoleted in v5.3.
> > Please use "
> > +                        "'Icelake-Server-v1' CPU model",
> 
> What's the difference between "deprecated" and "obsoleted"?
> 
Forgive my non-native understanding on English word:-D
Here is my understanding:
'Deprecate' is to express strong disapproval on the usage; but, can
still be used if user insists.
'Obsolete' means not usable anymore.

You can feel free to reword the note words.
Perhaps substitute 'removed' for 'obsolete' will be better.

> > +            },
> >              {
> >                  .version = 2,
> >                  .note = "no TSX",
> > @@ -3477,6 +3482,9 @@ static X86CPUDefinition builtin_x86_defs[] =
> > {
> >                      { "rtm", "off" },
> >                      { /* end of list */ }
> >                  },
> > +                .deprecated = true,
> > +                .note = "Deprecated. Will be obsoleted in v5.3.
> > Please use "
> > +                        "'Icelake-Server-v2' CPU model",
> >              },
> >              { /* end of list */ }
> >          }
> > -- 
> > 1.8.3.1
> > 
> 
>