Re: [PATCH] drm/i915/perf: Initialise the dynamic sysfs attr

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

 



Quoting Lionel Landwerlin (2017-08-04 09:56:36)
> Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin@xxxxxxxxx>
> 
> I'm curious though, why hasn't it been a problem before??
> All of the configs we store in kernel don't seem to properly initialize 
> attr either, yet it hasn't yield any warning.
> Did we get it wrong all along there too?

The attr contains an embedded lock_class, but due to the design of
lockdep all lock_classes must be static. sysfs_attr_init() contains a
magic static lock_class - so instead of each attr now being an
independent lock class, all of the dynamic attr belong to the same
class. Don't try to use sysfs of one attr from inside the callback of
another!
-Chris
_______________________________________________
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