Re: [igt-dev] [PATCH v3 1/3] lib/rendercopy: Add AUX page table support

From: Imre Deak <imre.deak@intel.com>
To: Chris Wilson <chris@chris-wilson.co.uk>
Cc: igt-dev@lists.freedesktop.org, Brian Welty <brian.welty@intel.com>
Subject: Re: [igt-dev] [PATCH v3 1/3] lib/rendercopy: Add AUX page table support
Date: Thu, 7 Nov 2019 20:37:41 +0200	[thread overview]
Message-ID: <20191107183741.GA14350@ideak-desk.fi.intel.com> (raw)
In-Reply-To: <157313049529.1799.6842248427029326412@skylake-alporthouse-com>

On Thu, Nov 07, 2019 at 12:41:35PM +0000, Chris Wilson wrote:
> Quoting Chris Wilson (2019-11-06 21:25:03)
> > Quoting Imre Deak (2019-11-06 19:04:10)
> > > An API to create the pagetable at a higher level would be more efficient
> > > in any case. Are you ok implementing that as a follow-up and starting
> > > with the current approach?
> > 
> > I'm ok with accepting this as proof we can do end-to-end compression
> > onto scanout. But there's plenty of work if we want to try and break the
> > kernel :-p
> 
> Suffice to say, fix up the reloc delta, and have a 
> Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
> for the series. I did not check the shifts against spec, but the
> structure looks sane.

Thanks, I also tested now more the relocations.

> -Chris
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev