irqbalance has been broken for a long time. Its ability to properly detect msi irqs and to correctly identify interrupt types (net vs. storage vs. other, etc), has been based on some tenuous string comparison logic that was easily broken by administrative name changes for interfaces. I've recently submitted this patch: https://lkml.org/lkml/2011/9/19/176 Which lets us use sysfs exclusively for finding device interrupts, which in turns lets us definitavely identify irq types (legacy pci vs. msi), as well as properly classifying them using the pci device class value. Additionally, this patch rips out the code that attemtps to bias interrupt count volumes using network statistics, since theres no sane way to be certain a single network interrupt is responsible for the number of packets received on a given interface. Workload computation is now done on soley on irq count. This may change in the future, adding /proc/stat irq and softirq time to the biasing mechanism. Note that without the above kernel change, this doesn't work right. Irqbalance contains a self check in which it identifies MSI interrupts in /proc/interrupts still. If it sees MSI irqs in /proc/interrupts, but none in sysfs, then it will issue a loud warning about irqs being missclassified until the kernel is updated. Signed-off-by: Neil Horman <nhorman@tuxdriver.com>
116 lines
2.8 KiB
C
116 lines
2.8 KiB
C
/*
|
|
* Copyright (C) 2006, Intel Corporation
|
|
*
|
|
* This file is part of irqbalance
|
|
*
|
|
* This program file is free software; you can redistribute it and/or modify it
|
|
* under the terms of the GNU General Public License as published by the
|
|
* Free Software Foundation; version 2 of the License.
|
|
*
|
|
* This program is distributed in the hope that it will be useful, but WITHOUT
|
|
* ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
|
|
* FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
|
|
* for more details.
|
|
*
|
|
* You should have received a copy of the GNU General Public License
|
|
* along with this program in a file named COPYING; if not, write to the
|
|
* Free Software Foundation, Inc.,
|
|
* 51 Franklin Street, Fifth Floor,
|
|
* Boston, MA 02110-1301 USA
|
|
*/
|
|
#include "config.h"
|
|
#include <stdio.h>
|
|
#include <stdlib.h>
|
|
#include <unistd.h>
|
|
#include <stdint.h>
|
|
#include <string.h>
|
|
#include <syslog.h>
|
|
|
|
#include "cpumask.h"
|
|
#include "irqbalance.h"
|
|
|
|
#define LINESIZE 4096
|
|
|
|
static int proc_int_has_msi = 0;
|
|
static int msi_found_in_sysfs = 0;
|
|
|
|
void parse_proc_interrupts(void)
|
|
{
|
|
FILE *file;
|
|
char *line = NULL;
|
|
size_t size = 0;
|
|
int int_type;
|
|
|
|
file = fopen("/proc/interrupts", "r");
|
|
if (!file)
|
|
return;
|
|
|
|
/* first line is the header we don't need; nuke it */
|
|
if (getline(&line, &size, file)==0) {
|
|
free(line);
|
|
return;
|
|
}
|
|
|
|
while (!feof(file)) {
|
|
int cpunr;
|
|
int number;
|
|
uint64_t count;
|
|
char *c, *c2;
|
|
|
|
if (getline(&line, &size, file)==0)
|
|
break;
|
|
|
|
if (!proc_int_has_msi)
|
|
if (strstr(line, "MSI") != NULL)
|
|
proc_int_has_msi = 1;
|
|
|
|
/* lines with letters in front are special, like NMI count. Ignore */
|
|
if (!(line[0]==' ' || (line[0]>='0' && line[0]<='9')))
|
|
break;
|
|
c = strchr(line, ':');
|
|
if (!c)
|
|
continue;
|
|
*c = 0;
|
|
c++;
|
|
number = strtoul(line, NULL, 10);
|
|
count = 0;
|
|
cpunr = 0;
|
|
|
|
c2=NULL;
|
|
while (1) {
|
|
uint64_t C;
|
|
C = strtoull(c, &c2, 10);
|
|
if (c==c2) /* end of numbers */
|
|
break;
|
|
count += C;
|
|
c=c2;
|
|
cpunr++;
|
|
}
|
|
if (cpunr != core_count)
|
|
need_cpu_rescan = 1;
|
|
|
|
set_interrupt_count(number, count);
|
|
|
|
/* is interrupt MSI based? */
|
|
int_type = find_irq_integer_prop(number, IRQ_TYPE);
|
|
if ((int_type == IRQ_TYPE_MSI) || (int_type == IRQ_TYPE_MSIX)) {
|
|
msi_found_in_sysfs = 1;
|
|
/* Set numa node for irq if it was MSI */
|
|
if (debug_mode)
|
|
printf("Set MSI interrupt for %d\n", number);
|
|
set_msi_interrupt_numa(number, c);
|
|
}
|
|
}
|
|
if ((proc_int_has_msi) && (!msi_found_in_sysfs)) {
|
|
syslog(LOG_WARNING, "WARNING: MSI interrupts found in /proc/interrupts\n");
|
|
syslog(LOG_WARNING, "But none found in sysfs, you need to update your kernel\n");
|
|
syslog(LOG_WARNING, "Until then, IRQs will be improperly classified\n");
|
|
/*
|
|
* Set msi_foun_in_sysfs, so we don't get this error constantly
|
|
*/
|
|
msi_found_in_sysfs = 1;
|
|
}
|
|
fclose(file);
|
|
free(line);
|
|
}
|