* [PATCH v2] replay: notify CPU on event
@ 2021-02-16 9:34 Pavel Dovgalyuk
2021-02-24 8:26 ` Pavel Dovgalyuk
2021-03-15 6:01 ` Pavel Dovgalyuk
0 siblings, 2 replies; 7+ messages in thread
From: Pavel Dovgalyuk @ 2021-02-16 9:34 UTC (permalink / raw)
To: qemu-devel; +Cc: alex.bennee, pbonzini, pavel.dovgalyuk
This patch enables vCPU notification to wake it up
when new async event comes in replay mode.
The motivation of this patch is the following.
Consider recorded block async event. It is saved into the log
with one of the checkpoints. This checkpoint may be passed in
vCPU loop. In replay mode when this async event is read from
the log, and block thread task is not finished yet, vCPU thread
goes to sleep. That is why this patch adds waking up the vCPU
to process this finished event.
Signed-off-by: Pavel Dovgalyuk <Pavel.Dovgalyuk@ispras.ru>
---
replay/replay-events.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/replay/replay-events.c b/replay/replay-events.c
index a1c6bb934e..92dc800219 100644
--- a/replay/replay-events.c
+++ b/replay/replay-events.c
@@ -126,6 +126,7 @@ void replay_add_event(ReplayAsyncEventKind event_kind,
g_assert(replay_mutex_locked());
QTAILQ_INSERT_TAIL(&events_list, event, events);
+ qemu_clock_notify(QEMU_CLOCK_VIRTUAL);
}
void replay_bh_schedule_event(QEMUBH *bh)
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH v2] replay: notify CPU on event
2021-02-16 9:34 [PATCH v2] replay: notify CPU on event Pavel Dovgalyuk
@ 2021-02-24 8:26 ` Pavel Dovgalyuk
2021-03-15 6:01 ` Pavel Dovgalyuk
1 sibling, 0 replies; 7+ messages in thread
From: Pavel Dovgalyuk @ 2021-02-24 8:26 UTC (permalink / raw)
To: qemu-devel; +Cc: pbonzini, alex.bennee
ping
On 16.02.2021 12:34, Pavel Dovgalyuk wrote:
> This patch enables vCPU notification to wake it up
> when new async event comes in replay mode.
>
> The motivation of this patch is the following.
> Consider recorded block async event. It is saved into the log
> with one of the checkpoints. This checkpoint may be passed in
> vCPU loop. In replay mode when this async event is read from
> the log, and block thread task is not finished yet, vCPU thread
> goes to sleep. That is why this patch adds waking up the vCPU
> to process this finished event.
>
> Signed-off-by: Pavel Dovgalyuk <Pavel.Dovgalyuk@ispras.ru>
> ---
> replay/replay-events.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/replay/replay-events.c b/replay/replay-events.c
> index a1c6bb934e..92dc800219 100644
> --- a/replay/replay-events.c
> +++ b/replay/replay-events.c
> @@ -126,6 +126,7 @@ void replay_add_event(ReplayAsyncEventKind event_kind,
>
> g_assert(replay_mutex_locked());
> QTAILQ_INSERT_TAIL(&events_list, event, events);
> + qemu_clock_notify(QEMU_CLOCK_VIRTUAL);
> }
>
> void replay_bh_schedule_event(QEMUBH *bh)
>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2] replay: notify CPU on event
2021-02-16 9:34 [PATCH v2] replay: notify CPU on event Pavel Dovgalyuk
2021-02-24 8:26 ` Pavel Dovgalyuk
@ 2021-03-15 6:01 ` Pavel Dovgalyuk
1 sibling, 0 replies; 7+ messages in thread
From: Pavel Dovgalyuk @ 2021-03-15 6:01 UTC (permalink / raw)
To: qemu-devel; +Cc: pbonzini, alex.bennee
ping
On 16.02.2021 12:34, Pavel Dovgalyuk wrote:
> This patch enables vCPU notification to wake it up
> when new async event comes in replay mode.
>
> The motivation of this patch is the following.
> Consider recorded block async event. It is saved into the log
> with one of the checkpoints. This checkpoint may be passed in
> vCPU loop. In replay mode when this async event is read from
> the log, and block thread task is not finished yet, vCPU thread
> goes to sleep. That is why this patch adds waking up the vCPU
> to process this finished event.
>
> Signed-off-by: Pavel Dovgalyuk <Pavel.Dovgalyuk@ispras.ru>
> ---
> replay/replay-events.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/replay/replay-events.c b/replay/replay-events.c
> index a1c6bb934e..92dc800219 100644
> --- a/replay/replay-events.c
> +++ b/replay/replay-events.c
> @@ -126,6 +126,7 @@ void replay_add_event(ReplayAsyncEventKind event_kind,
>
> g_assert(replay_mutex_locked());
> QTAILQ_INSERT_TAIL(&events_list, event, events);
> + qemu_clock_notify(QEMU_CLOCK_VIRTUAL);
> }
>
> void replay_bh_schedule_event(QEMUBH *bh)
>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2] replay: notify CPU on event
2021-03-29 11:23 ` Pavel Dovgalyuk
@ 2021-03-29 12:10 ` Paolo Bonzini
0 siblings, 0 replies; 7+ messages in thread
From: Paolo Bonzini @ 2021-03-29 12:10 UTC (permalink / raw)
To: Pavel Dovgalyuk, qemu-devel; +Cc: alex.bennee
On 29/03/21 13:23, Pavel Dovgalyuk wrote:
>>
>> I still do not understand why the qemu_clock_notify function is used
>> to notify the CPU. It would be nice to have a description of the call
>> chain in a comment or to have a more appropriate API. I can queue the
>> patch since it's deep in replay-events.c and not in common code, but
>> if you post a short description here I'll integrate it.
>
> Maybe I missed something.
> This call is needed for the case when vCPU is waiting in rr_wait_io_event.
> You mean that we can use qemu_cpu_kick instead?
I have no idea. Using qemu_clock_notify however is definitely weird.
Paolo
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2] replay: notify CPU on event
2021-03-29 10:29 ` Paolo Bonzini
@ 2021-03-29 11:23 ` Pavel Dovgalyuk
2021-03-29 12:10 ` Paolo Bonzini
0 siblings, 1 reply; 7+ messages in thread
From: Pavel Dovgalyuk @ 2021-03-29 11:23 UTC (permalink / raw)
To: Paolo Bonzini, qemu-devel; +Cc: alex.bennee
On 29.03.2021 13:29, Paolo Bonzini wrote:
> On 29/03/21 09:43, Pavel Dovgalyuk wrote:
>> This patch enables vCPU notification to wake it up
>> when new async event comes in replay mode.
>>
>> The motivation of this patch is the following.
>> Consider recorded block async event. It is saved into the log
>> with one of the checkpoints. This checkpoint may be passed in
>> vCPU loop. In replay mode when this async event is read from
>> the log, and block thread task is not finished yet, vCPU thread
>> goes to sleep. That is why this patch adds waking up the vCPU
>> to process this finished event.
>>
>> Signed-off-by: Pavel Dovgalyuk <Pavel.Dovgalyuk@ispras.ru>
>> ---
>> replay/replay-events.c | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/replay/replay-events.c b/replay/replay-events.c
>> index a1c6bb934e..92dc800219 100644
>> --- a/replay/replay-events.c
>> +++ b/replay/replay-events.c
>> @@ -126,6 +126,7 @@ void replay_add_event(ReplayAsyncEventKind
>> event_kind,
>> g_assert(replay_mutex_locked());
>> QTAILQ_INSERT_TAIL(&events_list, event, events);
>> + qemu_clock_notify(QEMU_CLOCK_VIRTUAL);
>> }
>
> I still do not understand why the qemu_clock_notify function is used to
> notify the CPU. It would be nice to have a description of the call
> chain in a comment or to have a more appropriate API. I can queue the
> patch since it's deep in replay-events.c and not in common code, but if
> you post a short description here I'll integrate it.
Maybe I missed something.
This call is needed for the case when vCPU is waiting in rr_wait_io_event.
You mean that we can use qemu_cpu_kick instead?
Pavel Dovgalyuk
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2] replay: notify CPU on event
2021-03-29 7:43 Pavel Dovgalyuk
@ 2021-03-29 10:29 ` Paolo Bonzini
2021-03-29 11:23 ` Pavel Dovgalyuk
0 siblings, 1 reply; 7+ messages in thread
From: Paolo Bonzini @ 2021-03-29 10:29 UTC (permalink / raw)
To: Pavel Dovgalyuk, qemu-devel; +Cc: alex.bennee
On 29/03/21 09:43, Pavel Dovgalyuk wrote:
> This patch enables vCPU notification to wake it up
> when new async event comes in replay mode.
>
> The motivation of this patch is the following.
> Consider recorded block async event. It is saved into the log
> with one of the checkpoints. This checkpoint may be passed in
> vCPU loop. In replay mode when this async event is read from
> the log, and block thread task is not finished yet, vCPU thread
> goes to sleep. That is why this patch adds waking up the vCPU
> to process this finished event.
>
> Signed-off-by: Pavel Dovgalyuk <Pavel.Dovgalyuk@ispras.ru>
> ---
> replay/replay-events.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/replay/replay-events.c b/replay/replay-events.c
> index a1c6bb934e..92dc800219 100644
> --- a/replay/replay-events.c
> +++ b/replay/replay-events.c
> @@ -126,6 +126,7 @@ void replay_add_event(ReplayAsyncEventKind event_kind,
>
> g_assert(replay_mutex_locked());
> QTAILQ_INSERT_TAIL(&events_list, event, events);
> + qemu_clock_notify(QEMU_CLOCK_VIRTUAL);
> }
I still do not understand why the qemu_clock_notify function is used to
notify the CPU. It would be nice to have a description of the call
chain in a comment or to have a more appropriate API. I can queue the
patch since it's deep in replay-events.c and not in common code, but if
you post a short description here I'll integrate it.
Paolo
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH v2] replay: notify CPU on event
@ 2021-03-29 7:43 Pavel Dovgalyuk
2021-03-29 10:29 ` Paolo Bonzini
0 siblings, 1 reply; 7+ messages in thread
From: Pavel Dovgalyuk @ 2021-03-29 7:43 UTC (permalink / raw)
To: qemu-devel; +Cc: alex.bennee, pbonzini, pavel.dovgalyuk
This patch enables vCPU notification to wake it up
when new async event comes in replay mode.
The motivation of this patch is the following.
Consider recorded block async event. It is saved into the log
with one of the checkpoints. This checkpoint may be passed in
vCPU loop. In replay mode when this async event is read from
the log, and block thread task is not finished yet, vCPU thread
goes to sleep. That is why this patch adds waking up the vCPU
to process this finished event.
Signed-off-by: Pavel Dovgalyuk <Pavel.Dovgalyuk@ispras.ru>
---
replay/replay-events.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/replay/replay-events.c b/replay/replay-events.c
index a1c6bb934e..92dc800219 100644
--- a/replay/replay-events.c
+++ b/replay/replay-events.c
@@ -126,6 +126,7 @@ void replay_add_event(ReplayAsyncEventKind event_kind,
g_assert(replay_mutex_locked());
QTAILQ_INSERT_TAIL(&events_list, event, events);
+ qemu_clock_notify(QEMU_CLOCK_VIRTUAL);
}
void replay_bh_schedule_event(QEMUBH *bh)
^ permalink raw reply related [flat|nested] 7+ messages in thread
end of thread, other threads:[~2021-03-29 12:12 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-02-16 9:34 [PATCH v2] replay: notify CPU on event Pavel Dovgalyuk
2021-02-24 8:26 ` Pavel Dovgalyuk
2021-03-15 6:01 ` Pavel Dovgalyuk
2021-03-29 7:43 Pavel Dovgalyuk
2021-03-29 10:29 ` Paolo Bonzini
2021-03-29 11:23 ` Pavel Dovgalyuk
2021-03-29 12:10 ` Paolo Bonzini
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.