All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Daniel P. Berrangé" <berrange@redhat.com>
To: Gerd Hoffmann <kraxel@redhat.com>
Cc: qemu-devel@nongnu.org,
	Pierrick Bouvier <pierrick.bouvier@oss.qualcomm.com>,
	"Michael S. Tsirkin" <mst@redhat.com>
Subject: Re: [PATCH 1/2] pci id: clarify pci subsystem id usage
Date: Thu, 15 Oct 2026 11:28:00 +0100	[thread overview]
Message-ID: <atCqsLfQb3rkKcsP@redhat.com> (raw)
In-Reply-To: <atCmDyLUrH6D0JaS@nucbdw.ghoffman.csb.home.kraxel.org>

On Thu, Oct 15, 2026 at 12:20:59PM +0200, Gerd Hoffmann wrote:
> On Thu, Oct 15, 2026 at 09:45:45AM +0100, Daniel P. Berrangé wrote:
> > >  1af4:1100
> > > -  Used as PCI Subsystem ID for existing hardware devices emulated
> > > -  by QEMU.
> > > +  Used as PCI Subsystem ID for existing hardware devices emulated by
> > > +  QEMU, such as the intel chipset devices.  Also used for virtio
> > > +  devices and generic pci class devices (1b36:00xx, see below).
> > 
> > When talking virtio devices, does that include virtio devices
> > implemented outside QEMU via vhostuser, or only in-QEMU virtio
> > devices ?
> 
> For vhost devices the pci config space emulation is still handled by
> qemu.  On a quick check I don't see anything special done for vhost,
> so I expect the same subsystem ID being used.

So IIUC, the QEMU subsystem ID applies to every PCI device exposed
from QEMU, except for VFIO passthrough of host devices.

> 
> Maybe we should extend the vhost protocol to allow overriding the
> subsystem id, so linux kernel / rust-vmm  / ... can have their own
> subsystem ids?

Would be a good idea as a way to distinguish the QEMU impls for
virtio devices, from 3rd party ones accessed via vhostuser.

With regards,
Daniel
-- 
|: https://berrange.com       ~~        https://hachyderm.io/@berrange :|
|: https://libvirt.org          ~~          https://entangle-photo.org :|
|: https://pixelfed.art/berrange   ~~    https://fstop138.berrange.com :|



  reply	other threads:[~2026-10-15 10:37 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-10-15  8:15 [PATCH 0/2] pci id documentation updates Gerd Hoffmann
2026-10-15  8:15 ` [PATCH 1/2] pci id: clarify pci subsystem id usage Gerd Hoffmann
2026-10-15  8:45   ` Daniel P. Berrangé
2026-10-15 10:20     ` Gerd Hoffmann
2026-10-15 10:28       ` Daniel P. Berrangé [this message]
2026-10-15  8:15 ` [PATCH 2/2] pci id: assign pci subsystem id for osmates Gerd Hoffmann
2026-10-15  8:38   ` Daniel P. Berrangé

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=atCqsLfQb3rkKcsP@redhat.com \
    --to=berrange@redhat.com \
    --cc=kraxel@redhat.com \
    --cc=mst@redhat.com \
    --cc=pierrick.bouvier@oss.qualcomm.com \
    --cc=qemu-devel@nongnu.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.