Re: [PATCH] drm/i915/perf: Initialise dynamic sysfs group before creation

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 09/08/17 16:14, Lionel Landwerlin wrote:
On 09/08/17 16:02, Chris Wilson wrote:
Quoting Lionel Landwerlin (2017-08-09 15:57:59)
Right, that's exactly what I was asking in our previous exchange.
I just follow the traces, and am internally screaming at how much we are
stuffing inside drm_i915_private nowadays.
You need to leave the memset() I think, because we check further down
It is already zeroed.

that function that id == 0 to detect failure to recognize that the (Gen,
GT) is supported.
Unless there is a guarantee that dev_priv is fully memset()?
It is allocated using kzalloc() and we depend upon that in many, many
places.
-Chris


Cool,

Fixes: f89823c21224 ("drm/i915/perf: Implement I915_PERF_ADD/REMOVE_CONFIG interface")

The line above is wrong and not what I wanted to write... :

Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin@xxxxxxxxx>
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux