On Tue, Sep 28, 2021 at 9:53 AM Paolo Bonzini <pbonzini@redhat.com> wrote:
> I suspect no one has ever looked at socket_scm_helper.c, because when I
> did my reaction was just "why". The functionality of this 136-line program
> can be reproduced in Python with fewer lines of code than it takes to
> invoke it. Do it, and let it rest in peace.
>
> The only reason not to do that could be because of the upcoming switch
> to aqmp. For that, the new send_fd method has to be wrapped to use
> transport.get_extra_info('socket') and loop.run_in_executor. Let me
> know if you prefer to hold on this until later.
>
> Paolo
>
> Paolo Bonzini (4):
> python: stop using socket_scm_helper
> socket_scm_helper: remove
> python: raise OSError from send_fd_scm
> python: split the two sides of send_fd_scm
>
> python/qemu/machine/machine.py | 58 +++--------
> python/qemu/machine/qtest.py | 2 -
> python/qemu/qmp/__init__.py | 15 +++
> tests/Makefile.include | 5 +-
> tests/meson.build | 4 -
> tests/qemu-iotests/045 | 3 +-
> tests/qemu-iotests/147 | 3 +-
> tests/qemu-iotests/iotests.py | 3 -
> tests/qemu-iotests/meson.build | 5 -
> tests/qemu-iotests/socket_scm_helper.c | 136 -------------------------
> tests/qemu-iotests/testenv.py | 8 +-
> 11 files changed, 35 insertions(+), 207 deletions(-)
> delete mode 100644 tests/qemu-iotests/meson.build
> delete mode 100644 tests/qemu-iotests/socket_scm_helper.c
>
> --
> 2.31.1
>
We discussed this extremely briefly on IRC, but I have some similar
patches already floating around on the list, so I won't be taking these.
Nevertheless, we're in agreement about socket_scm_helper setting sail for
the great beyond!
Any additional cleanups made in these patches I'll just pull into my
ongoing series of Python cleanups and add additional authorship credits.
Thanks!
--js
(For patchew? I don't know if this works. Let's find out:)
NACK