qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Oleksandr Andrushchenko <andr2000@gmail.com>
To: Gerd Hoffmann <kraxel@redhat.com>
Cc: Oleksandr Andrushchenko <Oleksandr_Andrushchenko@epam.com>,
	Dongwon Kim <dongwon.kim@intel.com>,
	Tomeu Vizoso <tomeu.vizoso@collabora.com>,
	David Airlie <airlied@linux.ie>,
	open list <linux-kernel@vger.kernel.org>,
	dri-devel <dri-devel@lists.freedesktop.org>,
	qemu-devel@nongnu.org,
	"moderated list:DMA BUFFER SHARING FRAMEWORK"
	<linaro-mm-sig@lists.linaro.org>,
	"open list:DMA BUFFER SHARING FRAMEWORK"
	<linux-media@vger.kernel.org>,
	Matt Roper <matthew.d.roper@intel.com>
Subject: Re: [Qemu-devel] [RfC PATCH] Add udmabuf misc device
Date: Fri, 6 Apr 2018 15:36:03 +0300	[thread overview]
Message-ID: <7ef89a29-6584-d23c-efd1-f30d9b767a24@gmail.com> (raw)
In-Reply-To: <20180406115730.jtwcbz5okrphlxli@sirius.home.kraxel.org>

On 04/06/2018 02:57 PM, Gerd Hoffmann wrote:
>    Hi,
>
>>> I fail to see any common ground for xen-zcopy and udmabuf ...
>> Does the above mean you can assume that xen-zcopy and udmabuf
>> can co-exist as two different solutions?
> Well, udmabuf route isn't fully clear yet, but yes.
>
> See also gvt (intel vgpu), where the hypervisor interface is abstracted
> away into a separate kernel modules even though most of the actual vgpu
> emulation code is common.
Thank you for your input, I'm just trying to figure out
which of the three z-copy solutions intersect and how much
>> And what about hyper-dmabuf?
> No idea, didn't look at it in detail.
>
> Looks pretty complex from a distant view.  Maybe because it tries to
> build a communication framework using dma-bufs instead of a simple
> dma-buf passing mechanism.
Yes, I am looking at it now, trying to figure out the full story
and its implementation. BTW, Intel guys were about to share some
test application for hyper-dmabuf, maybe I have missed one.
It could probably better explain the use-cases and the complexity
they have in hyper-dmabuf.
>
> Like xen-zcopy it seems to depend on the idea that the hypervisor
> manages all memory it is easy for guests to share pages with the help of
> the hypervisor.
So, for xen-zcopy we were not trying to make it generic,
it just solves display (dumb) zero-copying use-cases for Xen.
We implemented it as a DRM helper driver because we can't see any
other use-cases as of now.
For example, we also have Xen para-virtualized sound driver, but
its buffer memory usage is not comparable to what display wants
and it works somewhat differently (e.g. there is no "frame done"
event, so one can't tell when the sound buffer can be "flipped").
At the same time, we do not use virtio-gpu, so this could probably
be one more candidate for shared dma-bufs some day.
>    Which simply isn't the case on kvm.
>
> hyper-dmabuf and xen-zcopy could maybe share code, or hyper-dmabuf build
> on top of xen-zcopy.
Hm, I can imagine that: xen-zcopy could be a library code for hyper-dmabuf
in terms of implementing all that page sharing fun in multiple directions,
e.g. Host->Guest, Guest->Host, Guest<->Guest.
But I'll let Matt and Dongwon to comment on that.

>
> cheers,
>    Gerd
>
Thank you,
Oleksandr

P.S. Sorry for making your original mail thread to discuss things much
broader than your RFC...

  reply	other threads:[~2018-04-06 12:36 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-03-13 15:48 [Qemu-devel] [RfC PATCH] Add udmabuf misc device Gerd Hoffmann
2018-03-13 16:10 ` Daniel Vetter
2018-03-14  8:03   ` Gerd Hoffmann
2018-04-05 20:32     ` Daniel Vetter
2018-04-06  0:11       ` Matt Roper
2018-04-06  6:55         ` Oleksandr Andrushchenko
2018-04-06  9:07           ` Gerd Hoffmann
2018-04-06  9:34             ` Oleksandr Andrushchenko
2018-04-06 11:17             ` Oleksandr Andrushchenko
2018-04-06 11:57               ` Gerd Hoffmann
2018-04-06 12:36                 ` Oleksandr Andrushchenko [this message]
2018-04-06 18:57                   ` Dongwon Kim
2018-04-10  6:37                     ` Oleksandr Andrushchenko
2018-04-10 17:26                       ` Dongwon Kim
2018-04-11  5:59                         ` Oleksandr Andrushchenko
2018-04-13 15:37                           ` Daniel Vetter
2018-04-16  7:16                             ` Oleksandr Andrushchenko
2018-04-16  7:43                               ` Daniel Vetter
2018-04-16  8:22                                 ` Oleksandr Andrushchenko
2018-04-16  9:32                                   ` Daniel Vetter
2018-04-16 10:14                                     ` Oleksandr Andrushchenko
2018-04-16 12:08                                       ` Daniel Vetter
2018-04-09  8:12         ` Daniel Vetter
2018-04-06  9:52     ` Daniel Stone
2018-04-06 10:54       ` Gerd Hoffmann
2018-04-09  8:00         ` Daniel Vetter
2018-04-10 14:22           ` Gerd Hoffmann

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=7ef89a29-6584-d23c-efd1-f30d9b767a24@gmail.com \
    --to=andr2000@gmail.com \
    --cc=Oleksandr_Andrushchenko@epam.com \
    --cc=airlied@linux.ie \
    --cc=dongwon.kim@intel.com \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=kraxel@redhat.com \
    --cc=linaro-mm-sig@lists.linaro.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-media@vger.kernel.org \
    --cc=matthew.d.roper@intel.com \
    --cc=qemu-devel@nongnu.org \
    --cc=tomeu.vizoso@collabora.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).