Re: [PATCH v10 00/10] Introduce the implementation of GVT context

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Also CC Daniel. I have got "Reviewed-by" and passed the ABAT tests. Is there anything I should do before patches get picked?

Thanks,
Zhi.

> -----Original Message-----
> From: Wang, Zhi A
> Sent: Monday, June 13, 2016 4:17 PM
> To: intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> Cc: chris@xxxxxxxxxxxxxxxxxx; Lv, Zhiyuan <zhiyuan.lv@xxxxxxxxx>; Tian, Kevin
> <kevin.tian@xxxxxxxxx>; tvrtko.ursulin@xxxxxxxxxxxxxxx;
> joonas.lahtinen@xxxxxxxxxxxxxxx
> Subject: RE: [PATCH v10 00/10] Introduce the implementation of GVT context
> 
> Hi Chris:
>     As Joonas is on vacation, can you pick my patches? or if you have some
> comments I can continue improving them.
> 
> Thanks,
> Zhi.
> 
> > -----Original Message-----
> > From: Wang, Zhi A
> > Sent: Thursday, June 09, 2016 9:44 PM
> > To: intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> > Cc: chris@xxxxxxxxxxxxxxxxxx; Lv, Zhiyuan <zhiyuan.lv@xxxxxxxxx>;
> > Tian, Kevin <kevin.tian@xxxxxxxxx>; tvrtko.ursulin@xxxxxxxxxxxxxxx;
> > joonas.lahtinen@xxxxxxxxxxxxxxx; Wang, Zhi A <zhi.a.wang@xxxxxxxxx>
> > Subject: [PATCH v10 00/10] Introduce the implementation of GVT context
> >
> > This patchset introduces the implementation of GVT context. GVT
> > context is a special GEM context used by GVT-g. GVT-g uses it as the
> > shadow context.It doesn't have a drm client nor a PPGTT. And it
> > requires a larger ring buffer with several special features need by
> > GVT-g workload scheduler like context status change notification, context
> single submission...
> >
> > ABAT results and link:
> >
> > Series: Introduce the implementation of GVT context
> > URL   : https://patchwork.freedesktop.org/series/8470/
> > State : success
> >
> > v10:
> >
> > - Take Joonas' comments.
> >
> > v9:
> >
> > - Take Chris' comments.
> >
> > v8:
> >
> > - Take Joonas/Chris's comments.
> >
> > v7:
> >
> > - Take Joonas comments.
> >
> > v6:
> >
> > - Take Chris comments.
> >
> > v5:
> >
> > - Drop PPGTT related patches.
> > - Let most functions take struct drm_i915_private *
> > - Fixed some misspelled words in Kconfig
> > - Only complied some feature when CONFIG_DRM_I915_GVT=y
> > - Drop the fecne related changes, will send it after this series.
> >
> > v4:
> >
> > - Based on the latest drm-intel-nightly branch.
> > - Drop PPGTT refactor patches. (GVT-g will use LRI to load PDPs)
> > - Drop i915_gem_context() refactor patches, reuse kernel context functions.
> >   (Dave Gordon)
> > - Drop context allocation params and refactor as the lrc deferred
> >   allocation function has been refactored in another styles.
> > - Re-wrtie GVT context creation function
> >
> > Difference from community release
> > ---------------------------------
> >
> > This patchset is different from regular iGVT-g code release[4], which
> > is still based on old host-mediated architecture. Furthermore, this
> > patchset only supports BDW whereas code release supports HSW/BDW/SKL.
> > We will add SKL support later based on this RFC code and HSW support
> > will be dropped.
> >
> > Internally we tested this RFC patchset with both linux and windows VM
> > and the architecture changes work fine.
> >
> > Acknowledgment
> > ---------------
> >
> > iGVT-g implementation is several years effort and many people
> > contributed to the code. There names are not here yet. In later formal
> > patchset we will reflect individual's contribution.
> >
> > Meanwhile, in the previous iGVT-g related discussion, Daniel, Chris
> > and Joonas ever gave very good inputs. We appreciate them and look
> > forward to more comments/suggestions from community.
> >
> > We are trying to get more familiar with i915 but may still have gaps.
> > We are willing to adopt suggestions to keep improving. We hope to work
> > with community together to make iGVT-g a great component in i915 to
> > support graphics virtualization. Thanks!
> >
> > Reference
> > ---------
> >
> > [1] https://01.org/igvt-g
> > [2]
> > http://lists.freedesktop.org/archives/intel-gfx/2014-September/053098.
> > html
> > [3]
> > http://lists.freedesktop.org/archives/intel-gfx/2015-September/075397.
> > html
> >
> >
> > Bing Niu (1):
> >   drm/i915: Introduce host graphics memory partition for GVT-g
> >
> > Zhi Wang (9):
> >   drm/i915: Factor out i915_pvinfo.h
> >   drm/i915: Use offsetof() to calculate the offset of members in PVINFO
> >     page
> >   drm/i915: Fold vGPU active check into inner functions
> >   drm/i915: gvt: Introduce the basic architecture of GVT-g
> >   drm/i915: Make ring buffer size of a LRC context configurable
> >   drm/i915: Make addressing mode bits in context descriptor configurable
> >   drm/i915: Introduce execlist context status change notification
> >   drm/i915: Support LRC context single submission
> >   drm/i915: Introduce GVT context creation API
> >
> >  drivers/gpu/drm/i915/Kconfig            |  22 +++++
> >  drivers/gpu/drm/i915/Makefile           |   5 ++
> >  drivers/gpu/drm/i915/gvt/Makefile       |   5 ++
> >  drivers/gpu/drm/i915/gvt/debug.h        |  34 ++++++++
> >  drivers/gpu/drm/i915/gvt/gvt.c          | 145
> > ++++++++++++++++++++++++++++++++
> >  drivers/gpu/drm/i915/gvt/gvt.h          |  69 +++++++++++++++
> >  drivers/gpu/drm/i915/gvt/hypercall.h    |  38 +++++++++
> >  drivers/gpu/drm/i915/gvt/mpt.h          |  49 +++++++++++
> >  drivers/gpu/drm/i915/i915_dma.c         |  16 +++-
> >  drivers/gpu/drm/i915/i915_drv.h         |  16 ++++
> >  drivers/gpu/drm/i915/i915_gem_context.c |  38 +++++++++
> >  drivers/gpu/drm/i915/i915_gem_gtt.c     |  11 +--
> >  drivers/gpu/drm/i915/i915_params.c      |   5 ++
> >  drivers/gpu/drm/i915/i915_params.h      |   1 +
> >  drivers/gpu/drm/i915/i915_pvinfo.h      | 113
> > +++++++++++++++++++++++++
> >  drivers/gpu/drm/i915/i915_reg.h         |  12 +++
> >  drivers/gpu/drm/i915/i915_vgpu.c        |  32 +++++--
> >  drivers/gpu/drm/i915/i915_vgpu.h        |  90 +-------------------
> >  drivers/gpu/drm/i915/intel_gvt.c        | 100
> ++++++++++++++++++++++
> >  drivers/gpu/drm/i915/intel_gvt.h        |  81 ++++++++++++++++++
> >  drivers/gpu/drm/i915/intel_lrc.c        |  54 +++++++++---
> >  drivers/gpu/drm/i915/intel_lrc.h        |   5 ++
> >  22 files changed, 821 insertions(+), 120 deletions(-)  create mode
> > 100644 drivers/gpu/drm/i915/gvt/Makefile  create mode 100644
> > drivers/gpu/drm/i915/gvt/debug.h  create mode 100644
> > drivers/gpu/drm/i915/gvt/gvt.c  create mode 100644
> > drivers/gpu/drm/i915/gvt/gvt.h  create mode 100644
> > drivers/gpu/drm/i915/gvt/hypercall.h
> >  create mode 100644 drivers/gpu/drm/i915/gvt/mpt.h  create mode
> 100644
> > drivers/gpu/drm/i915/i915_pvinfo.h
> >  create mode 100644 drivers/gpu/drm/i915/intel_gvt.c  create mode
> > 100644 drivers/gpu/drm/i915/intel_gvt.h
> >
> > --
> > 1.9.1

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




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux