public inbox for intel-gfx@lists.freedesktop.org
 help / color / mirror / Atom feed
From: Jike Song <jike.song@intel.com>
To: Daniel Vetter <daniel@ffwll.ch>
Cc: daniel.vetter@ffwll.ch, intel-gfx@lists.freedesktop.org
Subject: Re: [RFC PATCH 0/8] Add host i915 support for vGPU
Date: Thu, 23 Oct 2014 11:13:51 +0800	[thread overview]
Message-ID: <5448726F.60401@intel.com> (raw)
In-Reply-To: <20141022094832.GS26941@phenom.ffwll.local>

On 10/22/2014 05:48 PM, Daniel Vetter wrote:
> So on a very high level I don't understand this design. For the guest side
> it's completely clear that we need a bunch of hooks over the driver to
> make paravirtualization work.
>
> But on the host side I expect the driver to be in full control of the
> hardware. I haven't really seen the other side but it looks like with vgt
> we actually have two drivers fighting over the hardware, which requires
> the various hooks to avoid disaster. The drm subsystem is littered with
> such attempts, and they all didn't end up in a pretty way.
>
> So way can't we have the vgt support for guests sit on top of i915, using
> the i915 functions to set up pagetables, contexts and reserve gtt areas
> for the guests? Then we'd have just one driver in control of the hardware,
> and vgt on the host side would just look like a really crazy interace
> layer between virtual hosts and the low-level driver, similar to who the
> execbuf ioctl is a really crazy interface between userspace and the
> low-level driver.

  Yes we can do this, but that also means lots of pollution to existing i915
codes, only for virtualization purpose. Currently vgt has pretty abstractions
for both host i915 and guest graphics drivers, mixing vgt and host i915 means
breaking that.  I believe a vgt-integrated repository will be better to look
at :)

>
> Cheers, Daniel
>

--
Thanks,
Jike

  reply	other threads:[~2014-10-23  3:16 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-09-30 10:05 [RFC PATCH 0/8] Add host i915 support for vGPU Jike Song
2014-09-30 10:05 ` [RFC PATCH 1/8] drm/i915: introduce a new modparam: enable_vgt Jike Song
2014-09-30 10:05 ` [RFC PATCH 2/8] drm/i915: introduce the skeleton of vgt Jike Song
2014-09-30 10:05 ` [RFC PATCH 3/8] drm/i915: add the vgt implementation of MMIO/GTT mediations Jike Song
2014-09-30 16:34   ` Tian, Kevin
2014-10-01 10:58     ` Jike Song
2014-09-30 10:05 ` [RFC PATCH 4/8] drm/i915: redirect MMIO accesses to vgt if enabled Jike Song
2014-09-30 10:05 ` [RFC PATCH 5/8] drm/i915: GTT access abstraction Jike Song
2014-09-30 10:05 ` [RFC PATCH 6/8] drm/i915: redirect GTT accesses to vgt if enabled Jike Song
2014-09-30 10:05 ` [RFC PATCH 7/8] drm/i915: vgt irq mediation - via a tasklet based mechanism Jike Song
2014-09-30 10:30   ` Daniel Vetter
2014-09-30 16:26     ` Tian, Kevin
2014-10-22  7:34       ` Jike Song
2014-10-28  6:59         ` Tian, Kevin
2014-09-30 10:05 ` [RFC PATCH 8/8] drm/i915: enable vgt if specified by module param Jike Song
2014-10-22  9:48 ` [RFC PATCH 0/8] Add host i915 support for vGPU Daniel Vetter
2014-10-23  3:13   ` Jike Song [this message]
2014-10-23  8:56     ` Daniel Vetter
2014-10-23 11:01       ` Gerd Hoffmann
2014-10-23 12:10         ` Daniel Vetter
2014-10-23 12:32           ` Gerd Hoffmann
2014-10-28  8:45           ` Tian, Kevin
2014-10-28  8:25       ` Tian, Kevin

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=5448726F.60401@intel.com \
    --to=jike.song@intel.com \
    --cc=daniel.vetter@ffwll.ch \
    --cc=daniel@ffwll.ch \
    --cc=intel-gfx@lists.freedesktop.org \
    /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