[PATCH] content: Reserve ID 50 for video encoder/decoder device

[PATCH] content: Reserve ID 50 for video encoder/decoder device

[Alexander Gordeev]

Virtio video encoder/decoder device provides a stateful API for video
encoding and decoding.

IDs 30 and 31 are already reserved for devices providing only one of
the two functions. There are use-cases for a device providing both
functions, so a new ID is reserved after a discussion on the ML ([1]).

[1] https://lore.kernel.org/virtio-comment/20241223055543-mutt-send-email-mst@kernel.org/

Signed-off-by: Alexander Gordeev <alexander.gordeev@oss.qualcomm.com>
---
 content.tex | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/content.tex b/content.tex
index 5de811f..243ce2d 100644
--- a/content.tex
+++ b/content.tex
@@ -803,6 +803,8 @@ \chapter{Device Types}\label{sec:Device Types}
 \hline
 49         &   USB controller \\
 \hline
+50         &   Video encoder/decoder device \\
+\hline
 \end{longtable}
 
 Some of the devices above are unspecified by this document,
-- 
2.43.0

Re: [PATCH] content: Reserve ID 50 for video encoder/decoder device

[Manos Pitsidianakis]

On Mon, Oct 27, 2025 at 10:34 PM Alexander Gordeev
<alexander.gordeev@oss.qualcomm.com> wrote:
>
> Virtio video encoder/decoder device provides a stateful API for video
> encoding and decoding.
>
> IDs 30 and 31 are already reserved for devices providing only one of
> the two functions. There are use-cases for a device providing both
> functions, so a new ID is reserved after a discussion on the ML ([1]).
>
> [1] https://lore.kernel.org/virtio-comment/20241223055543-mutt-send-email-mst@kernel.org/
>
> Signed-off-by: Alexander Gordeev <alexander.gordeev@oss.qualcomm.com>
> ---

Reviewed-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>

>  content.tex | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/content.tex b/content.tex
> index 5de811f..243ce2d 100644
> --- a/content.tex
> +++ b/content.tex
> @@ -803,6 +803,8 @@ \chapter{Device Types}\label{sec:Device Types}
>  \hline
>  49         &   USB controller \\
>  \hline
> +50         &   Video encoder/decoder device \\
> +\hline
>  \end{longtable}
>
>  Some of the devices above are unspecified by this document,
> --
> 2.43.0
>

Re: [PATCH] content: Reserve ID 50 for video encoder/decoder device

[Matias Ezequiel Vara Larsen]

On Mon, Oct 27, 2025 at 09:33:56PM +0100, Alexander Gordeev wrote:
> Virtio video encoder/decoder device provides a stateful API for video
> encoding and decoding.
> 
> IDs 30 and 31 are already reserved for devices providing only one of
> the two functions. There are use-cases for a device providing both
> functions, so a new ID is reserved after a discussion on the ML ([1]).
> 
> [1] https://lore.kernel.org/virtio-comment/20241223055543-mutt-send-email-mst@kernel.org/
> 
> Signed-off-by: Alexander Gordeev <alexander.gordeev@oss.qualcomm.com>
> ---

Reviewed-by: Matias Ezequiel Vara Larsen <mvaralar@redhat.com

Re: [PATCH] content: Reserve ID 50 for video encoder/decoder device

[Alexander Gordeev]

On 27/10/2025 21:33, Alexander Gordeev wrote:
> Virtio video encoder/decoder device provides a stateful API for video
> encoding and decoding.
> 
> IDs 30 and 31 are already reserved for devices providing only one of
> the two functions. There are use-cases for a device providing both
> functions, so a new ID is reserved after a discussion on the ML ([1]).
> 
> [1] https://lore.kernel.org/virtio-comment/20241223055543-mutt-send-email-mst@kernel.org/
> 
> Signed-off-by: Alexander Gordeev <alexander.gordeev@oss.qualcomm.com>
> ---
>  content.tex | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/content.tex b/content.tex
> index 5de811f..243ce2d 100644
> --- a/content.tex
> +++ b/content.tex
> @@ -803,6 +803,8 @@ \chapter{Device Types}\label{sec:Device Types}
>  \hline
>  49         &   USB controller \\
>  \hline
> +50         &   Video encoder/decoder device \\
> +\hline
>  \end{longtable}
>  
>  Some of the devices above are unspecified by this document,

Thanks for the reviews!
Could we please open a voting ballot for these changes?

Fixes: https://github.com/oasis-tcs/virtio-spec/issues/235