Re: [Intel-gfx] [PATCH] drm/i915/guc: Reserve the upper end of the Global GTT for the GuC

2016-12-22 Thread Srivatsa, Anusha
>-Original Message- >From: Intel-gfx [mailto:intel-gfx-boun...@lists.freedesktop.org] On Behalf Of >Daniele Ceraolo Spurio >Sent: Thursday, December 22, 2016 10:45 AM >To: intel-gfx@lists.freedesktop.org >Subject: Re: [Intel-gfx] [PATCH] drm/i915/guc: Reserve the upper

Re: [Intel-gfx] [PATCH] drm/i915/guc: Reserve the upper end of the Global GTT for the GuC

2016-12-22 Thread Daniele Ceraolo Spurio
On 22/12/16 08:38, Arkadiusz Hiler wrote: On Thu, Dec 22, 2016 at 03:18:08PM +, Chris Wilson wrote: On Thu, Dec 22, 2016 at 03:53:15PM +0100, Arkadiusz Hiler wrote: On Wed, Dec 21, 2016 at 07:35:04PM +0100, Srivatsa, Anusha wrote: With enable_guc_loading=2 and enable_guc_submission=0 I ge

Re: [Intel-gfx] [PATCH] drm/i915/guc: Reserve the upper end of the Global GTT for the GuC

2016-12-22 Thread Arkadiusz Hiler
On Thu, Dec 22, 2016 at 03:18:08PM +, Chris Wilson wrote: > On Thu, Dec 22, 2016 at 03:53:15PM +0100, Arkadiusz Hiler wrote: > > On Wed, Dec 21, 2016 at 07:35:04PM +0100, Srivatsa, Anusha wrote: > > > With enable_guc_loading=2 and enable_guc_submission=0 I get HuC > > > authentication failure a

Re: [Intel-gfx] [PATCH] drm/i915/guc: Reserve the upper end of the Global GTT for the GuC

2016-12-22 Thread Chris Wilson
On Thu, Dec 22, 2016 at 03:53:15PM +0100, Arkadiusz Hiler wrote: > On Wed, Dec 21, 2016 at 07:35:04PM +0100, Srivatsa, Anusha wrote: > > With enable_guc_loading=2 and enable_guc_submission=0 I get HuC > > authentication failure and with enable_guc_loading and > > enable_guc_submisssion both set to

Re: [Intel-gfx] [PATCH] drm/i915/guc: Reserve the upper end of the Global GTT for the GuC

2016-12-22 Thread Arkadiusz Hiler
on ; intel-gfx@lists.freedesktop.org; > >Hiler, Arkadiusz > >Subject: Re: [Intel-gfx] [PATCH] drm/i915/guc: Reserve the upper end of the > >Global GTT for the GuC > > > >On Wed, Dec 21, 2016 at 04:03:30PM +, Chris Wilson wrote: > >> On Wed, Dec 21, 2016 at 04:30:1

Re: [Intel-gfx] [PATCH] drm/i915/guc: Reserve the upper end of the Global GTT for the GuC

2016-12-22 Thread Kamble, Sagar A
On 12/22/2016 2:40 AM, Daniele Ceraolo Spurio wrote: On 21/12/16 06:11, Chris Wilson wrote: The GuC would like to own the upper portion of the GTT for itself, so exclude it from our drm_mm to prevent us using it. I had a quick chat with a GuC dev and he is pretty sure that GuC can't reser

Re: [Intel-gfx] [PATCH] drm/i915/guc: Reserve the upper end of the Global GTT for the GuC

2016-12-21 Thread Daniele Ceraolo Spurio
On 21/12/16 06:11, Chris Wilson wrote: The GuC would like to own the upper portion of the GTT for itself, so exclude it from our drm_mm to prevent us using it. I had a quick chat with a GuC dev and he is pretty sure that GuC can't reserve any GTT areas for itself (and that is why we do the a

Re: [Intel-gfx] [PATCH] drm/i915/guc: Reserve the upper end of the Global GTT for the GuC

2016-12-21 Thread Srivatsa, Anusha
op.org] On Behalf Of >Michal Wajdeczko >Sent: Wednesday, December 21, 2016 8:31 AM >To: Chris Wilson ; intel-gfx@lists.freedesktop.org; >Hiler, Arkadiusz >Subject: Re: [Intel-gfx] [PATCH] drm/i915/guc: Reserve the upper end of the >Global GTT for the GuC > >On Wed, Dec 21, 2

Re: [Intel-gfx] [PATCH] drm/i915/guc: Reserve the upper end of the Global GTT for the GuC

2016-12-21 Thread Michal Wajdeczko
On Wed, Dec 21, 2016 at 04:03:30PM +, Chris Wilson wrote: > On Wed, Dec 21, 2016 at 04:30:14PM +0100, Michal Wajdeczko wrote: > > On Wed, Dec 21, 2016 at 02:11:26PM +, Chris Wilson wrote: > > > The GuC would like to own the upper portion of the GTT for itself, so > > > exclude it from our d

Re: [Intel-gfx] [PATCH] drm/i915/guc: Reserve the upper end of the Global GTT for the GuC

2016-12-21 Thread Chris Wilson
On Wed, Dec 21, 2016 at 04:30:14PM +0100, Michal Wajdeczko wrote: > On Wed, Dec 21, 2016 at 02:11:26PM +, Chris Wilson wrote: > > The GuC would like to own the upper portion of the GTT for itself, so > > exclude it from our drm_mm to prevent us using it. > > > > Signed-off-by: Chris Wilson >

Re: [Intel-gfx] [PATCH] drm/i915/guc: Reserve the upper end of the Global GTT for the GuC

2016-12-21 Thread Michal Wajdeczko
On Wed, Dec 21, 2016 at 02:11:26PM +, Chris Wilson wrote: > The GuC would like to own the upper portion of the GTT for itself, so > exclude it from our drm_mm to prevent us using it. > > Signed-off-by: Chris Wilson > Cc: Michal Wajdeczko > Cc: Arkadiusz Hiler > --- > drivers/gpu/drm/i915/i

Re: [Intel-gfx] [PATCH] drm/i915/guc: Reserve the upper end of the Global GTT for the GuC

2016-12-21 Thread Arkadiusz Hiler
On Wed, Dec 21, 2016 at 02:11:26PM +, Chris Wilson wrote: > The GuC would like to own the upper portion of the GTT for itself, so > exclude it from our drm_mm to prevent us using it. > > Signed-off-by: Chris Wilson > Cc: Michal Wajdeczko > Cc: Arkadiusz Hiler Tested-by: Arkadiusz Hiler Fi

Re: [Intel-gfx] [PATCH] drm/i915/guc: Reserve the upper end of the Global GTT for the GuC

2016-12-21 Thread Michał Winiarski
On Wed, Dec 21, 2016 at 02:11:26PM +, Chris Wilson wrote: > The GuC would like to own the upper portion of the GTT for itself, so > exclude it from our drm_mm to prevent us using it. Fixes: 2947e40 ("drm/i915/execlists: Request the kernel context be pinned high") Reviewed-by: Michał Winiarski

[Intel-gfx] [PATCH] drm/i915/guc: Reserve the upper end of the Global GTT for the GuC

2016-12-21 Thread Chris Wilson
The GuC would like to own the upper portion of the GTT for itself, so exclude it from our drm_mm to prevent us using it. Signed-off-by: Chris Wilson Cc: Michal Wajdeczko Cc: Arkadiusz Hiler --- drivers/gpu/drm/i915/i915_gem_gtt.c | 5 + drivers/gpu/drm/i915/intel_guc_fwif.h | 3 +++ 2 fi