Re: [PATCH] dump-guest-memory: Use BQL to protect dump finalize process

2021-11-16 Thread Peter Xu
On Tue, Nov 16, 2021 at 12:20:37PM +0100, Laszlo Ersek wrote: > On 11/16/21 04:22, Peter Xu wrote: > > When finalizing the dump-guest-memory with detached mode, we'll first set > > dump > > status to either FAIL or COMPLETE before doing the cleanup, however right > > after > > the dump status cha

Re: [PATCH] dump-guest-memory: Use BQL to protect dump finalize process

2021-11-16 Thread Laszlo Ersek
On 11/16/21 04:22, Peter Xu wrote: > When finalizing the dump-guest-memory with detached mode, we'll first set dump > status to either FAIL or COMPLETE before doing the cleanup, however right > after > the dump status change it's possible that another dump-guest-memory qmp > command > is sent so

Re: [PATCH] dump-guest-memory: Use BQL to protect dump finalize process

2021-11-15 Thread Marc-André Lureau
Hi On Tue, Nov 16, 2021 at 7:22 AM Peter Xu wrote: > > When finalizing the dump-guest-memory with detached mode, we'll first set dump > status to either FAIL or COMPLETE before doing the cleanup, however right > after > the dump status change it's possible that another dump-guest-memory qmp > c

[PATCH] dump-guest-memory: Use BQL to protect dump finalize process

2021-11-15 Thread Peter Xu
When finalizing the dump-guest-memory with detached mode, we'll first set dump status to either FAIL or COMPLETE before doing the cleanup, however right after the dump status change it's possible that another dump-guest-memory qmp command is sent so both the main thread and dump thread (which is du