public inbox for kvm@vger.kernel.org
 help / color / mirror / Atom feed
From: Alex Williamson <alex@shazbot.org>
To: David Matlack <dmatlack@google.com>
Cc: Alex Williamson <alex.williamson@redhat.com>,
	Jason Gunthorpe <jgg@nvidia.com>, Josh Hilke <jrhilke@google.com>,
	kvm@vger.kernel.org, Vipin Sharma <vipinsh@google.com>
Subject: Re: [PATCH 00/12] vfio: selftests: Support for multi-device tests
Date: Wed, 5 Nov 2025 14:54:21 -0700	[thread overview]
Message-ID: <20251105145421.216ae95c.alex@shazbot.org> (raw)
In-Reply-To: <CALzav=cMyD=SOUnpy3XnFn5cmn7xb8_YnTJHFyfCgqNpECCQsA@mail.gmail.com>

On Wed, 5 Nov 2025 13:03:51 -0800
David Matlack <dmatlack@google.com> wrote:

> On Wed, Nov 5, 2025 at 11:06 AM Alex Williamson <alex@shazbot.org> wrote:
> >
> > On Wed,  8 Oct 2025 23:25:19 +0000
> > David Matlack <dmatlack@google.com> wrote:
> >  
> > > This series adds support for tests that use multiple devices, and adds
> > > one new test, vfio_pci_device_init_perf_test, which measures parallel
> > > device initialization time to demonstrate the improvement from commit
> > > e908f58b6beb ("vfio/pci: Separate SR-IOV VF dev_set").
> > >
> > > This series also breaks apart the monolithic vfio_util.h and
> > > vfio_pci_device.c into separate files, to account for all the new code.
> > > This required some code motion so the diffstat looks large. The final
> > > layout is more granular and provides a better separation of the IOMMU
> > > code from the device code.  
> >
> > Hi David,
> >
> > This series doesn't apply to mainline currently and I see you have some
> > self-comments that suggests this is still a WIP, so I'll drop it and
> > look for a v2.  
> 
> Yes, I am going to send a v2 to address my self-comments. I'm also
> looking at getting reviews of this series from some fellow Googlers on
> the mailing list. Feel free to hold off for now.
> 
> >  I believe
> > https://lore.kernel.org/kvm/20250912222525.2515416-2-dmatlack@google.com/
> > is still in play though and does apply.  Thanks,  
> 
> This series will need another revision as well to address Sean's
> feedback, and I think can be merged via KVM maintainers.
> 
> That being said,
> https://lore.kernel.org/kvm/20250922224857.2528737-1-dmatlack@google.com/
> should apply and be ready to merge. It is a small prep patch for the
> KVM selftests integration.

Oops, yes, that's the one I meant to reference.  It links to the prior
one and I grabbed the wrong link from my local branch.  Thanks for the
confirmation.

Alex

      reply	other threads:[~2025-11-05 21:54 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-10-08 23:25 [PATCH 00/12] vfio: selftests: Support for multi-device tests David Matlack
2025-10-08 23:25 ` [PATCH 01/12] vfio: selftests: Split run.sh into separate scripts David Matlack
2025-11-10  3:21   ` Raghavendra Rao Ananta
2025-11-10 18:19     ` David Matlack
2025-10-08 23:25 ` [PATCH 02/12] vfio: selftests: Allow passing multiple BDFs on the command line David Matlack
2025-11-10  3:45   ` Raghavendra Rao Ananta
2025-11-10 18:31     ` David Matlack
2025-10-08 23:25 ` [PATCH 03/12] vfio: selftests: Rename struct vfio_iommu_mode to iommu_mode David Matlack
2025-10-08 23:25 ` [PATCH 04/12] vfio: selftests: Introduce struct iommu David Matlack
2025-10-08 23:25 ` [PATCH 05/12] vfio: selftests: Support multiple devices in the same container/iommufd David Matlack
2025-10-27 16:21   ` David Matlack
2025-11-07  3:37     ` Josh Hilke
2025-10-08 23:25 ` [PATCH 06/12] vfio: selftests: Eliminate overly chatty logging David Matlack
2025-10-08 23:25 ` [PATCH 07/12] vfio: selftests: Prefix logs with device BDF where relevant David Matlack
2025-11-10  4:54   ` Raghavendra Rao Ananta
2025-11-10 18:28     ` David Matlack
2025-10-08 23:25 ` [PATCH 08/12] vfio: selftests: Rename struct vfio_dma_region to dma_region David Matlack
2025-10-08 23:25 ` [PATCH 09/12] vfio: selftests: Move iommu_*() functions into iommu.c David Matlack
2025-10-08 23:25 ` [PATCH 10/12] vfio: selftests: Rename vfio_util.h to libvfio.h David Matlack
2025-10-08 23:25 ` [PATCH 11/12] vfio: selftests: Split libvfio.h into separate header files David Matlack
2025-10-08 23:25 ` [PATCH 12/12] vfio: selftests: Add vfio_pci_device_init_perf_test David Matlack
2025-10-16 16:12   ` David Matlack
2025-11-05 19:06 ` [PATCH 00/12] vfio: selftests: Support for multi-device tests Alex Williamson
2025-11-05 21:03   ` David Matlack
2025-11-05 21:54     ` Alex Williamson [this message]

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=20251105145421.216ae95c.alex@shazbot.org \
    --to=alex@shazbot.org \
    --cc=alex.williamson@redhat.com \
    --cc=dmatlack@google.com \
    --cc=jgg@nvidia.com \
    --cc=jrhilke@google.com \
    --cc=kvm@vger.kernel.org \
    --cc=vipinsh@google.com \
    /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