* [PATCH] s390x/cpu_model: use official name for 8562
@ 2021-03-11 13:27 Cornelia Huck
2021-03-11 13:30 ` David Hildenbrand
` (2 more replies)
0 siblings, 3 replies; 6+ messages in thread
From: Cornelia Huck @ 2021-03-11 13:27 UTC (permalink / raw)
To: David Hildenbrand, Richard Henderson, Thomas Huth,
Christian Borntraeger, Halil Pasic
Cc: qemu-s390x, Cornelia Huck, qemu-devel
The single-frame z15 is called "z15 T02".
Signed-off-by: Cornelia Huck <cohuck@redhat.com>
---
target/s390x/cpu_models.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/target/s390x/cpu_models.c b/target/s390x/cpu_models.c
index dd474c5e9ad1..b59ed4135615 100644
--- a/target/s390x/cpu_models.c
+++ b/target/s390x/cpu_models.c
@@ -87,7 +87,7 @@ static S390CPUDef s390_cpu_defs[] = {
CPUDEF_INIT(0x3906, 14, 2, 47, 0x08000000U, "z14.2", "IBM z14 GA2"),
CPUDEF_INIT(0x3907, 14, 1, 47, 0x08000000U, "z14ZR1", "IBM z14 Model ZR1 GA1"),
CPUDEF_INIT(0x8561, 15, 1, 47, 0x08000000U, "gen15a", "IBM z15 GA1"),
- CPUDEF_INIT(0x8562, 15, 1, 47, 0x08000000U, "gen15b", "IBM 8562 GA1"),
+ CPUDEF_INIT(0x8562, 15, 1, 47, 0x08000000U, "gen15b", "IBM z15 T02 GA1"),
};
#define QEMU_MAX_CPU_TYPE 0x2964
--
2.26.2
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] s390x/cpu_model: use official name for 8562
2021-03-11 13:27 [PATCH] s390x/cpu_model: use official name for 8562 Cornelia Huck
@ 2021-03-11 13:30 ` David Hildenbrand
2021-03-12 13:29 ` Cornelia Huck
2021-03-12 13:48 ` Christian Borntraeger
2 siblings, 0 replies; 6+ messages in thread
From: David Hildenbrand @ 2021-03-11 13:30 UTC (permalink / raw)
To: Cornelia Huck, Richard Henderson, Thomas Huth,
Christian Borntraeger, Halil Pasic
Cc: qemu-s390x, qemu-devel
On 11.03.21 14:27, Cornelia Huck wrote:
> The single-frame z15 is called "z15 T02".
>
> Signed-off-by: Cornelia Huck <cohuck@redhat.com>
> ---
> target/s390x/cpu_models.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/target/s390x/cpu_models.c b/target/s390x/cpu_models.c
> index dd474c5e9ad1..b59ed4135615 100644
> --- a/target/s390x/cpu_models.c
> +++ b/target/s390x/cpu_models.c
> @@ -87,7 +87,7 @@ static S390CPUDef s390_cpu_defs[] = {
> CPUDEF_INIT(0x3906, 14, 2, 47, 0x08000000U, "z14.2", "IBM z14 GA2"),
> CPUDEF_INIT(0x3907, 14, 1, 47, 0x08000000U, "z14ZR1", "IBM z14 Model ZR1 GA1"),
> CPUDEF_INIT(0x8561, 15, 1, 47, 0x08000000U, "gen15a", "IBM z15 GA1"),
> - CPUDEF_INIT(0x8562, 15, 1, 47, 0x08000000U, "gen15b", "IBM 8562 GA1"),
> + CPUDEF_INIT(0x8562, 15, 1, 47, 0x08000000U, "gen15b", "IBM z15 T02 GA1"),
Reviewed-by: David Hildenbrand <david@redhat.com>
--
Thanks,
David / dhildenb
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] s390x/cpu_model: use official name for 8562
2021-03-11 13:27 [PATCH] s390x/cpu_model: use official name for 8562 Cornelia Huck
2021-03-11 13:30 ` David Hildenbrand
@ 2021-03-12 13:29 ` Cornelia Huck
2021-03-12 13:48 ` Christian Borntraeger
2 siblings, 0 replies; 6+ messages in thread
From: Cornelia Huck @ 2021-03-12 13:29 UTC (permalink / raw)
To: David Hildenbrand, Richard Henderson, Thomas Huth,
Christian Borntraeger, Halil Pasic
Cc: qemu-s390x, qemu-devel
On Thu, 11 Mar 2021 14:27:46 +0100
Cornelia Huck <cohuck@redhat.com> wrote:
> The single-frame z15 is called "z15 T02".
>
> Signed-off-by: Cornelia Huck <cohuck@redhat.com>
> ---
> target/s390x/cpu_models.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/target/s390x/cpu_models.c b/target/s390x/cpu_models.c
> index dd474c5e9ad1..b59ed4135615 100644
> --- a/target/s390x/cpu_models.c
> +++ b/target/s390x/cpu_models.c
> @@ -87,7 +87,7 @@ static S390CPUDef s390_cpu_defs[] = {
> CPUDEF_INIT(0x3906, 14, 2, 47, 0x08000000U, "z14.2", "IBM z14 GA2"),
> CPUDEF_INIT(0x3907, 14, 1, 47, 0x08000000U, "z14ZR1", "IBM z14 Model ZR1 GA1"),
> CPUDEF_INIT(0x8561, 15, 1, 47, 0x08000000U, "gen15a", "IBM z15 GA1"),
> - CPUDEF_INIT(0x8562, 15, 1, 47, 0x08000000U, "gen15b", "IBM 8562 GA1"),
> + CPUDEF_INIT(0x8562, 15, 1, 47, 0x08000000U, "gen15b", "IBM z15 T02 GA1"),
> };
>
> #define QEMU_MAX_CPU_TYPE 0x2964
Queued to s390-next.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] s390x/cpu_model: use official name for 8562
2021-03-11 13:27 [PATCH] s390x/cpu_model: use official name for 8562 Cornelia Huck
2021-03-11 13:30 ` David Hildenbrand
2021-03-12 13:29 ` Cornelia Huck
@ 2021-03-12 13:48 ` Christian Borntraeger
2021-03-12 13:55 ` Cornelia Huck
2 siblings, 1 reply; 6+ messages in thread
From: Christian Borntraeger @ 2021-03-12 13:48 UTC (permalink / raw)
To: Cornelia Huck, David Hildenbrand, Richard Henderson, Thomas Huth,
Halil Pasic
Cc: qemu-s390x, qemu-devel
On 11.03.21 14:27, Cornelia Huck wrote:
> The single-frame z15 is called "z15 T02".
>
> Signed-off-by: Cornelia Huck <cohuck@redhat.com>
Acked-by: Christian Borntraeger <borntraeger@de.ibm.com>
> ---
> target/s390x/cpu_models.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/target/s390x/cpu_models.c b/target/s390x/cpu_models.c
> index dd474c5e9ad1..b59ed4135615 100644
> --- a/target/s390x/cpu_models.c
> +++ b/target/s390x/cpu_models.c
> @@ -87,7 +87,7 @@ static S390CPUDef s390_cpu_defs[] = {
> CPUDEF_INIT(0x3906, 14, 2, 47, 0x08000000U, "z14.2", "IBM z14 GA2"),
> CPUDEF_INIT(0x3907, 14, 1, 47, 0x08000000U, "z14ZR1", "IBM z14 Model ZR1 GA1"),
> CPUDEF_INIT(0x8561, 15, 1, 47, 0x08000000U, "gen15a", "IBM z15 GA1"),
you could also change that formn "z15" to "z15 T01"
> - CPUDEF_INIT(0x8562, 15, 1, 47, 0x08000000U, "gen15b", "IBM 8562 GA1"),
> + CPUDEF_INIT(0x8562, 15, 1, 47, 0x08000000U, "gen15b", "IBM z15 T02 GA1"),
> };
>
> #define QEMU_MAX_CPU_TYPE 0x2964
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] s390x/cpu_model: use official name for 8562
2021-03-12 13:48 ` Christian Borntraeger
@ 2021-03-12 13:55 ` Cornelia Huck
2021-03-12 14:12 ` David Hildenbrand
0 siblings, 1 reply; 6+ messages in thread
From: Cornelia Huck @ 2021-03-12 13:55 UTC (permalink / raw)
To: Christian Borntraeger
Cc: Thomas Huth, David Hildenbrand, Richard Henderson, qemu-devel,
Halil Pasic, qemu-s390x
On Fri, 12 Mar 2021 14:48:17 +0100
Christian Borntraeger <borntraeger@de.ibm.com> wrote:
> On 11.03.21 14:27, Cornelia Huck wrote:
> > The single-frame z15 is called "z15 T02".
> >
> > Signed-off-by: Cornelia Huck <cohuck@redhat.com>
>
> Acked-by: Christian Borntraeger <borntraeger@de.ibm.com>
> > ---
> > target/s390x/cpu_models.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/target/s390x/cpu_models.c b/target/s390x/cpu_models.c
> > index dd474c5e9ad1..b59ed4135615 100644
> > --- a/target/s390x/cpu_models.c
> > +++ b/target/s390x/cpu_models.c
> > @@ -87,7 +87,7 @@ static S390CPUDef s390_cpu_defs[] = {
> > CPUDEF_INIT(0x3906, 14, 2, 47, 0x08000000U, "z14.2", "IBM z14 GA2"),
> > CPUDEF_INIT(0x3907, 14, 1, 47, 0x08000000U, "z14ZR1", "IBM z14 Model ZR1 GA1"),
> > CPUDEF_INIT(0x8561, 15, 1, 47, 0x08000000U, "gen15a", "IBM z15 GA1"),
>
> you could also change that formn "z15" to "z15 T01"
> > - CPUDEF_INIT(0x8562, 15, 1, 47, 0x08000000U, "gen15b", "IBM 8562 GA1"),
> > + CPUDEF_INIT(0x8562, 15, 1, 47, 0x08000000U, "gen15b", "IBM z15 T02 GA1"),
> > };
> >
> > #define QEMU_MAX_CPU_TYPE 0x2964
> >
>
I now have:
Author: Cornelia Huck <cohuck@redhat.com>
Date: Thu Mar 11 14:27:46 2021 +0100
s390x/cpu_model: use official name for 8562
The single-frame z15 is called "z15 T02" (and the multi-frame z15
"z15 T01").
Signed-off-by: Cornelia Huck <cohuck@redhat.com>
Reviewed-by: David Hildenbrand <david@redhat.com>
Acked-by: Christian Borntraeger <borntraeger@de.ibm.com>
Message-Id: <20210311132746.1777754-1-cohuck@redhat.com>
diff --git a/target/s390x/cpu_models.c b/target/s390x/cpu_models.c
index dd474c5e9ad1..050dcf2d42d2 100644
--- a/target/s390x/cpu_models.c
+++ b/target/s390x/cpu_models.c
@@ -86,8 +86,8 @@ static S390CPUDef s390_cpu_defs[] = {
CPUDEF_INIT(0x3906, 14, 1, 47, 0x08000000U, "z14", "IBM z14 GA1"),
CPUDEF_INIT(0x3906, 14, 2, 47, 0x08000000U, "z14.2", "IBM z14 GA2"),
CPUDEF_INIT(0x3907, 14, 1, 47, 0x08000000U, "z14ZR1", "IBM z14 Model ZR1 GA1"),
- CPUDEF_INIT(0x8561, 15, 1, 47, 0x08000000U, "gen15a", "IBM z15 GA1"),
- CPUDEF_INIT(0x8562, 15, 1, 47, 0x08000000U, "gen15b", "IBM 8562 GA1"),
+ CPUDEF_INIT(0x8561, 15, 1, 47, 0x08000000U, "gen15a", "IBM z15 T01 GA1"),
+ CPUDEF_INIT(0x8562, 15, 1, 47, 0x08000000U, "gen15b", "IBM z15 T02 GA1"),
};
#define QEMU_MAX_CPU_TYPE 0x2964
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] s390x/cpu_model: use official name for 8562
2021-03-12 13:55 ` Cornelia Huck
@ 2021-03-12 14:12 ` David Hildenbrand
0 siblings, 0 replies; 6+ messages in thread
From: David Hildenbrand @ 2021-03-12 14:12 UTC (permalink / raw)
To: Cornelia Huck, Christian Borntraeger
Cc: Halil Pasic, Thomas Huth, Richard Henderson, qemu-s390x, qemu-devel
On 12.03.21 14:55, Cornelia Huck wrote:
> On Fri, 12 Mar 2021 14:48:17 +0100
> Christian Borntraeger <borntraeger@de.ibm.com> wrote:
>
>> On 11.03.21 14:27, Cornelia Huck wrote:
>>> The single-frame z15 is called "z15 T02".
>>>
>>> Signed-off-by: Cornelia Huck <cohuck@redhat.com>
>>
>> Acked-by: Christian Borntraeger <borntraeger@de.ibm.com>
>>> ---
>>> target/s390x/cpu_models.c | 2 +-
>>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/target/s390x/cpu_models.c b/target/s390x/cpu_models.c
>>> index dd474c5e9ad1..b59ed4135615 100644
>>> --- a/target/s390x/cpu_models.c
>>> +++ b/target/s390x/cpu_models.c
>>> @@ -87,7 +87,7 @@ static S390CPUDef s390_cpu_defs[] = {
>>> CPUDEF_INIT(0x3906, 14, 2, 47, 0x08000000U, "z14.2", "IBM z14 GA2"),
>>> CPUDEF_INIT(0x3907, 14, 1, 47, 0x08000000U, "z14ZR1", "IBM z14 Model ZR1 GA1"),
>>> CPUDEF_INIT(0x8561, 15, 1, 47, 0x08000000U, "gen15a", "IBM z15 GA1"),
>>
>> you could also change that formn "z15" to "z15 T01"
>>> - CPUDEF_INIT(0x8562, 15, 1, 47, 0x08000000U, "gen15b", "IBM 8562 GA1"),
>>> + CPUDEF_INIT(0x8562, 15, 1, 47, 0x08000000U, "gen15b", "IBM z15 T02 GA1"),
>>> };
>>>
>>> #define QEMU_MAX_CPU_TYPE 0x2964
>>>
>>
>
> I now have:
>
> Author: Cornelia Huck <cohuck@redhat.com>
> Date: Thu Mar 11 14:27:46 2021 +0100
>
> s390x/cpu_model: use official name for 8562
>
> The single-frame z15 is called "z15 T02" (and the multi-frame z15
> "z15 T01").
>
> Signed-off-by: Cornelia Huck <cohuck@redhat.com>
> Reviewed-by: David Hildenbrand <david@redhat.com>
> Acked-by: Christian Borntraeger <borntraeger@de.ibm.com>
> Message-Id: <20210311132746.1777754-1-cohuck@redhat.com>
>
> diff --git a/target/s390x/cpu_models.c b/target/s390x/cpu_models.c
> index dd474c5e9ad1..050dcf2d42d2 100644
> --- a/target/s390x/cpu_models.c
> +++ b/target/s390x/cpu_models.c
> @@ -86,8 +86,8 @@ static S390CPUDef s390_cpu_defs[] = {
> CPUDEF_INIT(0x3906, 14, 1, 47, 0x08000000U, "z14", "IBM z14 GA1"),
> CPUDEF_INIT(0x3906, 14, 2, 47, 0x08000000U, "z14.2", "IBM z14 GA2"),
> CPUDEF_INIT(0x3907, 14, 1, 47, 0x08000000U, "z14ZR1", "IBM z14 Model ZR1 GA1"),
> - CPUDEF_INIT(0x8561, 15, 1, 47, 0x08000000U, "gen15a", "IBM z15 GA1"),
> - CPUDEF_INIT(0x8562, 15, 1, 47, 0x08000000U, "gen15b", "IBM 8562 GA1"),
> + CPUDEF_INIT(0x8561, 15, 1, 47, 0x08000000U, "gen15a", "IBM z15 T01 GA1"),
> + CPUDEF_INIT(0x8562, 15, 1, 47, 0x08000000U, "gen15b", "IBM z15 T02 GA1"),
> };
>
> #define QEMU_MAX_CPU_TYPE 0x2964
>
LGTM
--
Thanks,
David / dhildenb
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2021-03-12 14:46 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-03-11 13:27 [PATCH] s390x/cpu_model: use official name for 8562 Cornelia Huck
2021-03-11 13:30 ` David Hildenbrand
2021-03-12 13:29 ` Cornelia Huck
2021-03-12 13:48 ` Christian Borntraeger
2021-03-12 13:55 ` Cornelia Huck
2021-03-12 14:12 ` David Hildenbrand
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.