* [PATCH v2] content: Reserve ID 52 for QPSFI device
@ 2026-10-21 18:00 Peter Hilber
2026-10-21 19:25 ` Demi Marie Obenour
` (2 more replies)
0 siblings, 3 replies; 29+ messages in thread
From: Peter Hilber @ 2026-10-21 18:00 UTC (permalink / raw)
To: virtio-comment
Cc: Jan Kiszka, Stefan Hajnoczi, Michael S . Tsirkin, Trilok Soni,
Peter Hilber
Reserve a device ID for QPSFI devices (Qualcomm Platform Subsystem
Firmware Interface). QPSFI devices expose functionality across SoC
subsystems through a common Virtio-based protocol. There are no plans to
standardize the devices.
The Linux DCP display driver series is planned as the first user of this
device ID:
https://lore.kernel.org/all/20261015-jshekhar-drm-msm-dcp-v1-v1-10-48aad0fe4f72@oss.qualcomm.com/
Device ID 51 is already reserved for the Virtio HAB device:
https://lore.kernel.org/virtio-comment/20260702140939.3535968-1-zeloren@qti.qualcomm.com/
Signed-off-by: Peter Hilber <peter.hilber@oss.qualcomm.com>
---
v2:
- Rename the device from "Snapdragon device" to "QPSFI device".
- Refer to DCP Linux driver patch as planned first QPSFI user.
v1:
- Was posted as "content: Reserve ID 52 for Snapdragon device":
https://lore.kernel.org/virtio-comment/20260826115504.1955-1-peter.hilber@oss.qualcomm.com/
content.tex | 2 ++
1 file changed, 2 insertions(+)
diff --git a/content.tex b/content.tex
index 243ce2dde36b..7416939e67d3 100644
--- a/content.tex
+++ b/content.tex
@@ -805,6 +805,8 @@ \chapter{Device Types}\label{sec:Device Types}
\hline
50 & Video encoder/decoder device \\
\hline
+52 & QPSFI device \\
+\hline
\end{longtable}
Some of the devices above are unspecified by this document,
--
2.43.0
^ permalink raw reply related [flat|nested] 29+ messages in thread* Re: [PATCH v2] content: Reserve ID 52 for QPSFI device 2026-10-21 18:00 [PATCH v2] content: Reserve ID 52 for QPSFI device Peter Hilber @ 2026-10-21 19:25 ` Demi Marie Obenour 2026-10-21 19:50 ` Michael S. Tsirkin 2026-10-21 19:41 ` Michael S. Tsirkin 2026-10-22 13:56 ` Dmitry Baryshkov 2 siblings, 1 reply; 29+ messages in thread From: Demi Marie Obenour @ 2026-10-21 19:25 UTC (permalink / raw) To: Peter Hilber, virtio-comment Cc: Jan Kiszka, Stefan Hajnoczi, Michael S . Tsirkin, Trilok Soni [-- Attachment #1.1: Type: text/plain, Size: 1781 bytes --] On 10/21/26 14:00, Peter Hilber wrote: > Reserve a device ID for QPSFI devices (Qualcomm Platform Subsystem > Firmware Interface). QPSFI devices expose functionality across SoC > subsystems through a common Virtio-based protocol. There are no plans to > standardize the devices. > > The Linux DCP display driver series is planned as the first user of this > device ID: > > https://lore.kernel.org/all/20261015-jshekhar-drm-msm-dcp-v1-v1-10-48aad0fe4f72@oss.qualcomm.com/ > > Device ID 51 is already reserved for the Virtio HAB device: > > https://lore.kernel.org/virtio-comment/20260702140939.3535968-1-zeloren@qti.qualcomm.com/ > > Signed-off-by: Peter Hilber <peter.hilber@oss.qualcomm.com> > --- > > v2: > > - Rename the device from "Snapdragon device" to "QPSFI device". > > - Refer to DCP Linux driver patch as planned first QPSFI user. > > v1: > > - Was posted as "content: Reserve ID 52 for Snapdragon device": > > https://lore.kernel.org/virtio-comment/20260826115504.1955-1-peter.hilber@oss.qualcomm.com/ > > content.tex | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/content.tex b/content.tex > index 243ce2dde36b..7416939e67d3 100644 > --- a/content.tex > +++ b/content.tex > @@ -805,6 +805,8 @@ \chapter{Device Types}\label{sec:Device Types} > \hline > 50 & Video encoder/decoder device \\ > \hline > +52 & QPSFI device \\ > +\hline > \end{longtable} > > Some of the devices above are unspecified by this document, I think it would be best to have a single ID for vendor-specific devices. I believe most of the ID space has already been used, at least for virtio-PCI. -- Sincerely, Demi Marie Obenour (she/her/hers) [-- Attachment #2: OpenPGP digital signature --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH v2] content: Reserve ID 52 for QPSFI device 2026-10-21 19:25 ` Demi Marie Obenour @ 2026-10-21 19:50 ` Michael S. Tsirkin 2026-10-22 4:35 ` Cornelia Huck 0 siblings, 1 reply; 29+ messages in thread From: Michael S. Tsirkin @ 2026-10-21 19:50 UTC (permalink / raw) To: Demi Marie Obenour Cc: Peter Hilber, virtio-comment, Jan Kiszka, Stefan Hajnoczi, Trilok Soni On Wed, Oct 21, 2026 at 03:25:31PM -0400, Demi Marie Obenour wrote: > On 10/21/26 14:00, Peter Hilber wrote: > > Reserve a device ID for QPSFI devices (Qualcomm Platform Subsystem > > Firmware Interface). QPSFI devices expose functionality across SoC > > subsystems through a common Virtio-based protocol. There are no plans to > > standardize the devices. > > > > The Linux DCP display driver series is planned as the first user of this > > device ID: > > > > https://lore.kernel.org/all/20261015-jshekhar-drm-msm-dcp-v1-v1-10-48aad0fe4f72@oss.qualcomm.com/ > > > > Device ID 51 is already reserved for the Virtio HAB device: > > > > https://lore.kernel.org/virtio-comment/20260702140939.3535968-1-zeloren@qti.qualcomm.com/ > > > > Signed-off-by: Peter Hilber <peter.hilber@oss.qualcomm.com> > > --- > > > > v2: > > > > - Rename the device from "Snapdragon device" to "QPSFI device". > > > > - Refer to DCP Linux driver patch as planned first QPSFI user. > > > > v1: > > > > - Was posted as "content: Reserve ID 52 for Snapdragon device": > > > > https://lore.kernel.org/virtio-comment/20260826115504.1955-1-peter.hilber@oss.qualcomm.com/ > > > > content.tex | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/content.tex b/content.tex > > index 243ce2dde36b..7416939e67d3 100644 > > --- a/content.tex > > +++ b/content.tex > > @@ -805,6 +805,8 @@ \chapter{Device Types}\label{sec:Device Types} > > \hline > > 50 & Video encoder/decoder device \\ > > \hline > > +52 & QPSFI device \\ > > +\hline > > \end{longtable} > > > > Some of the devices above are unspecified by this document, > > I think it would be best to have a single ID for vendor-specific devices. > I believe most of the ID space has already been used, at least for virtio-PCI. > -- > Sincerely, > Demi Marie Obenour (she/her/hers) Sounds like a good idea to me. And then qualcomm can do what they want, like check subsystem id for pci or whatever? Only issue is there is no way to distinguish for mmio/ccw. Maybe, add vendor specific id as first e.g. 8 bytes ro in the config space? Peter? -- MST ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH v2] content: Reserve ID 52 for QPSFI device 2026-10-21 19:50 ` Michael S. Tsirkin @ 2026-10-22 4:35 ` Cornelia Huck 2026-10-22 7:07 ` Demi Marie Obenour 0 siblings, 1 reply; 29+ messages in thread From: Cornelia Huck @ 2026-10-22 4:35 UTC (permalink / raw) To: Michael S. Tsirkin, Demi Marie Obenour Cc: Peter Hilber, virtio-comment, Jan Kiszka, Stefan Hajnoczi, Trilok Soni On Wed, Oct 21 2026, "Michael S. Tsirkin" <mst@redhat.com> wrote: > On Wed, Oct 21, 2026 at 03:25:31PM -0400, Demi Marie Obenour wrote: >> On 10/21/26 14:00, Peter Hilber wrote: >> > Reserve a device ID for QPSFI devices (Qualcomm Platform Subsystem >> > Firmware Interface). QPSFI devices expose functionality across SoC >> > subsystems through a common Virtio-based protocol. There are no plans to >> > standardize the devices. >> > >> > The Linux DCP display driver series is planned as the first user of this >> > device ID: >> > >> > https://lore.kernel.org/all/20261015-jshekhar-drm-msm-dcp-v1-v1-10-48aad0fe4f72@oss.qualcomm.com/ >> > >> > Device ID 51 is already reserved for the Virtio HAB device: >> > >> > https://lore.kernel.org/virtio-comment/20260702140939.3535968-1-zeloren@qti.qualcomm.com/ >> > >> > Signed-off-by: Peter Hilber <peter.hilber@oss.qualcomm.com> >> > --- >> > >> > v2: >> > >> > - Rename the device from "Snapdragon device" to "QPSFI device". >> > >> > - Refer to DCP Linux driver patch as planned first QPSFI user. >> > >> > v1: >> > >> > - Was posted as "content: Reserve ID 52 for Snapdragon device": >> > >> > https://lore.kernel.org/virtio-comment/20260826115504.1955-1-peter.hilber@oss.qualcomm.com/ >> > >> > content.tex | 2 ++ >> > 1 file changed, 2 insertions(+) >> > >> > diff --git a/content.tex b/content.tex >> > index 243ce2dde36b..7416939e67d3 100644 >> > --- a/content.tex >> > +++ b/content.tex >> > @@ -805,6 +805,8 @@ \chapter{Device Types}\label{sec:Device Types} >> > \hline >> > 50 & Video encoder/decoder device \\ >> > \hline >> > +52 & QPSFI device \\ >> > +\hline >> > \end{longtable} >> > >> > Some of the devices above are unspecified by this document, >> >> I think it would be best to have a single ID for vendor-specific devices. >> I believe most of the ID space has already been used, at least for virtio-PCI. >> -- >> Sincerely, >> Demi Marie Obenour (she/her/hers) > > Sounds like a good idea to me. And then qualcomm can do what they > want, like check subsystem id for pci or whatever? > Only issue is there is no way to distinguish for mmio/ccw. > Maybe, add vendor specific id as first e.g. 8 bytes ro in the config space? > Peter? Using some information in the config space looks like the most flexible way to me. Are we still happy with how the device ids are organized in general (i.e. when do we want a new one, and when do we want to extend an existing device?) ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH v2] content: Reserve ID 52 for QPSFI device 2026-10-22 4:35 ` Cornelia Huck @ 2026-10-22 7:07 ` Demi Marie Obenour 2026-10-22 7:24 ` Michael S. Tsirkin 0 siblings, 1 reply; 29+ messages in thread From: Demi Marie Obenour @ 2026-10-22 7:07 UTC (permalink / raw) To: Cornelia Huck, Michael S. Tsirkin Cc: Peter Hilber, virtio-comment, Jan Kiszka, Stefan Hajnoczi, Trilok Soni [-- Attachment #1.1: Type: text/plain, Size: 3078 bytes --] On 10/22/26 00:35, Cornelia Huck wrote: > On Wed, Oct 21 2026, "Michael S. Tsirkin" <mst@redhat.com> wrote: > >> On Wed, Oct 21, 2026 at 03:25:31PM -0400, Demi Marie Obenour wrote: >>> On 10/21/26 14:00, Peter Hilber wrote: >>>> Reserve a device ID for QPSFI devices (Qualcomm Platform Subsystem >>>> Firmware Interface). QPSFI devices expose functionality across SoC >>>> subsystems through a common Virtio-based protocol. There are no plans to >>>> standardize the devices. >>>> >>>> The Linux DCP display driver series is planned as the first user of this >>>> device ID: >>>> >>>> https://lore.kernel.org/all/20261015-jshekhar-drm-msm-dcp-v1-v1-10-48aad0fe4f72@oss.qualcomm.com/ >>>> >>>> Device ID 51 is already reserved for the Virtio HAB device: >>>> >>>> https://lore.kernel.org/virtio-comment/20260702140939.3535968-1-zeloren@qti.qualcomm.com/ >>>> >>>> Signed-off-by: Peter Hilber <peter.hilber@oss.qualcomm.com> >>>> --- >>>> >>>> v2: >>>> >>>> - Rename the device from "Snapdragon device" to "QPSFI device". >>>> >>>> - Refer to DCP Linux driver patch as planned first QPSFI user. >>>> >>>> v1: >>>> >>>> - Was posted as "content: Reserve ID 52 for Snapdragon device": >>>> >>>> https://lore.kernel.org/virtio-comment/20260826115504.1955-1-peter.hilber@oss.qualcomm.com/ >>>> >>>> content.tex | 2 ++ >>>> 1 file changed, 2 insertions(+) >>>> >>>> diff --git a/content.tex b/content.tex >>>> index 243ce2dde36b..7416939e67d3 100644 >>>> --- a/content.tex >>>> +++ b/content.tex >>>> @@ -805,6 +805,8 @@ \chapter{Device Types}\label{sec:Device Types} >>>> \hline >>>> 50 & Video encoder/decoder device \\ >>>> \hline >>>> +52 & QPSFI device \\ >>>> +\hline >>>> \end{longtable} >>>> >>>> Some of the devices above are unspecified by this document, >>> >>> I think it would be best to have a single ID for vendor-specific devices. >>> I believe most of the ID space has already been used, at least for virtio-PCI. >>> -- >>> Sincerely, >>> Demi Marie Obenour (she/her/hers) >> >> Sounds like a good idea to me. And then qualcomm can do what they >> want, like check subsystem id for pci or whatever? >> Only issue is there is no way to distinguish for mmio/ccw. >> Maybe, add vendor specific id as first e.g. 8 bytes ro in the config space? >> Peter? > > Using some information in the config space looks like the most flexible > way to me. Are we still happy with how the device ids are organized in > general (i.e. when do we want a new one, and when do we want to extend > an existing device?) For a platform device like this, I'd prefer to pass the information out of band. After all, this device will only ever be found on Qualcomm SoCs, and presumably the kernel knows exactly which PCI device is supposed to be the QPSFI device. This means that the kernel can just special-case it. Would it make sense to put this in the ACPI tables or device tree? -- Sincerely, Demi Marie Obenour (she/her/hers) [-- Attachment #2: OpenPGP digital signature --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH v2] content: Reserve ID 52 for QPSFI device 2026-10-22 7:07 ` Demi Marie Obenour @ 2026-10-22 7:24 ` Michael S. Tsirkin 2026-10-22 8:41 ` Manos Pitsidianakis 0 siblings, 1 reply; 29+ messages in thread From: Michael S. Tsirkin @ 2026-10-22 7:24 UTC (permalink / raw) To: Demi Marie Obenour Cc: Cornelia Huck, Peter Hilber, virtio-comment, Jan Kiszka, Stefan Hajnoczi, Trilok Soni On Thu, Oct 22, 2026 at 03:07:41AM -0400, Demi Marie Obenour wrote: > On 10/22/26 00:35, Cornelia Huck wrote: > > On Wed, Oct 21 2026, "Michael S. Tsirkin" <mst@redhat.com> wrote: > > > >> On Wed, Oct 21, 2026 at 03:25:31PM -0400, Demi Marie Obenour wrote: > >>> On 10/21/26 14:00, Peter Hilber wrote: > >>>> Reserve a device ID for QPSFI devices (Qualcomm Platform Subsystem > >>>> Firmware Interface). QPSFI devices expose functionality across SoC > >>>> subsystems through a common Virtio-based protocol. There are no plans to > >>>> standardize the devices. > >>>> > >>>> The Linux DCP display driver series is planned as the first user of this > >>>> device ID: > >>>> > >>>> https://lore.kernel.org/all/20261015-jshekhar-drm-msm-dcp-v1-v1-10-48aad0fe4f72@oss.qualcomm.com/ > >>>> > >>>> Device ID 51 is already reserved for the Virtio HAB device: > >>>> > >>>> https://lore.kernel.org/virtio-comment/20260702140939.3535968-1-zeloren@qti.qualcomm.com/ > >>>> > >>>> Signed-off-by: Peter Hilber <peter.hilber@oss.qualcomm.com> > >>>> --- > >>>> > >>>> v2: > >>>> > >>>> - Rename the device from "Snapdragon device" to "QPSFI device". > >>>> > >>>> - Refer to DCP Linux driver patch as planned first QPSFI user. > >>>> > >>>> v1: > >>>> > >>>> - Was posted as "content: Reserve ID 52 for Snapdragon device": > >>>> > >>>> https://lore.kernel.org/virtio-comment/20260826115504.1955-1-peter.hilber@oss.qualcomm.com/ > >>>> > >>>> content.tex | 2 ++ > >>>> 1 file changed, 2 insertions(+) > >>>> > >>>> diff --git a/content.tex b/content.tex > >>>> index 243ce2dde36b..7416939e67d3 100644 > >>>> --- a/content.tex > >>>> +++ b/content.tex > >>>> @@ -805,6 +805,8 @@ \chapter{Device Types}\label{sec:Device Types} > >>>> \hline > >>>> 50 & Video encoder/decoder device \\ > >>>> \hline > >>>> +52 & QPSFI device \\ > >>>> +\hline > >>>> \end{longtable} > >>>> > >>>> Some of the devices above are unspecified by this document, > >>> > >>> I think it would be best to have a single ID for vendor-specific devices. > >>> I believe most of the ID space has already been used, at least for virtio-PCI. > >>> -- > >>> Sincerely, > >>> Demi Marie Obenour (she/her/hers) > >> > >> Sounds like a good idea to me. And then qualcomm can do what they > >> want, like check subsystem id for pci or whatever? > >> Only issue is there is no way to distinguish for mmio/ccw. > >> Maybe, add vendor specific id as first e.g. 8 bytes ro in the config space? > >> Peter? > > > > Using some information in the config space looks like the most flexible > > way to me. Are we still happy with how the device ids are organized in > > general (i.e. when do we want a new one, and when do we want to extend > > an existing device?) > > For a platform device like this, I'd prefer to pass the information out > of band. After all, this device will only ever be found on Qualcomm > SoCs, and presumably the kernel knows exactly which PCI device is > supposed to be the QPSFI device. This means that the kernel can just > special-case it. Yes but we are discussing making this id a general "vendor specific device", as opposed to a "qualcomm device". Could you share, e.g. if this device has config space, and whether reading X bytes at offset 0 of this device is harmless, for some value of X? > Would it make sense to put this in the ACPI tables or device tree? As long as it's vendor specific. > -- > Sincerely, > Demi Marie Obenour (she/her/hers) ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH v2] content: Reserve ID 52 for QPSFI device 2026-10-22 7:24 ` Michael S. Tsirkin @ 2026-10-22 8:41 ` Manos Pitsidianakis 2026-10-22 13:19 ` Michael S. Tsirkin 0 siblings, 1 reply; 29+ messages in thread From: Manos Pitsidianakis @ 2026-10-22 8:41 UTC (permalink / raw) To: Michael S. Tsirkin Cc: Demi Marie Obenour, Cornelia Huck, Peter Hilber, virtio-comment, Jan Kiszka, Stefan Hajnoczi, Trilok Soni On Thu, Oct 22, 2026 at 10:24 AM Michael S. Tsirkin <mst@redhat.com> wrote: > > On Thu, Oct 22, 2026 at 03:07:41AM -0400, Demi Marie Obenour wrote: > > On 10/22/26 00:35, Cornelia Huck wrote: > > > On Wed, Oct 21 2026, "Michael S. Tsirkin" <mst@redhat.com> wrote: > > > > > >> On Wed, Oct 21, 2026 at 03:25:31PM -0400, Demi Marie Obenour wrote: > > >>> On 10/21/26 14:00, Peter Hilber wrote: > > >>>> Reserve a device ID for QPSFI devices (Qualcomm Platform Subsystem > > >>>> Firmware Interface). QPSFI devices expose functionality across SoC > > >>>> subsystems through a common Virtio-based protocol. There are no plans to > > >>>> standardize the devices. > > >>>> > > >>>> The Linux DCP display driver series is planned as the first user of this > > >>>> device ID: > > >>>> > > >>>> https://lore.kernel.org/all/20261015-jshekhar-drm-msm-dcp-v1-v1-10-48aad0fe4f72@oss.qualcomm.com/ > > >>>> > > >>>> Device ID 51 is already reserved for the Virtio HAB device: > > >>>> > > >>>> https://lore.kernel.org/virtio-comment/20260702140939.3535968-1-zeloren@qti.qualcomm.com/ > > >>>> > > >>>> Signed-off-by: Peter Hilber <peter.hilber@oss.qualcomm.com> > > >>>> --- > > >>>> > > >>>> v2: > > >>>> > > >>>> - Rename the device from "Snapdragon device" to "QPSFI device". > > >>>> > > >>>> - Refer to DCP Linux driver patch as planned first QPSFI user. > > >>>> > > >>>> v1: > > >>>> > > >>>> - Was posted as "content: Reserve ID 52 for Snapdragon device": > > >>>> > > >>>> https://lore.kernel.org/virtio-comment/20260826115504.1955-1-peter.hilber@oss.qualcomm.com/ > > >>>> > > >>>> content.tex | 2 ++ > > >>>> 1 file changed, 2 insertions(+) > > >>>> > > >>>> diff --git a/content.tex b/content.tex > > >>>> index 243ce2dde36b..7416939e67d3 100644 > > >>>> --- a/content.tex > > >>>> +++ b/content.tex > > >>>> @@ -805,6 +805,8 @@ \chapter{Device Types}\label{sec:Device Types} > > >>>> \hline > > >>>> 50 & Video encoder/decoder device \\ > > >>>> \hline > > >>>> +52 & QPSFI device \\ > > >>>> +\hline > > >>>> \end{longtable} > > >>>> > > >>>> Some of the devices above are unspecified by this document, > > >>> > > >>> I think it would be best to have a single ID for vendor-specific devices. > > >>> I believe most of the ID space has already been used, at least for virtio-PCI. > > >>> -- > > >>> Sincerely, > > >>> Demi Marie Obenour (she/her/hers) > > >> > > >> Sounds like a good idea to me. And then qualcomm can do what they > > >> want, like check subsystem id for pci or whatever? > > >> Only issue is there is no way to distinguish for mmio/ccw. > > >> Maybe, add vendor specific id as first e.g. 8 bytes ro in the config space? > > >> Peter? > > > > > > Using some information in the config space looks like the most flexible > > > way to me. Are we still happy with how the device ids are organized in > > > general (i.e. when do we want a new one, and when do we want to extend > > > an existing device?) > > > > For a platform device like this, I'd prefer to pass the information out > > of band. After all, this device will only ever be found on Qualcomm > > SoCs, and presumably the kernel knows exactly which PCI device is > > supposed to be the QPSFI device. This means that the kernel can just > > special-case it. > > Yes but we are discussing making this id a general "vendor specific > device", as opposed to a "qualcomm device". Is there a reason why a generic VIRTIO_ID_VENDOR wouldn't work? Instead of adding more and more IDs for different devices that will never been upstreamed. Then, a kernel can discover all vendor devices when enumerating the transport, ignore them unless it knows specifically about them through ACPI or device tree. It doesn't make sense to me that HAB/QPSFI need an ID when they are unusable to anyone except for qualcomm kernel drivers so maybe we should require that an ID allocation always needs an explicit justification (i.e. "the spec will be upstreamed", or "we cannot use VIRTIO_ID_VENDOR because X/Y/Z"). -- Manos Pitsidianakis Emulation and Virtualization Engineer at Linaro Ltd ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH v2] content: Reserve ID 52 for QPSFI device 2026-10-22 8:41 ` Manos Pitsidianakis @ 2026-10-22 13:19 ` Michael S. Tsirkin 2026-10-22 14:07 ` Dmitry Baryshkov 2026-10-22 14:50 ` Zelong Ren 0 siblings, 2 replies; 29+ messages in thread From: Michael S. Tsirkin @ 2026-10-22 13:19 UTC (permalink / raw) To: Manos Pitsidianakis Cc: Demi Marie Obenour, Cornelia Huck, Peter Hilber, virtio-comment, Jan Kiszka, Stefan Hajnoczi, Trilok Soni On Thu, Oct 22, 2026 at 11:41:17AM +0300, Manos Pitsidianakis wrote: > On Thu, Oct 22, 2026 at 10:24 AM Michael S. Tsirkin <mst@redhat.com> wrote: > > > > On Thu, Oct 22, 2026 at 03:07:41AM -0400, Demi Marie Obenour wrote: > > > On 10/22/26 00:35, Cornelia Huck wrote: > > > > On Wed, Oct 21 2026, "Michael S. Tsirkin" <mst@redhat.com> wrote: > > > > > > > >> On Wed, Oct 21, 2026 at 03:25:31PM -0400, Demi Marie Obenour wrote: > > > >>> On 10/21/26 14:00, Peter Hilber wrote: > > > >>>> Reserve a device ID for QPSFI devices (Qualcomm Platform Subsystem > > > >>>> Firmware Interface). QPSFI devices expose functionality across SoC > > > >>>> subsystems through a common Virtio-based protocol. There are no plans to > > > >>>> standardize the devices. > > > >>>> > > > >>>> The Linux DCP display driver series is planned as the first user of this > > > >>>> device ID: > > > >>>> > > > >>>> https://lore.kernel.org/all/20261015-jshekhar-drm-msm-dcp-v1-v1-10-48aad0fe4f72@oss.qualcomm.com/ > > > >>>> > > > >>>> Device ID 51 is already reserved for the Virtio HAB device: > > > >>>> > > > >>>> https://lore.kernel.org/virtio-comment/20260702140939.3535968-1-zeloren@qti.qualcomm.com/ > > > >>>> > > > >>>> Signed-off-by: Peter Hilber <peter.hilber@oss.qualcomm.com> > > > >>>> --- > > > >>>> > > > >>>> v2: > > > >>>> > > > >>>> - Rename the device from "Snapdragon device" to "QPSFI device". > > > >>>> > > > >>>> - Refer to DCP Linux driver patch as planned first QPSFI user. > > > >>>> > > > >>>> v1: > > > >>>> > > > >>>> - Was posted as "content: Reserve ID 52 for Snapdragon device": > > > >>>> > > > >>>> https://lore.kernel.org/virtio-comment/20260826115504.1955-1-peter.hilber@oss.qualcomm.com/ > > > >>>> > > > >>>> content.tex | 2 ++ > > > >>>> 1 file changed, 2 insertions(+) > > > >>>> > > > >>>> diff --git a/content.tex b/content.tex > > > >>>> index 243ce2dde36b..7416939e67d3 100644 > > > >>>> --- a/content.tex > > > >>>> +++ b/content.tex > > > >>>> @@ -805,6 +805,8 @@ \chapter{Device Types}\label{sec:Device Types} > > > >>>> \hline > > > >>>> 50 & Video encoder/decoder device \\ > > > >>>> \hline > > > >>>> +52 & QPSFI device \\ > > > >>>> +\hline > > > >>>> \end{longtable} > > > >>>> > > > >>>> Some of the devices above are unspecified by this document, > > > >>> > > > >>> I think it would be best to have a single ID for vendor-specific devices. > > > >>> I believe most of the ID space has already been used, at least for virtio-PCI. > > > >>> -- > > > >>> Sincerely, > > > >>> Demi Marie Obenour (she/her/hers) > > > >> > > > >> Sounds like a good idea to me. And then qualcomm can do what they > > > >> want, like check subsystem id for pci or whatever? > > > >> Only issue is there is no way to distinguish for mmio/ccw. > > > >> Maybe, add vendor specific id as first e.g. 8 bytes ro in the config space? > > > >> Peter? > > > > > > > > Using some information in the config space looks like the most flexible > > > > way to me. Are we still happy with how the device ids are organized in > > > > general (i.e. when do we want a new one, and when do we want to extend > > > > an existing device?) > > > > > > For a platform device like this, I'd prefer to pass the information out > > > of band. After all, this device will only ever be found on Qualcomm > > > SoCs, and presumably the kernel knows exactly which PCI device is > > > supposed to be the QPSFI device. This means that the kernel can just > > > special-case it. > > > > Yes but we are discussing making this id a general "vendor specific > > device", as opposed to a "qualcomm device". > > Is there a reason why a generic VIRTIO_ID_VENDOR wouldn't work? > > Instead of adding more and more IDs for different devices that will > never been upstreamed. > > Then, a kernel can discover all vendor devices when enumerating the > transport, ignore them unless it knows specifically about them through > ACPI or device tree. > > It doesn't make sense to me that HAB/QPSFI need an ID when they are > unusable to anyone except for qualcomm kernel drivers so maybe we > should require that an ID allocation always needs an explicit > justification (i.e. "the spec will be upstreamed", or "we cannot use > VIRTIO_ID_VENDOR because X/Y/Z"). Right so let's just reserve ID 52 as VIRTIO_ID_VENDOR for now. And document precisely what you are saying. And let's add a chapter for it please: "A driver is not automatically loaded based on VIRTIO_ID_VENDOR alone, instead, the platform probes the device, verifies it is the correct vendor specific driver and loads it, all in a platform specific way". and conformance clauses stating the same. -- MST ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH v2] content: Reserve ID 52 for QPSFI device 2026-10-22 13:19 ` Michael S. Tsirkin @ 2026-10-22 14:07 ` Dmitry Baryshkov 2026-10-22 15:05 ` Cornelia Huck 2026-10-22 15:44 ` Michael S. Tsirkin 2026-10-22 14:50 ` Zelong Ren 1 sibling, 2 replies; 29+ messages in thread From: Dmitry Baryshkov @ 2026-10-22 14:07 UTC (permalink / raw) To: Michael S. Tsirkin Cc: Manos Pitsidianakis, Demi Marie Obenour, Cornelia Huck, Peter Hilber, virtio-comment, Jan Kiszka, Stefan Hajnoczi, Trilok Soni On Thu, Oct 22, 2026 at 09:19:16AM -0400, Michael S. Tsirkin wrote: > On Thu, Oct 22, 2026 at 11:41:17AM +0300, Manos Pitsidianakis wrote: > > On Thu, Oct 22, 2026 at 10:24 AM Michael S. Tsirkin <mst@redhat.com> wrote: > > > > > > On Thu, Oct 22, 2026 at 03:07:41AM -0400, Demi Marie Obenour wrote: > > > > On 10/22/26 00:35, Cornelia Huck wrote: > > > > > On Wed, Oct 21 2026, "Michael S. Tsirkin" <mst@redhat.com> wrote: > > > > > > > > > >> On Wed, Oct 21, 2026 at 03:25:31PM -0400, Demi Marie Obenour wrote: > > > > >>> On 10/21/26 14:00, Peter Hilber wrote: > > > > >>>> Reserve a device ID for QPSFI devices (Qualcomm Platform Subsystem > > > > >>>> Firmware Interface). QPSFI devices expose functionality across SoC > > > > >>>> subsystems through a common Virtio-based protocol. There are no plans to > > > > >>>> standardize the devices. > > > > >>>> > > > > >>>> The Linux DCP display driver series is planned as the first user of this > > > > >>>> device ID: > > > > >>>> > > > > >>>> https://lore.kernel.org/all/20261015-jshekhar-drm-msm-dcp-v1-v1-10-48aad0fe4f72@oss.qualcomm.com/ > > > > >>>> > > > > >>>> Device ID 51 is already reserved for the Virtio HAB device: > > > > >>>> > > > > >>>> https://lore.kernel.org/virtio-comment/20260702140939.3535968-1-zeloren@qti.qualcomm.com/ > > > > >>>> > > > > >>>> Signed-off-by: Peter Hilber <peter.hilber@oss.qualcomm.com> > > > > >>>> --- > > > > >>>> > > > > >>>> v2: > > > > >>>> > > > > >>>> - Rename the device from "Snapdragon device" to "QPSFI device". > > > > >>>> > > > > >>>> - Refer to DCP Linux driver patch as planned first QPSFI user. > > > > >>>> > > > > >>>> v1: > > > > >>>> > > > > >>>> - Was posted as "content: Reserve ID 52 for Snapdragon device": > > > > >>>> > > > > >>>> https://lore.kernel.org/virtio-comment/20260826115504.1955-1-peter.hilber@oss.qualcomm.com/ > > > > >>>> > > > > >>>> content.tex | 2 ++ > > > > >>>> 1 file changed, 2 insertions(+) > > > > >>>> > > > > >>>> diff --git a/content.tex b/content.tex > > > > >>>> index 243ce2dde36b..7416939e67d3 100644 > > > > >>>> --- a/content.tex > > > > >>>> +++ b/content.tex > > > > >>>> @@ -805,6 +805,8 @@ \chapter{Device Types}\label{sec:Device Types} > > > > >>>> \hline > > > > >>>> 50 & Video encoder/decoder device \\ > > > > >>>> \hline > > > > >>>> +52 & QPSFI device \\ > > > > >>>> +\hline > > > > >>>> \end{longtable} > > > > >>>> > > > > >>>> Some of the devices above are unspecified by this document, > > > > >>> > > > > >>> I think it would be best to have a single ID for vendor-specific devices. > > > > >>> I believe most of the ID space has already been used, at least for virtio-PCI. > > > > >>> -- > > > > >>> Sincerely, > > > > >>> Demi Marie Obenour (she/her/hers) > > > > >> > > > > >> Sounds like a good idea to me. And then qualcomm can do what they > > > > >> want, like check subsystem id for pci or whatever? > > > > >> Only issue is there is no way to distinguish for mmio/ccw. > > > > >> Maybe, add vendor specific id as first e.g. 8 bytes ro in the config space? > > > > >> Peter? > > > > > > > > > > Using some information in the config space looks like the most flexible > > > > > way to me. Are we still happy with how the device ids are organized in > > > > > general (i.e. when do we want a new one, and when do we want to extend > > > > > an existing device?) > > > > > > > > For a platform device like this, I'd prefer to pass the information out > > > > of band. After all, this device will only ever be found on Qualcomm > > > > SoCs, and presumably the kernel knows exactly which PCI device is > > > > supposed to be the QPSFI device. This means that the kernel can just > > > > special-case it. > > > > > > Yes but we are discussing making this id a general "vendor specific > > > device", as opposed to a "qualcomm device". > > > > Is there a reason why a generic VIRTIO_ID_VENDOR wouldn't work? > > > > Instead of adding more and more IDs for different devices that will > > never been upstreamed. The driver for the QPSFI (well, at least for the display controller) has been posted and it targets being upstreamed. > > > > Then, a kernel can discover all vendor devices when enumerating the > > transport, ignore them unless it knows specifically about them through > > ACPI or device tree. In case of QPSFI the device is visible to the host through the VirtIO-MMIO on steroids. However, nothing prevents one from passing that device into the guest VM (e.g. in QEMU) where it then becomes a VirtIO-PCI device. The same issues would apply to any other vendor trying to implement the driver for this VirtIO device (and trying to run the generic enough kernel which supports multiple "vendor" devices). > > > > It doesn't make sense to me that HAB/QPSFI need an ID when they are > > unusable to anyone except for qualcomm kernel drivers so maybe we > > should require that an ID allocation always needs an explicit > > justification (i.e. "the spec will be upstreamed", or "we cannot use > > VIRTIO_ID_VENDOR because X/Y/Z"). > > Right so let's just reserve ID 52 as VIRTIO_ID_VENDOR for now. > And document precisely what you are saying. > > And let's add a chapter for it please: > > "A driver is not automatically loaded based on VIRTIO_ID_VENDOR alone, > instead, the platform probes the device, verifies it is the > correct vendor specific driver and loads it, all in > a platform specific way". From my point of view this would be very fragile as different vendors would implement different mechanisms to identify the device. It would make it pretty much impossible to support multiple "vendor" devices in the upstream kernel. Let's define some standard way for the vendors to declare that this device is "Qualcomm", "NXP" or "hobbyist which doesn't own PCI or USB id block". > > and conformance clauses stating the same. -- With best wishes Dmitry ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH v2] content: Reserve ID 52 for QPSFI device 2026-10-22 14:07 ` Dmitry Baryshkov @ 2026-10-22 15:05 ` Cornelia Huck 2026-10-22 15:35 ` Parav Pandit 2026-10-22 15:44 ` Michael S. Tsirkin 1 sibling, 1 reply; 29+ messages in thread From: Cornelia Huck @ 2026-10-22 15:05 UTC (permalink / raw) To: Dmitry Baryshkov, Michael S. Tsirkin Cc: Manos Pitsidianakis, Demi Marie Obenour, Peter Hilber, virtio-comment, Jan Kiszka, Stefan Hajnoczi, Trilok Soni On Thu, Oct 22 2026, Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com> wrote: > Let's define some standard way for the vendors to declare that this > device is "Qualcomm", "NXP" or "hobbyist which doesn't own PCI or USB id > block". Are we able to use the first <n> bytes of config space for that? I guess that is the only way that can possibly work for any transport. Then, maybe something like one or two bytes for a vendor id (including one for "testing purposes" or so), and the rest to be used in a vendor-specific way. ^ permalink raw reply [flat|nested] 29+ messages in thread
* RE: [PATCH v2] content: Reserve ID 52 for QPSFI device 2026-10-22 15:05 ` Cornelia Huck @ 2026-10-22 15:35 ` Parav Pandit 2026-10-22 15:41 ` Michael S. Tsirkin 0 siblings, 1 reply; 29+ messages in thread From: Parav Pandit @ 2026-10-22 15:35 UTC (permalink / raw) To: Cornelia Huck, Dmitry Baryshkov, Michael S. Tsirkin Cc: Manos Pitsidianakis, Demi Marie Obenour, Peter Hilber, virtio-comment@lists.linux.dev, Jan Kiszka, Stefan Hajnoczi, Trilok Soni > From: Cornelia Huck <cohuck@redhat.com> > Sent: 22 October 2026 08:35 PM > > On Thu, Oct 22 2026, Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com> wrote: > > > Let's define some standard way for the vendors to declare that this > > device is "Qualcomm", "NXP" or "hobbyist which doesn't own PCI or USB id > > block". > > Are we able to use the first <n> bytes of config space for that? I guess > that is the only way that can possibly work for any transport. > > Then, maybe something like one or two bytes for a vendor id (including > one for "testing purposes" or so), and the rest to be used in a > vendor-specific way. > device ids are independent of the transport. but Peter already announced that: " Reserve a device ID for QPSFI devices (Qualcomm Platform Subsystem Firmware Interface). QPSFI devices expose functionality across SoC subsystems through a common Virtio-based protocol. There are no plans to standardize the devices." So, I am unable to understand why should it be part of vendor agnostic virtio spec? Why a vendor specific implementation of device and driver is not sufficient? ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH v2] content: Reserve ID 52 for QPSFI device 2026-10-22 15:35 ` Parav Pandit @ 2026-10-22 15:41 ` Michael S. Tsirkin 2026-10-22 15:49 ` Parav Pandit 0 siblings, 1 reply; 29+ messages in thread From: Michael S. Tsirkin @ 2026-10-22 15:41 UTC (permalink / raw) To: Parav Pandit Cc: Cornelia Huck, Dmitry Baryshkov, Manos Pitsidianakis, Demi Marie Obenour, Peter Hilber, virtio-comment@lists.linux.dev, Jan Kiszka, Stefan Hajnoczi, Trilok Soni On Thu, Oct 22, 2026 at 03:35:51PM +0000, Parav Pandit wrote: > > > From: Cornelia Huck <cohuck@redhat.com> > > Sent: 22 October 2026 08:35 PM > > > > On Thu, Oct 22 2026, Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com> wrote: > > > > > Let's define some standard way for the vendors to declare that this > > > device is "Qualcomm", "NXP" or "hobbyist which doesn't own PCI or USB id > > > block". > > > > Are we able to use the first <n> bytes of config space for that? I guess > > that is the only way that can possibly work for any transport. > > > > Then, maybe something like one or two bytes for a vendor id (including > > one for "testing purposes" or so), and the rest to be used in a > > vendor-specific way. > > > device ids are independent of the transport. > > but Peter already announced that: > " Reserve a device ID for QPSFI devices (Qualcomm Platform Subsystem > Firmware Interface). QPSFI devices expose functionality across SoC > subsystems through a common Virtio-based protocol. There are no plans to > standardize the devices." > > So, I am unable to understand why should it be part of vendor agnostic virtio spec? > Why a vendor specific implementation of device and driver is not sufficient? I don't really think we should vendors burn up device IDs every times they want something non standard. Reserving ID 52 as "vendor specific" so vendors can go wild sounds good, but a bit more needs to be done than just reserving: document what is expected, e.g. no auto load based on ID alone. -- MST ^ permalink raw reply [flat|nested] 29+ messages in thread
* RE: [PATCH v2] content: Reserve ID 52 for QPSFI device 2026-10-22 15:41 ` Michael S. Tsirkin @ 2026-10-22 15:49 ` Parav Pandit 2026-10-22 16:02 ` Michael S. Tsirkin 0 siblings, 1 reply; 29+ messages in thread From: Parav Pandit @ 2026-10-22 15:49 UTC (permalink / raw) To: Michael S. Tsirkin Cc: Cornelia Huck, Dmitry Baryshkov, Manos Pitsidianakis, Demi Marie Obenour, Peter Hilber, virtio-comment@lists.linux.dev, Jan Kiszka, Stefan Hajnoczi, Trilok Soni > From: Michael S. Tsirkin <mst@redhat.com> > Sent: 22 October 2026 09:12 PM > > On Thu, Oct 22, 2026 at 03:35:51PM +0000, Parav Pandit wrote: > > > > > From: Cornelia Huck <cohuck@redhat.com> > > > Sent: 22 October 2026 08:35 PM > > > > > > On Thu, Oct 22 2026, Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com> wrote: > > > > > > > Let's define some standard way for the vendors to declare that this > > > > device is "Qualcomm", "NXP" or "hobbyist which doesn't own PCI or USB id > > > > block". > > > > > > Are we able to use the first <n> bytes of config space for that? I guess > > > that is the only way that can possibly work for any transport. > > > > > > Then, maybe something like one or two bytes for a vendor id (including > > > one for "testing purposes" or so), and the rest to be used in a > > > vendor-specific way. > > > > > device ids are independent of the transport. > > > > but Peter already announced that: > > " Reserve a device ID for QPSFI devices (Qualcomm Platform Subsystem > > Firmware Interface). QPSFI devices expose functionality across SoC > > subsystems through a common Virtio-based protocol. There are no plans to > > standardize the devices." > > > > So, I am unable to understand why should it be part of vendor agnostic virtio spec? > > Why a vendor specific implementation of device and driver is not sufficient? > > I don't really think we should vendors burn up device IDs every > times they want something non standard. Reserving ID 52 as > "vendor specific" so vendors can go wild sounds good, > but a bit more needs to be done than just reserving: > document what is expected, e.g. no auto load based on ID > alone. First, you indicate to open virtio to vendor specific extension. This is huge in itself. that should be first discussed. If you suggest this, may be every device type should be able to extend, and not just demux at device id level. Second, one reserved id is not enough, because a given system may have multiple such different vendors devices installed. So why to bend things so much and re-invent the wheel? Why for say PCI transport's any vendor and device id are not sufficient to indicate a vendor and device? and if so, why to even document it in the virtio spec? I guess, I am missing the basic motivation part. ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH v2] content: Reserve ID 52 for QPSFI device 2026-10-22 15:49 ` Parav Pandit @ 2026-10-22 16:02 ` Michael S. Tsirkin 2026-10-22 16:20 ` Parav Pandit 0 siblings, 1 reply; 29+ messages in thread From: Michael S. Tsirkin @ 2026-10-22 16:02 UTC (permalink / raw) To: Parav Pandit Cc: Cornelia Huck, Dmitry Baryshkov, Manos Pitsidianakis, Demi Marie Obenour, Peter Hilber, virtio-comment@lists.linux.dev, Jan Kiszka, Stefan Hajnoczi, Trilok Soni On Thu, Oct 22, 2026 at 03:49:51PM +0000, Parav Pandit wrote: > > > From: Michael S. Tsirkin <mst@redhat.com> > > Sent: 22 October 2026 09:12 PM > > > > On Thu, Oct 22, 2026 at 03:35:51PM +0000, Parav Pandit wrote: > > > > > > > From: Cornelia Huck <cohuck@redhat.com> > > > > Sent: 22 October 2026 08:35 PM > > > > > > > > On Thu, Oct 22 2026, Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com> wrote: > > > > > > > > > Let's define some standard way for the vendors to declare that this > > > > > device is "Qualcomm", "NXP" or "hobbyist which doesn't own PCI or USB id > > > > > block". > > > > > > > > Are we able to use the first <n> bytes of config space for that? I guess > > > > that is the only way that can possibly work for any transport. > > > > > > > > Then, maybe something like one or two bytes for a vendor id (including > > > > one for "testing purposes" or so), and the rest to be used in a > > > > vendor-specific way. > > > > > > > device ids are independent of the transport. > > > > > > but Peter already announced that: > > > " Reserve a device ID for QPSFI devices (Qualcomm Platform Subsystem > > > Firmware Interface). QPSFI devices expose functionality across SoC > > > subsystems through a common Virtio-based protocol. There are no plans to > > > standardize the devices." > > > > > > So, I am unable to understand why should it be part of vendor agnostic virtio spec? > > > Why a vendor specific implementation of device and driver is not sufficient? > > > > I don't really think we should vendors burn up device IDs every > > times they want something non standard. Reserving ID 52 as > > "vendor specific" so vendors can go wild sounds good, > > but a bit more needs to be done than just reserving: > > document what is expected, e.g. no auto load based on ID > > alone. > > First, you indicate to open virtio to vendor specific extension. This is huge in itself. Not as such. It's just reserving an id. People always do things like implement some crazy spec saying "we will upstream later" and later they do not. And then the ID conflicts with a standard one. Suboptimal. So if we can tell everyone "always use this ID, and use platform specific tool dor discovery, until your device is in the spec" then that's not too bad. > that should be first discussed. > If you suggest this, may be every device type should be able to extend, and not just demux at device id level. > > Second, one reserved id is not enough, because a given system may have multiple such different vendors devices installed. I don't think we should care. > So why to bend things so much and re-invent the wheel? > Why for say PCI transport's any vendor and device id are not sufficient to indicate a vendor and device? > > and if so, why to even document it in the virtio spec? > > I guess, I am missing the basic motivation part. For the virtio spec? Above. For Qualcomm since they decided never to upstream - donnu. Don't care much, either... ^ permalink raw reply [flat|nested] 29+ messages in thread
* RE: [PATCH v2] content: Reserve ID 52 for QPSFI device 2026-10-22 16:02 ` Michael S. Tsirkin @ 2026-10-22 16:20 ` Parav Pandit 2026-10-22 16:29 ` Michael S. Tsirkin 0 siblings, 1 reply; 29+ messages in thread From: Parav Pandit @ 2026-10-22 16:20 UTC (permalink / raw) To: Michael S. Tsirkin Cc: Cornelia Huck, Dmitry Baryshkov, Manos Pitsidianakis, Demi Marie Obenour, Peter Hilber, virtio-comment@lists.linux.dev, Jan Kiszka, Stefan Hajnoczi, Trilok Soni > From: Michael S. Tsirkin <mst@redhat.com> > Sent: 22 October 2026 09:32 PM > > On Thu, Oct 22, 2026 at 03:49:51PM +0000, Parav Pandit wrote: > > > > > From: Michael S. Tsirkin <mst@redhat.com> > > > Sent: 22 October 2026 09:12 PM > > > > > > On Thu, Oct 22, 2026 at 03:35:51PM +0000, Parav Pandit wrote: > > > > > > > > > From: Cornelia Huck <cohuck@redhat.com> > > > > > Sent: 22 October 2026 08:35 PM > > > > > > > > > > On Thu, Oct 22 2026, Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com> wrote: > > > > > > > > > > > Let's define some standard way for the vendors to declare that this > > > > > > device is "Qualcomm", "NXP" or "hobbyist which doesn't own PCI or USB id > > > > > > block". > > > > > > > > > > Are we able to use the first <n> bytes of config space for that? I guess > > > > > that is the only way that can possibly work for any transport. > > > > > > > > > > Then, maybe something like one or two bytes for a vendor id (including > > > > > one for "testing purposes" or so), and the rest to be used in a > > > > > vendor-specific way. > > > > > > > > > device ids are independent of the transport. > > > > > > > > but Peter already announced that: > > > > " Reserve a device ID for QPSFI devices (Qualcomm Platform Subsystem > > > > Firmware Interface). QPSFI devices expose functionality across SoC > > > > subsystems through a common Virtio-based protocol. There are no plans to > > > > standardize the devices." > > > > > > > > So, I am unable to understand why should it be part of vendor agnostic virtio spec? > > > > Why a vendor specific implementation of device and driver is not sufficient? > > > > > > I don't really think we should vendors burn up device IDs every > > > times they want something non standard. Reserving ID 52 as > > > "vendor specific" so vendors can go wild sounds good, > > > but a bit more needs to be done than just reserving: > > > document what is expected, e.g. no auto load based on ID > > > alone. > > > > First, you indicate to open virtio to vendor specific extension. This is huge in itself. > > Not as such. It's just reserving an id. People always do things like > implement some crazy spec saying "we will upstream later" and later they > do not. > And then the ID conflicts with a standard one. Suboptimal. > > So if we can tell everyone "always use this ID, and use platform > specific tool dor discovery, until your device is in the spec" > then that's not too bad. > > > > > that should be first discussed. > > If you suggest this, may be every device type should be able to extend, and not just demux at device id level. > > > > Second, one reserved id is not enough, because a given system may have multiple such different vendors devices installed. > > I don't think we should care. > than all 0xffff as one experimental id once for all is good enough. than vendor FOO, BAR, differentiation is not needed. And if it is not for experimental purpose, not sure why would vendor stop at only one device. > > So why to bend things so much and re-invent the wheel? > > Why for say PCI transport's any vendor and device id are not sufficient to indicate a vendor and device? > > > > and if so, why to even document it in the virtio spec? > > > > I guess, I am missing the basic motivation part. > > For the virtio spec? Above. For Qualcomm since they decided never to > upstream - donnu. Don't care much, either... ok. so yeah, let vendor's platform device be in the vendor's OSS zone. ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH v2] content: Reserve ID 52 for QPSFI device 2026-10-22 16:20 ` Parav Pandit @ 2026-10-22 16:29 ` Michael S. Tsirkin 2026-10-22 22:39 ` Peter Hilber 0 siblings, 1 reply; 29+ messages in thread From: Michael S. Tsirkin @ 2026-10-22 16:29 UTC (permalink / raw) To: Parav Pandit Cc: Cornelia Huck, Dmitry Baryshkov, Manos Pitsidianakis, Demi Marie Obenour, Peter Hilber, virtio-comment@lists.linux.dev, Jan Kiszka, Stefan Hajnoczi, Trilok Soni On Thu, Oct 22, 2026 at 04:20:51PM +0000, Parav Pandit wrote: > > > From: Michael S. Tsirkin <mst@redhat.com> > > Sent: 22 October 2026 09:32 PM > > > > On Thu, Oct 22, 2026 at 03:49:51PM +0000, Parav Pandit wrote: > > > > > > > From: Michael S. Tsirkin <mst@redhat.com> > > > > Sent: 22 October 2026 09:12 PM > > > > > > > > On Thu, Oct 22, 2026 at 03:35:51PM +0000, Parav Pandit wrote: > > > > > > > > > > > From: Cornelia Huck <cohuck@redhat.com> > > > > > > Sent: 22 October 2026 08:35 PM > > > > > > > > > > > > On Thu, Oct 22 2026, Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com> wrote: > > > > > > > > > > > > > Let's define some standard way for the vendors to declare that this > > > > > > > device is "Qualcomm", "NXP" or "hobbyist which doesn't own PCI or USB id > > > > > > > block". > > > > > > > > > > > > Are we able to use the first <n> bytes of config space for that? I guess > > > > > > that is the only way that can possibly work for any transport. > > > > > > > > > > > > Then, maybe something like one or two bytes for a vendor id (including > > > > > > one for "testing purposes" or so), and the rest to be used in a > > > > > > vendor-specific way. > > > > > > > > > > > device ids are independent of the transport. > > > > > > > > > > but Peter already announced that: > > > > > " Reserve a device ID for QPSFI devices (Qualcomm Platform Subsystem > > > > > Firmware Interface). QPSFI devices expose functionality across SoC > > > > > subsystems through a common Virtio-based protocol. There are no plans to > > > > > standardize the devices." > > > > > > > > > > So, I am unable to understand why should it be part of vendor agnostic virtio spec? > > > > > Why a vendor specific implementation of device and driver is not sufficient? > > > > > > > > I don't really think we should vendors burn up device IDs every > > > > times they want something non standard. Reserving ID 52 as > > > > "vendor specific" so vendors can go wild sounds good, > > > > but a bit more needs to be done than just reserving: > > > > document what is expected, e.g. no auto load based on ID > > > > alone. > > > > > > First, you indicate to open virtio to vendor specific extension. This is huge in itself. > > > > Not as such. It's just reserving an id. People always do things like > > implement some crazy spec saying "we will upstream later" and later they > > do not. > > And then the ID conflicts with a standard one. Suboptimal. > > > > So if we can tell everyone "always use this ID, and use platform > > specific tool dor discovery, until your device is in the spec" > > then that's not too bad. > > > > > > > > > that should be first discussed. > > > If you suggest this, may be every device type should be able to extend, and not just demux at device id level. > > > > > > Second, one reserved id is not enough, because a given system may have multiple such different vendors devices installed. > > > > I don't think we should care. > > > than all 0xffff as one experimental id once for all is good enough. > than vendor FOO, BAR, differentiation is not needed. I don't think we want 0xffff, a lot of systems treat that value as special, pci comes to mind. > And if it is not for experimental purpose, not sure why would vendor stop at only one device. They can reuse that id as many times as they want and differentiate with a device specific or out of band methods. First X bytes of config space was mentioned. or whatever. We can add a recommentation maybe if vendors desire that. > > > So why to bend things so much and re-invent the wheel? > > > Why for say PCI transport's any vendor and device id are not sufficient to indicate a vendor and device? > > > > > > and if so, why to even document it in the virtio spec? > > > > > > I guess, I am missing the basic motivation part. > > > > For the virtio spec? Above. For Qualcomm since they decided never to > > upstream - donnu. Don't care much, either... > > ok. so yeah, let vendor's platform device be in the vendor's OSS zone. ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH v2] content: Reserve ID 52 for QPSFI device 2026-10-22 16:29 ` Michael S. Tsirkin @ 2026-10-22 22:39 ` Peter Hilber 2026-10-22 23:28 ` Michael S. Tsirkin ` (2 more replies) 0 siblings, 3 replies; 29+ messages in thread From: Peter Hilber @ 2026-10-22 22:39 UTC (permalink / raw) To: Michael S. Tsirkin Cc: Parav Pandit, Cornelia Huck, Dmitry Baryshkov, Manos Pitsidianakis, Demi Marie Obenour, virtio-comment@lists.linux.dev, Jan Kiszka, Stefan Hajnoczi, Trilok Soni On Thu, Oct 22, 2026 at 12:29:31PM -0400, Michael S. Tsirkin wrote: > On Thu, Oct 22, 2026 at 04:20:51PM +0000, Parav Pandit wrote: > > > > > From: Michael S. Tsirkin <mst@redhat.com> > > > Sent: 22 October 2026 09:32 PM > > > > > > On Thu, Oct 22, 2026 at 03:49:51PM +0000, Parav Pandit wrote: > > > > > > > > > From: Michael S. Tsirkin <mst@redhat.com> > > > > > Sent: 22 October 2026 09:12 PM > > > > > > > > > > On Thu, Oct 22, 2026 at 03:35:51PM +0000, Parav Pandit wrote: > > > > > > > > > > > > > From: Cornelia Huck <cohuck@redhat.com> > > > > > > > Sent: 22 October 2026 08:35 PM > > > > > > > > > > > > > > On Thu, Oct 22 2026, Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com> wrote: > > > > > > > > > > > > > > > Let's define some standard way for the vendors to declare that this > > > > > > > > device is "Qualcomm", "NXP" or "hobbyist which doesn't own PCI or USB id > > > > > > > > block". > > > > > > > > > > > > > > Are we able to use the first <n> bytes of config space for that? I guess > > > > > > > that is the only way that can possibly work for any transport. > > > > > > > > > > > > > > Then, maybe something like one or two bytes for a vendor id (including > > > > > > > one for "testing purposes" or so), and the rest to be used in a > > > > > > > vendor-specific way. > > > > > > > > > > > > > device ids are independent of the transport. > > > > > > > > > > > > but Peter already announced that: > > > > > > " Reserve a device ID for QPSFI devices (Qualcomm Platform Subsystem > > > > > > Firmware Interface). QPSFI devices expose functionality across SoC > > > > > > subsystems through a common Virtio-based protocol. There are no plans to > > > > > > standardize the devices." > > > > > > > > > > > > So, I am unable to understand why should it be part of vendor agnostic virtio spec? > > > > > > Why a vendor specific implementation of device and driver is not sufficient? > > > > > > > > > > I don't really think we should vendors burn up device IDs every > > > > > times they want something non standard. Reserving ID 52 as > > > > > "vendor specific" so vendors can go wild sounds good, > > > > > but a bit more needs to be done than just reserving: > > > > > document what is expected, e.g. no auto load based on ID > > > > > alone. > > > > > > > > First, you indicate to open virtio to vendor specific extension. This is huge in itself. > > > > > > Not as such. It's just reserving an id. People always do things like > > > implement some crazy spec saying "we will upstream later" and later they > > > do not. > > > And then the ID conflicts with a standard one. Suboptimal. > > > > > > So if we can tell everyone "always use this ID, and use platform > > > specific tool dor discovery, until your device is in the spec" > > > then that's not too bad. > > > > > > > > > > > > > that should be first discussed. > > > > If you suggest this, may be every device type should be able to extend, and not just demux at device id level. > > > > > > > > Second, one reserved id is not enough, because a given system may have multiple such different vendors devices installed. > > > > > > I don't think we should care. > > > > > than all 0xffff as one experimental id once for all is good enough. > > than vendor FOO, BAR, differentiation is not needed. > > I don't think we want 0xffff, a lot of systems treat that value as > special, pci comes to mind. > > > And if it is not for experimental purpose, not sure why would vendor stop at only one device. > > They can reuse that id as many times as they want and differentiate > with a device specific or out of band methods. > First X bytes of config space was mentioned. or whatever. > > We can add a recommentation maybe if vendors desire that. > Thank you to everybody contributing to the discussion and to the concept of a vendor-specific device. Just for clarity, Qualcomm intends to upstream Linux kernel drivers for this new device type, so would definitely want to use a registered device ID. In case of an in-band approach, I suppose we would then have an explicit vendor list in the spec where vendors have to register? Best regards, Peter > > > > So why to bend things so much and re-invent the wheel? > > > > Why for say PCI transport's any vendor and device id are not sufficient to indicate a vendor and device? > > > > > > > > and if so, why to even document it in the virtio spec? > > > > > > > > I guess, I am missing the basic motivation part. > > > > > > For the virtio spec? Above. For Qualcomm since they decided never to > > > upstream - donnu. Don't care much, either... > > > > ok. so yeah, let vendor's platform device be in the vendor's OSS zone. > ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH v2] content: Reserve ID 52 for QPSFI device 2026-10-22 22:39 ` Peter Hilber @ 2026-10-22 23:28 ` Michael S. Tsirkin 2026-10-23 3:32 ` Parav Pandit 2026-10-23 5:37 ` Manos Pitsidianakis 2 siblings, 0 replies; 29+ messages in thread From: Michael S. Tsirkin @ 2026-10-22 23:28 UTC (permalink / raw) To: Peter Hilber Cc: Parav Pandit, Cornelia Huck, Dmitry Baryshkov, Manos Pitsidianakis, Demi Marie Obenour, virtio-comment@lists.linux.dev, Jan Kiszka, Stefan Hajnoczi, Trilok Soni On Fri, Oct 23, 2026 at 12:39:48AM +0200, Peter Hilber wrote: > On Thu, Oct 22, 2026 at 12:29:31PM -0400, Michael S. Tsirkin wrote: > > On Thu, Oct 22, 2026 at 04:20:51PM +0000, Parav Pandit wrote: > > > > > > > From: Michael S. Tsirkin <mst@redhat.com> > > > > Sent: 22 October 2026 09:32 PM > > > > > > > > On Thu, Oct 22, 2026 at 03:49:51PM +0000, Parav Pandit wrote: > > > > > > > > > > > From: Michael S. Tsirkin <mst@redhat.com> > > > > > > Sent: 22 October 2026 09:12 PM > > > > > > > > > > > > On Thu, Oct 22, 2026 at 03:35:51PM +0000, Parav Pandit wrote: > > > > > > > > > > > > > > > From: Cornelia Huck <cohuck@redhat.com> > > > > > > > > Sent: 22 October 2026 08:35 PM > > > > > > > > > > > > > > > > On Thu, Oct 22 2026, Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com> wrote: > > > > > > > > > > > > > > > > > Let's define some standard way for the vendors to declare that this > > > > > > > > > device is "Qualcomm", "NXP" or "hobbyist which doesn't own PCI or USB id > > > > > > > > > block". > > > > > > > > > > > > > > > > Are we able to use the first <n> bytes of config space for that? I guess > > > > > > > > that is the only way that can possibly work for any transport. > > > > > > > > > > > > > > > > Then, maybe something like one or two bytes for a vendor id (including > > > > > > > > one for "testing purposes" or so), and the rest to be used in a > > > > > > > > vendor-specific way. > > > > > > > > > > > > > > > device ids are independent of the transport. > > > > > > > > > > > > > > but Peter already announced that: > > > > > > > " Reserve a device ID for QPSFI devices (Qualcomm Platform Subsystem > > > > > > > Firmware Interface). QPSFI devices expose functionality across SoC > > > > > > > subsystems through a common Virtio-based protocol. There are no plans to > > > > > > > standardize the devices." > > > > > > > > > > > > > > So, I am unable to understand why should it be part of vendor agnostic virtio spec? > > > > > > > Why a vendor specific implementation of device and driver is not sufficient? > > > > > > > > > > > > I don't really think we should vendors burn up device IDs every > > > > > > times they want something non standard. Reserving ID 52 as > > > > > > "vendor specific" so vendors can go wild sounds good, > > > > > > but a bit more needs to be done than just reserving: > > > > > > document what is expected, e.g. no auto load based on ID > > > > > > alone. > > > > > > > > > > First, you indicate to open virtio to vendor specific extension. This is huge in itself. > > > > > > > > Not as such. It's just reserving an id. People always do things like > > > > implement some crazy spec saying "we will upstream later" and later they > > > > do not. > > > > And then the ID conflicts with a standard one. Suboptimal. > > > > > > > > So if we can tell everyone "always use this ID, and use platform > > > > specific tool dor discovery, until your device is in the spec" > > > > then that's not too bad. > > > > > > > > > > > > > > > > > that should be first discussed. > > > > > If you suggest this, may be every device type should be able to extend, and not just demux at device id level. > > > > > > > > > > Second, one reserved id is not enough, because a given system may have multiple such different vendors devices installed. > > > > > > > > I don't think we should care. > > > > > > > than all 0xffff as one experimental id once for all is good enough. > > > than vendor FOO, BAR, differentiation is not needed. > > > > I don't think we want 0xffff, a lot of systems treat that value as > > special, pci comes to mind. > > > > > And if it is not for experimental purpose, not sure why would vendor stop at only one device. > > > > They can reuse that id as many times as they want and differentiate > > with a device specific or out of band methods. > > First X bytes of config space was mentioned. or whatever. > > > > We can add a recommentation maybe if vendors desire that. > > > > Thank you to everybody contributing to the discussion and to the concept > of a vendor-specific device. Just for clarity, Qualcomm intends to > upstream Linux kernel drivers for this new device type, so would > definitely want to use a registered device ID. > > In case of an in-band approach, I suppose we would then have an explicit > vendor list in the spec where vendors have to register? > > Best regards, > > Peter Just why are you bothering with virtio spec at all then? Just use your own vendor id for this. > > > > > > So why to bend things so much and re-invent the wheel? > > > > > Why for say PCI transport's any vendor and device id are not sufficient to indicate a vendor and device? > > > > > > > > > > and if so, why to even document it in the virtio spec? > > > > > > > > > > I guess, I am missing the basic motivation part. > > > > > > > > For the virtio spec? Above. For Qualcomm since they decided never to > > > > upstream - donnu. Don't care much, either... > > > > > > ok. so yeah, let vendor's platform device be in the vendor's OSS zone. > > ^ permalink raw reply [flat|nested] 29+ messages in thread
* RE: [PATCH v2] content: Reserve ID 52 for QPSFI device 2026-10-22 22:39 ` Peter Hilber 2026-10-22 23:28 ` Michael S. Tsirkin @ 2026-10-23 3:32 ` Parav Pandit 2026-10-23 5:37 ` Manos Pitsidianakis 2 siblings, 0 replies; 29+ messages in thread From: Parav Pandit @ 2026-10-23 3:32 UTC (permalink / raw) To: Peter Hilber, Michael S. Tsirkin Cc: Cornelia Huck, Dmitry Baryshkov, Manos Pitsidianakis, Demi Marie Obenour, virtio-comment@lists.linux.dev, Jan Kiszka, Stefan Hajnoczi, Trilok Soni > From: Peter Hilber <peter.hilber@oss.qualcomm.com> > Sent: 23 October 2026 04:10 AM > > On Thu, Oct 22, 2026 at 12:29:31PM -0400, Michael S. Tsirkin wrote: > > On Thu, Oct 22, 2026 at 04:20:51PM +0000, Parav Pandit wrote: > > > > > > > From: Michael S. Tsirkin <mst@redhat.com> > > > > Sent: 22 October 2026 09:32 PM > > > > > > > > On Thu, Oct 22, 2026 at 03:49:51PM +0000, Parav Pandit wrote: > > > > > > > > > > > From: Michael S. Tsirkin <mst@redhat.com> > > > > > > Sent: 22 October 2026 09:12 PM > > > > > > > > > > > > On Thu, Oct 22, 2026 at 03:35:51PM +0000, Parav Pandit wrote: > > > > > > > > > > > > > > > From: Cornelia Huck <cohuck@redhat.com> > > > > > > > > Sent: 22 October 2026 08:35 PM > > > > > > > > > > > > > > > > On Thu, Oct 22 2026, Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com> wrote: > > > > > > > > > > > > > > > > > Let's define some standard way for the vendors to declare that this > > > > > > > > > device is "Qualcomm", "NXP" or "hobbyist which doesn't own PCI or USB id > > > > > > > > > block". > > > > > > > > > > > > > > > > Are we able to use the first <n> bytes of config space for that? I guess > > > > > > > > that is the only way that can possibly work for any transport. > > > > > > > > > > > > > > > > Then, maybe something like one or two bytes for a vendor id (including > > > > > > > > one for "testing purposes" or so), and the rest to be used in a > > > > > > > > vendor-specific way. > > > > > > > > > > > > > > > device ids are independent of the transport. > > > > > > > > > > > > > > but Peter already announced that: > > > > > > > " Reserve a device ID for QPSFI devices (Qualcomm Platform Subsystem > > > > > > > Firmware Interface). QPSFI devices expose functionality across SoC > > > > > > > subsystems through a common Virtio-based protocol. There are no plans to > > > > > > > standardize the devices." > > > > > > > > > > > > > > So, I am unable to understand why should it be part of vendor agnostic virtio spec? > > > > > > > Why a vendor specific implementation of device and driver is not sufficient? > > > > > > > > > > > > I don't really think we should vendors burn up device IDs every > > > > > > times they want something non standard. Reserving ID 52 as > > > > > > "vendor specific" so vendors can go wild sounds good, > > > > > > but a bit more needs to be done than just reserving: > > > > > > document what is expected, e.g. no auto load based on ID > > > > > > alone. > > > > > > > > > > First, you indicate to open virtio to vendor specific extension. This is huge in itself. > > > > > > > > Not as such. It's just reserving an id. People always do things like > > > > implement some crazy spec saying "we will upstream later" and later they > > > > do not. > > > > And then the ID conflicts with a standard one. Suboptimal. > > > > > > > > So if we can tell everyone "always use this ID, and use platform > > > > specific tool dor discovery, until your device is in the spec" > > > > then that's not too bad. > > > > > > > > > > > > > > > > > that should be first discussed. > > > > > If you suggest this, may be every device type should be able to extend, and not just demux at device id level. > > > > > > > > > > Second, one reserved id is not enough, because a given system may have multiple such different vendors devices installed. > > > > > > > > I don't think we should care. > > > > > > > than all 0xffff as one experimental id once for all is good enough. > > > than vendor FOO, BAR, differentiation is not needed. > > > > I don't think we want 0xffff, a lot of systems treat that value as > > special, pci comes to mind. > > > > > And if it is not for experimental purpose, not sure why would vendor stop at only one device. > > > > They can reuse that id as many times as they want and differentiate > > with a device specific or out of band methods. > > First X bytes of config space was mentioned. or whatever. > > > > We can add a recommentation maybe if vendors desire that. > > > > Thank you to everybody contributing to the discussion and to the concept > of a vendor-specific device. Just for clarity, Qualcomm intends to > upstream Linux kernel drivers for this new device type, so would > definitely want to use a registered device ID. > > In case of an in-band approach, I suppose we would then have an explicit > vendor list in the spec where vendors have to register? > Given this is vendor specific device, I am unable to find the value of virtio-community to maintain the device and it spec. Motivation is unclear. virtio spec is something that multi-vendors can implement. Hence, can you please keep the driver and the spec outside of the virtio spec, and keep that in Qualcomm domain? > Best regards, > > Peter > > > > > > > So why to bend things so much and re-invent the wheel? > > > > > Why for say PCI transport's any vendor and device id are not sufficient to indicate a vendor and device? > > > > > > > > > > and if so, why to even document it in the virtio spec? > > > > > > > > > > I guess, I am missing the basic motivation part. > > > > > > > > For the virtio spec? Above. For Qualcomm since they decided never to > > > > upstream - donnu. Don't care much, either... > > > > > > ok. so yeah, let vendor's platform device be in the vendor's OSS zone. > > ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH v2] content: Reserve ID 52 for QPSFI device 2026-10-22 22:39 ` Peter Hilber 2026-10-22 23:28 ` Michael S. Tsirkin 2026-10-23 3:32 ` Parav Pandit @ 2026-10-23 5:37 ` Manos Pitsidianakis 2026-10-23 5:40 ` Demi Marie Obenour 2 siblings, 1 reply; 29+ messages in thread From: Manos Pitsidianakis @ 2026-10-23 5:37 UTC (permalink / raw) To: Peter Hilber Cc: Michael S. Tsirkin, Parav Pandit, Cornelia Huck, Dmitry Baryshkov, Demi Marie Obenour, virtio-comment@lists.linux.dev, Jan Kiszka, Stefan Hajnoczi, Trilok Soni On Fri, Oct 23, 2026 at 1:39 AM Peter Hilber <peter.hilber@oss.qualcomm.com> wrote: > > On Thu, Oct 22, 2026 at 12:29:31PM -0400, Michael S. Tsirkin wrote: > > On Thu, Oct 22, 2026 at 04:20:51PM +0000, Parav Pandit wrote: > > > > > > > From: Michael S. Tsirkin <mst@redhat.com> > > > > Sent: 22 October 2026 09:32 PM > > > > > > > > On Thu, Oct 22, 2026 at 03:49:51PM +0000, Parav Pandit wrote: > > > > > > > > > > > From: Michael S. Tsirkin <mst@redhat.com> > > > > > > Sent: 22 October 2026 09:12 PM > > > > > > > > > > > > On Thu, Oct 22, 2026 at 03:35:51PM +0000, Parav Pandit wrote: > > > > > > > > > > > > > > > From: Cornelia Huck <cohuck@redhat.com> > > > > > > > > Sent: 22 October 2026 08:35 PM > > > > > > > > > > > > > > > > On Thu, Oct 22 2026, Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com> wrote: > > > > > > > > > > > > > > > > > Let's define some standard way for the vendors to declare that this > > > > > > > > > device is "Qualcomm", "NXP" or "hobbyist which doesn't own PCI or USB id > > > > > > > > > block". > > > > > > > > > > > > > > > > Are we able to use the first <n> bytes of config space for that? I guess > > > > > > > > that is the only way that can possibly work for any transport. > > > > > > > > > > > > > > > > Then, maybe something like one or two bytes for a vendor id (including > > > > > > > > one for "testing purposes" or so), and the rest to be used in a > > > > > > > > vendor-specific way. > > > > > > > > > > > > > > > device ids are independent of the transport. > > > > > > > > > > > > > > but Peter already announced that: > > > > > > > " Reserve a device ID for QPSFI devices (Qualcomm Platform Subsystem > > > > > > > Firmware Interface). QPSFI devices expose functionality across SoC > > > > > > > subsystems through a common Virtio-based protocol. There are no plans to > > > > > > > standardize the devices." > > > > > > > > > > > > > > So, I am unable to understand why should it be part of vendor agnostic virtio spec? > > > > > > > Why a vendor specific implementation of device and driver is not sufficient? > > > > > > > > > > > > I don't really think we should vendors burn up device IDs every > > > > > > times they want something non standard. Reserving ID 52 as > > > > > > "vendor specific" so vendors can go wild sounds good, > > > > > > but a bit more needs to be done than just reserving: > > > > > > document what is expected, e.g. no auto load based on ID > > > > > > alone. > > > > > > > > > > First, you indicate to open virtio to vendor specific extension. This is huge in itself. > > > > > > > > Not as such. It's just reserving an id. People always do things like > > > > implement some crazy spec saying "we will upstream later" and later they > > > > do not. > > > > And then the ID conflicts with a standard one. Suboptimal. > > > > > > > > So if we can tell everyone "always use this ID, and use platform > > > > specific tool dor discovery, until your device is in the spec" > > > > then that's not too bad. > > > > > > > > > > > > > > > > > that should be first discussed. > > > > > If you suggest this, may be every device type should be able to extend, and not just demux at device id level. > > > > > > > > > > Second, one reserved id is not enough, because a given system may have multiple such different vendors devices installed. > > > > > > > > I don't think we should care. > > > > > > > than all 0xffff as one experimental id once for all is good enough. > > > than vendor FOO, BAR, differentiation is not needed. > > > > I don't think we want 0xffff, a lot of systems treat that value as > > special, pci comes to mind. > > > > > And if it is not for experimental purpose, not sure why would vendor stop at only one device. > > > > They can reuse that id as many times as they want and differentiate > > with a device specific or out of band methods. > > First X bytes of config space was mentioned. or whatever. > > > > We can add a recommentation maybe if vendors desire that. > > Hello Peter, > Thank you to everybody contributing to the discussion and to the concept > of a vendor-specific device. I'd like to echo the sentiment that it unfortunately doesn't seem necessary to call vendor devices virtio at all. > Just for clarity, Qualcomm intends to > upstream Linux kernel drivers for this new device type, so would > definitely want to use a registered device ID. But no upstream specification, correct? There's no point without an open device specification to being included in a specification. People can only implement frontend drivers for this vendor device, and only by reading the upstream code - no spec to follow. Similar to a vendor SoC in concept. > > In case of an in-band approach, I suppose we would then have an explicit > vendor list in the spec where vendors have to register? > > Best regards, > > Peter > > > > > > > So why to bend things so much and re-invent the wheel? > > > > > Why for say PCI transport's any vendor and device id are not sufficient to indicate a vendor and device? > > > > > > > > > > and if so, why to even document it in the virtio spec? > > > > > > > > > > I guess, I am missing the basic motivation part. > > > > > > > > For the virtio spec? Above. For Qualcomm since they decided never to > > > > upstream - donnu. Don't care much, either... > > > > > > ok. so yeah, let vendor's platform device be in the vendor's OSS zone. > > ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH v2] content: Reserve ID 52 for QPSFI device 2026-10-23 5:37 ` Manos Pitsidianakis @ 2026-10-23 5:40 ` Demi Marie Obenour 2026-10-23 20:56 ` Peter Hilber 0 siblings, 1 reply; 29+ messages in thread From: Demi Marie Obenour @ 2026-10-23 5:40 UTC (permalink / raw) To: Manos Pitsidianakis, Peter Hilber Cc: Michael S. Tsirkin, Parav Pandit, Cornelia Huck, Dmitry Baryshkov, virtio-comment@lists.linux.dev, Jan Kiszka, Stefan Hajnoczi, Trilok Soni [-- Attachment #1.1: Type: text/plain, Size: 5431 bytes --] On 10/23/26 01:37, Manos Pitsidianakis wrote: > On Fri, Oct 23, 2026 at 1:39 AM Peter Hilber > <peter.hilber@oss.qualcomm.com> wrote: >> >> On Thu, Oct 22, 2026 at 12:29:31PM -0400, Michael S. Tsirkin wrote: >>> On Thu, Oct 22, 2026 at 04:20:51PM +0000, Parav Pandit wrote: >>>> >>>>> From: Michael S. Tsirkin <mst@redhat.com> >>>>> Sent: 22 October 2026 09:32 PM >>>>> >>>>> On Thu, Oct 22, 2026 at 03:49:51PM +0000, Parav Pandit wrote: >>>>>> >>>>>>> From: Michael S. Tsirkin <mst@redhat.com> >>>>>>> Sent: 22 October 2026 09:12 PM >>>>>>> >>>>>>> On Thu, Oct 22, 2026 at 03:35:51PM +0000, Parav Pandit wrote: >>>>>>>> >>>>>>>>> From: Cornelia Huck <cohuck@redhat.com> >>>>>>>>> Sent: 22 October 2026 08:35 PM >>>>>>>>> >>>>>>>>> On Thu, Oct 22 2026, Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com> wrote: >>>>>>>>> >>>>>>>>>> Let's define some standard way for the vendors to declare that this >>>>>>>>>> device is "Qualcomm", "NXP" or "hobbyist which doesn't own PCI or USB id >>>>>>>>>> block". >>>>>>>>> >>>>>>>>> Are we able to use the first <n> bytes of config space for that? I guess >>>>>>>>> that is the only way that can possibly work for any transport. >>>>>>>>> >>>>>>>>> Then, maybe something like one or two bytes for a vendor id (including >>>>>>>>> one for "testing purposes" or so), and the rest to be used in a >>>>>>>>> vendor-specific way. >>>>>>>>> >>>>>>>> device ids are independent of the transport. >>>>>>>> >>>>>>>> but Peter already announced that: >>>>>>>> " Reserve a device ID for QPSFI devices (Qualcomm Platform Subsystem >>>>>>>> Firmware Interface). QPSFI devices expose functionality across SoC >>>>>>>> subsystems through a common Virtio-based protocol. There are no plans to >>>>>>>> standardize the devices." >>>>>>>> >>>>>>>> So, I am unable to understand why should it be part of vendor agnostic virtio spec? >>>>>>>> Why a vendor specific implementation of device and driver is not sufficient? >>>>>>> >>>>>>> I don't really think we should vendors burn up device IDs every >>>>>>> times they want something non standard. Reserving ID 52 as >>>>>>> "vendor specific" so vendors can go wild sounds good, >>>>>>> but a bit more needs to be done than just reserving: >>>>>>> document what is expected, e.g. no auto load based on ID >>>>>>> alone. >>>>>> >>>>>> First, you indicate to open virtio to vendor specific extension. This is huge in itself. >>>>> >>>>> Not as such. It's just reserving an id. People always do things like >>>>> implement some crazy spec saying "we will upstream later" and later they >>>>> do not. >>>>> And then the ID conflicts with a standard one. Suboptimal. >>>>> >>>>> So if we can tell everyone "always use this ID, and use platform >>>>> specific tool dor discovery, until your device is in the spec" >>>>> then that's not too bad. >>>>> >>>>> >>>>> >>>>>> that should be first discussed. >>>>>> If you suggest this, may be every device type should be able to extend, and not just demux at device id level. >>>>>> >>>>>> Second, one reserved id is not enough, because a given system may have multiple such different vendors devices installed. >>>>> >>>>> I don't think we should care. >>>>> >>>> than all 0xffff as one experimental id once for all is good enough. >>>> than vendor FOO, BAR, differentiation is not needed. >>> >>> I don't think we want 0xffff, a lot of systems treat that value as >>> special, pci comes to mind. >>> >>>> And if it is not for experimental purpose, not sure why would vendor stop at only one device. >>> >>> They can reuse that id as many times as they want and differentiate >>> with a device specific or out of band methods. >>> First X bytes of config space was mentioned. or whatever. >>> >>> We can add a recommentation maybe if vendors desire that. >>> > > > Hello Peter, > >> Thank you to everybody contributing to the discussion and to the concept >> of a vendor-specific device. > > I'd like to echo the sentiment that it unfortunately doesn't seem > necessary to call vendor devices virtio at all. It acts like a virtio device, but otherwise I agree. > > Just for clarity, Qualcomm intends to >> upstream Linux kernel drivers for this new device type, so would >> definitely want to use a registered device ID. > > But no upstream specification, correct? There's no point without an > open device specification to being included in a specification. People > can only implement frontend drivers for this vendor device, and only > by reading the upstream code - no spec to follow. Similar to a vendor > SoC in concept. > >> >> In case of an in-band approach, I suppose we would then have an explicit >> vendor list in the spec where vendors have to register? >> >> Best regards, >> >> Peter >> >> >>>>>> So why to bend things so much and re-invent the wheel? >>>>>> Why for say PCI transport's any vendor and device id are not sufficient to indicate a vendor and device? >>>>>> >>>>>> and if so, why to even document it in the virtio spec? >>>>>> >>>>>> I guess, I am missing the basic motivation part. >>>>> >>>>> For the virtio spec? Above. For Qualcomm since they decided never to >>>>> upstream - donnu. Don't care much, either... >>>> >>>> ok. so yeah, let vendor's platform device be in the vendor's OSS zone. >>> -- Sincerely, Demi Marie Obenour (she/her/hers) [-- Attachment #2: OpenPGP digital signature --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH v2] content: Reserve ID 52 for QPSFI device 2026-10-23 5:40 ` Demi Marie Obenour @ 2026-10-23 20:56 ` Peter Hilber 2026-10-23 23:47 ` Michael S. Tsirkin 2026-10-26 14:31 ` Cornelia Huck 0 siblings, 2 replies; 29+ messages in thread From: Peter Hilber @ 2026-10-23 20:56 UTC (permalink / raw) To: Demi Marie Obenour Cc: Manos Pitsidianakis, Michael S. Tsirkin, Parav Pandit, Cornelia Huck, Dmitry Baryshkov, virtio-comment@lists.linux.dev, Jan Kiszka, Stefan Hajnoczi, Trilok Soni On Fri, Oct 23, 2026 at 01:40:13AM -0400, Demi Marie Obenour wrote: > On 10/23/26 01:37, Manos Pitsidianakis wrote: > > On Fri, Oct 23, 2026 at 1:39 AM Peter Hilber > > <peter.hilber@oss.qualcomm.com> wrote: > >> > >> On Thu, Oct 22, 2026 at 12:29:31PM -0400, Michael S. Tsirkin wrote: > >>> On Thu, Oct 22, 2026 at 04:20:51PM +0000, Parav Pandit wrote: > >>>> > >>>>> From: Michael S. Tsirkin <mst@redhat.com> > >>>>> Sent: 22 October 2026 09:32 PM > >>>>> > >>>>> On Thu, Oct 22, 2026 at 03:49:51PM +0000, Parav Pandit wrote: > >>>>>> > >>>>>>> From: Michael S. Tsirkin <mst@redhat.com> > >>>>>>> Sent: 22 October 2026 09:12 PM > >>>>>>> > >>>>>>> On Thu, Oct 22, 2026 at 03:35:51PM +0000, Parav Pandit wrote: > >>>>>>>> > >>>>>>>>> From: Cornelia Huck <cohuck@redhat.com> > >>>>>>>>> Sent: 22 October 2026 08:35 PM > >>>>>>>>> > >>>>>>>>> On Thu, Oct 22 2026, Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com> wrote: > >>>>>>>>> > >>>>>>>>>> Let's define some standard way for the vendors to declare that this > >>>>>>>>>> device is "Qualcomm", "NXP" or "hobbyist which doesn't own PCI or USB id > >>>>>>>>>> block". > >>>>>>>>> > >>>>>>>>> Are we able to use the first <n> bytes of config space for that? I guess > >>>>>>>>> that is the only way that can possibly work for any transport. > >>>>>>>>> > >>>>>>>>> Then, maybe something like one or two bytes for a vendor id (including > >>>>>>>>> one for "testing purposes" or so), and the rest to be used in a > >>>>>>>>> vendor-specific way. > >>>>>>>>> > >>>>>>>> device ids are independent of the transport. > >>>>>>>> > >>>>>>>> but Peter already announced that: > >>>>>>>> " Reserve a device ID for QPSFI devices (Qualcomm Platform Subsystem > >>>>>>>> Firmware Interface). QPSFI devices expose functionality across SoC > >>>>>>>> subsystems through a common Virtio-based protocol. There are no plans to > >>>>>>>> standardize the devices." > >>>>>>>> > >>>>>>>> So, I am unable to understand why should it be part of vendor agnostic virtio spec? > >>>>>>>> Why a vendor specific implementation of device and driver is not sufficient? > >>>>>>> > >>>>>>> I don't really think we should vendors burn up device IDs every > >>>>>>> times they want something non standard. Reserving ID 52 as > >>>>>>> "vendor specific" so vendors can go wild sounds good, > >>>>>>> but a bit more needs to be done than just reserving: > >>>>>>> document what is expected, e.g. no auto load based on ID > >>>>>>> alone. > >>>>>> > >>>>>> First, you indicate to open virtio to vendor specific extension. This is huge in itself. > >>>>> > >>>>> Not as such. It's just reserving an id. People always do things like > >>>>> implement some crazy spec saying "we will upstream later" and later they > >>>>> do not. > >>>>> And then the ID conflicts with a standard one. Suboptimal. > >>>>> > >>>>> So if we can tell everyone "always use this ID, and use platform > >>>>> specific tool dor discovery, until your device is in the spec" > >>>>> then that's not too bad. > >>>>> > >>>>> > >>>>> > >>>>>> that should be first discussed. > >>>>>> If you suggest this, may be every device type should be able to extend, and not just demux at device id level. > >>>>>> > >>>>>> Second, one reserved id is not enough, because a given system may have multiple such different vendors devices installed. > >>>>> > >>>>> I don't think we should care. > >>>>> > >>>> than all 0xffff as one experimental id once for all is good enough. > >>>> than vendor FOO, BAR, differentiation is not needed. > >>> > >>> I don't think we want 0xffff, a lot of systems treat that value as > >>> special, pci comes to mind. > >>> > >>>> And if it is not for experimental purpose, not sure why would vendor stop at only one device. > >>> > >>> They can reuse that id as many times as they want and differentiate > >>> with a device specific or out of band methods. > >>> First X bytes of config space was mentioned. or whatever. > >>> > >>> We can add a recommentation maybe if vendors desire that. > >>> > > > > > > Hello Peter, > > > >> Thank you to everybody contributing to the discussion and to the concept > >> of a vendor-specific device. > > > > I'd like to echo the sentiment that it unfortunately doesn't seem > > necessary to call vendor devices virtio at all. > > It acts like a virtio device, but otherwise I agree. > > > > Just for clarity, Qualcomm intends to > >> upstream Linux kernel drivers for this new device type, so would > >> definitely want to use a registered device ID. Qualcomm's intent is to upstream a Linux kernel Virtio driver for a non-standardized device using a Virtio transport [1]. It should be able to work in a vanilla Linux kernel. I think this requires a new "vendor-specific" or similar device. The driver needs to avoid collisions with other devices and must not take an unassigned ID. I think "vendor-specific" does not capture all possible use cases for such a device, though. It could be a "self-assigned" device, e.g. for prototypes to be standardized later. In the discussion, it was proposed to distinguish "vendor-specific" devices through common configuration space fields. I would suggest to use a UUID at the start of the configuration space of the new device 52 to distinguish "vendor-specific"/"self-assigned" devices. Everybody can generate a UUID when needed. This would - allow everybody to create new devices without ID collisions due to using unregistered IDs - allow using UUIDs for prototyping, with a new one available for each breaking change - be a way to resolve the device ID scarcity - relieve Virtio TC from some device ID registration work. Best regards, Peter [1] https://lore.kernel.org/all/20261015-jshekhar-drm-msm-dcp-v1-v1-10-48aad0fe4f72@oss.qualcomm.com/ > > > > But no upstream specification, correct? There's no point without an > > open device specification to being included in a specification. People > > can only implement frontend drivers for this vendor device, and only > > by reading the upstream code - no spec to follow. Similar to a vendor > > SoC in concept. > > > >> > >> In case of an in-band approach, I suppose we would then have an explicit > >> vendor list in the spec where vendors have to register? > >> > >> Best regards, > >> > >> Peter > >> > >> > >>>>>> So why to bend things so much and re-invent the wheel? > >>>>>> Why for say PCI transport's any vendor and device id are not sufficient to indicate a vendor and device? > >>>>>> > >>>>>> and if so, why to even document it in the virtio spec? > >>>>>> > >>>>>> I guess, I am missing the basic motivation part. > >>>>> > >>>>> For the virtio spec? Above. For Qualcomm since they decided never to > >>>>> upstream - donnu. Don't care much, either... > >>>> > >>>> ok. so yeah, let vendor's platform device be in the vendor's OSS zone. > >>> > -- > Sincerely, > Demi Marie Obenour (she/her/hers) ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH v2] content: Reserve ID 52 for QPSFI device 2026-10-23 20:56 ` Peter Hilber @ 2026-10-23 23:47 ` Michael S. Tsirkin 2026-10-26 14:31 ` Cornelia Huck 1 sibling, 0 replies; 29+ messages in thread From: Michael S. Tsirkin @ 2026-10-23 23:47 UTC (permalink / raw) To: Peter Hilber Cc: Demi Marie Obenour, Manos Pitsidianakis, Parav Pandit, Cornelia Huck, Dmitry Baryshkov, virtio-comment@lists.linux.dev, Jan Kiszka, Stefan Hajnoczi, Trilok Soni On Fri, Oct 23, 2026 at 10:56:24PM +0200, Peter Hilber wrote: > - relieve Virtio TC from some device ID registration work. At the cost of easing pressure on vendors to specify their devices. -- MST ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH v2] content: Reserve ID 52 for QPSFI device 2026-10-23 20:56 ` Peter Hilber 2026-10-23 23:47 ` Michael S. Tsirkin @ 2026-10-26 14:31 ` Cornelia Huck 2026-10-28 14:22 ` Peter Hilber 1 sibling, 1 reply; 29+ messages in thread From: Cornelia Huck @ 2026-10-26 14:31 UTC (permalink / raw) To: Peter Hilber, Demi Marie Obenour Cc: Manos Pitsidianakis, Michael S. Tsirkin, Parav Pandit, Dmitry Baryshkov, virtio-comment@lists.linux.dev, Jan Kiszka, Stefan Hajnoczi, Trilok Soni On Fri, Oct 23 2026, Peter Hilber <peter.hilber@oss.qualcomm.com> wrote: > Qualcomm's intent is to upstream a Linux kernel Virtio driver for a > non-standardized device using a Virtio transport [1]. It should be able > to work in a vanilla Linux kernel. I think this requires a new > "vendor-specific" or similar device. The driver needs to avoid > collisions with other devices and must not take an unassigned ID. > > I think "vendor-specific" does not capture all possible use cases for > such a device, though. It could be a "self-assigned" device, e.g. for > prototypes to be standardized later. > > In the discussion, it was proposed to distinguish "vendor-specific" > devices through common configuration space fields. I would suggest to > use a UUID at the start of the configuration space of the new device 52 > to distinguish "vendor-specific"/"self-assigned" devices. Everybody can > generate a UUID when needed. This would > > - allow everybody to create new devices without ID collisions due to > using unregistered IDs > > - allow using UUIDs for prototyping, with a new one available for each > breaking change I would prefer to have a namespace for this kind of experimentation, i.e. <testing prefix><uuid>, with vendor or purpose-specific devices using <vendor/purpose prefix><uuid>, so that... > > - be a way to resolve the device ID scarcity ...this would still hold (I think the most limiting factor for this is actually ccw? not sure how much we care about implementing new devices on ccw, though), and... > > - relieve Virtio TC from some device ID registration work. ...this would also still somewhat hold (no need to reserve an ID for everything, but if you're serious, get a vendor-or-purpose prefix.) I'd still want to encourage everyone creating a serious device to go and try to standardize it. ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH v2] content: Reserve ID 52 for QPSFI device 2026-10-26 14:31 ` Cornelia Huck @ 2026-10-28 14:22 ` Peter Hilber 0 siblings, 0 replies; 29+ messages in thread From: Peter Hilber @ 2026-10-28 14:22 UTC (permalink / raw) To: Cornelia Huck Cc: Demi Marie Obenour, Manos Pitsidianakis, Michael S. Tsirkin, Parav Pandit, Dmitry Baryshkov, virtio-comment@lists.linux.dev, Jan Kiszka, Stefan Hajnoczi, Trilok Soni On Mon, Oct 26, 2026 at 03:31:16PM +0100, Cornelia Huck wrote: > On Fri, Oct 23 2026, Peter Hilber <peter.hilber@oss.qualcomm.com> wrote: > > > Qualcomm's intent is to upstream a Linux kernel Virtio driver for a > > non-standardized device using a Virtio transport [1]. It should be able > > to work in a vanilla Linux kernel. I think this requires a new > > "vendor-specific" or similar device. The driver needs to avoid > > collisions with other devices and must not take an unassigned ID. > > > > I think "vendor-specific" does not capture all possible use cases for > > such a device, though. It could be a "self-assigned" device, e.g. for > > prototypes to be standardized later. > > > > In the discussion, it was proposed to distinguish "vendor-specific" > > devices through common configuration space fields. I would suggest to > > use a UUID at the start of the configuration space of the new device 52 > > to distinguish "vendor-specific"/"self-assigned" devices. Everybody can > > generate a UUID when needed. This would > > > > - allow everybody to create new devices without ID collisions due to > > using unregistered IDs > > > > - allow using UUIDs for prototyping, with a new one available for each > > breaking change > > I would prefer to have a namespace for this kind of experimentation, > i.e. <testing prefix><uuid>, with vendor or purpose-specific devices > using <vendor/purpose prefix><uuid>, so that... > > > > > - be a way to resolve the device ID scarcity > > ...this would still hold (I think the most limiting factor for this is > actually ccw? not sure how much we care about implementing new devices > on ccw, though), and... > > > > > - relieve Virtio TC from some device ID registration work. > > ...this would also still somewhat hold (no need to reserve an ID for > everything, but if you're serious, get a vendor-or-purpose prefix.) > I'd still want to encourage everyone creating a serious device to go and > try to standardize it. > I am wondering how these namespaces would be managed and used. Would introducing a vendor/purpose prefix require an approval process in the Virtio TC? Would new standardized devices have a "standard" prefix when using ID 52? Could a device UUID transparently transition from testing to vendor/purpose, so that drivers would only match on the UUID? Could the namespaces be seen as a quality gate so that testing-prefix devices may be refused in some circumstances? ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH v2] content: Reserve ID 52 for QPSFI device 2026-10-22 14:07 ` Dmitry Baryshkov 2026-10-22 15:05 ` Cornelia Huck @ 2026-10-22 15:44 ` Michael S. Tsirkin 1 sibling, 0 replies; 29+ messages in thread From: Michael S. Tsirkin @ 2026-10-22 15:44 UTC (permalink / raw) To: Dmitry Baryshkov Cc: Manos Pitsidianakis, Demi Marie Obenour, Cornelia Huck, Peter Hilber, virtio-comment, Jan Kiszka, Stefan Hajnoczi, Trilok Soni On Thu, Oct 22, 2026 at 05:07:09PM +0300, Dmitry Baryshkov wrote: > Let's define some standard way for the vendors to declare that this > device is "Qualcomm", "NXP" or "hobbyist which doesn't own PCI or USB id > block". That is best, yes. Is this acceptable for Qualcomm? You will be able to make it work with your SOCs? -- MST ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH v2] content: Reserve ID 52 for QPSFI device 2026-10-22 13:19 ` Michael S. Tsirkin 2026-10-22 14:07 ` Dmitry Baryshkov @ 2026-10-22 14:50 ` Zelong Ren 1 sibling, 0 replies; 29+ messages in thread From: Zelong Ren @ 2026-10-22 14:50 UTC (permalink / raw) To: Michael S . Tsirkin Cc: Cornelia Huck, Demi Marie Obenour, Jan Kiszka, Manos Pitsidianakis, Peter Hilber, Stefan Hajnoczi, Trilok Soni, lxia, virtio-comment Hi Manos, Michael, > What about the HAB btw? It was said the spec will be upstreamed > did I miss that? > It doesn't make sense to me that HAB/QPSFI need an ID when they are > unusable to anyone except for qualcomm kernel drivers so maybe we > should require that an ID allocation always needs an explicit > justification (i.e. "the spec will be upstreamed", or "we cannot use > VIRTIO_ID_VENDOR because X/Y/Z"). We are already working on virtio-hab specification, linux kernel RFC patch and qemu patch as the first review package. It is not intended to remain a Qualcomm-only device or driver. In fact, even the downstream virtio-hab hab vendor specific implementation, it has already been used by many external Qualcomm customers as an easy-to-use and reliable generic inter-VM shared-memory IPC mechanism. Current work is trimming down the vendor specific implementation and add support for more common arch(x86, arm), more common vmm(qemu, crosvm) based on kvm(and Xen follows). We need some time to complete this work but it will be soon. We will need everybody's support on this. Just to repeat the functionality of virtio-hab is to provide a generic inter-VM shared-memory IPC mechanism for user-space applications on multi-VM systems. HAB provides shared-memory support as the primary data path, including complete shared-memory lifecycle management. HAB also provides a simple messaging mechanism as the control path for connection management and simple synchronization between the frontend and backend. So for virtio-hab, the existing approved device id 51 is more appropriate for everyone. This is very different than a generic VIRTIO_ID_VENDOR. Please let us know if you have any more questions. Best regards, Zelong ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH v2] content: Reserve ID 52 for QPSFI device 2026-10-21 18:00 [PATCH v2] content: Reserve ID 52 for QPSFI device Peter Hilber 2026-10-21 19:25 ` Demi Marie Obenour @ 2026-10-21 19:41 ` Michael S. Tsirkin 2026-10-22 13:56 ` Dmitry Baryshkov 2 siblings, 0 replies; 29+ messages in thread From: Michael S. Tsirkin @ 2026-10-21 19:41 UTC (permalink / raw) To: Peter Hilber; +Cc: virtio-comment, Jan Kiszka, Stefan Hajnoczi, Trilok Soni On Wed, Oct 21, 2026 at 08:00:31PM +0200, Peter Hilber wrote: > Reserve a device ID for QPSFI devices (Qualcomm Platform Subsystem > Firmware Interface). QPSFI devices expose functionality across SoC > subsystems through a common Virtio-based protocol. There are no plans to > standardize the devices. > > The Linux DCP display driver series is planned as the first user of this > device ID: > > https://lore.kernel.org/all/20261015-jshekhar-drm-msm-dcp-v1-v1-10-48aad0fe4f72@oss.qualcomm.com/ > > Device ID 51 is already reserved for the Virtio HAB device: > > https://lore.kernel.org/virtio-comment/20260702140939.3535968-1-zeloren@qti.qualcomm.com/ > > Signed-off-by: Peter Hilber <peter.hilber@oss.qualcomm.com> What about the HAB btw? It was said the spec will be upstreamed did I miss that? > --- > > v2: > > - Rename the device from "Snapdragon device" to "QPSFI device". > > - Refer to DCP Linux driver patch as planned first QPSFI user. > > v1: > > - Was posted as "content: Reserve ID 52 for Snapdragon device": > > https://lore.kernel.org/virtio-comment/20260826115504.1955-1-peter.hilber@oss.qualcomm.com/ > > content.tex | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/content.tex b/content.tex > index 243ce2dde36b..7416939e67d3 100644 > --- a/content.tex > +++ b/content.tex > @@ -805,6 +805,8 @@ \chapter{Device Types}\label{sec:Device Types} > \hline > 50 & Video encoder/decoder device \\ > \hline > +52 & QPSFI device \\ > +\hline > \end{longtable} > > Some of the devices above are unspecified by this document, > -- > 2.43.0 ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH v2] content: Reserve ID 52 for QPSFI device 2026-10-21 18:00 [PATCH v2] content: Reserve ID 52 for QPSFI device Peter Hilber 2026-10-21 19:25 ` Demi Marie Obenour 2026-10-21 19:41 ` Michael S. Tsirkin @ 2026-10-22 13:56 ` Dmitry Baryshkov 2 siblings, 0 replies; 29+ messages in thread From: Dmitry Baryshkov @ 2026-10-22 13:56 UTC (permalink / raw) To: Peter Hilber Cc: virtio-comment, Jan Kiszka, Stefan Hajnoczi, Michael S . Tsirkin, Trilok Soni On Wed, Oct 21, 2026 at 08:00:31PM +0200, Peter Hilber wrote: > Reserve a device ID for QPSFI devices (Qualcomm Platform Subsystem > Firmware Interface). QPSFI devices expose functionality across SoC > subsystems through a common Virtio-based protocol. There are no plans to > standardize the devices. > > The Linux DCP display driver series is planned as the first user of this > device ID: What exactly does this mean? What other users are planned for this device? How do we plan to differentiate between them on the kernel side? What exactly is planned for device 52? If the QPSFI device is passed into the Guest VM, what should that VM do? > > https://lore.kernel.org/all/20261015-jshekhar-drm-msm-dcp-v1-v1-10-48aad0fe4f72@oss.qualcomm.com/ > > Device ID 51 is already reserved for the Virtio HAB device: > > https://lore.kernel.org/virtio-comment/20260702140939.3535968-1-zeloren@qti.qualcomm.com/ > > Signed-off-by: Peter Hilber <peter.hilber@oss.qualcomm.com> > --- > > v2: > > - Rename the device from "Snapdragon device" to "QPSFI device". > > - Refer to DCP Linux driver patch as planned first QPSFI user. > > v1: > > - Was posted as "content: Reserve ID 52 for Snapdragon device": > > https://lore.kernel.org/virtio-comment/20260826115504.1955-1-peter.hilber@oss.qualcomm.com/ > > content.tex | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/content.tex b/content.tex > index 243ce2dde36b..7416939e67d3 100644 > --- a/content.tex > +++ b/content.tex > @@ -805,6 +805,8 @@ \chapter{Device Types}\label{sec:Device Types} > \hline > 50 & Video encoder/decoder device \\ > \hline > +52 & QPSFI device \\ > +\hline > \end{longtable} > > Some of the devices above are unspecified by this document, > -- > 2.43.0 > -- With best wishes Dmitry ^ permalink raw reply [flat|nested] 29+ messages in thread
end of thread, other threads:[~2026-10-28 14:22 UTC | newest] Thread overview: 29+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2026-10-21 18:00 [PATCH v2] content: Reserve ID 52 for QPSFI device Peter Hilber 2026-10-21 19:25 ` Demi Marie Obenour 2026-10-21 19:50 ` Michael S. Tsirkin 2026-10-22 4:35 ` Cornelia Huck 2026-10-22 7:07 ` Demi Marie Obenour 2026-10-22 7:24 ` Michael S. Tsirkin 2026-10-22 8:41 ` Manos Pitsidianakis 2026-10-22 13:19 ` Michael S. Tsirkin 2026-10-22 14:07 ` Dmitry Baryshkov 2026-10-22 15:05 ` Cornelia Huck 2026-10-22 15:35 ` Parav Pandit 2026-10-22 15:41 ` Michael S. Tsirkin 2026-10-22 15:49 ` Parav Pandit 2026-10-22 16:02 ` Michael S. Tsirkin 2026-10-22 16:20 ` Parav Pandit 2026-10-22 16:29 ` Michael S. Tsirkin 2026-10-22 22:39 ` Peter Hilber 2026-10-22 23:28 ` Michael S. Tsirkin 2026-10-23 3:32 ` Parav Pandit 2026-10-23 5:37 ` Manos Pitsidianakis 2026-10-23 5:40 ` Demi Marie Obenour 2026-10-23 20:56 ` Peter Hilber 2026-10-23 23:47 ` Michael S. Tsirkin 2026-10-26 14:31 ` Cornelia Huck 2026-10-28 14:22 ` Peter Hilber 2026-10-22 15:44 ` Michael S. Tsirkin 2026-10-22 14:50 ` Zelong Ren 2026-10-21 19:41 ` Michael S. Tsirkin 2026-10-22 13:56 ` Dmitry Baryshkov
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.