All of lore.kernel.org
 help / color / mirror / Atom feed
From: Yinghai Lu <yinghai@kernel.org>
To: Andrew Morton <akpm@linux-foundation.org>,
	Mike Travis <travis@sgi.com>,
	Thomas Gleixner <tglx@linutronix.de>
Cc: "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: [PATCH] irq: remove smp_affinity_list when unregister irq proc
Date: Wed, 25 May 2011 22:09:40 -0700	[thread overview]
Message-ID: <4DDDE094.6050505@kernel.org> (raw)


got:
[  274.239264] ------------[ cut here ]------------
[  274.239500] WARNING: at fs/proc/generic.c:850 remove_proc_entry+0x24c/0x27a()
[  274.251761] remove_proc_entry: removing non-empty directory 'irq/184', leaking at least 'smp_affinity_list'
[  274.271696] Modules linked in:
[  274.271880] Pid: 1, comm: swapper Tainted: G        W   2.6.39-tip-yh-07723-gf48dc10-dirty #1051
[  274.291734] Call Trace:
[  274.291892]  [<ffffffff8107cfa8>] warn_slowpath_common+0x85/0x9d
[  274.311582]  [<ffffffff8107d063>] warn_slowpath_fmt+0x46/0x48
[  274.311874]  [<ffffffff8118994e>] remove_proc_entry+0x24c/0x27a
[  274.331549]  [<ffffffff813336ca>] ? vsnprintf+0x3ed/0x429
[  274.331798]  [<ffffffff81333746>] ? sprintf+0x40/0x42
[  274.351430]  [<ffffffff810cde65>] unregister_irq_proc+0xb5/0xd2
[  274.351715]  [<ffffffff810ca64b>] free_desc+0x22/0x61
[  274.371461]  [<ffffffff810ca6c5>] irq_free_descs+0x3b/0x73
[  274.371738]  [<ffffffff81053bb3>] free_irq_at+0x31/0x36
[  274.391409]  [<ffffffff810553e6>] destroy_irq+0x74/0x7c
[  274.391662]  [<ffffffff81055619>] native_teardown_msi_irq+0xe/0x10
[  274.411407]  [<ffffffff8136239c>] default_teardown_msi_irqs+0x57/0x79
[  274.411735]  [<ffffffff81361c49>] free_msi_irqs+0x84/0xe2
[  274.431394]  [<ffffffff8136294d>] pci_disable_msi+0x35/0x39
[  274.431678]  [<ffffffff816574e2>] e1000e_reset_interrupt_capability+0x55/0x63
[  274.451420]  [<ffffffff81657701>] e1000_test_msi_interrupt+0x30/0x114
[  274.451749]  [<ffffffff81657853>] e1000_test_msi+0x6e/0xc5
[  274.471429]  [<ffffffff81658add>] e1000_open+0x10f/0x1e2
[  274.471731]  [<ffffffff81afed63>] __dev_open+0x9c/0xca
[  274.491282]  [<ffffffff81afef9d>] __dev_change_flags+0xad/0x130
[  274.491561]  [<ffffffff81aff0a1>] dev_change_flags+0x21/0x56
[  274.511370]  [<ffffffff82737d6b>] ic_open_devs+0xe3/0x233
[  274.511637]  [<ffffffff827394cf>] ip_auto_config+0xa5/0x2c0
[  274.531432]  [<ffffffff8273942a>] ? root_nfs_parse_addr+0xb5/0xb5
[  274.531780]  [<ffffffff810002cf>] do_one_initcall+0x57/0x134
[  274.551599]  [<ffffffff826ecf91>] kernel_init+0x137/0x1bb
[  274.551884]  [<ffffffff81c2e354>] kernel_thread_helper+0x4/0x10
[  274.571423]  [<ffffffff81c268c0>] ? retint_restore_args+0xe/0xe
[  274.591165]  [<ffffffff826ece5a>] ? start_kernel+0x3d3/0x3d3
[  274.591483]  [<ffffffff81c2e350>] ? gs_change+0xb/0xb
[  274.611129] ---[ end trace e07c035be3c974e5 ]---

caused by:
| commit 4b060420a596095869a6d7849caa798d23839cd1
| Author: Mike Travis <travis@sgi.com>
| Date:   Tue May 24 17:13:12 2011 -0700
|
|    bitmap, irq: add smp_affinity_list interface to /proc/irq

it forgets to remove it in exit path

Signed-off-by: Yinghai Lu <yinghai@kernel.org>

---
 kernel/irq/proc.c |    1 +
 1 file changed, 1 insertion(+)

Index: linux-2.6/kernel/irq/proc.c
===================================================================
--- linux-2.6.orig/kernel/irq/proc.c
+++ linux-2.6/kernel/irq/proc.c
@@ -352,6 +352,7 @@ void unregister_irq_proc(unsigned int ir
 #ifdef CONFIG_SMP
 	remove_proc_entry("smp_affinity", desc->dir);
 	remove_proc_entry("affinity_hint", desc->dir);
+	remove_proc_entry("smp_affinity_list", desc->dir);
 	remove_proc_entry("node", desc->dir);
 #endif
 	remove_proc_entry("spurious", desc->dir);

             reply	other threads:[~2011-05-26  5:10 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-05-26  5:09 Yinghai Lu [this message]
2011-05-26 11:18 ` [tip:irq/urgent] irq: Remove smp_affinity_list when unregister irq proc tip-bot for Yinghai Lu
2011-05-26 14:05 ` [PATCH] irq: remove " Mike Travis

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4DDDE094.6050505@kernel.org \
    --to=yinghai@kernel.org \
    --cc=akpm@linux-foundation.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=tglx@linutronix.de \
    --cc=travis@sgi.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.