From: Greg Ungerer <gerg@uclinux.org>
To: Linux Kernel Development <linux-kernel@vger.kernel.org>,
Linux/m68k <linux-m68k@vger.kernel.org>
Subject: rcu alignment warning tripping on m68k
Date: Thu, 29 May 2014 12:08:32 +1000 [thread overview]
Message-ID: <538696A0.5090601@uclinux.org> (raw)
Hi All,
Inside kernel/rcy/tree.c in __call_rcu() it does an alignment check on
the head pointer passed in. This trips on m68k systems, because they only
need alignment of 32bit quantities to 16bit boundaries.
...
UDP hash table entries: 512 (order: 0, 8192 bytes)
UDP-Lite hash table entries: 512 (order: 0, 8192 bytes)
NET: Registered protocol family 1
ROMFS MTD (C) 2007 Red Hat, Inc.
------------[ cut here ]------------
WARNING: CPU: 0 PID: 18 at kernel/rcu/tree.c:2470 0x400577e0()
Modules linked in:
CPU: 0 PID: 18 Comm: kworker/u2:0 Not tainted 3.15.0-rc4-00013-g1c07b64-dirty #14
Stack from 418ada9c:
418ada9c 401e8336 400258e0 401c69c4 418ac000 418a86f6 4003bc0a 418a8390
418a86f6 4002590e 401e176b 000009a6 400577e0 00000009 00000000 400577e0
401e176b 000009a6 401c69c4 418ac000 418a8390 418a8390 418ac000 40057928
418a86f6 40025bcc 40218fe0 00000000 401ddb56 401e17ad 000002af 400261c2
418a86f6 40025bcc 00000013 00000030 00000010 00000013 418ac000 418adf88
418a8390 418adf70 400263f0 418a8000 418adf18 41885950 00000000 00000000
Call Trace:
[<400258e0>] 0x400258e0
[<401c69c4>] 0x401c69c4
[<4003bc0a>] 0x4003bc0a
[<4002590e>] 0x4002590e
[<400577e0>] 0x400577e0
What is the best solution to fix?
Make the alignment check be only 16bits?
--- a/kernel/rcu/tree.c
+++ b/kernel/rcu/tree.c
@@ -2467,7 +2467,7 @@ __call_rcu(struct rcu_head *head, void (*func)(struct rcu_
unsigned long flags;
struct rcu_data *rdp;
- WARN_ON_ONCE((unsigned long)head & 0x3); /* Misaligned rcu_head! */
+ WARN_ON_ONCE((unsigned long)head & 0x1); /* Misaligned rcu_head! */
if (debug_rcu_head_queue(head)) {
/* Probable double call_rcu(), so leak the callback. */
ACCESS_ONCE(head->func) = rcu_leak_callback;
Or should we just get rid of this check altogether?
Regards
Greg
next reply other threads:[~2014-05-29 2:19 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-05-29 2:08 Greg Ungerer [this message]
2014-05-29 13:11 ` rcu alignment warning tripping on m68k One Thousand Gnomes
2014-05-30 1:29 ` Greg Ungerer
2014-06-06 18:46 ` Paul E. McKenney
2014-06-07 13:17 ` Mikael Pettersson
2014-06-09 15:24 ` Paul E. McKenney
2014-06-10 6:22 ` Greg Ungerer
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=538696A0.5090601@uclinux.org \
--to=gerg@uclinux.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-m68k@vger.kernel.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