qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [RFC PATCH v8 08/21] cpu: replay instructions sequence


From: Pavel Dovgaluk
Subject: Re: [Qemu-devel] [RFC PATCH v8 08/21] cpu: replay instructions sequence
Date: Mon, 16 Feb 2015 15:26:04 +0300

> From: Paolo Bonzini [mailto:address@hidden On Behalf Of Paolo Bonzini
> On 02/02/2015 13:42, Pavel Dovgaluk wrote:
> >> From: Paolo Bonzini [mailto:address@hidden
> >> On 02/02/2015 13:28, Pavel Dovgaluk wrote:
> >>>>>>>                              cpu->exception_index = EXCP_INTERRUPT;
> >>>>>>>                              next_tb = 0;
> >>>>>>> +                            qemu_notify_event();
> >>>>>
> >>>>> Why is this needed?
> >>> It is needed to wake up iothread in replay mode.
> >>> Otherwise it waits for additional time and replay becomes too slow.
> >>
> >> What event (something from a timerlist?) is ready, that has not been
> >> notified to the iothread yet?  qemu_notify_event() should never be
> >> needed in common case.  It's probably missing somewhere else.
> >
> > I think in this case there are no events at all - just reading timers values
> > that were made while recording.
> > We have to replay these reads by waking iothread.
> 
> I think the right place for this is in replay_read_next_clock then.

It doesn't fit. Log file is not read until all instructions are executed.
And the next read from the file should be performed by iothread which should
be notified and waked up.

Pavel Dovgalyuk




reply via email to

[Prev in Thread] Current Thread [Next in Thread]