On Thu, Feb 03, 2022 at 12:10:17PM +0000, Peter Griffin wrote:
Hi Michael,
Apologies for the delaying in replying to you, I didn't see your email until now whilst replying to Alex review feedback.
On Tue, 11 Jan 2022, Michael S. Tsirkin wrote:
On Thu, Dec 09, 2021 at 02:55:53PM +0000, Peter Griffin wrote:
This series adds support for virtio-video decoder devices in Qemu and also provides a vhost-user-video vmm implementation.
The vhost-user-video vmm currently parses virtio-vido v3 protocol (as that is what the Linux frontend driver implements). It then converts that to a v4l2 mem2mem stateful decoder device. Currently this has been tested using v4l2 vicodec test driver in Linux [1] but it is intended to be used with Arm SoCs which often implement v4l2 stateful decoders/encoders drivers for their video accelerators.
The primary goal so far has been to allow continuing development of virtio-video Linux frontend driver and testing with Qemu. Using vicodec on the host allows a purely virtual dev env, and allows for ci integration in the future by kernelci etc.
This series also adds the virtio_video.h header and adds the FWHT format which is used by vicodec driver.
I have tested this VMM using v4l2-ctl from v4l2 utils in the guest to do a video decode to a file. This can then be validated using ffplay v4l2-compliance tool in the guest has also been run which stresses the interface and issues lots of syscall level tests
See the README.md for example commands on how to configure guest kernel and do a video decode using Qemu, vicodec using this VMM.
Linux virtio-video frontend driver code: https://github.com/petegriffin/linux/commits/v5.10-virtio-video-latest
Are you going to post this btw?
Do you think there is much value in posting this version of the driver again?
As it is more or less the same as what Opensynergy folks posted to linux-media ml plus a few fixups that have landed in the ChromeOS tree over time from using it in their products and a few extra fixups from me to move it to a newer kernel and fix a few bugs I found whilst working on the Qemu backend.
I guess it needs to be updated to v5 spec then?
Qemu vmm code: https://github.com/petegriffin/qemu/tree/vhost-virtio-video-master-v1
This is part of a wider initiative by Linaro called "project Stratos" for which you can find information here:
https://collaborate.linaro.org/display/STR/Stratos+Home
Applies cleanly to git://git.qemu.org/qemu.git master(a3607def89).
Thanks,
Peter.
And the spec status? Does this match this spec: https://lore.kernel.org/linux-media/CAPBb6MUD_oYeUt8_bGRwAPCPam40Jtktz2F7+A3... ? A year ago a new revision was coming RSN ...
No unfortunately this VMM currently implements the v3 spec. The next task is to update both the VMM and the frontend driver above to the v5 spec you linked to above.
We would like to get the v5 spec voted on and integrated into the standard so any advice you can give me as to the process for doing that would be appreciated :)
kind regards,
Peter.
I think as with most cases where the TC lacks experts in the specific field, we'd like to get acks from experts, one way to do that is by posting the driver and soliciting acks. Ideally we'd like to see an open source driver and device implementations alongside the spec (preferably qemu as it's still more popular than rust vmm and more people read C than rust).
Peter Griffin (8): vhost-user-video: Add a README.md with cheat sheet of commands MAINTAINERS: Add virtio-video section vhost-user-video: boiler plate code for vhost-user-video device vhost-user-video: add meson subdir build logic standard-headers: Add virtio_video.h virtio_video: Add Fast Walsh-Hadamard Transform format hw/display: add vhost-user-video-pci tools/vhost-user-video: Add initial vhost-user-video vmm
MAINTAINERS | 8 + hw/display/Kconfig | 5 + hw/display/meson.build | 3 + hw/display/vhost-user-video-pci.c | 82 + hw/display/vhost-user-video.c | 386 ++++ include/hw/virtio/vhost-user-video.h | 41 + include/standard-headers/linux/virtio_video.h | 484 +++++ tools/meson.build | 9 + tools/vhost-user-video/50-qemu-rpmb.json.in | 5 + tools/vhost-user-video/README.md | 98 + tools/vhost-user-video/main.c | 1680 ++++++++++++++++ tools/vhost-user-video/meson.build | 10 + tools/vhost-user-video/v4l2_backend.c | 1777 +++++++++++++++++ tools/vhost-user-video/v4l2_backend.h | 99 + tools/vhost-user-video/virtio_video_helpers.c | 462 +++++ tools/vhost-user-video/virtio_video_helpers.h | 166 ++ tools/vhost-user-video/vuvideo.h | 43 + 17 files changed, 5358 insertions(+) create mode 100644 hw/display/vhost-user-video-pci.c create mode 100644 hw/display/vhost-user-video.c create mode 100644 include/hw/virtio/vhost-user-video.h create mode 100644 include/standard-headers/linux/virtio_video.h create mode 100644 tools/vhost-user-video/50-qemu-rpmb.json.in create mode 100644 tools/vhost-user-video/README.md create mode 100644 tools/vhost-user-video/main.c create mode 100644 tools/vhost-user-video/meson.build create mode 100644 tools/vhost-user-video/v4l2_backend.c create mode 100644 tools/vhost-user-video/v4l2_backend.h create mode 100644 tools/vhost-user-video/virtio_video_helpers.c create mode 100644 tools/vhost-user-video/virtio_video_helpers.h create mode 100644 tools/vhost-user-video/vuvideo.h
-- 2.25.1