[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v3 2/2] Mark Icelake-Client CPU models deprecated
From: |
Robert Hoo |
Subject: |
Re: [PATCH v3 2/2] Mark Icelake-Client CPU models deprecated |
Date: |
Fri, 18 Sep 2020 10:18:56 +0800 |
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
> >
>
>
Re: [PATCH v3 1/2] Introduce (x86) CPU model deprecation API, Eduardo Habkost, 2020/09/17