Message ID | 20220316095156.2613419-1-marcandre.lureau@redhat.com |
---|---|
Headers | show |
Series | Misc fixes and cleanups for 7.0? | expand |
On Wed, Mar 16, 2022 at 01:51:56PM +0400, marcandre.lureau@redhat.com wrote: > From: Marc-André Lureau <marcandre.lureau@redhat.com> > > Hi, > > Various clean-up and fixes patches that might be worth it for 7.0. > Some of the patches have been posted and reviewed before. FYI, something went wrong with this posting, as it hasn't set mail threading, thus every patch appears as a standalone message. With regards, Daniel
Hi On Wed, Mar 16, 2022 at 5:09 PM Daniel P. Berrangé <berrange@redhat.com> wrote: > > On Wed, Mar 16, 2022 at 01:51:56PM +0400, marcandre.lureau@redhat.com wrote: > > From: Marc-André Lureau <marcandre.lureau@redhat.com> > > > > Hi, > > > > Various clean-up and fixes patches that might be worth it for 7.0. > > Some of the patches have been posted and reviewed before. > > FYI, something went wrong with this posting, as it hasn't set mail > threading, thus every patch appears as a standalone message. > Thanks I didn't notice with gmail. I used git-publish with --separate-send, because I wanted cccmd to apply on each patch. I thought it would respect the threading though.. Stefan, what do you recommend instead?
On 16/3/22 10:51, marcandre.lureau@redhat.com wrote: > From: Marc-André Lureau <marcandre.lureau@redhat.com> > > Hi, > > Various clean-up and fixes patches that might be worth it for 7.0. > Some of the patches have been posted and reviewed before. > > Marc-André Lureau (27): > m68k/nios2-semi: fix gettimeofday() result check > meson: remove bsd_oses > meson: use chardev_ss dependencies > meson: add util dependency for oslib-posix on freebsd > char: move qemu_openpty_raw from util/ to char/ > Drop qemu_foo() socket API wrapper > Replace GCC_FMT_ATTR with G_GNUC_PRINTF > compiler.h: replace QEMU_WARN_UNUSED_RESULT with > G_GNUC_WARN_UNUSED_RESULT > compiler.h: replace QEMU_SENTINEL with G_GNUC_NULL_TERMINATED > Replace config-time define HOST_WORDS_BIGENDIAN > osdep: poison HOST_WORDS_BIGENDIAN > Simplify HOST_LONG_BITS > Move HOST_LONG_BITS to compiler.h > scripts/modinfo-collect: remove unused/dead code > util: remove needless includes > util: remove the net/net.h dependency > qapi: remove needless include > meson: move int128 checks from configure > meson: fix CONFIG_ATOMIC128 check > qapi: remove needless include > qga: remove bswap.h include > error-report: replace error progname with glib functions > tests: remove needless include > Remove trailing ; after G_DEFINE_AUTO macro > include/qapi: add g_autoptr support for qobject types > tests: replace free_all() usage with g_auto > qapi: remove needless include FYI git-publish eases sending patches to mailing list: https://github.com/stefanha/git-publish
On Wed, Mar 16, 2022 at 05:13:49PM +0400, Marc-André Lureau wrote: > Hi > > On Wed, Mar 16, 2022 at 5:09 PM Daniel P. Berrangé <berrange@redhat.com> wrote: > > > > On Wed, Mar 16, 2022 at 01:51:56PM +0400, marcandre.lureau@redhat.com wrote: > > > From: Marc-André Lureau <marcandre.lureau@redhat.com> > > > > > > Hi, > > > > > > Various clean-up and fixes patches that might be worth it for 7.0. > > > Some of the patches have been posted and reviewed before. > > > > FYI, something went wrong with this posting, as it hasn't set mail > > threading, thus every patch appears as a standalone message. > > > > Thanks I didn't notice with gmail. I used git-publish with > --separate-send, because I wanted cccmd to apply on each patch. I > thought it would respect the threading though.. Stefan, what do you > recommend instead? You can use --in-reply-to to send each email as a reply to the same parent. Nothing fancier than that is supported by git-publish at the moment. Stefan
From: Marc-André Lureau <marcandre.lureau@redhat.com> Hi, Various clean-up and fixes patches that might be worth it for 7.0. Some of the patches have been posted and reviewed before. Marc-André Lureau (27): m68k/nios2-semi: fix gettimeofday() result check meson: remove bsd_oses meson: use chardev_ss dependencies meson: add util dependency for oslib-posix on freebsd char: move qemu_openpty_raw from util/ to char/ Drop qemu_foo() socket API wrapper Replace GCC_FMT_ATTR with G_GNUC_PRINTF compiler.h: replace QEMU_WARN_UNUSED_RESULT with G_GNUC_WARN_UNUSED_RESULT compiler.h: replace QEMU_SENTINEL with G_GNUC_NULL_TERMINATED Replace config-time define HOST_WORDS_BIGENDIAN osdep: poison HOST_WORDS_BIGENDIAN Simplify HOST_LONG_BITS Move HOST_LONG_BITS to compiler.h scripts/modinfo-collect: remove unused/dead code util: remove needless includes util: remove the net/net.h dependency qapi: remove needless include meson: move int128 checks from configure meson: fix CONFIG_ATOMIC128 check qapi: remove needless include qga: remove bswap.h include error-report: replace error progname with glib functions tests: remove needless include Remove trailing ; after G_DEFINE_AUTO macro include/qapi: add g_autoptr support for qobject types tests: replace free_all() usage with g_auto qapi: remove needless include configure | 49 +-------- meson.build | 52 +++++++-- accel/tcg/atomic_template.h | 4 +- audio/audio.h | 6 +- block/qcow2.h | 2 +- bsd-user/qemu.h | 2 +- hw/display/pl110_template.h | 6 +- hw/display/qxl.h | 2 +- hw/net/can/ctucan_core.h | 2 +- hw/net/rocker/rocker.h | 2 +- hw/net/vmxnet3.h | 4 +- hw/xen/xen_pt.h | 2 +- include/chardev/char-fe.h | 2 +- include/disas/dis-asm.h | 2 +- include/exec/cpu-all.h | 4 +- include/exec/cpu-common.h | 2 +- include/exec/memop.h | 2 +- include/exec/memory.h | 2 +- include/fpu/softfloat-types.h | 2 +- include/hw/acpi/aml-build.h | 12 +- include/hw/core/cpu.h | 4 +- include/hw/hw.h | 2 +- include/hw/i386/intel_iommu.h | 6 +- include/hw/i386/x86-iommu.h | 4 +- include/hw/virtio/virtio-access.h | 6 +- include/hw/virtio/virtio-gpu-bswap.h | 2 +- include/hw/virtio/virtio.h | 2 +- include/hw/xen/xen-bus-helper.h | 4 +- include/hw/xen/xen-bus.h | 4 +- include/hw/xen/xen_common.h | 2 +- include/hw/xen/xen_pvdev.h | 2 +- include/libdecnumber/dconfig.h | 2 +- include/monitor/monitor.h | 4 +- include/net/eth.h | 2 +- include/qapi/error.h | 20 ++-- include/qapi/qmp/qbool.h | 6 + include/qapi/qmp/qdict.h | 6 + include/qapi/qmp/qjson.h | 8 +- include/qapi/qmp/qlist.h | 8 +- include/qapi/qmp/qnull.h | 6 + include/qapi/qmp/qnum.h | 6 + include/qapi/qmp/qstring.h | 6 + include/qemu-common.h | 24 +--- include/qemu/bswap.h | 8 +- include/qemu/buffer.h | 2 +- include/qemu/compiler.h | 20 ++-- include/qemu/error-report.h | 26 ++--- include/qemu/host-utils.h | 2 +- include/qemu/int128.h | 2 +- include/qemu/log-for-trace.h | 2 +- include/qemu/log.h | 2 +- include/qemu/osdep.h | 15 +-- include/qemu/qemu-print.h | 8 +- include/qemu/range.h | 4 +- include/qemu/readline.h | 2 +- include/qom/object.h | 6 +- include/ui/qemu-pixman.h | 2 +- net/util.h | 2 +- qga/guest-agent-core.h | 2 +- qga/vss-win32/requester.h | 2 +- scripts/cocci-macro-file.h | 6 +- target/arm/cpu.h | 8 +- target/arm/translate-a64.h | 2 +- target/arm/vec_internal.h | 2 +- target/i386/cpu.h | 2 +- target/mips/cpu.h | 2 +- target/ppc/cpu.h | 2 +- target/s390x/tcg/vec.h | 2 +- target/xtensa/cpu.h | 2 +- tests/fp/platform.h | 4 +- tests/qtest/libqos/libqtest.h | 42 +++---- tests/qtest/libqtest-single.h | 2 +- tests/qtest/migration-helpers.h | 6 +- accel/kvm/kvm-all.c | 4 +- audio/alsaaudio.c | 4 +- audio/dbusaudio.c | 2 +- audio/dsoundaudio.c | 4 +- audio/ossaudio.c | 4 +- audio/paaudio.c | 2 +- audio/sdlaudio.c | 2 +- block/blkverify.c | 2 +- block/qcow2-refcount.c | 20 ++-- block/ssh.c | 4 +- chardev/char-pty.c | 111 +++++++++++++++++++ crypto/cipher-afalg.c | 4 +- crypto/hash-afalg.c | 4 +- disas.c | 2 +- fsdev/9p-marshal.c | 2 +- fsdev/virtfs-proxy-helper.c | 2 +- gdbstub.c | 2 +- hw/9pfs/9p.c | 2 +- hw/acpi/aml-build.c | 4 +- hw/core/loader.c | 4 +- hw/display/artist.c | 6 +- hw/display/pxa2xx_lcd.c | 2 +- hw/display/vga.c | 12 +- hw/display/virtio-gpu-gl.c | 2 +- hw/mips/fuloong2e.c | 2 +- hw/mips/malta.c | 2 +- hw/net/rtl8139.c | 2 +- hw/s390x/event-facility.c | 2 +- hw/virtio/vhost.c | 2 +- hw/virtio/virtio.c | 2 +- io/channel-socket.c | 6 +- io/channel-websock.c | 2 +- linux-user/arm/nwfpe/double_cpdo.c | 4 +- linux-user/arm/nwfpe/fpa11_cpdt.c | 4 +- linux-user/ppc/signal.c | 3 +- linux-user/syscall.c | 6 +- monitor/hmp.c | 4 +- nbd/server.c | 12 +- net/announce.c | 13 +++ net/net.c | 4 +- net/socket.c | 24 ++-- qapi/qapi-forward-visitor.c | 1 - qapi/qmp-dispatch.c | 1 - qapi/string-output-visitor.c | 1 - qemu-img.c | 4 +- qemu-io.c | 12 +- qga/main.c | 1 - qobject/json-parser.c | 2 +- softmmu/qtest.c | 4 +- softmmu/vl.c | 2 +- storage-daemon/qemu-storage-daemon.c | 2 +- target/alpha/translate.c | 2 +- target/arm/crypto_helper.c | 2 +- target/arm/helper.c | 2 +- target/arm/kvm64.c | 4 +- target/arm/neon_helper.c | 2 +- target/arm/sve_helper.c | 4 +- target/arm/translate-sve.c | 6 +- target/arm/translate-vfp.c | 2 +- target/arm/translate.c | 2 +- target/hppa/translate.c | 2 +- target/i386/tcg/translate.c | 2 +- target/m68k/m68k-semi.c | 2 +- target/mips/tcg/lmmi_helper.c | 2 +- target/mips/tcg/msa_helper.c | 54 ++++----- target/nios2/nios2-semi.c | 2 +- target/ppc/arch_dump.c | 2 +- target/ppc/int_helper.c | 22 ++-- target/ppc/kvm.c | 4 +- target/ppc/mem_helper.c | 2 +- target/riscv/vector_helper.c | 2 +- target/s390x/tcg/translate.c | 2 +- target/sparc/vis_helper.c | 4 +- tcg/tcg-op.c | 4 +- tcg/tcg.c | 12 +- tests/qtest/e1000e-test.c | 4 +- tests/qtest/libqtest.c | 6 +- tests/qtest/npcm7xx_emc-test.c | 4 +- tests/qtest/test-filter-mirror.c | 4 +- tests/qtest/test-filter-redirector.c | 8 +- tests/qtest/vhost-user-blk-test.c | 2 +- tests/qtest/virtio-blk-test.c | 2 +- tests/qtest/virtio-net-test.c | 10 +- tests/unit/check-qobject.c | 128 +++++++--------------- tests/unit/socket-helpers.c | 2 +- tests/unit/test-qobject-input-visitor.c | 4 +- trace/control.c | 2 +- ui/vdagent.c | 2 +- ui/vnc.c | 2 +- util/bitmap.c | 2 +- util/cutils.c | 16 --- util/host-utils.c | 2 +- util/osdep.c | 4 +- util/qemu-error.c | 24 +--- util/qemu-openpty.c | 139 ------------------------ util/qemu-sockets.c | 10 +- target/ppc/translate/vmx-impl.c.inc | 4 +- target/ppc/translate/vsx-impl.c.inc | 2 +- target/riscv/insn_trans/trans_rvv.c.inc | 4 +- target/s390x/tcg/translate_vx.c.inc | 2 +- tcg/aarch64/tcg-target.c.inc | 4 +- tcg/arm/tcg-target.c.inc | 4 +- tcg/mips/tcg-target.c.inc | 2 +- tcg/ppc/tcg-target.c.inc | 10 +- tcg/riscv/tcg-target.c.inc | 4 +- audio/coreaudio.m | 4 +- chardev/meson.build | 4 +- scripts/checkpatch.pl | 6 +- scripts/modinfo-collect.py | 5 - util/meson.build | 7 +- 183 files changed, 646 insertions(+), 754 deletions(-) delete mode 100644 util/qemu-openpty.c