| Message ID | 20241002083646.2893078-1-marcandre.lureau@redhat.com |
|---|---|
| State | New |
| Headers | show |
On 02/10/2024 10.36, marcandre.lureau@redhat.com wrote: > From: Marc-André Lureau <marcandre.lureau@redhat.com> > > The following changes since commit 718780d20470c66a3a36d036b29148d5809dc855: > > Merge tag 'pull-nvme-20241001' of https://gitlab.com/birkelund/qemu into staging (2024-10-01 11:34:07 +0100) > > are available in the Git repository at: > > https://gitlab.com/marcandre.lureau/qemu.git tags/warn-pull-request > > for you to fetch changes up to baad82ecfb27474f17f4318502725622ab7170fc: > > qom/object: fix -Werror=maybe-uninitialized (2024-10-02 11:36:12 +0400) > > ---------------------------------------------------------------- > -Werror=maybe-uninitialized fixes > > ---------------------------------------------------------------- > > Marc-André Lureau (22): > util/coroutine: fix -Werror=maybe-uninitialized false-positive > util/timer: fix -Werror=maybe-uninitialized false-positive > hw/qxl: fix -Werror=maybe-uninitialized false-positives > nbd: fix -Werror=maybe-uninitialized false-positive > block/mirror: fix -Werror=maybe-uninitialized false-positive > block/mirror: fix -Werror=maybe-uninitialized false-positive > block/stream: fix -Werror=maybe-uninitialized false-positives > hw/ahci: fix -Werror=maybe-uninitialized false-positive > hw/vhost-scsi: fix -Werror=maybe-uninitialized > hw/sdhci: fix -Werror=maybe-uninitialized false-positive > block/block-copy: fix -Werror=maybe-uninitialized false-positive > migration: fix -Werror=maybe-uninitialized false-positives > hw/virtio-blk: fix -Werror=maybe-uninitialized false-positive > migration: fix -Werror=maybe-uninitialized false-positive > linux-user/hppa: fix -Werror=maybe-uninitialized false-positive > target/loongarch: fix -Werror=maybe-uninitialized false-positive > tests: fix -Werror=maybe-uninitialized false-positive > hw/virtio: fix -Werror=maybe-uninitialized > hw/virtio: freeing leaked memory from vhost_svq_get_buf in > vhost_svq_poll > block: fix -Werror=maybe-uninitialized false-positive > fsdep/9p: fix -Werror=maybe-uninitialized false-positive > qom/object: fix -Werror=maybe-uninitialized > > block/block-copy.c | 2 +- > block/file-posix.c | 2 +- > block/mirror.c | 8 ++++---- > block/stream.c | 6 +++--- > fsdev/9p-iov-marshal.c | 15 ++++++++++++--- > hw/block/virtio-blk.c | 2 +- > hw/display/qxl.c | 4 ++-- > hw/ide/ahci.c | 3 ++- > hw/scsi/vhost-scsi.c | 2 +- > hw/sd/sdhci.c | 2 +- > hw/virtio/vhost-shadow-virtqueue.c | 6 ++++-- > linux-user/hppa/cpu_loop.c | 2 ++ > migration/dirtyrate.c | 4 ++-- > migration/migration.c | 2 +- > migration/ram.c | 2 +- > nbd/client-connection.c | 2 +- > qom/object.c | 2 +- > target/loongarch/gdbstub.c | 26 ++++++++++++++------------ > tests/unit/test-bdrv-drain.c | 2 +- > tests/unit/test-block-iothread.c | 2 +- > util/qemu-coroutine.c | 2 +- > util/qemu-timer.c | 6 +++--- > roms/openbios | 2 +- Replying here as well, so that Peter has a chance to see it: The update to roms/openbios in patch 03 seems to have slipped in by accident, so I'd suggest to fix that first before merging this. Thomas
Hi On Wed, Oct 2, 2024 at 12:43 PM Thomas Huth <thuth@redhat.com> wrote: > On 02/10/2024 10.36, marcandre.lureau@redhat.com wrote: > > From: Marc-André Lureau <marcandre.lureau@redhat.com> > > > > The following changes since commit > 718780d20470c66a3a36d036b29148d5809dc855: > > > > Merge tag 'pull-nvme-20241001' of https://gitlab.com/birkelund/qemu > into staging (2024-10-01 11:34:07 +0100) > > > > are available in the Git repository at: > > > > https://gitlab.com/marcandre.lureau/qemu.git tags/warn-pull-request > > > > for you to fetch changes up to baad82ecfb27474f17f4318502725622ab7170fc: > > > > qom/object: fix -Werror=maybe-uninitialized (2024-10-02 11:36:12 > +0400) > > > > ---------------------------------------------------------------- > > -Werror=maybe-uninitialized fixes > > > > ---------------------------------------------------------------- > > > > Marc-André Lureau (22): > > util/coroutine: fix -Werror=maybe-uninitialized false-positive > > util/timer: fix -Werror=maybe-uninitialized false-positive > > hw/qxl: fix -Werror=maybe-uninitialized false-positives > > nbd: fix -Werror=maybe-uninitialized false-positive > > block/mirror: fix -Werror=maybe-uninitialized false-positive > > block/mirror: fix -Werror=maybe-uninitialized false-positive > > block/stream: fix -Werror=maybe-uninitialized false-positives > > hw/ahci: fix -Werror=maybe-uninitialized false-positive > > hw/vhost-scsi: fix -Werror=maybe-uninitialized > > hw/sdhci: fix -Werror=maybe-uninitialized false-positive > > block/block-copy: fix -Werror=maybe-uninitialized false-positive > > migration: fix -Werror=maybe-uninitialized false-positives > > hw/virtio-blk: fix -Werror=maybe-uninitialized false-positive > > migration: fix -Werror=maybe-uninitialized false-positive > > linux-user/hppa: fix -Werror=maybe-uninitialized false-positive > > target/loongarch: fix -Werror=maybe-uninitialized false-positive > > tests: fix -Werror=maybe-uninitialized false-positive > > hw/virtio: fix -Werror=maybe-uninitialized > > hw/virtio: freeing leaked memory from vhost_svq_get_buf in > > vhost_svq_poll > > block: fix -Werror=maybe-uninitialized false-positive > > fsdep/9p: fix -Werror=maybe-uninitialized false-positive > > qom/object: fix -Werror=maybe-uninitialized > > > > block/block-copy.c | 2 +- > > block/file-posix.c | 2 +- > > block/mirror.c | 8 ++++---- > > block/stream.c | 6 +++--- > > fsdev/9p-iov-marshal.c | 15 ++++++++++++--- > > hw/block/virtio-blk.c | 2 +- > > hw/display/qxl.c | 4 ++-- > > hw/ide/ahci.c | 3 ++- > > hw/scsi/vhost-scsi.c | 2 +- > > hw/sd/sdhci.c | 2 +- > > hw/virtio/vhost-shadow-virtqueue.c | 6 ++++-- > > linux-user/hppa/cpu_loop.c | 2 ++ > > migration/dirtyrate.c | 4 ++-- > > migration/migration.c | 2 +- > > migration/ram.c | 2 +- > > nbd/client-connection.c | 2 +- > > qom/object.c | 2 +- > > target/loongarch/gdbstub.c | 26 ++++++++++++++------------ > > tests/unit/test-bdrv-drain.c | 2 +- > > tests/unit/test-block-iothread.c | 2 +- > > util/qemu-coroutine.c | 2 +- > > util/qemu-timer.c | 6 +++--- > > roms/openbios | 2 +- > > Replying here as well, so that Peter has a chance to see it: > > The update to roms/openbios in patch 03 seems to have slipped in by > accident, so I'd suggest to fix that first before merging this. > > \o/ submodules :) Peter, do you want a v2? thanks
On Wed, 2 Oct 2024 at 09:45, Marc-André Lureau <marcandre.lureau@gmail.com> wrote: > > Hi > > On Wed, Oct 2, 2024 at 12:43 PM Thomas Huth <thuth@redhat.com> wrote: >> >> On 02/10/2024 10.36, marcandre.lureau@redhat.com wrote: >> > From: Marc-André Lureau <marcandre.lureau@redhat.com> >> > >> > The following changes since commit 718780d20470c66a3a36d036b29148d5809dc855: >> > >> > Merge tag 'pull-nvme-20241001' of https://gitlab.com/birkelund/qemu into staging (2024-10-01 11:34:07 +0100) >> > >> > are available in the Git repository at: >> > >> > https://gitlab.com/marcandre.lureau/qemu.git tags/warn-pull-request >> > >> > for you to fetch changes up to baad82ecfb27474f17f4318502725622ab7170fc: >> > >> > qom/object: fix -Werror=maybe-uninitialized (2024-10-02 11:36:12 +0400) >> > >> > ---------------------------------------------------------------- >> > -Werror=maybe-uninitialized fixes >> > >> > ---------------------------------------------------------------- >> > >> > Marc-André Lureau (22): >> > util/coroutine: fix -Werror=maybe-uninitialized false-positive >> > util/timer: fix -Werror=maybe-uninitialized false-positive >> > hw/qxl: fix -Werror=maybe-uninitialized false-positives >> > nbd: fix -Werror=maybe-uninitialized false-positive >> > block/mirror: fix -Werror=maybe-uninitialized false-positive >> > block/mirror: fix -Werror=maybe-uninitialized false-positive >> > block/stream: fix -Werror=maybe-uninitialized false-positives >> > hw/ahci: fix -Werror=maybe-uninitialized false-positive >> > hw/vhost-scsi: fix -Werror=maybe-uninitialized >> > hw/sdhci: fix -Werror=maybe-uninitialized false-positive >> > block/block-copy: fix -Werror=maybe-uninitialized false-positive >> > migration: fix -Werror=maybe-uninitialized false-positives >> > hw/virtio-blk: fix -Werror=maybe-uninitialized false-positive >> > migration: fix -Werror=maybe-uninitialized false-positive >> > linux-user/hppa: fix -Werror=maybe-uninitialized false-positive >> > target/loongarch: fix -Werror=maybe-uninitialized false-positive >> > tests: fix -Werror=maybe-uninitialized false-positive >> > hw/virtio: fix -Werror=maybe-uninitialized >> > hw/virtio: freeing leaked memory from vhost_svq_get_buf in >> > vhost_svq_poll >> > block: fix -Werror=maybe-uninitialized false-positive >> > fsdep/9p: fix -Werror=maybe-uninitialized false-positive >> > qom/object: fix -Werror=maybe-uninitialized >> > >> > block/block-copy.c | 2 +- >> > block/file-posix.c | 2 +- >> > block/mirror.c | 8 ++++---- >> > block/stream.c | 6 +++--- >> > fsdev/9p-iov-marshal.c | 15 ++++++++++++--- >> > hw/block/virtio-blk.c | 2 +- >> > hw/display/qxl.c | 4 ++-- >> > hw/ide/ahci.c | 3 ++- >> > hw/scsi/vhost-scsi.c | 2 +- >> > hw/sd/sdhci.c | 2 +- >> > hw/virtio/vhost-shadow-virtqueue.c | 6 ++++-- >> > linux-user/hppa/cpu_loop.c | 2 ++ >> > migration/dirtyrate.c | 4 ++-- >> > migration/migration.c | 2 +- >> > migration/ram.c | 2 +- >> > nbd/client-connection.c | 2 +- >> > qom/object.c | 2 +- >> > target/loongarch/gdbstub.c | 26 ++++++++++++++------------ >> > tests/unit/test-bdrv-drain.c | 2 +- >> > tests/unit/test-block-iothread.c | 2 +- >> > util/qemu-coroutine.c | 2 +- >> > util/qemu-timer.c | 6 +++--- >> > roms/openbios | 2 +- >> >> Replying here as well, so that Peter has a chance to see it: >> >> The update to roms/openbios in patch 03 seems to have slipped in by >> accident, so I'd suggest to fix that first before merging this. >> > > \o/ submodules :) > > Peter, do you want a v2? > thanks Yes, you'll need to resend the pullreq. (You can send just the cover letter for a trivial tweak like this.) -- PMM
From: Marc-André Lureau <marcandre.lureau@redhat.com> The following changes since commit 718780d20470c66a3a36d036b29148d5809dc855: Merge tag 'pull-nvme-20241001' of https://gitlab.com/birkelund/qemu into staging (2024-10-01 11:34:07 +0100) are available in the Git repository at: https://gitlab.com/marcandre.lureau/qemu.git tags/warn-pull-request for you to fetch changes up to baad82ecfb27474f17f4318502725622ab7170fc: qom/object: fix -Werror=maybe-uninitialized (2024-10-02 11:36:12 +0400) ---------------------------------------------------------------- -Werror=maybe-uninitialized fixes ---------------------------------------------------------------- Marc-André Lureau (22): util/coroutine: fix -Werror=maybe-uninitialized false-positive util/timer: fix -Werror=maybe-uninitialized false-positive hw/qxl: fix -Werror=maybe-uninitialized false-positives nbd: fix -Werror=maybe-uninitialized false-positive block/mirror: fix -Werror=maybe-uninitialized false-positive block/mirror: fix -Werror=maybe-uninitialized false-positive block/stream: fix -Werror=maybe-uninitialized false-positives hw/ahci: fix -Werror=maybe-uninitialized false-positive hw/vhost-scsi: fix -Werror=maybe-uninitialized hw/sdhci: fix -Werror=maybe-uninitialized false-positive block/block-copy: fix -Werror=maybe-uninitialized false-positive migration: fix -Werror=maybe-uninitialized false-positives hw/virtio-blk: fix -Werror=maybe-uninitialized false-positive migration: fix -Werror=maybe-uninitialized false-positive linux-user/hppa: fix -Werror=maybe-uninitialized false-positive target/loongarch: fix -Werror=maybe-uninitialized false-positive tests: fix -Werror=maybe-uninitialized false-positive hw/virtio: fix -Werror=maybe-uninitialized hw/virtio: freeing leaked memory from vhost_svq_get_buf in vhost_svq_poll block: fix -Werror=maybe-uninitialized false-positive fsdep/9p: fix -Werror=maybe-uninitialized false-positive qom/object: fix -Werror=maybe-uninitialized block/block-copy.c | 2 +- block/file-posix.c | 2 +- block/mirror.c | 8 ++++---- block/stream.c | 6 +++--- fsdev/9p-iov-marshal.c | 15 ++++++++++++--- hw/block/virtio-blk.c | 2 +- hw/display/qxl.c | 4 ++-- hw/ide/ahci.c | 3 ++- hw/scsi/vhost-scsi.c | 2 +- hw/sd/sdhci.c | 2 +- hw/virtio/vhost-shadow-virtqueue.c | 6 ++++-- linux-user/hppa/cpu_loop.c | 2 ++ migration/dirtyrate.c | 4 ++-- migration/migration.c | 2 +- migration/ram.c | 2 +- nbd/client-connection.c | 2 +- qom/object.c | 2 +- target/loongarch/gdbstub.c | 26 ++++++++++++++------------ tests/unit/test-bdrv-drain.c | 2 +- tests/unit/test-block-iothread.c | 2 +- util/qemu-coroutine.c | 2 +- util/qemu-timer.c | 6 +++--- roms/openbios | 2 +- 23 files changed, 61 insertions(+), 45 deletions(-)