qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Alex Williamson <alex.williamson@redhat.com>
To: Gerd Hoffmann <kraxel@redhat.com>
Cc: qemu-devel@nongnu.org, Tina Zhang <tina.zhang@intel.com>,
	intel-gvt-dev@lists.freedesktop.org,
	Kirti Wankhede <kwankhede@nvidia.com>
Subject: Re: [Qemu-devel] [RfC PATCH v2 0/5] vfio: add display support
Date: Wed, 31 Jan 2018 13:20:30 -0700	[thread overview]
Message-ID: <20180131132030.4bd5344f@w520.home> (raw)
In-Reply-To: <20180131121217.13557-1-kraxel@redhat.com>

On Wed, 31 Jan 2018 13:12:12 +0100
Gerd Hoffmann <kraxel@redhat.com> wrote:

> This series adds support for a vgpu display to the qemu vfio code.
> For now only regions are supported, dmabufs will follow later.
> 
> The vfio API update is done, queued in drm-next, should land in the
> upstream kernel during the 4.16 merge window.  So the 4.16-rc1 kernel
> header sync should bring the header changes needed for this series.
> 
> Patch #1 of this series has the vfio.h updates too, for testing
> convinience, but I don't plan to include that patch in the final
> patch submission.
> 
> plese test and review,

Hi Gerd,

The vfio bits look reasonable to me, it'd be nice if we could operate
on the VFIODevice rather than VFIOPCIDevice to make this universal for
all vfio devices, but that's just a nit since non-PCI graphics devices
may never come to fruition.  As for testing, region support was
included for NVIDIA, dmabuf for Intel... is this testing request mainly
for the Kirti and others at NVIDIA?  Thanks,

Alex

  parent reply	other threads:[~2018-01-31 20:20 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-01-31 12:12 [Qemu-devel] [RfC PATCH v2 0/5] vfio: add display support Gerd Hoffmann
2018-01-31 12:12 ` [Qemu-devel] [RfC PATCH v2 1/5] headers: update linux-headers/linux/vfio.h (intel-gvt kernel patches, v17) Gerd Hoffmann
2018-01-31 12:12 ` [Qemu-devel] [RfC PATCH v2 2/5] headers: add drm/drm_fourcc.h to standard-headers Gerd Hoffmann
2018-01-31 12:12 ` [Qemu-devel] [RfC PATCH v2 3/5] ui/pixman: add qemu_drm_format_to_pixman() Gerd Hoffmann
2018-01-31 12:12 ` [Qemu-devel] [RfC PATCH v2 4/5] vfio/display: core & wireup Gerd Hoffmann
2018-01-31 23:42   ` Zhang, Tina
2018-02-01  0:01     ` Alex Williamson
2018-01-31 12:12 ` [Qemu-devel] [RfC PATCH v2 5/5] vfio/display: adding region support Gerd Hoffmann
2018-01-31 20:20 ` Alex Williamson [this message]
2018-02-01  8:24   ` [Qemu-devel] [RfC PATCH v2 0/5] vfio: add display support Gerd Hoffmann
2018-02-03 19:10     ` Kirti Wankhede

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=20180131132030.4bd5344f@w520.home \
    --to=alex.williamson@redhat.com \
    --cc=intel-gvt-dev@lists.freedesktop.org \
    --cc=kraxel@redhat.com \
    --cc=kwankhede@nvidia.com \
    --cc=qemu-devel@nongnu.org \
    --cc=tina.zhang@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 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).