[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 09/10] python/aqmp: fix race condition in legacy.py
From: |
Daniel P . Berrangé |
Subject: |
Re: [PATCH 09/10] python/aqmp: fix race condition in legacy.py |
Date: |
Fri, 4 Mar 2022 18:01:03 +0000 |
User-agent: |
Mutt/2.1.5 (2021-12-30) |
On Fri, Feb 25, 2022 at 03:59:47PM -0500, John Snow wrote:
> With all of that refactoring out of the way, this is extraordinarily
> simple.
I think it'd be useful to explain in some detail what the
race condition was, because 1 year later no one will
remember what scenario we were fixing here.
>
> Signed-off-by: John Snow <jsnow@redhat.com>
> ---
> python/qemu/aqmp/legacy.py | 7 ++-----
> 1 file changed, 2 insertions(+), 5 deletions(-)
Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
- Re: [PATCH 09/10] python/aqmp: fix race condition in legacy.py,
Daniel P . Berrangé <=