[virtio-comment] [PATCH v1 1/2] virtio-gpu: Rename GFXSTREAM context capset to GFXSTREAM_VULKAN

[virtio-comment] [PATCH v1 1/2] virtio-gpu: Rename GFXSTREAM context capset to GFXSTREAM_VULKAN

[Dmitry Osipenko]

Initially, the plan was to use a single GFXSTREAM context for all of
GLES/GL/VK gfxstream sub-contexts. Over time the plan changed and
GFXSTREAM context became used exclusively for Vulkan, while GLES/GL
contexts were moved to a new own experimental virtio-gpu contexts.
Rename GFXSTREAM capset to GFXSTREAM_VULKAN.

Link: https://lore.kernel.org/dri-devel/CAAfnVB=Mki1zEyjByJ_7tRrib74E4KSbjezqQcb_Uia8EQEXHw@mail.gmail.com/
Link: https://chromium.googlesource.com/chromiumos/platform/crosvm/+/e4c1878733937042111fca58899a3a94002bfef0
Signed-off-by: Dmitry Osipenko <dmitry.osipenko@collabora.com>
---
 device-types/gpu/description.tex | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/device-types/gpu/description.tex b/device-types/gpu/description.tex
index 443524851a05..4b8dd7733c31 100644
--- a/device-types/gpu/description.tex
+++ b/device-types/gpu/description.tex
@@ -530,8 +530,8 @@ \subsubsection{Device Operation: controlq}\label{sec:Device Types / GPU Device /
 	the first edition of Virgl (Gallium OpenGL) protocol.
   \item \href{https://gitlab.freedesktop.org/virgl/virglrenderer/-/blob/master/src/virgl_hw.h#L550}{VIRTIO_GPU_CAPSET_VIRGL2} --
 	the second edition of Virgl (Gallium OpenGL) protocol after the capset fix.
-  \item \href{https://android.googlesource.com/device/generic/vulkan-cereal/+/refs/heads/master/protocols/}{VIRTIO_GPU_CAPSET_GFXSTREAM} --
-	gfxtream's (mostly) autogenerated GLES and Vulkan streaming protocols.
+  \item \href{https://android.googlesource.com/device/generic/vulkan-cereal/+/refs/heads/android13-release/protocols/vulkan/}{VIRTIO_GPU_CAPSET_GFXSTREAM_VULKAN} --
+	gfxtream's (mostly) autogenerated Vulkan streaming protocol.
   \item \href{https://gitlab.freedesktop.org/olv/venus-protocol}{VIRTIO_GPU_CAPSET_VENUS} --
 	Mesa's (mostly) autogenerated Vulkan protocol.
   \item \href{https://chromium.googlesource.com/chromiumos/platform/crosvm/+/refs/heads/main/rutabaga_gfx/src/cross_domain/cross_domain_protocol.rs}{VIRTIO_GPU_CAPSET_CROSS_DOMAIN} --
@@ -547,7 +547,7 @@ \subsubsection{Device Operation: controlq}\label{sec:Device Types / GPU Device /
 
 #define VIRTIO_GPU_CAPSET_VIRGL 1
 #define VIRTIO_GPU_CAPSET_VIRGL2 2
-#define VIRTIO_GPU_CAPSET_GFXSTREAM 3
+#define VIRTIO_GPU_CAPSET_GFXSTREAM_VULKAN 3
 #define VIRTIO_GPU_CAPSET_VENUS 4
 #define VIRTIO_GPU_CAPSET_CROSS_DOMAIN 5
 struct virtio_gpu_resp_capset_info {
-- 
2.41.0


This publicly archived list offers a means to provide input to the
OASIS Virtual I/O Device (VIRTIO) TC.

In order to verify user consent to the Feedback License terms and
to minimize spam in the list archive, subscription is required
before posting.

Subscribe: virtio-comment-subscribe@lists.oasis-open.org
Unsubscribe: virtio-comment-unsubscribe@lists.oasis-open.org
List help: virtio-comment-help@lists.oasis-open.org
List archive: https://lists.oasis-open.org/archives/virtio-comment/
Feedback License: https://www.oasis-open.org/who/ipr/feedback_license.pdf
List Guidelines: https://www.oasis-open.org/policies-guidelines/mailing-lists
Committee: https://www.oasis-open.org/committees/virtio/
Join OASIS: https://www.oasis-open.org/join/

[virtio-comment] [PATCH v1 2/2] virtio-gpu: Document VIRTIO_GPU_CAPSET_DRM

[Dmitry Osipenko]

Document VIRTIO_GPU_CAPSET_DRM that is capset for the DRM native context
protocols which implement Linux DRM UAPIs over VirtIO-GPU, effectively
exposing host GPU to a Linux guest like it's a native/baremetal GPU.

Link: https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/14900
Link: https://gitlab.freedesktop.org/virgl/virglrenderer/-/merge_requests/693
Link: https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/21658
Link: https://gitlab.freedesktop.org/virgl/virglrenderer/-/merge_requests/1274
Link: https://gitlab.freedesktop.org/digetx/mesa/-/commits/native-context-iris
Link: https://chromium.googlesource.com/chromiumos/platform/crosvm/+/69ecfb0e3913a4cd9048056f9a37cc496c7b9235
Signed-off-by: Dmitry Osipenko <dmitry.osipenko@collabora.com>
---
 device-types/gpu/description.tex | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/device-types/gpu/description.tex b/device-types/gpu/description.tex
index 4b8dd7733c31..29c8a97c3a3f 100644
--- a/device-types/gpu/description.tex
+++ b/device-types/gpu/description.tex
@@ -536,6 +536,8 @@ \subsubsection{Device Operation: controlq}\label{sec:Device Types / GPU Device /
 	Mesa's (mostly) autogenerated Vulkan protocol.
   \item \href{https://chromium.googlesource.com/chromiumos/platform/crosvm/+/refs/heads/main/rutabaga_gfx/src/cross_domain/cross_domain_protocol.rs}{VIRTIO_GPU_CAPSET_CROSS_DOMAIN} --
 	protocol for display virtualization via Wayland proxying.
+  \item \href{https://gitlab.freedesktop.org/virgl/virglrenderer/-/tree/master/src/drm}{VIRTIO_GPU_CAPSET_DRM} --
+	DRM native context protocols.
   \end{itemize*}
 
 \begin{lstlisting}
@@ -550,6 +552,7 @@ \subsubsection{Device Operation: controlq}\label{sec:Device Types / GPU Device /
 #define VIRTIO_GPU_CAPSET_GFXSTREAM_VULKAN 3
 #define VIRTIO_GPU_CAPSET_VENUS 4
 #define VIRTIO_GPU_CAPSET_CROSS_DOMAIN 5
+#define VIRTIO_GPU_CAPSET_DRM 6
 struct virtio_gpu_resp_capset_info {
         struct virtio_gpu_ctrl_hdr hdr;
         le32 capset_id;
-- 
2.41.0


This publicly archived list offers a means to provide input to the
OASIS Virtual I/O Device (VIRTIO) TC.

In order to verify user consent to the Feedback License terms and
to minimize spam in the list archive, subscription is required
before posting.

Subscribe: virtio-comment-subscribe@lists.oasis-open.org
Unsubscribe: virtio-comment-unsubscribe@lists.oasis-open.org
List help: virtio-comment-help@lists.oasis-open.org
List archive: https://lists.oasis-open.org/archives/virtio-comment/
Feedback License: https://www.oasis-open.org/who/ipr/feedback_license.pdf
List Guidelines: https://www.oasis-open.org/policies-guidelines/mailing-lists
Committee: https://www.oasis-open.org/committees/virtio/
Join OASIS: https://www.oasis-open.org/join/

[virtio-comment] Re: [PATCH v1 1/2] virtio-gpu: Rename GFXSTREAM context capset to GFXSTREAM_VULKAN

[Huang Rui]

On Mon, Nov 06, 2023 at 05:17:27AM +0800, Dmitry Osipenko wrote:
> Initially, the plan was to use a single GFXSTREAM context for all of
> GLES/GL/VK gfxstream sub-contexts. Over time the plan changed and
> GFXSTREAM context became used exclusively for Vulkan, while GLES/GL
> contexts were moved to a new own experimental virtio-gpu contexts.
> Rename GFXSTREAM capset to GFXSTREAM_VULKAN.
> 
> Link: https://lore.kernel.org/dri-devel/CAAfnVB=Mki1zEyjByJ_7tRrib74E4KSbjezqQcb_Uia8EQEXHw@mail.gmail.com/
> Link: https://chromium.googlesource.com/chromiumos/platform/crosvm/+/e4c1878733937042111fca58899a3a94002bfef0
> Signed-off-by: Dmitry Osipenko <dmitry.osipenko@collabora.com>

These two patches are Acked-by: Huang Rui <ray.huang@amd.com>

(Re-send, because just subscribe virtio-comment)

> ---
>  device-types/gpu/description.tex | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/device-types/gpu/description.tex b/device-types/gpu/description.tex
> index 443524851a05..4b8dd7733c31 100644
> --- a/device-types/gpu/description.tex
> +++ b/device-types/gpu/description.tex
> @@ -530,8 +530,8 @@ \subsubsection{Device Operation: controlq}\label{sec:Device Types / GPU Device /
>  	the first edition of Virgl (Gallium OpenGL) protocol.
>    \item \href{https://gitlab.freedesktop.org/virgl/virglrenderer/-/blob/master/src/virgl_hw.h#L550}{VIRTIO_GPU_CAPSET_VIRGL2} --
>  	the second edition of Virgl (Gallium OpenGL) protocol after the capset fix.
> -  \item \href{https://android.googlesource.com/device/generic/vulkan-cereal/+/refs/heads/master/protocols/}{VIRTIO_GPU_CAPSET_GFXSTREAM} --
> -	gfxtream's (mostly) autogenerated GLES and Vulkan streaming protocols.
> +  \item \href{https://android.googlesource.com/device/generic/vulkan-cereal/+/refs/heads/android13-release/protocols/vulkan/}{VIRTIO_GPU_CAPSET_GFXSTREAM_VULKAN} --
> +	gfxtream's (mostly) autogenerated Vulkan streaming protocol.
>    \item \href{https://gitlab.freedesktop.org/olv/venus-protocol}{VIRTIO_GPU_CAPSET_VENUS} --
>  	Mesa's (mostly) autogenerated Vulkan protocol.
>    \item \href{https://chromium.googlesource.com/chromiumos/platform/crosvm/+/refs/heads/main/rutabaga_gfx/src/cross_domain/cross_domain_protocol.rs}{VIRTIO_GPU_CAPSET_CROSS_DOMAIN} --
> @@ -547,7 +547,7 @@ \subsubsection{Device Operation: controlq}\label{sec:Device Types / GPU Device /
>  
>  #define VIRTIO_GPU_CAPSET_VIRGL 1
>  #define VIRTIO_GPU_CAPSET_VIRGL2 2
> -#define VIRTIO_GPU_CAPSET_GFXSTREAM 3
> +#define VIRTIO_GPU_CAPSET_GFXSTREAM_VULKAN 3
>  #define VIRTIO_GPU_CAPSET_VENUS 4
>  #define VIRTIO_GPU_CAPSET_CROSS_DOMAIN 5
>  struct virtio_gpu_resp_capset_info {
> -- 
> 2.41.0
> 

This publicly archived list offers a means to provide input to the
OASIS Virtual I/O Device (VIRTIO) TC.

In order to verify user consent to the Feedback License terms and
to minimize spam in the list archive, subscription is required
before posting.

Subscribe: virtio-comment-subscribe@lists.oasis-open.org
Unsubscribe: virtio-comment-unsubscribe@lists.oasis-open.org
List help: virtio-comment-help@lists.oasis-open.org
List archive: https://lists.oasis-open.org/archives/virtio-comment/
Feedback License: https://www.oasis-open.org/who/ipr/feedback_license.pdf
List Guidelines: https://www.oasis-open.org/policies-guidelines/mailing-lists
Committee: https://www.oasis-open.org/committees/virtio/
Join OASIS: https://www.oasis-open.org/join/