[PATCH 0/9] drm/i915/gvt: Refine the gtt shadowing
Du, Changbin
changbin.du at intel.com
Tue Dec 26 03:23:48 UTC 2017
On Tue, Dec 26, 2017 at 11:10:12AM +0800, Zhenyu Wang wrote:
> On 2017.12.25 21:17:26 +0800, Zhi Wang wrote:
> > Well, I think that's all my comments for this series. Since Zhenyu invites
> > me to review this patches, Zhenyu you can also give rb/ack based on your
> > opinion.
> >
>
> Several ones still seems not easy to review, e.g 1/9, even it's generally
> ok, but still can be refined to help reviewer, e.g change one part of struct
> at each time and clearly split those rename part. For those big change, we
> definitely like to see step by step change, although which means more effort
> on developer side.
>
Myabe an alternate solution is that we can review these two big one in a meeting.
Split it takes me several times efforts and it's really to rewrite them again
and again.
> And for other internal data structure change, need at least some macro benchmark
> comparison or even micro bench to understand its impact.
>
yeah, the data will be in next revise.
> >
> > On 12/25/17 17:11, changbin.du at intel.com wrote:
> > > From: Changbin Du <changbin.du at intel.com>
> > >
> > > This is the first part of patch set "drm/i915/gvt: Add support for huge gtt (2M/64K)".
> > > The GTT related code are refined. I just need a clean code base to add new
> > > feature.
> > >
> > > patch #1 and #7 are fat patch, please take care.
> > >
> > > Changbin Du (9):
> > > drm/i915/gvt: Rework shadow graphic memory management code
> > > drm/i915/gvt: Add verbose gtt shadow logs
> > > drm/i915/gvt: Rename ggtt related functions to be more specific
> > > drm/i915/gvt: Factor out
> > > intel_vgpu_{get_or_create_ppgtt_mm,find_destroy_ppgtt_mm} interfaces
> > > drm/i915/gvt: Use standard pte bit definition
> > > drm/i915/gvt: Refine pte shadowing process
> > > drm/i915/gvt: Rework shadow page management code
> > > drm/i915/gvt: Manage shadow pages with radix tree
> > > drm/i915/gvt: Define PTE addr mask with GENMASK_ULL
> > >
> > > drivers/gpu/drm/i915/gvt/Makefile | 2 +-
> > > drivers/gpu/drm/i915/gvt/gtt.c | 1427 +++++++++++++++------------------
> > > drivers/gpu/drm/i915/gvt/gtt.h | 182 ++---
> > > drivers/gpu/drm/i915/gvt/gvt.c | 2 +-
> > > drivers/gpu/drm/i915/gvt/gvt.h | 2 +
> > > drivers/gpu/drm/i915/gvt/handlers.c | 18 +-
> > > drivers/gpu/drm/i915/gvt/mmio.c | 9 +-
> > > drivers/gpu/drm/i915/gvt/mpt.h | 36 +-
> > > drivers/gpu/drm/i915/gvt/page_track.c | 181 +++++
> > > drivers/gpu/drm/i915/gvt/page_track.h | 54 ++
> > > drivers/gpu/drm/i915/gvt/scheduler.c | 48 +-
> > > drivers/gpu/drm/i915/gvt/trace.h | 38 +-
> > > drivers/gpu/drm/i915/gvt/vgpu.c | 1 +
> > > 13 files changed, 1002 insertions(+), 998 deletions(-)
> > > create mode 100644 drivers/gpu/drm/i915/gvt/page_track.c
> > > create mode 100644 drivers/gpu/drm/i915/gvt/page_track.h
> > >
> >
> > _______________________________________________
> > intel-gvt-dev mailing list
> > intel-gvt-dev at lists.freedesktop.org
> > https://lists.freedesktop.org/mailman/listinfo/intel-gvt-dev
>
> --
> Open Source Technology Center, Intel ltd.
>
> $gpg --keyserver wwwkeys.pgp.net --recv-keys 4D781827
--
Thanks,
Changbin Du
More information about the intel-gvt-dev
mailing list