On Tue, 13 Sep 2016, Craig Gallek wrote:
> +What: /sys/kernel/irq/<irq>/name
> +Date: September 2016
> +KernelVersion: 4.9
> +Contact: Craig Gallek <kr...@google.com>
> +Description: Human-readable flow handler name as defined by the device
> + driver.
That's ambigous. device driver reads like a the driver of the device which
uses this irq. I fixed that up.
> static void free_desc(unsigned int irq)
> {
> struct irq_desc *desc = irq_to_desc(irq);
>
> unregister_irq_proc(irq, desc);
> + kobject_del(&desc->kobj);
You forgot to move this under the sparse lock protection as well. There is
the same race against a concurrent initcall.
Thanks,
tglx
