Re: [PATCH V3 1/2] irq: Track the interrupt timings

2016-02-22 Thread Daniel Lezcano
On 02/22/2016 03:48 PM, Shreyas B Prabhu wrote: [ ... ] + + free_timings(desc); + free_masks(desc); + free_percpu(desc->kstat_irqs); + kfree(desc); } I think this is a rebase error. If I am not wrong, instead of this you need to add free_timings(desc) to delayed_free

Re: [PATCH V3 1/2] irq: Track the interrupt timings

2016-02-22 Thread Shreyas B Prabhu
Hi Daniel, On 02/16/2016 09:13 PM, Daniel Lezcano wrote: > diff --git a/kernel/irq/irqdesc.c b/kernel/irq/irqdesc.c > index 0ccd028..577686b 100644 > --- a/kernel/irq/irqdesc.c > +++ b/kernel/irq/irqdesc.c > @@ -174,6 +174,9 @@ static struct irq_desc *alloc_desc(int irq, int node, > struct module

Re: [PATCH V3 1/2] irq: Track the interrupt timings

2016-02-16 Thread Nicolas Pitre
On Tue, 16 Feb 2016, Daniel Lezcano wrote: > The interrupt framework gives a lot of information about each interrupt. > It does not keep track of when those interrupts occur though. > > This patch provides a mean to record the elapsed time between successive > interrupt occurrences in a per-IRQ p