adding support for using irq assignments not numbers

This is to give the option of a more readable key for the irq, like munin.
Thanks for the updates guys. I'll get it working with multi-cpu. All the
servers I tested had unique allocations so I'm not sure what to do about
the irq's with identical allocations. Would sending the metrics from both
allocations just double the results? If so is that a bad thing?

g.



On 5 September 2014 09:32, Florian Forster <notifications@github.com> wrote:

> In src/irq.c:
>
> > @@ -157,6 +170,13 @@ static int irq_read (void)
> >
> >  		irq_name[irq_name_len - 1] = 0;
> >  		irq_name_len--;
> > +                /* Use the last column instead of the irq number */
> > +                if (named_irq == 1)
> > +                {
> > +                        /* Some first column names are already non digits */
> > +                        if ( isdigit(irq_name[0]) )
> > +                                irq_name = fields[4];
>
> fields[4] is wrong and should probably be fields[fields_num - 1] instead.
> You should also add a check along the lines of fields_num > (cpu_num + 1)
> to check that the counter columns are actually followed by an additional
> column.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/collectd/collectd/pull/721/files#r17161669>.
>



-- 
Giles Westwood