qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: "Michael S. Tsirkin" <mst@redhat.com>
To: Richard Henderson <richard.henderson@linaro.org>
Cc: Peter Maydell <peter.maydell@linaro.org>,
	Jason Wang <jasowang@redhat.com>,
	qemu-devel@nongnu.org
Subject: Re: [PULL 00/57] pc,pci,virtio: features, fixes
Date: Tue, 5 Oct 2021 17:32:33 -0400	[thread overview]
Message-ID: <20211005173128-mutt-send-email-mst@kernel.org> (raw)
In-Reply-To: <20fc357d-586f-5178-6d2d-d44769f0a233@linaro.org>

On Tue, Oct 05, 2021 at 10:21:43AM -0700, Richard Henderson wrote:
> On 10/5/21 9:00 AM, Michael S. Tsirkin wrote:
> > The following changes since commit 9618c5badaa8eed25259cf095ff880efb939fbe7:
> > 
> >    Merge remote-tracking branch 'remotes/vivier/tags/trivial-branch-for-6.2-pull-request' into staging (2021-10-04 16:27:35 -0700)
> > 
> > are available in the Git repository at:
> > 
> >    git://git.kernel.org/pub/scm/virt/kvm/mst/qemu.git tags/for_upstream
> > 
> > for you to fetch changes up to c7d2f59cf940b8c8c52c29d5fa25613fe662f7b6:
> > 
> >    hw/i386/amd_iommu: Add description/category to TYPE_AMD_IOMMU_PCI (2021-10-05 11:46:45 -0400)
> > 
> > ----------------------------------------------------------------
> > pc,pci,virtio: features, fixes
> > 
> > VDPA multiqueue support.
> > A huge acpi refactoring.
> > Fixes, cleanups all over the place.
> > 
> > Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
> > 
> > ----------------------------------------------------------------
> > Ani Sinha (3):
> >        bios-tables-test: allow changes in DSDT ACPI tables for q35
> >        hw/i386/acpi: fix conflicting IO address range for acpi pci hotplug in q35
> >        bios-tables-test: Update ACPI DSDT table golden blobs for q35
> > 
> > Dr. David Alan Gilbert (1):
> >        virtio-balloon: Fix page-poison subsection name
> > 
> > Igor Mammedov (35):
> >        acpi: add helper routines to initialize ACPI tables
> >        acpi: build_rsdt: use acpi_table_begin()/acpi_table_end() instead of build_header()
> >        acpi: build_xsdt: use acpi_table_begin()/acpi_table_end() instead of build_header()
> >        acpi: build_slit: use acpi_table_begin()/acpi_table_end() instead of build_header()
> >        acpi: build_fadt: use acpi_table_begin()/acpi_table_end() instead of build_header()
> >        acpi: build_tpm2: use acpi_table_begin()/acpi_table_end() instead of build_header()
> >        acpi: acpi_build_hest: use acpi_table_begin()/acpi_table_end() instead of build_header()
> >        acpi: build_mcfg: use acpi_table_begin()/acpi_table_end() instead of build_header()
> >        acpi: build_hmat: use acpi_table_begin()/acpi_table_end() instead of build_header()
> >        acpi: nvdimm_build_nfit: use acpi_table_begin()/acpi_table_end() instead of build_header()
> >        acpi: nvdimm_build_ssdt: use acpi_table_begin()/acpi_table_end() instead of build_header()
> >        acpi: vmgenid_build_acpi: use acpi_table_begin()/acpi_table_end() instead of build_header()
> >        acpi: x86: build_dsdt: use acpi_table_begin()/acpi_table_end() instead of build_header()
> >        acpi: build_hpet: use acpi_table_begin()/acpi_table_end() instead of build_header()
> >        acpi: build_tpm_tcpa: use acpi_table_begin()/acpi_table_end() instead of build_header()
> >        acpi: arm/x86: build_srat: use acpi_table_begin()/acpi_table_end() instead of build_header()
> >        acpi: use build_append_int_noprefix() API to compose SRAT table
> >        acpi: build_dmar_q35: use acpi_table_begin()/acpi_table_end() instead of build_header()
> >        acpi: build_waet: use acpi_table_begin()/acpi_table_end() instead of build_header()
> >        acpi: build_amd_iommu: use acpi_table_begin()/acpi_table_end() instead of build_header()
> >        acpi: madt: arm/x86: use acpi_table_begin()/acpi_table_end() instead of build_header()
> >        acpi: x86: remove dead code
> >        acpi: x86: set enabled when composing _MAT entries
> >        acpi: x86: madt: use build_append_int_noprefix() API to compose MADT table
> >        acpi: arm/virt: madt: use build_append_int_noprefix() API to compose MADT table
> >        acpi: build_dsdt_microvm: use acpi_table_begin()/acpi_table_end() instead of build_header()
> >        acpi: arm: virt: build_dsdt: use acpi_table_begin()/acpi_table_end() instead of build_header()
> >        acpi: arm: virt: build_iort: use acpi_table_begin()/acpi_table_end() instead of build_header()
> >        acpi: arm/virt: convert build_iort() to endian agnostic build_append_FOO() API
> >        acpi: arm/virt: build_spcr: fix invalid cast
> >        acpi: arm/virt: build_spcr: use acpi_table_begin()/acpi_table_end() instead of build_header()
> >        acpi: arm/virt: build_gtdt: use acpi_table_begin()/acpi_table_end() instead of build_header()
> >        acpi: build_facs: use build_append_int_noprefix() API to compose table
> >        acpi: remove no longer used build_header()
> >        acpi: AcpiGenericAddress no longer used to map/access fields of MMIO, drop packed attribute
> > 
> > Jason Wang (10):
> >        vhost-vdpa: open device fd in net_init_vhost_vdpa()
> >        vhost-vdpa: classify one time request
> >        vhost-vdpa: prepare for the multiqueue support
> >        vhost-vdpa: let net_vhost_vdpa_init() returns NetClientState *
> >        net: introduce control client
> >        vhost-net: control virtqueue support
> >        virtio-net: use "queue_pairs" instead of "queues" when possible
> >        vhost: record the last virtqueue index for the virtio device
> >        virtio-net: vhost control virtqueue support
> >        vhost-vdpa: multiqueue support
> > 
> > Li Zhijian (1):
> >        nvdimm: release the correct device list
> > 
> > Philippe Mathieu-Daudé (5):
> >        hw/virtio: Acquire RCU read lock in virtqueue_packed_drop_all()
> >        hw/virtio: Have virtqueue_get_avail_bytes() pass caches arg to callees
> >        hw/i386/amd_iommu: Rename amdviPCI TypeInfo
> >        hw/i386/amd_iommu: Rename SysBus specific functions as amdvi_sysbus_X()
> >        hw/i386/amd_iommu: Add description/category to TYPE_AMD_IOMMU_PCI
> > 
> > Stefano Garzarella (2):
> >        vhost-vsock: fix migration issue when seqpacket is supported
> >        vhost-vsock: handle common features in vhost-vsock-common
> > 
> >   include/hw/acpi/acpi-defs.h            | 528 +------------------------------
> >   include/hw/acpi/acpi_dev_interface.h   |   3 +-
> >   include/hw/acpi/aml-build.h            |  37 ++-
> >   include/hw/acpi/ich9.h                 |   2 +-
> >   include/hw/i386/pc.h                   |   6 +-
> >   include/hw/virtio/vhost-vdpa.h         |   1 +
> >   include/hw/virtio/vhost-vsock-common.h |   5 +
> >   include/hw/virtio/vhost.h              |   2 +
> >   include/hw/virtio/virtio-net.h         |   5 +-
> >   include/net/net.h                      |   5 +
> >   include/net/vhost_net.h                |   6 +-
> >   hw/acpi/acpi-x86-stub.c                |   3 +-
> >   hw/acpi/aml-build.c                    | 193 +++++++-----
> >   hw/acpi/cpu.c                          |  17 +-
> >   hw/acpi/ghes.c                         |  10 +-
> >   hw/acpi/hmat.c                         |  14 +-
> >   hw/acpi/nvdimm.c                       |  76 ++---
> >   hw/acpi/pci.c                          |  18 +-
> >   hw/acpi/vmgenid.c                      |  13 +-
> >   hw/arm/virt-acpi-build.c               | 553 +++++++++++++++++++--------------
> >   hw/core/machine.c                      |   5 +-
> >   hw/i386/acpi-build.c                   | 292 +++++++++--------
> >   hw/i386/acpi-common.c                  | 160 +++++-----
> >   hw/i386/acpi-microvm.c                 |  13 +-
> >   hw/i386/amd_iommu.c                    |  41 ++-
> >   hw/net/vhost_net.c                     |  55 +++-
> >   hw/net/virtio-net.c                    | 165 +++++-----
> >   hw/virtio/vhost-user-vsock.c           |   4 +-
> >   hw/virtio/vhost-vdpa.c                 |  56 +++-
> >   hw/virtio/vhost-vsock-common.c         |  31 ++
> >   hw/virtio/vhost-vsock.c                |  11 +-
> >   hw/virtio/virtio-balloon.c             |   2 +-
> >   hw/virtio/virtio.c                     |  31 +-
> >   net/net.c                              |  24 +-
> >   net/vhost-vdpa.c                       | 127 +++++++-
> >   tests/data/acpi/q35/DSDT               | Bin 8289 -> 8289 bytes
> >   tests/data/acpi/q35/DSDT.acpihmat      | Bin 9614 -> 9614 bytes
> >   tests/data/acpi/q35/DSDT.bridge        | Bin 11003 -> 11003 bytes
> >   tests/data/acpi/q35/DSDT.cphp          | Bin 8753 -> 8753 bytes
> >   tests/data/acpi/q35/DSDT.dimmpxm       | Bin 9943 -> 9943 bytes
> >   tests/data/acpi/q35/DSDT.ipmibt        | Bin 8364 -> 8364 bytes
> >   tests/data/acpi/q35/DSDT.memhp         | Bin 9648 -> 9648 bytes
> >   tests/data/acpi/q35/DSDT.mmio64        | Bin 9419 -> 9419 bytes
> >   tests/data/acpi/q35/DSDT.nohpet        | Bin 8147 -> 8147 bytes
> >   tests/data/acpi/q35/DSDT.numamem       | Bin 8295 -> 8295 bytes
> >   tests/data/acpi/q35/DSDT.tis.tpm12     | Bin 8894 -> 8894 bytes
> >   tests/data/acpi/q35/DSDT.tis.tpm2      | Bin 8894 -> 8894 bytes
> >   47 files changed, 1202 insertions(+), 1312 deletions(-)
> 
> You missed updating the stub version of these functions:
> 
> ../src/hw/net/vhost_net-stub.c:34:5: error: conflicting types for ‘vhost_net_start’
>    34 | int vhost_net_start(VirtIODevice *dev,
>       |     ^~~~~~~~~~~~~~~
> In file included from ../src/hw/net/vhost_net-stub.c:19:
> /home/rth/qemu-publish/src/include/net/vhost_net.h:24:5: note: previous
> declaration of ‘vhost_net_start’ was here
>    24 | int vhost_net_start(VirtIODevice *dev, NetClientState *ncs,
>       |     ^~~~~~~~~~~~~~~
> ../src/hw/net/vhost_net-stub.c:40:6: error: conflicting types for ‘vhost_net_stop’
>    40 | void vhost_net_stop(VirtIODevice *dev,
>       |      ^~~~~~~~~~~~~~
> In file included from ../src/hw/net/vhost_net-stub.c:19:
> /home/rth/qemu-publish/src/include/net/vhost_net.h:26:6: note: previous
> declaration of ‘vhost_net_stop’ was here
>    26 | void vhost_net_stop(VirtIODevice *dev, NetClientState *ncs,
>       |      ^~~~~~~~~~~~~~
> 
> 

Indeed. I dropped these patches for now.
Could you refetch from same tag and confirm it's ok?
Jason, I think you want to respin.

-- 
MST



  reply	other threads:[~2021-10-05 21:34 UTC|newest]

Thread overview: 62+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-10-05 16:00 [PULL 00/57] pc,pci,virtio: features, fixes Michael S. Tsirkin
2021-10-05 16:00 ` [PULL 01/57] hw/virtio: Acquire RCU read lock in virtqueue_packed_drop_all() Michael S. Tsirkin
2021-10-05 16:00 ` [PULL 02/57] hw/virtio: Have virtqueue_get_avail_bytes() pass caches arg to callees Michael S. Tsirkin
2021-10-05 16:00 ` [PULL 03/57] vhost-vdpa: open device fd in net_init_vhost_vdpa() Michael S. Tsirkin
2021-10-05 16:00 ` [PULL 04/57] vhost-vdpa: classify one time request Michael S. Tsirkin
2021-10-05 16:00 ` [PULL 05/57] vhost-vdpa: prepare for the multiqueue support Michael S. Tsirkin
2021-10-05 16:01 ` [PULL 06/57] vhost-vdpa: let net_vhost_vdpa_init() returns NetClientState * Michael S. Tsirkin
2021-10-05 16:01 ` [PULL 07/57] net: introduce control client Michael S. Tsirkin
2021-10-05 16:01 ` [PULL 08/57] vhost-net: control virtqueue support Michael S. Tsirkin
2021-10-05 16:01 ` [PULL 09/57] virtio-net: use "queue_pairs" instead of "queues" when possible Michael S. Tsirkin
2021-10-05 16:01 ` [PULL 10/57] vhost: record the last virtqueue index for the virtio device Michael S. Tsirkin
2021-10-05 16:01 ` [PULL 11/57] virtio-net: vhost control virtqueue support Michael S. Tsirkin
2021-10-05 16:01 ` [PULL 12/57] vhost-vdpa: multiqueue support Michael S. Tsirkin
2021-10-05 16:01 ` [PULL 13/57] vhost-vsock: fix migration issue when seqpacket is supported Michael S. Tsirkin
2021-10-05 16:01 ` [PULL 14/57] vhost-vsock: handle common features in vhost-vsock-common Michael S. Tsirkin
2021-10-05 16:01 ` [PULL 15/57] acpi: add helper routines to initialize ACPI tables Michael S. Tsirkin
2021-10-05 16:01 ` [PULL 16/57] acpi: build_rsdt: use acpi_table_begin()/acpi_table_end() instead of build_header() Michael S. Tsirkin
2021-10-05 16:01 ` [PULL 17/57] acpi: build_xsdt: " Michael S. Tsirkin
2021-10-05 16:01 ` [PULL 18/57] acpi: build_slit: " Michael S. Tsirkin
2021-10-05 16:01 ` [PULL 19/57] acpi: build_fadt: " Michael S. Tsirkin
2021-10-05 16:01 ` [PULL 20/57] acpi: build_tpm2: " Michael S. Tsirkin
2021-10-05 16:01 ` [PULL 21/57] acpi: acpi_build_hest: " Michael S. Tsirkin
2021-10-05 16:02 ` [PULL 22/57] acpi: build_mcfg: " Michael S. Tsirkin
2021-10-05 16:02 ` [PULL 23/57] acpi: build_hmat: " Michael S. Tsirkin
2021-10-05 16:02 ` [PULL 24/57] acpi: nvdimm_build_nfit: " Michael S. Tsirkin
2021-10-05 16:02 ` [PULL 25/57] acpi: nvdimm_build_ssdt: " Michael S. Tsirkin
2021-10-05 16:02 ` [PULL 26/57] acpi: vmgenid_build_acpi: " Michael S. Tsirkin
2021-10-05 16:02 ` [PULL 27/57] acpi: x86: build_dsdt: " Michael S. Tsirkin
2021-10-05 16:02 ` [PULL 28/57] acpi: build_hpet: " Michael S. Tsirkin
2021-10-05 16:02 ` [PULL 29/57] acpi: build_tpm_tcpa: " Michael S. Tsirkin
2021-10-05 16:02 ` [PULL 30/57] acpi: arm/x86: build_srat: " Michael S. Tsirkin
2021-10-05 16:02 ` [PULL 31/57] acpi: use build_append_int_noprefix() API to compose SRAT table Michael S. Tsirkin
2021-10-05 16:03 ` [PULL 32/57] acpi: build_dmar_q35: use acpi_table_begin()/acpi_table_end() instead of build_header() Michael S. Tsirkin
2021-10-05 16:03 ` [PULL 33/57] acpi: build_waet: " Michael S. Tsirkin
2021-10-05 16:03 ` [PULL 34/57] acpi: build_amd_iommu: " Michael S. Tsirkin
2021-10-05 16:03 ` [PULL 35/57] acpi: madt: arm/x86: " Michael S. Tsirkin
2021-10-05 16:03 ` [PULL 36/57] acpi: x86: remove dead code Michael S. Tsirkin
2021-10-05 16:03 ` [PULL 37/57] acpi: x86: set enabled when composing _MAT entries Michael S. Tsirkin
2021-10-05 16:03 ` [PULL 38/57] acpi: x86: madt: use build_append_int_noprefix() API to compose MADT table Michael S. Tsirkin
2021-10-05 16:03 ` [PULL 39/57] acpi: arm/virt: " Michael S. Tsirkin
2021-10-05 16:03 ` [PULL 40/57] acpi: build_dsdt_microvm: use acpi_table_begin()/acpi_table_end() instead of build_header() Michael S. Tsirkin
2021-10-05 16:03 ` [PULL 41/57] acpi: arm: virt: build_dsdt: " Michael S. Tsirkin
2021-10-05 16:03 ` [PULL 42/57] acpi: arm: virt: build_iort: " Michael S. Tsirkin
2021-10-05 16:03 ` [PULL 43/57] acpi: arm/virt: convert build_iort() to endian agnostic build_append_FOO() API Michael S. Tsirkin
2021-10-05 16:03 ` [PULL 44/57] acpi: arm/virt: build_spcr: fix invalid cast Michael S. Tsirkin
2021-10-05 16:04 ` [PULL 45/57] acpi: arm/virt: build_spcr: use acpi_table_begin()/acpi_table_end() instead of build_header() Michael S. Tsirkin
2021-10-05 16:04 ` [PULL 46/57] acpi: arm/virt: build_gtdt: " Michael S. Tsirkin
2021-10-05 16:04 ` [PULL 47/57] acpi: build_facs: use build_append_int_noprefix() API to compose table Michael S. Tsirkin
2021-10-05 16:04 ` [PULL 48/57] acpi: remove no longer used build_header() Michael S. Tsirkin
2021-10-05 16:04 ` [PULL 49/57] acpi: AcpiGenericAddress no longer used to map/access fields of MMIO, drop packed attribute Michael S. Tsirkin
2021-10-05 16:04 ` [PULL 50/57] bios-tables-test: allow changes in DSDT ACPI tables for q35 Michael S. Tsirkin
2021-10-05 16:04 ` [PULL 51/57] hw/i386/acpi: fix conflicting IO address range for acpi pci hotplug in q35 Michael S. Tsirkin
2021-10-05 16:04 ` [PULL 52/57] bios-tables-test: Update ACPI DSDT table golden blobs for q35 Michael S. Tsirkin
2021-10-05 16:04 ` [PULL 53/57] virtio-balloon: Fix page-poison subsection name Michael S. Tsirkin
2021-10-05 16:04 ` [PULL 54/57] nvdimm: release the correct device list Michael S. Tsirkin
2021-10-05 16:04 ` [PULL 55/57] hw/i386/amd_iommu: Rename amdviPCI TypeInfo Michael S. Tsirkin
2021-10-05 16:04 ` [PULL 56/57] hw/i386/amd_iommu: Rename SysBus specific functions as amdvi_sysbus_X() Michael S. Tsirkin
2021-10-05 16:04 ` [PULL 57/57] hw/i386/amd_iommu: Add description/category to TYPE_AMD_IOMMU_PCI Michael S. Tsirkin
2021-10-05 17:21 ` [PULL 00/57] pc,pci,virtio: features, fixes Richard Henderson
2021-10-05 21:32   ` Michael S. Tsirkin [this message]
2021-10-05 23:36     ` Richard Henderson
2021-10-11  3:32     ` Jason Wang

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=20211005173128-mutt-send-email-mst@kernel.org \
    --to=mst@redhat.com \
    --cc=jasowang@redhat.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.org \
    --cc=richard.henderson@linaro.org \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).