Re: [PATCH 1/2] Revert "drm/i915: Introduce private PAT management"

Wang, Zhi A
2019-07-12T08:45:22Z

It's quite a long time ago. I cannot fully remember the whole picture now. I remembered the last status was to push some unit tests and with the last part of patch. Currently I think you can revert this if it causes troubles, we can purpose this again if we need this in future.

Thanks,
Zhi.

-----Original Message-----
From: Zhenyu Wang [mailto:zhenyuw@linux.intel.com]
Sent: Friday, July 12, 2019 11:29 AM
To: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Cc: Winiarski, Michal <michal.winiarski@intel.com>; Chris Wilson <chris@chris-wilson.co.uk>; intel-gfx@lists.freedesktop.org; Zhenyu Wang <zhenyuw@linux.intel.com>; Vivi, Rodrigo <rodrigo.vivi@intel.com>; Wang, Zhi A <zhi.a.wang@intel.com>
Subject: Re: [PATCH 1/2] Revert "drm/i915: Introduce private PAT management"

On 2019.07.12 11:06:01 +0300, Joonas Lahtinen wrote:
> + Zhenyu as FYI
>
> Quoting Chris Wilson (2019-07-02 14:56:04)
> > Quoting Michał Winiarski (2019-07-02 12:31:48)
> > > This reverts commit 4395890a48551982549d222d1923e2833dac47cf.
> > >
> > > It's been over a year since this was merged, and the actual users
> > > of intel_ppat_get / intel_ppat_put never materialized.
> > >
> > > Time to remove it!
> > >
> > > v2: Unbreak suspend (Chris)
> > > v3: Rebase, drop fixes tag to avoid confusion
> > >
> > > Signed-off-by: Michał Winiarski <michal.winiarski@intel.com>
> > > Cc: Chris Wilson <chris@chris-wilson.co.uk>
> > > Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
> > > Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
> > > Cc: Zhi Wang <zhi.a.wang@intel.com>
> > Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
> >
> > While I would appreciate an ack from Zhi (being the original
> > author), since we've already sent this a few times, there's no
> > reason to delay waiting for a response.

Well I also depend on Zhi for comment here, as previous idea was to manage PPAT allocation between VM/host, but I'm not sure what's left work status there..

--
Open Source Technology Center, Intel ltd.

$gpg --keyserver wwwkeys.pgp.net --recv-keys 4D781827