Re: [Spice-devel] [PATCH RFC v4 0/1] Virtio Video Device Specification

2020-07-08 Thread Tomasz Figa
On Wed, Jul 8, 2020 at 6:35 AM Alexandre Courbot wrote: > > On Fri, Jul 3, 2020 at 6:55 PM Tomasz Figa wrote: > > > > On Fri, Jul 3, 2020 at 11:27 AM Alexandre Courbot > > wrote: > > > > > > On Fri, Jul 3, 2020 at 6:18 PM Michael S. Tsirkin wrote: > > > > > > > > On Fri, Jul 03, 2020 at 02:45:

Re: [Spice-devel] [PATCH RFC v4 0/1] Virtio Video Device Specification

2020-07-07 Thread Alexandre Courbot
On Fri, Jul 3, 2020 at 6:55 PM Tomasz Figa wrote: > > On Fri, Jul 3, 2020 at 11:27 AM Alexandre Courbot > wrote: > > > > On Fri, Jul 3, 2020 at 6:18 PM Michael S. Tsirkin wrote: > > > > > > On Fri, Jul 03, 2020 at 02:45:15PM +0900, Alexandre Courbot wrote: > > > > Hi Dmitry, > > > > > > > > On

Re: [Spice-devel] [PATCH RFC v4 0/1] Virtio Video Device Specification

2020-07-06 Thread Dmitry Sepp
Hi Alexandre, Tomasz, Thank you very much for your feedback. Yes, unfortunately we cannot disable dmabuf mode as it is currently mandatory for Android. AFAIU the work to have this ready in the main v4l2 spec requires time. But in the virtio-video spec we indeed can mention something like that

Re: [Spice-devel] [PATCH RFC v4 0/1] Virtio Video Device Specification

2020-07-03 Thread Michael S. Tsirkin
On Fri, Jul 03, 2020 at 02:45:15PM +0900, Alexandre Courbot wrote: > Hi Dmitry, > > On Thu, Jul 2, 2020 at 10:47 PM Dmitry Sepp > wrote: > > > > Hi Keiichi, > > > > Thanks for the clarification. I believe we should explicitly describe this > > in > > the VIRTIO_VIDEO_CMD_RESOURCE_ATTACH section

Re: [Spice-devel] [PATCH RFC v4 0/1] Virtio Video Device Specification

2020-07-03 Thread Tomasz Figa
On Fri, Jul 3, 2020 at 11:27 AM Alexandre Courbot wrote: > > On Fri, Jul 3, 2020 at 6:18 PM Michael S. Tsirkin wrote: > > > > On Fri, Jul 03, 2020 at 02:45:15PM +0900, Alexandre Courbot wrote: > > > Hi Dmitry, > > > > > > On Thu, Jul 2, 2020 at 10:47 PM Dmitry Sepp > > > wrote: > > > > > > > >

Re: [Spice-devel] [PATCH RFC v4 0/1] Virtio Video Device Specification

2020-07-03 Thread Alexandre Courbot
On Fri, Jul 3, 2020 at 6:18 PM Michael S. Tsirkin wrote: > > On Fri, Jul 03, 2020 at 02:45:15PM +0900, Alexandre Courbot wrote: > > Hi Dmitry, > > > > On Thu, Jul 2, 2020 at 10:47 PM Dmitry Sepp > > wrote: > > > > > > Hi Keiichi, > > > > > > Thanks for the clarification. I believe we should expl

Re: [Spice-devel] [PATCH RFC v4 0/1] Virtio Video Device Specification

2020-07-02 Thread Alexandre Courbot
Hi Dmitry, On Thu, Jul 2, 2020 at 10:47 PM Dmitry Sepp wrote: > > Hi Keiichi, > > Thanks for the clarification. I believe we should explicitly describe this in > the VIRTIO_VIDEO_CMD_RESOURCE_ATTACH section. And I also still see a problem > there. If it is a guest allocated resource, we cannot co

Re: [Spice-devel] [PATCH RFC v4 0/1] Virtio Video Device Specification

2020-07-02 Thread Keiichi Watanabe
Hi Dmitry, On Thu, Jul 2, 2020 at 4:32 PM Dmitry Sepp wrote: > > Hi Keiichi, > > Thank you very much for the hard work to update the spec and to summarize all > of the recent proposals! > > I want to again raise a topic that was discussed earlier and unfortunately the > latest proposal cannot res

Re: [Spice-devel] [PATCH RFC v4 0/1] Virtio Video Device Specification

2020-07-02 Thread Dmitry Sepp
Hi Keiichi, Thanks for the clarification. I believe we should explicitly describe this in the VIRTIO_VIDEO_CMD_RESOURCE_ATTACH section. And I also still see a problem there. If it is a guest allocated resource, we cannot consider it to be free until the device really releases it. And it won't h

Re: [Spice-devel] [PATCH RFC v4 0/1] Virtio Video Device Specification

2020-07-02 Thread Dmitry Sepp
Hi Keiichi, Thank you very much for the hard work to update the spec and to summarize all of the recent proposals! I want to again raise a topic that was discussed earlier and unfortunately the latest proposal cannot resolve the problem. I hope together with upstream people we'll be able to fi

[Spice-devel] [PATCH RFC v4 0/1] Virtio Video Device Specification

2020-06-23 Thread Keiichi Watanabe
This is the v4 RFC of virtio video device specification. PDF versions are available at [1, 2]. Note that this patch depends on a recent patchset "Cross-device resource sharing" [3]. Here is a list of major changes from v3: * Redesingned struct definitions for each command and request based on d