All of lore.kernel.org
 help / color / mirror / Atom feed
From: Alex Williamson <alex.williamson@redhat.com>
To: Gerd Hoffmann <kraxel@redhat.com>
Cc: Tina Zhang <tina.zhang@intel.com>,
	chris@chris-wilson.co.uk, joonas.lahtinen@linux.intel.com,
	zhenyuw@linux.intel.com, zhiyuan.lv@intel.com,
	zhi.a.wang@intel.com, kevin.tian@intel.com, daniel@ffwll.ch,
	kwankhede@nvidia.com, hang.yuan@intel.com,
	intel-gfx@lists.freedesktop.org,
	intel-gvt-dev@lists.freedesktop.org,
	linux-kernel@vger.kernel.org,
	Daniel Vetter <daniel.vetter@ffwll.ch>
Subject: Re: [PATCH v17 5/6] vfio: ABI for mdev display dma-buf operation
Date: Thu, 9 Nov 2017 13:54:57 -0700	[thread overview]
Message-ID: <20171109135457.4f7514d3@t450s.home> (raw)
In-Reply-To: <20171109183514.w3tbqumovglr4sri@sirius.home.kraxel.org>

On Thu, 9 Nov 2017 19:35:14 +0100
Gerd Hoffmann <kraxel@redhat.com> wrote:

>   Hi,
> 
> > struct vfio_device_gfx_plane_info lacks the head field we've been
> > discussing.  Thanks,  
> 
> Adding multihead support turned out to not be that easy.  There are
> corner cases like a single framebuffer spawning both heads.  Also it
> would be useful to somehow hint to the guest which heads it should use.
> 
> In short:  Proper multihead support is more complex than just adding a
> head field for later use.  So in a short private discussion with Tina we
> came to the conclusion that it will be better add multihead support to
> the API when the first driver wants use it, so we can actually test the
> interface and make sure we didn't miss anything.  Adding a incomplete
> multihead API now doesn't help anybody.

Do you think we can enable multi-head and preserve backwards
compatibility within this API proposed here?  Thanks,

Alex

WARNING: multiple messages have this Message-ID (diff)
From: Alex Williamson <alex.williamson@redhat.com>
To: Gerd Hoffmann <kraxel@redhat.com>
Cc: hang.yuan@intel.com, Daniel Vetter <daniel.vetter@ffwll.ch>,
	intel-gfx@lists.freedesktop.org, linux-kernel@vger.kernel.org,
	kwankhede@nvidia.com, zhiyuan.lv@intel.com,
	intel-gvt-dev@lists.freedesktop.org
Subject: Re: [PATCH v17 5/6] vfio: ABI for mdev display dma-buf operation
Date: Thu, 9 Nov 2017 13:54:57 -0700	[thread overview]
Message-ID: <20171109135457.4f7514d3@t450s.home> (raw)
In-Reply-To: <20171109183514.w3tbqumovglr4sri@sirius.home.kraxel.org>

On Thu, 9 Nov 2017 19:35:14 +0100
Gerd Hoffmann <kraxel@redhat.com> wrote:

>   Hi,
> 
> > struct vfio_device_gfx_plane_info lacks the head field we've been
> > discussing.  Thanks,  
> 
> Adding multihead support turned out to not be that easy.  There are
> corner cases like a single framebuffer spawning both heads.  Also it
> would be useful to somehow hint to the guest which heads it should use.
> 
> In short:  Proper multihead support is more complex than just adding a
> head field for later use.  So in a short private discussion with Tina we
> came to the conclusion that it will be better add multihead support to
> the API when the first driver wants use it, so we can actually test the
> interface and make sure we didn't miss anything.  Adding a incomplete
> multihead API now doesn't help anybody.

Do you think we can enable multi-head and preserve backwards
compatibility within this API proposed here?  Thanks,

Alex
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

  reply	other threads:[~2017-11-09 20:55 UTC|newest]

Thread overview: 31+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-11-09  9:33 [PATCH v17 0/6] drm/i915/gvt: Dma-buf support for GVT-g Tina Zhang
2017-11-09  9:33 ` Tina Zhang
2017-11-09  9:33 ` [PATCH v17 1/6] drm/i915/gvt: Add framebuffer decoder support Tina Zhang
2017-11-09  9:33   ` Tina Zhang
2017-11-09  9:33 ` [PATCH v17 2/6] drm: Introduce RGB 64-bit 16:16:16:16 float format Tina Zhang
2017-11-09  9:33   ` Tina Zhang
2017-11-09  9:33 ` [PATCH v17 3/6] drm/i915/gvt: Add " Tina Zhang
2017-11-09  9:33   ` Tina Zhang
2017-11-09  9:34 ` [PATCH v17 4/6] drm/i915/gvt: Add opregion support Tina Zhang
2017-11-09  9:34   ` Tina Zhang
2017-11-09  9:34 ` [PATCH v17 5/6] vfio: ABI for mdev display dma-buf operation Tina Zhang
2017-11-09  9:34   ` Tina Zhang
2017-11-09 15:29   ` Alex Williamson
2017-11-09 15:29     ` Alex Williamson
2017-11-09 18:35     ` Gerd Hoffmann
2017-11-09 18:35       ` Gerd Hoffmann
2017-11-09 20:54       ` Alex Williamson [this message]
2017-11-09 20:54         ` Alex Williamson
2017-11-10  7:02         ` Gerd Hoffmann
2017-11-10  7:02           ` Gerd Hoffmann
2017-11-11  2:04           ` Zhang, Tina
2017-11-11  2:04             ` Zhang, Tina
2017-11-09  9:34 ` [PATCH v17 6/6] drm/i915/gvt: Dmabuf support for GVT-g Tina Zhang
2017-11-09  9:34   ` Tina Zhang
2017-11-09 10:15 ` ✗ Fi.CI.BAT: failure for drm/i915/gvt: Dma-buf " Patchwork
2017-11-09 14:09 ` [PATCH v17 0/6] " Gerd Hoffmann
2017-11-09 14:09   ` Gerd Hoffmann
2017-11-09 23:19   ` Zhang, Tina
2017-11-09 23:19     ` Zhang, Tina
2017-11-09 23:27   ` Zhang, Tina
2017-11-09 23:27     ` Zhang, Tina

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=20171109135457.4f7514d3@t450s.home \
    --to=alex.williamson@redhat.com \
    --cc=chris@chris-wilson.co.uk \
    --cc=daniel.vetter@ffwll.ch \
    --cc=daniel@ffwll.ch \
    --cc=hang.yuan@intel.com \
    --cc=intel-gfx@lists.freedesktop.org \
    --cc=intel-gvt-dev@lists.freedesktop.org \
    --cc=joonas.lahtinen@linux.intel.com \
    --cc=kevin.tian@intel.com \
    --cc=kraxel@redhat.com \
    --cc=kwankhede@nvidia.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=tina.zhang@intel.com \
    --cc=zhenyuw@linux.intel.com \
    --cc=zhi.a.wang@intel.com \
    --cc=zhiyuan.lv@intel.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.