[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [qemu-s390x] [PATCH v2 4/7] s390x/tcg: SET CLOCK COMPARATOR can clea
From: |
Cornelia Huck |
Subject: |
Re: [qemu-s390x] [PATCH v2 4/7] s390x/tcg: SET CLOCK COMPARATOR can clear CKC interrupts |
Date: |
Thu, 21 Jun 2018 15:58:15 +0200 |
On Thu, 21 Jun 2018 14:09:28 +0200
Thomas Huth <address@hidden> wrote:
> On 20.06.2018 12:08, David Hildenbrand wrote:
> > Let's stop the timer and delete any pending CKC IRQ before doing
> > anything else.
> >
> > Signed-off-by: David Hildenbrand <address@hidden>
> > ---
> > target/s390x/misc_helper.c | 6 ++++++
> > 1 file changed, 6 insertions(+)
> >
> > diff --git a/target/s390x/misc_helper.c b/target/s390x/misc_helper.c
> > index 6ccbe1fe9a..d5f9f5e1d3 100644
> > --- a/target/s390x/misc_helper.c
> > +++ b/target/s390x/misc_helper.c
> > @@ -158,6 +158,12 @@ void HELPER(sckc)(CPUS390XState *env, uint64_t time)
> > Error *err = NULL;
> > S390TOD tod_base;
> >
> > + /* stop the timer and remove pending CKC IRQs */
> > + timer_del(env->tod_timer);
> > + qemu_mutex_lock_iothread();
> > + env->pending_int &= ~INTERRUPT_EXT_CLOCK_COMPARATOR;
> > + qemu_mutex_unlock_iothread();
>
> Reviewed-by: Thomas Huth <address@hidden>
>
> > if (time == -1ULL) {
>
> I wonder whether that check is still adequate? Is there really a way to
> disable the clock comparator like this? At least I haven't seen it in
> the PoP.
Me neither. It seems to have been in the code since the beginning...
>
> > return;
> > }
> >
>
[qemu-s390x] [PATCH v2 1/7] s390x/tod: factor out TOD into separate device, David Hildenbrand, 2018/06/20
[qemu-s390x] [PATCH v2 3/7] s390x/tcg: properly implement the TOD, David Hildenbrand, 2018/06/20
Re: [qemu-s390x] [PATCH v2 3/7] s390x/tcg: properly implement the TOD, Thomas Huth, 2018/06/21