Re: [Intel-gfx] [PATCH 07/13 v4] drm/i915: GuC submission setup, phase 1

2015-07-28 Thread O'Rourke, Tom
On Tue, Jul 28, 2015 at 08:40:58PM +0100, Dave Gordon wrote: > On 28/07/15 16:16, Dave Gordon wrote: > >On 28/07/15 00:12, O'Rourke, Tom wrote: > >>On Mon, Jul 27, 2015 at 03:41:31PM -0700, Yu Dai wrote: > >>> > >>>On 07/24/2015 03:31 PM, O'Rourke, Tom wrote: > [TOR:] When I see "phase 1" I als

Re: [Intel-gfx] [PATCH 07/13 v4] drm/i915: GuC submission setup, phase 1

2015-07-28 Thread O'Rourke, Tom
On Tue, Jul 28, 2015 at 04:16:03PM +0100, Dave Gordon wrote: > On 28/07/15 00:12, O'Rourke, Tom wrote: > >On Mon, Jul 27, 2015 at 03:41:31PM -0700, Yu Dai wrote: > >> > >>On 07/24/2015 03:31 PM, O'Rourke, Tom wrote: > >>>[TOR:] When I see "phase 1" I also look for "phase 2". > >>>A subject that bet

Re: [Intel-gfx] [PATCH 07/13 v4] drm/i915: GuC submission setup, phase 1

2015-07-28 Thread Dave Gordon
On 28/07/15 16:16, Dave Gordon wrote: On 28/07/15 00:12, O'Rourke, Tom wrote: On Mon, Jul 27, 2015 at 03:41:31PM -0700, Yu Dai wrote: On 07/24/2015 03:31 PM, O'Rourke, Tom wrote: [TOR:] When I see "phase 1" I also look for "phase 2". A subject that better describes the change in this patch wo

Re: [Intel-gfx] [PATCH 07/13 v4] drm/i915: GuC submission setup, phase 1

2015-07-28 Thread Dave Gordon
On 28/07/15 00:12, O'Rourke, Tom wrote: On Mon, Jul 27, 2015 at 03:41:31PM -0700, Yu Dai wrote: On 07/24/2015 03:31 PM, O'Rourke, Tom wrote: [TOR:] When I see "phase 1" I also look for "phase 2". A subject that better describes the change in this patch would help. On Thu, Jul 09, 2015 at 07:2

Re: [Intel-gfx] [PATCH 07/13 v4] drm/i915: GuC submission setup, phase 1

2015-07-27 Thread Yu Dai
On 07/27/2015 04:12 PM, O'Rourke, Tom wrote: On Mon, Jul 27, 2015 at 03:41:31PM -0700, Yu Dai wrote: > > > On 07/24/2015 03:31 PM, O'Rourke, Tom wrote: > >[TOR:] When I see "phase 1" I also look for "phase 2". > >A subject that better describes the change in this patch > >would help. > > > >On

Re: [Intel-gfx] [PATCH 07/13 v4] drm/i915: GuC submission setup, phase 1

2015-07-27 Thread O'Rourke, Tom
On Mon, Jul 27, 2015 at 03:41:31PM -0700, Yu Dai wrote: > > > On 07/24/2015 03:31 PM, O'Rourke, Tom wrote: > >[TOR:] When I see "phase 1" I also look for "phase 2". > >A subject that better describes the change in this patch > >would help. > > > >On Thu, Jul 09, 2015 at 07:29:08PM +0100, Dave Gor

Re: [Intel-gfx] [PATCH 07/13 v4] drm/i915: GuC submission setup, phase 1

2015-07-27 Thread Yu Dai
On 07/24/2015 03:31 PM, O'Rourke, Tom wrote: [TOR:] When I see "phase 1" I also look for "phase 2". A subject that better describes the change in this patch would help. On Thu, Jul 09, 2015 at 07:29:08PM +0100, Dave Gordon wrote: > From: Alex Dai > > This adds the first of the data structures

Re: [Intel-gfx] [PATCH 07/13 v4] drm/i915: GuC submission setup, phase 1

2015-07-24 Thread O'Rourke, Tom
[TOR:] When I see "phase 1" I also look for "phase 2". A subject that better describes the change in this patch would help. On Thu, Jul 09, 2015 at 07:29:08PM +0100, Dave Gordon wrote: > From: Alex Dai > > This adds the first of the data structures used to communicate with the > GuC (the pool

[Intel-gfx] [PATCH 07/13 v4] drm/i915: GuC submission setup, phase 1

2015-07-09 Thread Dave Gordon
From: Alex Dai This adds the first of the data structures used to communicate with the GuC (the pool of guc_context structures). We create a GuC-specific wrapper round the GEM object allocator as all GEM objects shared with the GuC must be pinned into GGTT space at an address that is NOT in the