Re: [PATCH] qapi: fix device-sync-config since-version

2024-11-08 Thread Michael S. Tsirkin
On Fri, Nov 08, 2024 at 09:35:08AM +0100, Markus Armbruster wrote: > Vladimir Sementsov-Ogievskiy writes: > > > Actually it comes in 9.2, not 9.1. > > > > Fixes: 3f98408e2e ("qapi: introduce device-sync-config") > > Signed-off-by: Vladimir Sementsov-Ogievskiy > > --- > > qapi/qdev.json | 2 +- >

Re: [PATCH v7 0/3] vhost-user-blk: live resize additional APIs

2024-11-06 Thread Michael S. Tsirkin
On Wed, Nov 06, 2024 at 02:18:34PM +0300, Vladimir Sementsov-Ogievskiy wrote: > v7: update QAPI version 9.1 -> 9.2 Not like this. ypur patches are merged, pls post a fix patch on top. Thanks! > Vladimir Sementsov-Ogievskiy (3): > qdev-monitor: add option to report GenericError from find_devic

Re: [PULL 23/65] qapi: introduce device-sync-config

2024-11-05 Thread Michael S. Tsirkin
On Tue, Nov 05, 2024 at 09:10:07AM +, Daniel P. Berrangé wrote: > On Mon, Nov 04, 2024 at 04:07:00PM -0500, Michael S. Tsirkin wrote: > > From: Vladimir Sementsov-Ogievskiy > > > > Add command to sync config from vhost-user backend to the device. It &g

[PULL 22/65] vhost-user-blk: split vhost_user_blk_sync_config()

2024-11-04 Thread Michael S. Tsirkin
.ru> Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin Reviewed-by: Stefano Garzarella --- hw/block/vhost-user-blk.c | 26 +++--- 1 file changed, 19 insertions(+), 7 deletions(-) diff --git a/hw/block/vhost-user-blk.c b/hw/block/vhost-user-blk.c index 5b7f

[PULL 23/65] qapi: introduce device-sync-config

2024-11-04 Thread Michael S. Tsirkin
allow it during migration. Let's not allow that. Signed-off-by: Vladimir Sementsov-Ogievskiy Reviewed-by: Markus Armbruster Acked-by: Raphael Norwitz Message-Id: <20240920094936.450987-4-vsement...@yandex-team.ru> Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin

Re: [PATCH for-9.2 v15 04/11] s390x/pci: Check for multifunction after device realization

2024-09-11 Thread Michael S. Tsirkin
On Wed, Sep 11, 2024 at 07:58:15PM +0900, Akihiko Odaki wrote: > On 2024/09/11 18:38, Cédric Le Goater wrote: > > +Matthew +Eric > > > > Side note for the maintainers : > > > > Before this change, the igb device, which is multifunction, was working > > fine under Linux. > > > > Was there a fix i

Re: [PATCH for-9.2 v15 00/11] hw/pci: SR-IOV related fixes and improvements

2024-09-11 Thread Michael S. Tsirkin
On Wed, Sep 11, 2024 at 12:05:46PM +0900, Akihiko Odaki wrote: > On 2024/09/11 0:27, Michael S. Tsirkin wrote: > > On Tue, Sep 10, 2024 at 04:13:14PM +0200, Cédric Le Goater wrote: > > > On 9/10/24 15:34, Michael S. Tsirkin wrote: > > > > On Tue, Sep 10, 2024 at 03:2

Re: [PATCH v5 0/3] vhost-user-blk: live resize additional APIs

2024-09-11 Thread Michael S. Tsirkin
On Tue, Jun 25, 2024 at 03:18:40PM +0300, Vladimir Sementsov-Ogievskiy wrote: > v5: > 03: drop extra check on is is runstate running Causes build failures when generating qdoc. https://gitlab.com/mstredhat/qemu/-/jobs/7792086965 > > Vladimir Sementsov-Ogievskiy (3): > qdev-monitor: add optio

Re: [PATCH] docs/devel: Prohibit calling object_unparent() for memory region

2024-09-10 Thread Michael S. Tsirkin
off-by: Akihiko Odaki Acked-by: Michael S. Tsirkin who's applying this? Paolo? > --- > docs/devel/memory.rst | 14 +++--- > 1 file changed, 7 insertions(+), 7 deletions(-) > > diff --git a/docs/devel/memory.rst b/docs/devel/memory.rst > index 69c5e3f914ac..8

Re: [PATCH for-9.2 v15 00/11] hw/pci: SR-IOV related fixes and improvements

2024-09-10 Thread Michael S. Tsirkin
On Tue, Sep 10, 2024 at 04:13:14PM +0200, Cédric Le Goater wrote: > On 9/10/24 15:34, Michael S. Tsirkin wrote: > > On Tue, Sep 10, 2024 at 03:21:54PM +0200, Cédric Le Goater wrote: > > > On 9/10/24 11:33, Akihiko Odaki wrote: > > > > On 2024/09/10 18:21, Michael S. T

Re: [PATCH for-9.2 v15 00/11] hw/pci: SR-IOV related fixes and improvements

2024-09-10 Thread Michael S. Tsirkin
On Tue, Sep 10, 2024 at 03:21:54PM +0200, Cédric Le Goater wrote: > On 9/10/24 11:33, Akihiko Odaki wrote: > > On 2024/09/10 18:21, Michael S. Tsirkin wrote: > > > On Fri, Aug 23, 2024 at 02:00:37PM +0900, Akihiko Odaki wrote: > > > > Supersedes: <20240714-ro

Re: [PATCH for-9.2 v15 00/11] hw/pci: SR-IOV related fixes and improvements

2024-09-10 Thread Michael S. Tsirkin
On Tue, Sep 10, 2024 at 06:33:01PM +0900, Akihiko Odaki wrote: > On 2024/09/10 18:21, Michael S. Tsirkin wrote: > > On Fri, Aug 23, 2024 at 02:00:37PM +0900, Akihiko Odaki wrote: > > > Supersedes: <20240714-rombar-v2-0-af1504ef5...@daynix.com> > > > ("[PATC

Re: [PATCH for-9.2 v15 00/11] hw/pci: SR-IOV related fixes and improvements

2024-09-10 Thread Michael S. Tsirkin
Do not reject VFs created after a PF". > - Added patch "hw/ppc/spapr_pci: Do not create DT for disabled PCI device". > - Added patch "hw/pci: Convert rom_bar into OnOffAuto". > - Dropped patch "hw/pci: Determine if rombar is explicitly enabled". > - Dropped

Re: [PATCH for-9.2 v12 00/11] hw/pci: SR-IOV related fixes and improvements

2024-08-05 Thread Michael S. Tsirkin
On Mon, Aug 05, 2024 at 04:10:02PM +0900, Akihiko Odaki wrote: > On 2024/08/05 15:39, Michael S. Tsirkin wrote: > > On Sun, Aug 04, 2024 at 06:01:36PM +0900, Akihiko Odaki wrote: > > > Supersedes: <20240714-rombar-v2-0-af1504ef5...@daynix.com> > > > ("[PATC

Re: [PATCH for-9.2 v12 00/11] hw/pci: SR-IOV related fixes and improvements

2024-08-04 Thread Michael S. Tsirkin
h "hw/qdev: Remove opts member". > - Link to v9: > https://lore.kernel.org/r/20240315-reuse-v9-0-67aa69af4...@daynix.com > > Changes in v9: > - Rebased. > - Restored '#include "qapi/error.h"' (Michael S. Tsirkin) > - Added patch "pcie_sriov: E

Re: [PATCH for-9.2 v11 06/11] pcie_sriov: Reuse SR-IOV VF device instances

2024-08-02 Thread Michael S. Tsirkin
On Fri, Aug 02, 2024 at 02:17:56PM +0900, Akihiko Odaki wrote: > Disable SR-IOV VF devices by reusing code to power down PCI devices > instead of removing them when the guest requests to disable VFs. This > allows to realize devices and report VF realization errors at PF > realization time. What k

Re: [PATCH for-9.2 v11 08/11] pcie_sriov: Remove num_vfs from PCIESriovPF

2024-08-02 Thread Michael S. Tsirkin
On Sat, Aug 03, 2024 at 12:38:10AM +0900, Akihiko Odaki wrote: > On 2024/08/02 21:58, Michael S. Tsirkin wrote: > > On Fri, Aug 02, 2024 at 02:17:58PM +0900, Akihiko Odaki wrote: > > > num_vfs is not migrated so use PCI_SRIOV_CTRL_VFE and PCI_SRIOV_NUM_VF > > > instead.

Re: [PATCH for-9.2 v11 08/11] pcie_sriov: Remove num_vfs from PCIESriovPF

2024-08-02 Thread Michael S. Tsirkin
On Fri, Aug 02, 2024 at 02:17:58PM +0900, Akihiko Odaki wrote: > num_vfs is not migrated so use PCI_SRIOV_CTRL_VFE and PCI_SRIOV_NUM_VF > instead. > > Signed-off-by: Akihiko Odaki > --- > include/hw/pci/pcie_sriov.h | 1 - > hw/pci/pcie_sriov.c | 28 > hw/pc

Re: [PATCH v2 0/4] hw/pci: Convert rom_bar into OnOffAuto

2024-08-01 Thread Michael S. Tsirkin
On Thu, Aug 01, 2024 at 12:59:57PM +0200, Markus Armbruster wrote: > Akihiko Odaki writes: > > > On 2024/07/31 17:32, Markus Armbruster wrote: > >> Akihiko Odaki writes: > >> > >>> rom_bar is tristate but was defined as uint32_t so convert it into > >>> OnOffAuto to clarify that. For compatibil

[PULL 13/19] Revert "pcie_sriov: Ensure VF function number does not overflow"

2024-08-01 Thread Michael S. Tsirkin
This reverts commit 77718701157f6ca77ea7a57b536fa0a22f676082. Signed-off-by: Michael S. Tsirkin --- docs/pcie_sriov.txt | 8 +++- include/hw/pci/pcie_sriov.h | 5 ++--- hw/net/igb.c| 13 +++-- hw/nvme/ctrl.c | 24 hw

Re: [PATCH v5 0/3] vhost-user-blk: live resize additional APIs

2024-08-01 Thread Michael S. Tsirkin
On Thu, Aug 01, 2024 at 11:35:19AM +0300, Vladimir Sementsov-Ogievskiy wrote: > On 01.07.24 23:55, Michael S. Tsirkin wrote: > > On Mon, Jul 01, 2024 at 08:42:39AM -0400, Raphael Norwitz wrote: > > > I have no issues with these APIs, but I'm not a QMP expert so others &g

Re: [PATCH v2 0/4] hw/pci: Convert rom_bar into OnOffAuto

2024-08-01 Thread Michael S. Tsirkin
On Thu, Aug 01, 2024 at 04:01:44PM +0900, Akihiko Odaki wrote: > rom_bar_set() and and underlying visit_type_str_preserving() are ugly, but > we can remove them once the deprecation period ends. On the other hand, if > we don't make this change, dev->opts will keep floating around, and we will > ev

Re: [PATCH v5 0/8] virtio-net: add support for SR-IOV emulation

2024-08-01 Thread Michael S. Tsirkin
On Thu, Aug 01, 2024 at 04:13:14PM +0900, Akihiko Odaki wrote: > I think it is a good idea to revert these patches for now OK I reverted the 2 patchsets. there were some bugfixes there but I had to revert them too due to the dependency. If appropriate, feel free to resubmit just the fixes. -- MS

Re: [PATCH RFC v4 0/7] virtio-net: add support for SR-IOV emulation

2024-07-31 Thread Michael S. Tsirkin
On Thu, Aug 01, 2024 at 02:37:55PM +0900, Akihiko Odaki wrote: > I don't know much about vDPA-capable device, but as a reference, igb only > has 8 VFs. modern vdpa capable devices have much more than 8 VFs, 8 is a very low number. -- MST

Re: [PATCH v2 0/4] hw/pci: Convert rom_bar into OnOffAuto

2024-07-31 Thread Michael S. Tsirkin
On Wed, Jul 31, 2024 at 10:32:19AM +0200, Markus Armbruster wrote: > Whether it would be worth your trouble and mine just to clean up > "rombar" seems highly dubious, though. Exactly. -- MST

Re: [PATCH v5 0/8] virtio-net: add support for SR-IOV emulation

2024-07-30 Thread Michael S. Tsirkin
On Tue, Jul 30, 2024 at 09:26:20PM +0900, Akihiko Odaki wrote: > On 2024/07/30 20:37, Michael S. Tsirkin wrote: > > On Mon, Jul 15, 2024 at 02:19:06PM +0900, Akihiko Odaki wrote: > > > Based-on: <20240714-rombar-v2-0-af1504ef5...@daynix.com> > > > ("[PATC

Re: [PATCH v5 0/8] virtio-net: add support for SR-IOV emulation

2024-07-30 Thread Michael S. Tsirkin
On Mon, Jul 15, 2024 at 02:19:06PM +0900, Akihiko Odaki wrote: > Based-on: <20240714-rombar-v2-0-af1504ef5...@daynix.com> > ("[PATCH v2 0/4] hw/pci: Convert rom_bar into OnOffAuto") OK I will revert this for now. We'll try again after the release, there will be time to address s390.

[PULL v2 61/61] hw/nvme: Add SPDM over DOE support

2024-07-23 Thread Michael S. Tsirkin
758-4-alistair.fran...@wdc.com> Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin --- include/hw/pci/pci_device.h | 7 ++ include/hw/pci/pcie_doe.h | 3 + hw/nvme/ctrl.c | 62 + docs/specs/index.rst| 1 + docs/specs/spdm.rst

[PULL v2 26/61] vhost,vhost-user: Add VIRTIO_F_IN_ORDER to vhost feature bits

2024-07-23 Thread Michael S. Tsirkin
if the backend does not support it. Acked-by: Eugenio Pérez Signed-off-by: Jonah Palmer Message-Id: <20240710125522.4168043-6-jonah.pal...@oracle.com> Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin --- hw/block/vhost-user-blk.c| 1 + hw/net/vhost_net.c

[PULL 62/63] hw/nvme: Add SPDM over DOE support

2024-07-21 Thread Michael S. Tsirkin
758-4-alistair.fran...@wdc.com> Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin --- include/hw/pci/pci_device.h | 7 ++ include/hw/pci/pcie_doe.h | 3 + hw/nvme/ctrl.c | 62 + docs/specs/index.rst| 1 + docs/specs/spdm.rst

[PULL 26/63] vhost,vhost-user: Add VIRTIO_F_IN_ORDER to vhost feature bits

2024-07-21 Thread Michael S. Tsirkin
if the backend does not support it. Acked-by: Eugenio Pérez Signed-off-by: Jonah Palmer Message-Id: <20240710125522.4168043-6-jonah.pal...@oracle.com> Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin --- hw/block/vhost-user-blk.c| 1 + hw/net/vhost_net.c

Re: [PATCH v4 6/6] virtio: Add VIRTIO_F_IN_ORDER property definition

2024-07-20 Thread Michael S. Tsirkin
On Wed, Jul 10, 2024 at 08:55:19AM -0400, Jonah Palmer wrote: > Extend the virtio device property definitions to include the > VIRTIO_F_IN_ORDER feature. > > The default state of this feature is disabled, allowing it to be > explicitly enabled where it's supported. > > Acked-by: Eugenio Pérez >

Re: [PATCH v10 06/12] pcie_sriov: Reuse SR-IOV VF device instances

2024-07-20 Thread Michael S. Tsirkin
On Sat, Jul 13, 2024 at 09:45:07PM +0900, Akihiko Odaki wrote: > On 2024/07/10 19:52, Michael S. Tsirkin wrote: > > On Wed, Jul 10, 2024 at 08:37:27AM +0200, Cédric Le Goater wrote: > > > Hello, > > > > > > This change introduced a regression on s390x. I could

Re: [PATCH v10 06/12] pcie_sriov: Reuse SR-IOV VF device instances

2024-07-10 Thread Michael S. Tsirkin
On Wed, Jul 10, 2024 at 08:37:27AM +0200, Cédric Le Goater wrote: > Hello, > > This change introduced a regression on s390x. I could have spotted it > earlier. Sorry about that. Here is the scenario, > > QEMU now creates automatically the PCI device objects representing the > VFs when the PF devi

Re: [PATCH v8 00/13] vhost-user: support any POSIX system (tested on macOS, FreeBSD, OpenBSD)

2024-07-05 Thread Michael S. Tsirkin
On Fri, Jul 05, 2024 at 10:39:33AM +0200, Stefano Garzarella wrote: > On Wed, Jul 03, 2024 at 06:49:30PM GMT, Michael S. Tsirkin wrote: > > On Tue, Jun 18, 2024 at 12:00:30PM +0200, Stefano Garzarella wrote: > > > As discussed with Michael and Markus [1], this version

Re: [PATCH v3] virtio: Implement Virtio Backend for SD/MMC in QEMU

2024-07-04 Thread Michael S. Tsirkin
On Thu, Jul 04, 2024 at 10:25:53AM +0300, Mikhail Krasheninnikov wrote: > > On Wed, 3 Jul 2024, Michael S. Tsirkin wrote: > > > On Wed, Jul 03, 2024 at 10:55:17PM +0300, Mikhail Krasheninnikov wrote: > > > > > > Hello, Alex! > > > > > > N

[PULL v3 80/85] pcie_sriov: Ensure VF function number does not overflow

2024-07-03 Thread Michael S. Tsirkin
From: Akihiko Odaki pci_new() aborts when creating a VF with a function number equals to or is greater than PCI_DEVFN_MAX. Signed-off-by: Akihiko Odaki Message-Id: <20240627-reuse-v10-5-7ca0b8ed3...@daynix.com> Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin ---

Re: [PATCH v8 00/13] vhost-user: support any POSIX system (tested on macOS, FreeBSD, OpenBSD)

2024-07-03 Thread Michael S. Tsirkin
On Tue, Jun 18, 2024 at 12:00:30PM +0200, Stefano Garzarella wrote: > As discussed with Michael and Markus [1], this version also includes the patch > on which v7 depended to simplify the merge in Michael's tree. > > The series is all reviewed, so if there are no new changes required, I would > as

[PULL v3 15/85] vhost-user: fix lost reconnect again

2024-07-03 Thread Michael S. Tsirkin
vhost-user devices have this issue, including vhost-user-blk/scsi. Fixes: 71e076a07d ("hw/virtio: generalise CHR_EVENT_CLOSED handling") Signed-off-by: Li Feng Message-Id: <20240516025753.130171-3-fen...@smartx.com> Reviewed-by: Raphael Norwitz Reviewed-by: Michael S. Tsirkin

[PULL v3 14/85] Revert "vhost-user: fix lost reconnect"

2024-07-03 Thread Michael S. Tsirkin
to false, resulting in subsequent reconnection not being executed. The next patch will completely fix this issue with a better approach. Signed-off-by: Li Feng Message-Id: <20240516025753.130171-2-fen...@smartx.com> Reviewed-by: Raphael Norwitz Reviewed-by: Michael S. Tsirkin Sign

[PULL v3 08/85] vhost/vhost-user: Add VIRTIO_F_NOTIFICATION_DATA to vhost feature bits

2024-07-03 Thread Michael S. Tsirkin
it can be disabled if the backend does not support it. Tested-by: Lei Yang Reviewed-by: Eugenio Pérez Signed-off-by: Jonah Palmer Message-Id: <20240315165557.26942-6-jonah.pal...@oracle.com> Acked-by: Srujana Challa Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin -

Re: [PATCH v3] virtio: Implement Virtio Backend for SD/MMC in QEMU

2024-07-03 Thread Michael S. Tsirkin
On Wed, Jul 03, 2024 at 10:55:17PM +0300, Mikhail Krasheninnikov wrote: > > Hello, Alex! > > No, there's no patch to the VirtIO specification yet. This is > proof-of-concept solution since I'm not sure that I did everything > correct with the design (and as folks' reviews show, for a good reaso

Re: [PATCH v10 11/12] hw/pci: Convert rom_bar into OnOffAuto

2024-07-03 Thread Michael S. Tsirkin
On Wed, Jul 03, 2024 at 01:00:21PM +0200, BALATON Zoltan wrote: > On Wed, 3 Jul 2024, Michael S. Tsirkin wrote: > > On Wed, Jul 03, 2024 at 04:15:23AM +0200, BALATON Zoltan wrote: > > > On Tue, 2 Jul 2024, Michael S. Tsirkin wrote: > > > > On Thu, Jun 27, 2024 at 0

Re: [PATCH v10 11/12] hw/pci: Convert rom_bar into OnOffAuto

2024-07-02 Thread Michael S. Tsirkin
On Wed, Jul 03, 2024 at 04:15:23AM +0200, BALATON Zoltan wrote: > On Tue, 2 Jul 2024, Michael S. Tsirkin wrote: > > On Thu, Jun 27, 2024 at 03:08:00PM +0900, Akihiko Odaki wrote: > > > rom_bar is tristate but was defined as uint32_t so convert it into > > > OnOffAuto.

[PULL v2 14/88] Revert "vhost-user: fix lost reconnect"

2024-07-02 Thread Michael S. Tsirkin
to false, resulting in subsequent reconnection not being executed. The next patch will completely fix this issue with a better approach. Signed-off-by: Li Feng Message-Id: <20240516025753.130171-2-fen...@smartx.com> Reviewed-by: Raphael Norwitz Reviewed-by: Michael S. Tsirkin Sign

Re: [PATCH v2] virtio: Implement Virtio Backend for SD/MMC in QEMU

2024-07-02 Thread Michael S. Tsirkin
> CC: qemu-block@nongnu.org > CC: Michael S. Tsirkin > CC: Kevin Wolf > CC: Stefan Hajnoczi > --- > > After a feedback, moved virtio.c from virtio core directory to hw/block. > >From what I see from the examples of virtio drivers, other files should > be where they

[PULL v2 83/88] pcie_sriov: Ensure VF function number does not overflow

2024-07-02 Thread Michael S. Tsirkin
From: Akihiko Odaki pci_new() aborts when creating a VF with a function number equals to or is greater than PCI_DEVFN_MAX. Signed-off-by: Akihiko Odaki Message-Id: <20240627-reuse-v10-5-7ca0b8ed3...@daynix.com> Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin ---

[PULL v2 15/88] vhost-user: fix lost reconnect again

2024-07-02 Thread Michael S. Tsirkin
vhost-user devices have this issue, including vhost-user-blk/scsi. Fixes: 71e076a07d ("hw/virtio: generalise CHR_EVENT_CLOSED handling") Signed-off-by: Li Feng Message-Id: <20240516025753.130171-3-fen...@smartx.com> Reviewed-by: Raphael Norwitz Reviewed-by: Michael S. Tsirkin

[PULL v2 53/88] vhost-user: enable frontends on any POSIX system

2024-07-02 Thread Michael S. Tsirkin
pe Mathieu-Daudé Tested-by: Philippe Mathieu-Daudé Acked-by: Stefan Hajnoczi Reviewed-by: David Hildenbrand Signed-off-by: Stefano Garzarella Message-Id: <20240618100455.145733-1-sgarz...@redhat.com> Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin --- hw/net/vhost_net

[PULL v2 15/91] vhost-user: fix lost reconnect again

2024-07-02 Thread Michael S. Tsirkin
vhost-user devices have this issue, including vhost-user-blk/scsi. Fixes: 71e076a07d ("hw/virtio: generalise CHR_EVENT_CLOSED handling") Signed-off-by: Li Feng Message-Id: <20240516025753.130171-3-fen...@smartx.com> Reviewed-by: Raphael Norwitz Reviewed-by: Michael S. Tsirkin

[PULL v2 08/88] vhost/vhost-user: Add VIRTIO_F_NOTIFICATION_DATA to vhost feature bits

2024-07-02 Thread Michael S. Tsirkin
it can be disabled if the backend does not support it. Tested-by: Lei Yang Reviewed-by: Eugenio Pérez Signed-off-by: Jonah Palmer Message-Id: <20240315165557.26942-6-jonah.pal...@oracle.com> Acked-by: Srujana Challa Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin -

[PULL v2 14/91] Revert "vhost-user: fix lost reconnect"

2024-07-02 Thread Michael S. Tsirkin
to false, resulting in subsequent reconnection not being executed. The next patch will completely fix this issue with a better approach. Signed-off-by: Li Feng Message-Id: <20240516025753.130171-2-fen...@smartx.com> Reviewed-by: Raphael Norwitz Reviewed-by: Michael S. Tsirkin Sign

[PULL v2 08/91] vhost/vhost-user: Add VIRTIO_F_NOTIFICATION_DATA to vhost feature bits

2024-07-02 Thread Michael S. Tsirkin
it can be disabled if the backend does not support it. Tested-by: Lei Yang Reviewed-by: Eugenio Pérez Signed-off-by: Jonah Palmer Message-Id: <20240315165557.26942-6-jonah.pal...@oracle.com> Acked-by: Srujana Challa Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin -

[PULL 86/91] pcie_sriov: Ensure VF function number does not overflow

2024-07-02 Thread Michael S. Tsirkin
From: Akihiko Odaki pci_new() aborts when creating a VF with a function number equals to or is greater than PCI_DEVFN_MAX. Signed-off-by: Akihiko Odaki Message-Id: <20240627-reuse-v10-5-7ca0b8ed3...@daynix.com> Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin ---

[PULL 53/91] vhost-user: enable frontends on any POSIX system

2024-07-02 Thread Michael S. Tsirkin
pe Mathieu-Daudé Tested-by: Philippe Mathieu-Daudé Acked-by: Stefan Hajnoczi Reviewed-by: David Hildenbrand Signed-off-by: Stefano Garzarella Message-Id: <20240618100455.145733-1-sgarz...@redhat.com> Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin --- hw/net/vhost_net

[PULL 15/91] vhost-user: fix lost reconnect again

2024-07-02 Thread Michael S. Tsirkin
vhost-user devices have this issue, including vhost-user-blk/scsi. Fixes: 71e076a07d ("hw/virtio: generalise CHR_EVENT_CLOSED handling") Signed-off-by: Li Feng Message-Id: <20240516025753.130171-3-fen...@smartx.com> Reviewed-by: Raphael Norwitz Reviewed-by: Michael S. Tsirkin

[PULL 14/91] Revert "vhost-user: fix lost reconnect"

2024-07-02 Thread Michael S. Tsirkin
to false, resulting in subsequent reconnection not being executed. The next patch will completely fix this issue with a better approach. Signed-off-by: Li Feng Message-Id: <20240516025753.130171-2-fen...@smartx.com> Reviewed-by: Raphael Norwitz Reviewed-by: Michael S. Tsirkin Sign

[PULL 08/91] vhost/vhost-user: Add VIRTIO_F_NOTIFICATION_DATA to vhost feature bits

2024-07-02 Thread Michael S. Tsirkin
it can be disabled if the backend does not support it. Tested-by: Lei Yang Reviewed-by: Eugenio Pérez Signed-off-by: Jonah Palmer Message-Id: <20240315165557.26942-6-jonah.pal...@oracle.com> Acked-by: Srujana Challa Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin -

Re: [PATCH v10 11/12] hw/pci: Convert rom_bar into OnOffAuto

2024-07-02 Thread Michael S. Tsirkin
On Thu, Jun 27, 2024 at 03:08:00PM +0900, Akihiko Odaki wrote: > rom_bar is tristate but was defined as uint32_t so convert it into > OnOffAuto. > > Signed-off-by: Akihiko Odaki Commit log should explain why this is an improvement, not just what's done. > diff --git a/docs/igd-assign.txt b/doc

Re: [PATCH v7 3/3] hw/nvme: Add SPDM over DOE support

2024-07-02 Thread Michael S. Tsirkin
On Fri, Jun 14, 2024 at 11:28:46AM +1000, Alistair Francis wrote: > From: Wilfred Mallawa > > Setup Data Object Exchance (DOE) as an extended capability for the NVME > controller and connect SPDM to it (CMA) to it. > > Signed-off-by: Wilfred Mallawa > Signed-off-by: Alistair Francis > Reviewed

Re: [PATCH v5 0/3] vhost-user-blk: live resize additional APIs

2024-07-01 Thread Michael S. Tsirkin
On Mon, Jul 01, 2024 at 08:42:39AM -0400, Raphael Norwitz wrote: > I have no issues with these APIs, but I'm not a QMP expert so others > should review those bits. > > For the vhost-user-blk code: > > Acked-by: Raphael Norwitz Could the relevant bits get ack from qapi maintainers please? > O

Re: [PATCH] virtio: Implement Virtio Backend for SD/MMC in QEMU

2024-07-01 Thread Michael S. Tsirkin
On Sun, Jun 30, 2024 at 01:43:48PM +, Mikhail Krasheninnikov wrote: > Add a Virtio backend for SD/MMC devices. Confirmed interoperability with > Linux. > > Signed-off-by: Mikhail Krasheninnikov > CC: Matwey Kornilov > CC: qemu-block@nongnu.org > CC: Michael S. Tsirki

Re: [PATCH 00/13] qapi: convert "Note" and "Example" sections to rST

2024-07-01 Thread Michael S. Tsirkin
with an amendment patch at next opportunity. > > Any feedback not implemented should be interpreted as evidence of a > forgetful (rather than a spiteful) mind. Please remind me where > appropriate. > > --js virtio things: Reviewed-by: Michael S. Tsirkin > John Snow (13):

Re: [PATCH v2 00/15] Fix check-qtest-ppc64 sanitizer errors

2024-07-01 Thread Michael S. Tsirkin
ned with the following change for GLib: > https://gitlab.gnome.org/GNOME/glib/-/merge_requests/4120 > > Signed-off-by: Akihiko Odaki Reviewed-by: Michael S. Tsirkin who's merging all this? > --- > Changes in v2: > - Rebased to "[PATCH] cpu: fix memleak of 'halt_c

Re: [PATCH 06/14] hw/virtio: Free vqs before vhost_dev_cleanup()

2024-06-26 Thread Michael S. Tsirkin
On Wed, Jun 26, 2024 at 08:06:29PM +0900, Akihiko Odaki wrote: > This suppresses LeakSanitizer warnings. > more specifically, is there a leak here this fixes? or a false positive warning? > Signed-off-by: Akihiko Odaki > --- > hw/virtio/vhost-user-base.c | 2 ++ > 1 file changed, 2 insertions(

Re: [PATCH RESEND v7 00/12] vhost-user: support any POSIX system (tested on macOS, FreeBSD, OpenBSD)

2024-06-17 Thread Michael S. Tsirkin
On Mon, Jun 17, 2024 at 02:59:14PM +0200, Stefano Garzarella wrote: > On Mon, Jun 17, 2024 at 02:02:30PM GMT, Markus Armbruster wrote: > > Stefano Garzarella writes: > > > > > Hi Michael, > > > > > > On Wed, Jun 12, 2024 at 03:01:28PM GMT, Stefano Garzarella wrote: > > > > This series should be

[PULL v3 14/41] Revert "vhost-user: fix lost reconnect"

2024-06-05 Thread Michael S. Tsirkin
to false, resulting in subsequent reconnection not being executed. The next patch will completely fix this issue with a better approach. Signed-off-by: Li Feng Message-Id: <20240516025753.130171-2-fen...@smartx.com> Reviewed-by: Raphael Norwitz Reviewed-by: Michael S. Tsirkin Sign

[PULL v3 15/41] vhost-user: fix lost reconnect again

2024-06-05 Thread Michael S. Tsirkin
vhost-user devices have this issue, including vhost-user-blk/scsi. Fixes: 71e076a07d ("hw/virtio: generalise CHR_EVENT_CLOSED handling") Signed-off-by: Li Feng Message-Id: <20240516025753.130171-3-fen...@smartx.com> Reviewed-by: Raphael Norwitz Reviewed-by: Michael S. Tsirkin

[PULL v3 08/41] vhost/vhost-user: Add VIRTIO_F_NOTIFICATION_DATA to vhost feature bits

2024-06-05 Thread Michael S. Tsirkin
it can be disabled if the backend does not support it. Tested-by: Lei Yang Reviewed-by: Eugenio Pérez Signed-off-by: Jonah Palmer Message-Id: <20240315165557.26942-6-jonah.pal...@oracle.com> Acked-by: Srujana Challa Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin -

[PULL 16/46] vhost-user: fix lost reconnect again

2024-06-04 Thread Michael S. Tsirkin
vhost-user devices have this issue, including vhost-user-blk/scsi. Fixes: 71e076a07d ("hw/virtio: generalise CHR_EVENT_CLOSED handling") Signed-off-by: Li Feng Message-Id: <20240516025753.130171-3-fen...@smartx.com> Reviewed-by: Raphael Norwitz Reviewed-by: Michael S. Tsirkin

[PULL 15/46] Revert "vhost-user: fix lost reconnect"

2024-06-04 Thread Michael S. Tsirkin
to false, resulting in subsequent reconnection not being executed. The next patch will completely fix this issue with a better approach. Signed-off-by: Li Feng Message-Id: <20240516025753.130171-2-fen...@smartx.com> Reviewed-by: Raphael Norwitz Reviewed-by: Michael S. Tsirkin Sign

[PULL 09/46] vhost/vhost-user: Add VIRTIO_F_NOTIFICATION_DATA to vhost feature bits

2024-06-04 Thread Michael S. Tsirkin
it can be disabled if the backend does not support it. Tested-by: Lei Yang Reviewed-by: Eugenio Pérez Signed-off-by: Jonah Palmer Message-Id: <20240315165557.26942-6-jonah.pal...@oracle.com> Acked-by: Srujana Challa Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin -

Re: [PATCH v8 00/15] hw/pci: SR-IOV related fixes and improvements

2024-06-04 Thread Michael S. Tsirkin
ynix.com > > Changes in v5: > - Added patch "hw/pci: Always call pcie_sriov_pf_reset()". > - Added patch "pcie_sriov: Reset SR-IOV extended capability". > - Removed a reference to PCI_SRIOV_CTRL_VFE in hw/nvme. > (Michael S. Tsirkin) > - Noted the impact

Re: [PATCH] Use "void *" as parameter for functions that are used for aio_set_event_notifier()

2024-06-02 Thread Michael S. Tsirkin
n "-fsanitize=undefined". > And this also prevents enabling the strict mode of CFI which is currently > disabled with -fsanitize-cfi-icall-generalize-pointers. Thus let us avoid > the problem by using "void *" as parameter in all spots where it is needed. > > Signed-of

Re: [PATCH] virtio-blk: remove SCSI passthrough functionality

2024-06-02 Thread Michael S. Tsirkin
On Wed, May 08, 2024 at 01:32:03PM +0200, Paolo Bonzini wrote: > The legacy SCSI passthrough functionality has never been enabled for > VIRTIO 1.0 and was deprecated more than four years ago. > > Get rid of it---almost, because QEMU is advertising it unconditionally > for legacy virtio-blk devices

Re: [PATCH for 9.1 v9 00/11] hw/pci: SR-IOV related fixes and improvements

2024-05-27 Thread Michael S. Tsirkin
cial hacks like on/off/auto. > [1]: https://patchew.org/QEMU/20231210-sriov-v2-0-b959e8a6d...@daynix.com/ > > Signed-off-by: Akihiko Odaki > --- > Changes in v9: > - Rebased. > - Restored '#include "qapi/error.h"' (Michael S. Tsirkin) > - Added pat

Re: [PATCH v2 1/2] Revert "vhost-user: fix lost reconnect"

2024-05-13 Thread Michael S. Tsirkin
On Mon, May 13, 2024 at 03:10:47PM +0800, Li Feng wrote: > This reverts commit f02a4b8e6431598612466f76aac64ab492849abf. > include subject of reverted commit and motivation for the revert pls. > Signed-off-by: Li Feng > --- > hw/block/vhost-user-blk.c | 2 +- > hw/scsi/vhost-user-scsi.

[PULL 6/7] vhost-user-blk: simplify and fix vhost_user_blk_handle_config_change

2024-04-09 Thread Michael S. Tsirkin
e do want to notify the guest unconditionally. So, no reason to create extra branches in the logic. Signed-off-by: Vladimir Sementsov-Ogievskiy Acked-by: Raphael Norwitz Message-Id: <20240329183758.3360733-2-vsement...@yandex-team.ru> Reviewed-by: Michael S. Tsirkin Signed-off-by: M

Re: [PATCH-for-9.1 v2 0/3] rdma: Remove RDMA subsystem and pvrdma device

2024-03-29 Thread Michael S. Tsirkin
On Thu, Mar 28, 2024 at 02:02:52PM +0100, Philippe Mathieu-Daudé wrote: > Since v1: > - split in 3 (Thomas) > - justify gluster removal Reviewed-by: Michael S. Tsirkin > Philippe Mathieu-Daudé (3): > hw/rdma: Remove pvrdma device and rdmacm-mux helper > migration: Re

Re: [PATCH-for-9.1] rdma: Remove RDMA subsystem and pvrdma device

2024-03-28 Thread Michael S. Tsirkin
On Thu, Mar 28, 2024 at 07:43:06AM +0100, Thomas Huth wrote: > On 27/03/2024 11.55, Philippe Mathieu-Daudé wrote: > > The whole RDMA subsystem was deprecated in commit e9a54265f5 > > ("hw/rdma: Deprecate the pvrdma device and the rdma subsystem") > > released in v8.2. Time to remove it. > > > > Ke

Re: [PATCH for-9.0 v3] vdpa-dev: Fix initialisation order to restore VDUSE compatibility

2024-03-18 Thread Michael S. Tsirkin
On Fri, Mar 15, 2024 at 04:59:49PM +0100, Kevin Wolf wrote: > VDUSE requires that virtqueues are first enabled before the DRIVER_OK > status flag is set; with the current API of the kernel module, it is > impossible to enable the opposite order in our block export code because > userspace is not no

Re: [PATCH for-9.0 v3] vdpa-dev: Fix initialisation order to restore VDUSE compatibility

2024-03-18 Thread Michael S. Tsirkin
On Mon, Mar 18, 2024 at 12:31:26PM +0800, Jason Wang wrote: > On Fri, Mar 15, 2024 at 11:59 PM Kevin Wolf wrote: > > > > VDUSE requires that virtqueues are first enabled before the DRIVER_OK > > status flag is set; with the current API of the kernel module, it is > > impossible to enable the oppos

[PULL 44/68] pcie_sriov: Reset SR-IOV extended capability

2024-03-12 Thread Michael S. Tsirkin
pcie_sriov_pf_reset(), which does not only disable VFs but also resets the capability. Signed-off-by: Akihiko Odaki Message-Id: <20240228-reuse-v8-3-282660281...@daynix.com> Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin Reviewed-by: Sriram Yagnaraman --- include/hw/pci/pcie_sriov.

[PULL 42/68] hw/nvme: Use pcie_sriov_num_vfs()

2024-03-12 Thread Michael S. Tsirkin
dd support for the Virtualization Management command") Suggested-by: Michael S. Tsirkin Signed-off-by: Akihiko Odaki Message-Id: <20240228-reuse-v8-1-282660281...@daynix.com> Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin --- hw/nvme/ctrl.c | 26 -

[PULL 46/68] hw/pci: Always call pcie_sriov_pf_reset()

2024-03-12 Thread Michael S. Tsirkin
From: Akihiko Odaki Call pcie_sriov_pf_reset() from pci_do_device_reset() just as we do for msi_reset() and msix_reset() to prevent duplicating code for each SR-IOV PF. Signed-off-by: Akihiko Odaki Message-Id: <20240228-reuse-v8-5-282660281...@daynix.com> Reviewed-by: Michael S. T

Re: [PATCH v6 3/3] hw/nvme: Add SPDM over DOE support

2024-03-12 Thread Michael S. Tsirkin
On Mon, Mar 11, 2024 at 11:15:37AM +1000, Alistair Francis wrote: > From: Wilfred Mallawa > > Setup Data Object Exchance (DOE) as an extended capability for the NVME > controller and connect SPDM to it (CMA) to it. > > Signed-off-by: Wilfred Mallawa > Signed-off-by: Alistair Francis > Reviewed

Re: [PATCH v8 00/15] hw/pci: SR-IOV related fixes and improvements

2024-03-12 Thread Michael S. Tsirkin
v: Do not manually unrealize". > - Restored patch "pcie_sriov: Release VFs failed to realize" that was > missed in v5. > - Link to v5: > https://lore.kernel.org/r/20240218-reuse-v5-0-e4fc1c19b...@daynix.com > > Changes in v5: > - Added patch "hw/pci: Always cal

Re: [PATCH v6 08/15] pcie_sriov: Reuse SR-IOV VF device instances

2024-03-12 Thread Michael S. Tsirkin
On Tue, Feb 20, 2024 at 09:24:43PM +0900, Akihiko Odaki wrote: > Disable SR-IOV VF devices by reusing code to power down PCI devices > instead of removing them when the guest requests to disable VFs. This > allows to realize devices and report VF realization errors at PF > realization time. > > Si

Re: [PATCH v6 07/15] pcie_sriov: Do not manually unrealize

2024-03-12 Thread Michael S. Tsirkin
On Tue, Feb 20, 2024 at 09:24:42PM +0900, Akihiko Odaki wrote: > A device gets automatically unrealized when being unparented. > > Signed-off-by: Akihiko Odaki > --- > hw/pci/pcie_sriov.c | 5 - > 1 file changed, 5 deletions(-) > > diff --git a/hw/pci/pcie_sriov.c b/hw/pci/pcie_sriov.c > in

Re: [PATCH v8 07/15] pcie_sriov: Do not manually unrealize

2024-03-12 Thread Michael S. Tsirkin
On Wed, Feb 28, 2024 at 08:33:18PM +0900, Akihiko Odaki wrote: > A device gets automatically unrealized when being unparented. > > Signed-off-by: Akihiko Odaki I was bisecting and when I bisected to this commit I got a build error: ../hw/pci/pcie_sriov.c: In function ‘register_vf’: ../hw/pci/pc

Re: [PATCH v2] virtio-blk: iothread-vq-mapping coroutine pool sizing

2024-03-12 Thread Michael S. Tsirkin
ed-by: Boaz Ben Shabat > Reported-by: Joe Mario > Signed-off-by: Stefan Hajnoczi Looks reasonable. Reviewed-by: Michael S. Tsirkin if you want me to merge it, let me know pls. > --- > v2: > - State the the tighter bounds reflect the fact that threads may only >

Re: [PATCH v2 00/15] hw/southbridge: Extract ICH9 QOM container model

2024-03-12 Thread Michael S. Tsirkin
On Mon, Feb 26, 2024 at 12:13:59PM +0100, Philippe Mathieu-Daudé wrote: > Since v1 [1]: > - Rebased on top of Bernhard patches > - Rename files with 'ich9_' prefix (Bernhard) > > Hi, > > I have a long standing southbridge QOM rework branches. Since > Bernhard is actively working on the PIIX, I'll

Re: [PATCH v1 2/8] virtio-pci: Lock ioeventfd state with VIRTIO_F_NOTIFICATION_DATA

2024-03-12 Thread Michael S. Tsirkin
On Tue, Mar 12, 2024 at 10:33:51AM -0400, Jonah Palmer wrote: > > > On 3/11/24 11:47 AM, Michael S. Tsirkin wrote: > > On Mon, Mar 11, 2024 at 10:53:25AM -0400, Jonah Palmer wrote: > > > > > > > > > On 3/8/24 2:19 PM, Michael S. Tsirkin wrote: > &

Re: [PATCH v1 2/8] virtio-pci: Lock ioeventfd state with VIRTIO_F_NOTIFICATION_DATA

2024-03-11 Thread Michael S. Tsirkin
On Mon, Mar 11, 2024 at 10:53:25AM -0400, Jonah Palmer wrote: > > > On 3/8/24 2:19 PM, Michael S. Tsirkin wrote: > > On Fri, Mar 08, 2024 at 12:45:13PM -0500, Jonah Palmer wrote: > > > > > > > > > On 3/8/24 12:36 PM, Eugenio Perez Martin wrote: > &g

Re: [PATCH v1 2/8] virtio-pci: Lock ioeventfd state with VIRTIO_F_NOTIFICATION_DATA

2024-03-08 Thread Michael S. Tsirkin
On Fri, Mar 08, 2024 at 12:45:13PM -0500, Jonah Palmer wrote: > > > On 3/8/24 12:36 PM, Eugenio Perez Martin wrote: > > On Fri, Mar 8, 2024 at 6: 01 PM Michael S. Tsirkin > > wrote: > > On Mon, Mar 04, 2024 at 02: 46: 06PM -0500, Jonah Palmer > > wrote: > &g

Re: [PATCH v1 2/8] virtio-pci: Lock ioeventfd state with VIRTIO_F_NOTIFICATION_DATA

2024-03-08 Thread Michael S. Tsirkin
On Mon, Mar 04, 2024 at 02:46:06PM -0500, Jonah Palmer wrote: > Prevent ioeventfd from being enabled/disabled when a virtio-pci > device has negotiated the VIRTIO_F_NOTIFICATION_DATA transport > feature. > > Due to ioeventfd not being able to carry the extra data associated with > this feature, th

Re: [PATCH v1 0/8] virtio,vhost: Add VIRTIO_F_NOTIFICATION_DATA support

2024-03-05 Thread Michael S. Tsirkin
On Wed, Mar 06, 2024 at 08:07:31AM +0100, Eugenio Perez Martin wrote: > On Wed, Mar 6, 2024 at 6:34 AM Jason Wang wrote: > > > > On Tue, Mar 5, 2024 at 3:46 AM Jonah Palmer wrote: > > > > > > The goal of these patches are to add support to a variety of virtio and > > > vhost devices for the VIRTI

Re: [PATCH v5 02/11] pcie_sriov: Validate NumVFs

2024-02-18 Thread Michael S. Tsirkin
On Sun, Feb 18, 2024 at 01:56:07PM +0900, Akihiko Odaki wrote: > The guest may write NumVFs greater than TotalVFs and that can lead > to buffer overflow in VF implementations. > > Cc: qemu-sta...@nongnu.org > Fixes: 7c0fa8dff811 ("pcie: Add support for Single Root I/O Virtualization > (SR/IOV)")

Re: [PATCH v5 05/11] vfio: Avoid inspecting option QDict for rombar

2024-02-18 Thread Michael S. Tsirkin
On Sun, Feb 18, 2024 at 01:56:10PM +0900, Akihiko Odaki wrote: > Use pci_rom_bar_explicitly_enabled() to determine if rombar is explicitly > enabled. > > Signed-off-by: Akihiko Odaki I see little point in all this reworks: QDict lookups are robust. But if Alex wants this change, I won't oppose

Re: [PATCH v4 9/9] hw/nvme: Refer to dev->exp.sriov_pf.num_vfs

2024-02-14 Thread Michael S. Tsirkin
On Thu, Feb 15, 2024 at 01:07:29AM +0900, Akihiko Odaki wrote: > On 2024/02/15 0:46, Michael S. Tsirkin wrote: > > On Wed, Feb 14, 2024 at 11:09:50PM +0900, Akihiko Odaki wrote: > > > On 2024/02/14 16:07, Michael S. Tsirkin wrote: > > > > On Wed, Feb 14, 2024 at 0

  1   2   3   4   5   6   7   8   9   >