* [PATCH v2] s390x/tcg: clear local interrupts on reset normal
@ 2019-12-06 13:54 Cornelia Huck
2019-12-06 14:26 ` Philippe Mathieu-Daudé
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Cornelia Huck @ 2019-12-06 13:54 UTC (permalink / raw)
To: Richard Henderson, David Hildenbrand
Cc: qemu-s390x, Cornelia Huck, qemu-devel
We neglected to clean up pending interrupts and emergency signals;
fix that.
Signed-off-by: Cornelia Huck <cohuck@redhat.com>
---
v1->v2:
- rebased on top of my s390-next branch; we can now move the fields
to be reset instead of clearing them manually
Further cleanup possible in a follow-up patch.
---
target/s390x/cpu.h | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/target/s390x/cpu.h b/target/s390x/cpu.h
index 7f5fa1d35b73..e195e5c7c8bb 100644
--- a/target/s390x/cpu.h
+++ b/target/s390x/cpu.h
@@ -98,10 +98,6 @@ struct CPUS390XState {
uint64_t cregs[16]; /* control registers */
- int pending_int;
- uint16_t external_call_addr;
- DECLARE_BITMAP(emergency_signals, S390_MAX_CPUS);
-
uint64_t ckc;
uint64_t cputm;
uint32_t todpr;
@@ -117,6 +113,10 @@ struct CPUS390XState {
struct {} start_normal_reset_fields;
uint8_t riccb[64]; /* runtime instrumentation control */
+ int pending_int;
+ uint16_t external_call_addr;
+ DECLARE_BITMAP(emergency_signals, S390_MAX_CPUS);
+
/* Fields up to this point are cleared by a CPU reset */
struct {} end_reset_fields;
--
2.21.0
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH v2] s390x/tcg: clear local interrupts on reset normal
2019-12-06 13:54 [PATCH v2] s390x/tcg: clear local interrupts on reset normal Cornelia Huck
@ 2019-12-06 14:26 ` Philippe Mathieu-Daudé
2019-12-06 14:27 ` David Hildenbrand
2019-12-10 10:11 ` Cornelia Huck
2 siblings, 0 replies; 4+ messages in thread
From: Philippe Mathieu-Daudé @ 2019-12-06 14:26 UTC (permalink / raw)
To: Cornelia Huck, Richard Henderson, David Hildenbrand
Cc: qemu-s390x, qemu-devel
On 12/6/19 2:54 PM, Cornelia Huck wrote:
> We neglected to clean up pending interrupts and emergency signals;
> fix that.
>
> Signed-off-by: Cornelia Huck <cohuck@redhat.com>
> ---
>
> v1->v2:
> - rebased on top of my s390-next branch; we can now move the fields
> to be reset instead of clearing them manually
Yep, much cleaner than v1 :)
> Further cleanup possible in a follow-up patch.
>
> ---
> target/s390x/cpu.h | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/target/s390x/cpu.h b/target/s390x/cpu.h
> index 7f5fa1d35b73..e195e5c7c8bb 100644
> --- a/target/s390x/cpu.h
> +++ b/target/s390x/cpu.h
> @@ -98,10 +98,6 @@ struct CPUS390XState {
>
> uint64_t cregs[16]; /* control registers */
>
> - int pending_int;
> - uint16_t external_call_addr;
> - DECLARE_BITMAP(emergency_signals, S390_MAX_CPUS);
> -
> uint64_t ckc;
> uint64_t cputm;
> uint32_t todpr;
> @@ -117,6 +113,10 @@ struct CPUS390XState {
> struct {} start_normal_reset_fields;
> uint8_t riccb[64]; /* runtime instrumentation control */
>
> + int pending_int;
> + uint16_t external_call_addr;
> + DECLARE_BITMAP(emergency_signals, S390_MAX_CPUS);
> +
> /* Fields up to this point are cleared by a CPU reset */
> struct {} end_reset_fields;
>
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v2] s390x/tcg: clear local interrupts on reset normal
2019-12-06 13:54 [PATCH v2] s390x/tcg: clear local interrupts on reset normal Cornelia Huck
2019-12-06 14:26 ` Philippe Mathieu-Daudé
@ 2019-12-06 14:27 ` David Hildenbrand
2019-12-10 10:11 ` Cornelia Huck
2 siblings, 0 replies; 4+ messages in thread
From: David Hildenbrand @ 2019-12-06 14:27 UTC (permalink / raw)
To: Cornelia Huck, Richard Henderson; +Cc: qemu-s390x, qemu-devel
On 06.12.19 14:54, Cornelia Huck wrote:
> We neglected to clean up pending interrupts and emergency signals;
> fix that.
>
> Signed-off-by: Cornelia Huck <cohuck@redhat.com>
> ---
>
> v1->v2:
> - rebased on top of my s390-next branch; we can now move the fields
> to be reset instead of clearing them manually
>
> Further cleanup possible in a follow-up patch.
>
> ---
> target/s390x/cpu.h | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/target/s390x/cpu.h b/target/s390x/cpu.h
> index 7f5fa1d35b73..e195e5c7c8bb 100644
> --- a/target/s390x/cpu.h
> +++ b/target/s390x/cpu.h
> @@ -98,10 +98,6 @@ struct CPUS390XState {
>
> uint64_t cregs[16]; /* control registers */
>
> - int pending_int;
> - uint16_t external_call_addr;
> - DECLARE_BITMAP(emergency_signals, S390_MAX_CPUS);
> -
> uint64_t ckc;
> uint64_t cputm;
> uint32_t todpr;
> @@ -117,6 +113,10 @@ struct CPUS390XState {
> struct {} start_normal_reset_fields;
> uint8_t riccb[64]; /* runtime instrumentation control */
>
> + int pending_int;
> + uint16_t external_call_addr;
> + DECLARE_BITMAP(emergency_signals, S390_MAX_CPUS);
> +
> /* Fields up to this point are cleared by a CPU reset */
> struct {} end_reset_fields;
>
>
Reviewed-by: David Hildenbrand <david@redhat.com>
--
Thanks,
David / dhildenb
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v2] s390x/tcg: clear local interrupts on reset normal
2019-12-06 13:54 [PATCH v2] s390x/tcg: clear local interrupts on reset normal Cornelia Huck
2019-12-06 14:26 ` Philippe Mathieu-Daudé
2019-12-06 14:27 ` David Hildenbrand
@ 2019-12-10 10:11 ` Cornelia Huck
2 siblings, 0 replies; 4+ messages in thread
From: Cornelia Huck @ 2019-12-10 10:11 UTC (permalink / raw)
To: Richard Henderson, David Hildenbrand; +Cc: qemu-s390x, qemu-devel
On Fri, 6 Dec 2019 14:54:04 +0100
Cornelia Huck <cohuck@redhat.com> wrote:
> We neglected to clean up pending interrupts and emergency signals;
> fix that.
>
> Signed-off-by: Cornelia Huck <cohuck@redhat.com>
> ---
>
> v1->v2:
> - rebased on top of my s390-next branch; we can now move the fields
> to be reset instead of clearing them manually
>
> Further cleanup possible in a follow-up patch.
>
> ---
> target/s390x/cpu.h | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
Queued to s390-next.
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2019-12-10 10:11 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-12-06 13:54 [PATCH v2] s390x/tcg: clear local interrupts on reset normal Cornelia Huck
2019-12-06 14:26 ` Philippe Mathieu-Daudé
2019-12-06 14:27 ` David Hildenbrand
2019-12-10 10:11 ` Cornelia Huck
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).