From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757472AbdELRE5 (ORCPT ); Fri, 12 May 2017 13:04:57 -0400 Received: from mx1.redhat.com ([209.132.183.28]:38532 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751323AbdELRE4 (ORCPT ); Fri, 12 May 2017 13:04:56 -0400 DMARC-Filter: OpenDMARC Filter v1.3.2 mx1.redhat.com 16EFC7D4E3 Authentication-Results: ext-mx04.extmail.prod.ext.phx2.redhat.com; dmarc=none (p=none dis=none) header.from=redhat.com Authentication-Results: ext-mx04.extmail.prod.ext.phx2.redhat.com; spf=pass smtp.mailfrom=alex.williamson@redhat.com DKIM-Filter: OpenDKIM Filter v2.11.0 mx1.redhat.com 16EFC7D4E3 Date: Fri, 12 May 2017 11:04:47 -0600 From: Alex Williamson To: "Chen, Xiaoguang" Cc: Gerd Hoffmann , "Tian, Kevin" , "intel-gfx@lists.freedesktop.org" , "linux-kernel@vger.kernel.org" , "zhenyuw@linux.intel.com" , "Lv, Zhiyuan" , "intel-gvt-dev@lists.freedesktop.org" , "Wang, Zhi A" Subject: Re: [RFC PATCH 6/6] drm/i915/gvt: support QEMU getting the dmabuf Message-ID: <20170512110447.63e3650a@t450s.home> In-Reply-To: References: <1493372130-27727-1-git-send-email-xiaoguang.chen@intel.com> <1493372130-27727-7-git-send-email-xiaoguang.chen@intel.com> <1493718658.8581.82.camel@redhat.com> <20170504100833.199bc8ba@t450s.home> <1493967331.371.53.camel@redhat.com> <20170505091115.7a680636@t450s.home> <1494509273.17970.12.camel@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.28]); Fri, 12 May 2017 17:04:56 +0000 (UTC) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, 12 May 2017 06:56:03 +0000 "Chen, Xiaoguang" wrote: > Hi Gerd, > > >-----Original Message----- > >From: intel-gvt-dev [mailto:intel-gvt-dev-bounces@lists.freedesktop.org] On > >Behalf Of Gerd Hoffmann > >Sent: Thursday, May 11, 2017 9:28 PM > >To: Chen, Xiaoguang > >Cc: Tian, Kevin ; intel-gfx@lists.freedesktop.org; linux- > >kernel@vger.kernel.org; zhenyuw@linux.intel.com; Alex Williamson > >; Lv, Zhiyuan ; intel-gvt- > >dev@lists.freedesktop.org; Wang, Zhi A > >Subject: Re: [RFC PATCH 6/6] drm/i915/gvt: support QEMU getting the dmabuf > > > > Hi, > > > >> While read the framebuffer region we have to tell the vendor driver which > >framebuffer we want to read? There are two framebuffers now in KVMGT that is > >primary and cursor. > >> There are two methods to implement this: > >> 1) write the plane id first and then read the framebuffer. > >> 2) create 2 vfio regions one for primary and one for cursor. > > > >(3) Place information for both planes into one vfio region. > > Which allows to fetch both with a single read() syscall. > That works too. Then using the ioctl to get the dmabuf fd if needed. And plane id can be ioctl's parameter. > > How about method 2 primary plane and cursor plane are different and should generate different dmabuf for each of them. > > > > >The question is how you'll get the file descriptor then. If the ioctl returns the > >dma-buf fd only you have a racy interface: Things can change between read(vfio- > >region) and ioctl(need-dmabuf-fd). > You are right. So when creating the dmabuf we may have to decode the framebuffer and create the dmabuf using the latest framebuffer information and we must return the framebuffer information together with the dmabuf fd. > > In the current implementation I saved the framebuffer information while user querying the framebuffer and generate the dmabuf using the saved information no error found yet but in theory there are sync problems. > > > > >ioctl(need-dma-buf) could return both dmabuf fd and plane info to fix the race, > >but then it is easier to go with ioctl only interface (simliar to the orginal one from > >dec last year) I think. > Yes. ioctl works for it. > But based on the mail last week. If I understand correctly Alex hope to query the framebuffer information by reading the vfio device region and then get the dmabuf fd using ioctl. No, I was explaining that I had questioned whether we could use a vfio region in place of a separate dmabuf fd. We can't. I have no particular desire to use a vfio region just for querying framebuffer info. I prefer the dmabuf manager fd idea that Gerd suggested. Thanks, Alex