From: "Cédric Le Goater" <clg@kaod.org>
To: linuxppc-dev@lists.ozlabs.org
Cc: "Benjamin Herrenschmidt" <benh@kernel.crashing.org>,
"Michael Ellerman" <mpe@ellerman.id.au>,
"Paul Mackerras" <paulus@samba.org>,
"David Gibson" <david@gibson.dropbear.id.au>,
"Cédric Le Goater" <clg@kaod.org>
Subject: [PATCH 10/10] powerpc/xive: fix the size of the cpumask used in xive_find_target_in_mask()
Date: Tue, 8 Aug 2017 11:02:49 +0200 [thread overview]
Message-ID: <1502182969-3180-1-git-send-email-clg@kaod.org> (raw)
When called from xive_irq_startup(), the size of the cpumask can be
larger than nr_cpu_ids. Most of time, its value is NR_CPUS (2048).
This can result in such WARNINGs in xive_find_target_in_mask():
[ 0.094480] WARNING: CPU: 10 PID: 1 at ../arch/powerpc/sysdev/xive/common.c:476 xive_find_target_in_mask+0x110/0x2f0
[ 0.094486] Modules linked in:
[ 0.094491] CPU: 10 PID: 1 Comm: swapper/0 Not tainted 4.12.0+ #3
[ 0.094496] task: c0000003fae4f200 task.stack: c0000003fe108000
[ 0.094501] NIP: c00000000008a310 LR: c00000000008a2e4 CTR: 000000000072ca34
[ 0.094506] REGS: c0000003fe10b360 TRAP: 0700 Not tainted (4.12.0+)
[ 0.094510] MSR: 8000000000029033 <SF,EE,ME,IR,DR,RI,LE>
[ 0.094515] CR: 88000222 XER: 20040008
[ 0.094521] CFAR: c00000000008a2cc SOFTE: 0
[ 0.094521] GPR00: c00000000008a274 c0000003fe10b5e0 c000000001428f00 0000000000000010
[ 0.094521] GPR04: 0000000000000010 0000000000000010 0000000000000010 0000000000000099
[ 0.094521] GPR08: 0000000000000010 0000000000000001 ffffffffffff0000 0000000000000000
[ 0.094521] GPR12: 0000000000000000 c00000000fff2d00 c00000000000d4d8 0000000000000000
[ 0.094521] GPR16: 0000000000000000 0000000000000000 0000000000000000 0000000000000000
[ 0.094521] GPR20: 0000000000000000 0000000000000000 0000000000000000 c000000000b451e8
[ 0.094521] GPR24: 00000000ffffffff c000000001462354 0000000000000800 00000000000007ff
[ 0.094521] GPR28: c000000001462354 0000000000000010 c0000003f857e418 0000000000000010
[ 0.094580] NIP [c00000000008a310] xive_find_target_in_mask+0x110/0x2f0
[ 0.094585] LR [c00000000008a2e4] xive_find_target_in_mask+0xe4/0x2f0
[ 0.094589] Call Trace:
[ 0.094593] [c0000003fe10b5e0] [c00000000008a274] xive_find_target_in_mask+0x74/0x2f0 (unreliable)
[ 0.094601] [c0000003fe10b690] [c00000000008abf0] xive_pick_irq_target.isra.1+0x200/0x230
[ 0.094608] [c0000003fe10b830] [c00000000008b250] xive_irq_startup+0x60/0x180
[ 0.094614] [c0000003fe10b8b0] [c0000000001608f0] irq_startup+0x70/0xd0
[ 0.094620] [c0000003fe10b8f0] [c00000000015df7c] __setup_irq+0x7bc/0x880
[ 0.094626] [c0000003fe10ba90] [c00000000015e30c] request_threaded_irq+0x14c/0x2c0
[ 0.094632] [c0000003fe10baf0] [c0000000000aeb00] request_event_sources_irqs+0x100/0x180
[ 0.094639] [c0000003fe10bc10] [c000000000e7d2f8] __machine_initcall_pseries_init_ras_IRQ+0x104/0x134
[ 0.094646] [c0000003fe10bc40] [c00000000000cc88] do_one_initcall+0x68/0x1d0
[ 0.094652] [c0000003fe10bd00] [c000000000e643c8] kernel_init_freeable+0x290/0x374
[ 0.094658] [c0000003fe10bdc0] [c00000000000d4f4] kernel_init+0x24/0x170
[ 0.094664] [c0000003fe10be30] [c00000000000b268] ret_from_kernel_thread+0x5c/0x74
[ 0.094669] Instruction dump:
[ 0.094673] 48586529 60000000 e8dc0002 393f0001 7f9b4800 7c7d07b4 7d3f07b4 409effcc
[ 0.094682] 7f9d3000 7d26e850 79290fe0 69290001 <0b090000> 409c0194 3f620004 3b7b8ec8
Fix this problem by using a minimum value.
Signed-off-by: Cédric Le Goater <clg@kaod.org>
---
arch/powerpc/sysdev/xive/common.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/powerpc/sysdev/xive/common.c b/arch/powerpc/sysdev/xive/common.c
index 536ee15f61fb..4dac7d560a42 100644
--- a/arch/powerpc/sysdev/xive/common.c
+++ b/arch/powerpc/sysdev/xive/common.c
@@ -463,7 +463,7 @@ static int xive_find_target_in_mask(const struct cpumask *mask,
int cpu, first, num, i;
/* Pick up a starting point CPU in the mask based on fuzz */
- num = cpumask_weight(mask);
+ num = min_t(int, cpumask_weight(mask), nr_cpu_ids);
first = fuzz % num;
/* Locate it */
--
2.7.5
next reply other threads:[~2017-08-08 9:03 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-08-08 9:02 Cédric Le Goater [this message]
2017-08-09 7:06 ` [PATCH 10/10] powerpc/xive: fix the size of the cpumask used in xive_find_target_in_mask() Michael Ellerman
2017-08-09 7:29 ` Benjamin Herrenschmidt
2017-08-09 7:30 ` Cédric Le Goater
2017-08-24 5:49 ` Michael Ellerman
2017-08-24 7:37 ` Cédric Le Goater
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=1502182969-3180-1-git-send-email-clg@kaod.org \
--to=clg@kaod.org \
--cc=benh@kernel.crashing.org \
--cc=david@gibson.dropbear.id.au \
--cc=linuxppc-dev@lists.ozlabs.org \
--cc=mpe@ellerman.id.au \
--cc=paulus@samba.org \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).