From: Markus Armbruster <armbru@redhat.com>
To: John Snow <jsnow@redhat.com>
Cc: Steve Sistare <steven.sistare@oracle.com>,
qemu-devel <qemu-devel@nongnu.org>,
Cleber Rosa <crosa@redhat.com>, Eric Blake <eblake@redhat.com>,
Paolo Bonzini <pbonzini@redhat.com>,
"Daniel P. Berrange" <berrange@redhat.com>,
Eduardo Habkost <eduardo@habkost.net>,
Fabiano Rosas <farosas@suse.de>,
Laurent Vivier <lvivier@redhat.com>,
Philippe Mathieu-Daude <philmd@linaro.org>
Subject: Re: [PATCH V4 2/3] python: use qom-list-get
Date: Fri, 08 Aug 2025 08:28:07 +0200 [thread overview]
Message-ID: <87o6sqmio8.fsf@pond.sub.org> (raw)
In-Reply-To: <CAFn=p-biuic_12kWEj==NmNp54MW6X4wf=2hu0edvqJF+deyaw@mail.gmail.com> (John Snow's message of "Thu, 7 Aug 2025 18:04:14 -0400")
John Snow <jsnow@redhat.com> writes:
> On Wed, Jul 16, 2025 at 4:32 AM Markus Armbruster <armbru@redhat.com> wrote:
>>
>> John Snow <jsnow@redhat.com> writes:
>>
>> > On Fri, Jul 11, 2025, 10:47 AM Markus Armbruster <armbru@redhat.com> wrote:
>>
>> [...]
>>
>> >> Exception ignored in: <function QEMUMonitorProtocol.__del__ at 0x7fcfcd080d60>
>> >> Traceback (most recent call last):
>> >> File "/work/armbru/qemu/scripts/qmp/../../python/qemu/qmp/legacy.py", line 310, in __del__
>> >> self.close()
>> >> File "/work/armbru/qemu/scripts/qmp/../../python/qemu/qmp/legacy.py", line 281, in close
>> >> self._sync(
>> >> File "/work/armbru/qemu/scripts/qmp/../../python/qemu/qmp/legacy.py", line 102, in _sync
>> >> return self._aloop.run_until_complete(
>> >> File "/usr/lib64/python3.13/asyncio/base_events.py", line 719, in run_until_complete
>> >> return future.result()
>> >> File "/usr/lib64/python3.13/asyncio/tasks.py", line 507, in wait_for
>> >> return await fut
>> >> File "/work/armbru/qemu/scripts/qmp/../../python/qemu/qmp/protocol.py", line 399, in disconnect
>> >> await self._wait_disconnect()
>> >> File "/work/armbru/qemu/scripts/qmp/../../python/qemu/qmp/protocol.py", line 719, in _wait_disconnect
>> >> await all_defined_tasks # Raise Exceptions from the bottom half.
>> >> File "/work/armbru/qemu/scripts/qmp/../../python/qemu/qmp/protocol.py", line 870, in _bh_loop_forever
>> >> await async_fn()
>> >> RuntimeError: cannot reuse already awaited coroutine
>> >
>> > Curious about this backtrace. It looks like something has gone
>> > fundamentally wrong in the internals and the error is being raised by the
>> > garbage collector which is not ideal.
>> >
>> > Can you help me reproduce this? Even if it's old/bad code, I don't want
>> > python-qemu-qmp faulting like this.
>>
>> Reproducer for Fedora 41, current master c079d3a31e4:
>>
>> Run
>>
>> $ qemu-system-x86_64 -S -display none -chardev socket,id=mon1,path=test-qmp,server=on,wait=off -mon mode=control,id=qmp,chardev=mon1
>>
>> and
>>
>> $ scripts/qmp/qom-tree -s test-qmp >/dev/null
>>
>> Questions?
>>
>
> Doesn't seem to trigger for me on Fedora 42 from the c079d3a31e4
> build. Is this a Python version difference thing rearing its head?
I have no idea.
Is there anything else I can do to help you?
next prev parent reply other threads:[~2025-08-08 6:28 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-07-10 16:24 [PATCH V4 0/3] fast qom tree get Steve Sistare
2025-07-10 16:24 ` [PATCH V4 1/3] qom: qom-list-get Steve Sistare
2025-07-11 14:35 ` Markus Armbruster
2025-07-11 15:22 ` Steven Sistare
2025-07-10 16:24 ` [PATCH V4 2/3] python: use qom-list-get Steve Sistare
2025-07-11 14:47 ` Markus Armbruster
2025-07-11 15:23 ` Steven Sistare
2025-07-11 16:50 ` Markus Armbruster
2025-07-11 17:08 ` Steven Sistare
2025-07-15 17:27 ` John Snow
2025-07-16 8:32 ` Markus Armbruster
2025-08-07 22:04 ` John Snow
2025-08-08 6:28 ` Markus Armbruster [this message]
2025-08-08 16:16 ` John Snow
2025-07-10 16:24 ` [PATCH V4 3/3] tests/qtest/qom-test: unit test for qom-list-getv Steve Sistare
2025-07-11 15:02 ` Markus Armbruster
2025-07-11 15:23 ` Steven Sistare
2025-07-11 15:06 ` [PATCH V4 0/3] fast qom tree get Markus Armbruster
2025-07-11 15:27 ` Steven Sistare
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87o6sqmio8.fsf@pond.sub.org \
--to=armbru@redhat.com \
--cc=berrange@redhat.com \
--cc=crosa@redhat.com \
--cc=eblake@redhat.com \
--cc=eduardo@habkost.net \
--cc=farosas@suse.de \
--cc=jsnow@redhat.com \
--cc=lvivier@redhat.com \
--cc=pbonzini@redhat.com \
--cc=philmd@linaro.org \
--cc=qemu-devel@nongnu.org \
--cc=steven.sistare@oracle.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.