Re: [Intel-gfx] [PATCH v6 00/19] 48-bit PPGTT

2015-08-03 Thread Michel Thierry
On 7/29/2015 5:23 PM, Michel Thierry wrote: Michel Thierry (19): drm/i915: Remove unnecessary gen8_clamp_pd drm/i915/gen8: Make pdp allocation more dynamic drm/i915/gen8: Abstract PDP usage drm/i915/gen8: Generalize PTE writing for GEN8 PPGTT drm/i915/gen8: Add dynamic page trace

Re: [Intel-gfx] [PATCH v6 00/19] 48-bit PPGTT

2015-07-30 Thread Michel Thierry
On 7/30/2015 12:26 PM, Chris Wilson wrote: On Wed, Jul 29, 2015 at 05:23:44PM +0100, Michel Thierry wrote: This clean-up version delays the 48-bit work to later patches and includes more review comments from Akash and Chris. The first 5 patches prepare the dynamic page allocation code to handle

Re: [Intel-gfx] [PATCH v6 00/19] 48-bit PPGTT

2015-07-30 Thread Chris Wilson
On Thu, Jul 30, 2015 at 12:52:19PM +0100, Michel Thierry wrote: On 7/30/2015 12:26 PM, Chris Wilson wrote: Just a head's up, I haven't root caused this yet, but with i915.enable_ppgtt=2 I started getting GPU hangs that didn't happen before this series... Sounds like I screwed up something

Re: [Intel-gfx] [PATCH v6 00/19] 48-bit PPGTT

2015-07-30 Thread Chris Wilson
On Thu, Jul 30, 2015 at 12:52:19PM +0100, Michel Thierry wrote: Sounds like I screwed up something in the first 4 patches or in the Wa32bit one. The rest of the changes are contained to 48-bit code. Have you find a way to reproduce it? Seems like no. Whatever happened this morning, it hasn't

Re: [Intel-gfx] [PATCH v6 00/19] 48-bit PPGTT

2015-07-30 Thread Chris Wilson
On Wed, Jul 29, 2015 at 05:23:44PM +0100, Michel Thierry wrote: This clean-up version delays the 48-bit work to later patches and includes more review comments from Akash and Chris. The first 5 patches prepare the dynamic page allocation code to handle independent pdps, but no specific code

[Intel-gfx] [PATCH v6 00/19] 48-bit PPGTT

2015-07-29 Thread Michel Thierry
This clean-up version delays the 48-bit work to later patches and includes more review comments from Akash and Chris. The first 5 patches prepare the dynamic page allocation code to handle independent pdps, but no specific code for 48-bit mode is added before the 5th patch. In order expand the