Re: [Intel-gfx] [PATCH 1/2] drm/i915/perf: Register sysctl path globally

2019-12-16 Thread Chris Wilson
Quoting Jani Nikula (2019-12-16 10:27:59) > On Fri, 13 Dec 2019, Venkata Sandeep Dhanalakota > wrote: > > We do not require to register the sysctl paths per instance, > > so making registration global. > > > > v2: make sysctl path register and unregister function driver > > specific (Tvrtko a

Re: [Intel-gfx] [PATCH 1/2] drm/i915/perf: Register sysctl path globally

2019-12-16 Thread Jani Nikula
On Fri, 13 Dec 2019, Venkata Sandeep Dhanalakota wrote: > We do not require to register the sysctl paths per instance, > so making registration global. > > v2: make sysctl path register and unregister function driver > specific (Tvrtko and Lucas). > > v3: remove the NULL-check as unregister_s

[Intel-gfx] [PATCH 1/2] drm/i915/perf: Register sysctl path globally

2019-12-13 Thread Venkata Sandeep Dhanalakota
We do not require to register the sysctl paths per instance, so making registration global. v2: make sysctl path register and unregister function driver specific (Tvrtko and Lucas). v3: remove the NULL-check as unregister_sysctl_table is null safe. (Chris and Lucas) Cc: Sudeep Dutt Cc:

Re: [Intel-gfx] [PATCH 1/2] drm/i915/perf: Register sysctl path globally

2019-12-13 Thread Lucas De Marchi
On Fri, Dec 13, 2019 at 04:09:20PM +, Chris Wilson wrote: Quoting Venkata Sandeep Dhanalakota (2019-12-13 15:51:51) We do not require to register the sysctl paths per instance, so making registration global. v2: make sysctl path register and unregister function driver specific (Tvrtko a

Re: [Intel-gfx] [PATCH 1/2] drm/i915/perf: Register sysctl path globally

2019-12-13 Thread Chris Wilson
Quoting Venkata Sandeep Dhanalakota (2019-12-13 15:51:51) > We do not require to register the sysctl paths per instance, > so making registration global. > > v2: make sysctl path register and unregister function driver > specific (Tvrtko and Lucas). > > Cc: Sudeep Dutt > Cc: Rodrigo Vivi >

[Intel-gfx] [PATCH 1/2] drm/i915/perf: Register sysctl path globally

2019-12-13 Thread Venkata Sandeep Dhanalakota
We do not require to register the sysctl paths per instance, so making registration global. v2: make sysctl path register and unregister function driver specific (Tvrtko and Lucas). Cc: Sudeep Dutt Cc: Rodrigo Vivi Cc: Daniel Vetter Cc: Chris Wilson Cc: Jani Nikula Signed-off-by: Venkata

[Intel-gfx] [PATCH 1/2] drm/i915/perf: Register sysctl path globally

2019-12-13 Thread Venkata Sandeep Dhanalakota
We do not require to register the sysctl paths per instance, so making registration global. v2: make sysctl path register and unregister function driver specific (Tvrtko and Lucas). Cc: Sudeep Dutt Cc: Rodrigo Vivi Cc: Daniel Vetter Cc: Chris Wilson Cc: Jani Nikula Signed-off-by: Venkata

Re: [Intel-gfx] [PATCH 1/2] drm/i915/perf: Register sysctl path globally

2019-12-12 Thread Venkata Sandeep Dhanalakota
On 19/12/12 01:34, Lucas De Marchi wrote: > On Wed, Dec 11, 2019 at 11:35:21PM -0800, Venkata Sandeep Dhanalakota wrote: > > We do not require to register the sysctl paths per instance, > > so making registration global. > > > > v2: make sysctl path register and unregister function driver > >s

Re: [Intel-gfx] [PATCH 1/2] drm/i915/perf: Register sysctl path globally

2019-12-12 Thread Lucas De Marchi
On Wed, Dec 11, 2019 at 11:35:21PM -0800, Venkata Sandeep Dhanalakota wrote: We do not require to register the sysctl paths per instance, so making registration global. v2: make sysctl path register and unregister function driver specific (Tvrtko and Lucas). Cc: Sudeep Dutt Cc: Rodrigo Vivi

[Intel-gfx] [PATCH 1/2] drm/i915/perf: Register sysctl path globally

2019-12-11 Thread Venkata Sandeep Dhanalakota
We do not require to register the sysctl paths per instance, so making registration global. v2: make sysctl path register and unregister function driver specific (Tvrtko and Lucas). Cc: Sudeep Dutt Cc: Rodrigo Vivi Cc: Daniel Vetter Cc: Chris Wilson Cc: Jani Nikula Signed-off-by: Venkata

Re: [Intel-gfx] [PATCH 1/2] drm/i915/perf: Register sysctl path globally

2019-12-11 Thread Lucas De Marchi
On Wed, Dec 11, 2019 at 09:13:18AM -0800, Venkata Sandeep Dhanalakota wrote: On 19/12/11 04:39, Tvrtko Ursulin wrote: On 11/12/2019 16:31, Tvrtko Ursulin wrote: > On 11/12/2019 16:07, Venkata Sandeep Dhanalakota wrote: > > We do not require to register the sysctl paths per instance, > > so maki

Re: [Intel-gfx] [PATCH 1/2] drm/i915/perf: Register sysctl path globally

2019-12-11 Thread Tvrtko Ursulin
On 11/12/2019 17:13, Venkata Sandeep Dhanalakota wrote: On 19/12/11 04:39, Tvrtko Ursulin wrote: On 11/12/2019 16:31, Tvrtko Ursulin wrote: On 11/12/2019 16:07, Venkata Sandeep Dhanalakota wrote: We do not require to register the sysctl paths per instance, so making registration global. Cc:

Re: [Intel-gfx] [PATCH 1/2] drm/i915/perf: Register sysctl path globally

2019-12-11 Thread Venkata Sandeep Dhanalakota
On 19/12/11 04:39, Tvrtko Ursulin wrote: > > On 11/12/2019 16:31, Tvrtko Ursulin wrote: > > On 11/12/2019 16:07, Venkata Sandeep Dhanalakota wrote: > > > We do not require to register the sysctl paths per instance, > > > so making registration global. > > > > > > Cc: Sudeep Dutt > > > Cc: Rodrig

Re: [Intel-gfx] [PATCH 1/2] drm/i915/perf: Register sysctl path globally

2019-12-11 Thread Tvrtko Ursulin
On 11/12/2019 16:31, Tvrtko Ursulin wrote: On 11/12/2019 16:07, Venkata Sandeep Dhanalakota wrote: We do not require to register the sysctl paths per instance, so making registration global. Cc: Sudeep Dutt Cc: Rodrigo Vivi Cc: Daniel Vetter Cc: Chris Wilson Cc: Jani Nikula Signed-off-by:

Re: [Intel-gfx] [PATCH 1/2] drm/i915/perf: Register sysctl path globally

2019-12-11 Thread Tvrtko Ursulin
On 11/12/2019 16:07, Venkata Sandeep Dhanalakota wrote: We do not require to register the sysctl paths per instance, so making registration global. Cc: Sudeep Dutt Cc: Rodrigo Vivi Cc: Daniel Vetter Cc: Chris Wilson Cc: Jani Nikula Signed-off-by: Venkata Sandeep Dhanalakota --- drivers

Re: [Intel-gfx] [PATCH 1/2] drm/i915/perf: Register sysctl path globally

2019-12-11 Thread Lionel Landwerlin
On 11/12/2019 18:07, Venkata Sandeep Dhanalakota wrote: We do not require to register the sysctl paths per instance, so making registration global. Cc: Sudeep Dutt Cc: Rodrigo Vivi Cc: Daniel Vetter Cc: Chris Wilson Cc: Jani Nikula Signed-off-by: Venkata Sandeep Dhanalakota Not sure wha

[Intel-gfx] [PATCH 1/2] drm/i915/perf: Register sysctl path globally

2019-12-11 Thread Venkata Sandeep Dhanalakota
We do not require to register the sysctl paths per instance, so making registration global. Cc: Sudeep Dutt Cc: Rodrigo Vivi Cc: Daniel Vetter Cc: Chris Wilson Cc: Jani Nikula Signed-off-by: Venkata Sandeep Dhanalakota --- drivers/gpu/drm/i915/i915_perf.c | 10 -- drivers/gpu/