Re: [Intel-gfx] [PATCH 2/2] drm/i915: Reduce context HW ID lifetime

2018-02-22 Thread Lionel Landwerlin
On 22/02/18 09:17, Chris Wilson wrote: Quoting Lionel Landwerlin (2018-02-21 01:12:46) On 09/02/18 20:47, Chris Wilson wrote: Quoting Daniele Ceraolo Spurio (2018-02-09 18:56:36) On 09/02/18 02:22, Chris Wilson wrote: Future gen reduce the number of bits we will have available to differentiat

Re: [Intel-gfx] [PATCH 2/2] drm/i915: Reduce context HW ID lifetime

2018-02-22 Thread Chris Wilson
Quoting Lionel Landwerlin (2018-02-21 01:12:46) > On 09/02/18 20:47, Chris Wilson wrote: > > Quoting Daniele Ceraolo Spurio (2018-02-09 18:56:36) > >> > >> On 09/02/18 02:22, Chris Wilson wrote: > >>> Future gen reduce the number of bits we will have available to > >>> differentiate between context

Re: [Intel-gfx] [PATCH 2/2] drm/i915: Reduce context HW ID lifetime

2018-02-20 Thread Lionel Landwerlin
On 09/02/18 20:47, Chris Wilson wrote: Quoting Daniele Ceraolo Spurio (2018-02-09 18:56:36) On 09/02/18 02:22, Chris Wilson wrote: Future gen reduce the number of bits we will have available to differentiate between contexts, so reduce the lifetime of the ID assignment from that of the context

Re: [Intel-gfx] [PATCH 2/2] drm/i915: Reduce context HW ID lifetime

2018-02-12 Thread Lionel Landwerlin
On 12/02/18 11:08, Chris Wilson wrote: Quoting Lionel Landwerlin (2018-02-12 11:00:10) On 12/02/18 10:41, Chris Wilson wrote: Quoting Lionel Landwerlin (2018-02-12 10:37:52) On 09/02/18 20:53, Chris Wilson wrote: Quoting Lionel Landwerlin (2018-02-09 17:47:44) Hey Chris, From the i915/p

Re: [Intel-gfx] [PATCH 2/2] drm/i915: Reduce context HW ID lifetime

2018-02-12 Thread Chris Wilson
Quoting Lionel Landwerlin (2018-02-12 11:00:10) > On 12/02/18 10:41, Chris Wilson wrote: > > Quoting Lionel Landwerlin (2018-02-12 10:37:52) > >> On 09/02/18 20:53, Chris Wilson wrote: > >>> Quoting Lionel Landwerlin (2018-02-09 17:47:44) > Hey Chris, > > From the i915/perf point

Re: [Intel-gfx] [PATCH 2/2] drm/i915: Reduce context HW ID lifetime

2018-02-12 Thread Lionel Landwerlin
On 12/02/18 10:41, Chris Wilson wrote: Quoting Lionel Landwerlin (2018-02-12 10:37:52) On 09/02/18 20:53, Chris Wilson wrote: Quoting Lionel Landwerlin (2018-02-09 17:47:44) Hey Chris, From the i915/perf point of view, I'm fine with this change. The pinning of the hw_id when monitoring a s

Re: [Intel-gfx] [PATCH 2/2] drm/i915: Reduce context HW ID lifetime

2018-02-12 Thread Chris Wilson
Quoting Lionel Landwerlin (2018-02-12 10:37:52) > On 09/02/18 20:53, Chris Wilson wrote: > > Quoting Lionel Landwerlin (2018-02-09 17:47:44) > >> Hey Chris, > >> > >> From the i915/perf point of view, I'm fine with this change. > >> The pinning of the hw_id when monitoring a single context (with

Re: [Intel-gfx] [PATCH 2/2] drm/i915: Reduce context HW ID lifetime

2018-02-12 Thread Lionel Landwerlin
On 09/02/18 20:53, Chris Wilson wrote: Quoting Lionel Landwerlin (2018-02-09 17:47:44) Hey Chris, From the i915/perf point of view, I'm fine with this change. The pinning of the hw_id when monitoring a single context (with OA) doesn't break the existing userspace (I can only think of Mesa).

Re: [Intel-gfx] [PATCH 2/2] drm/i915: Reduce context HW ID lifetime

2018-02-09 Thread Chris Wilson
Quoting Lionel Landwerlin (2018-02-09 17:47:44) > Hey Chris, > > From the i915/perf point of view, I'm fine with this change. > The pinning of the hw_id when monitoring a single context (with OA) > doesn't break the existing userspace (I can only think of Mesa). > > I'm also trying to build up

Re: [Intel-gfx] [PATCH 2/2] drm/i915: Reduce context HW ID lifetime

2018-02-09 Thread Chris Wilson
Quoting Daniele Ceraolo Spurio (2018-02-09 18:56:36) > > > On 09/02/18 02:22, Chris Wilson wrote: > > Future gen reduce the number of bits we will have available to > > differentiate between contexts, so reduce the lifetime of the ID > > assignment from that of the context to its current active c

Re: [Intel-gfx] [PATCH 2/2] drm/i915: Reduce context HW ID lifetime

2018-02-09 Thread Daniele Ceraolo Spurio
On 09/02/18 02:22, Chris Wilson wrote: Future gen reduce the number of bits we will have available to differentiate between contexts, so reduce the lifetime of the ID assignment from that of the context to its current active cycle (i.e. only while it is pinned for use by the HW, will it have a

Re: [Intel-gfx] [PATCH 2/2] drm/i915: Reduce context HW ID lifetime

2018-02-09 Thread Lionel Landwerlin
Hey Chris, From the i915/perf point of view, I'm fine with this change. The pinning of the hw_id when monitoring a single context (with OA) doesn't break the existing userspace (I can only think of Mesa). I'm also trying to build up a system wide monitoring feature in GPUTop with a timeline d

[Intel-gfx] [PATCH 2/2] drm/i915: Reduce context HW ID lifetime

2018-02-09 Thread Chris Wilson
Future gen reduce the number of bits we will have available to differentiate between contexts, so reduce the lifetime of the ID assignment from that of the context to its current active cycle (i.e. only while it is pinned for use by the HW, will it have a constant ID). This means that instead of a