On Mon, Jan 25, 2016 at 04:32:03PM +0800, Huang Rui wrote:
> +struct power_pmu {
> + raw_spinlock_t lock;
> + struct list_head active_list;Maybe a dumb question, but what is that list for? You only ever add/del events to/from it, you never iterate it. So why keep it? If you drop the list, the lock can go too I think, making all this stuff simpler.
