Re: [Intel-gfx] [PATCH v5 06/35] drm/i915: Start of GPU scheduler

2016-02-26 Thread John Harrison
On 26/02/2016 09:13, Joonas Lahtinen wrote: Hi, The below answers are reasonable. So v6 should be the version. Are you planning on looking at the other patches first or are you going to wait until v6 is posted? Thanks, John. ___ Intel-gfx mailing

Re: [Intel-gfx] [PATCH v5 06/35] drm/i915: Start of GPU scheduler

2016-02-26 Thread Joonas Lahtinen
Hi, The below answers are reasonable. So v6 should be the version. Regards, Joonas On pe, 2016-02-19 at 17:03 +, John Harrison wrote: > On 19/02/2016 13:03, Joonas Lahtinen wrote: > > > > Hi, > > > > Now the code is in reviewable chunks, excellent! > > > > I've added my comments below. A

Re: [Intel-gfx] [PATCH v5 06/35] drm/i915: Start of GPU scheduler

2016-02-19 Thread John Harrison
On 19/02/2016 13:03, Joonas Lahtinen wrote: Hi, Now the code is in reviewable chunks, excellent! I've added my comments below. A few repeats from last round, but now with more questions about the logic itself. On to, 2016-02-18 at 14:26 +, john.c.harri...@intel.com wrote: From: John Harri

Re: [Intel-gfx] [PATCH v5 06/35] drm/i915: Start of GPU scheduler

2016-02-19 Thread Joonas Lahtinen
Hi, Now the code is in reviewable chunks, excellent! I've added my comments below. A few repeats from last round, but now with more questions about the logic itself. On to, 2016-02-18 at 14:26 +, john.c.harri...@intel.com wrote: > From: John Harrison > > Initial creation of scheduler sourc

[Intel-gfx] [PATCH v5 06/35] drm/i915: Start of GPU scheduler

2016-02-18 Thread John . C . Harrison
From: John Harrison Initial creation of scheduler source files. Note that this patch implements most of the scheduler functionality but does not hook it in to the driver yet. It also leaves the scheduler code in 'pass through' mode so that even when it is hooked in, it will not actually do very m