From: Yao Fei Zhu <walkinair@cn.ibm.com>
To: Andrew Morton <akpm@osdl.org>
Cc: David Chinner <dgc@sgi.com>,
linux-kernel@vger.kernel.org, haveblue@us.ibm.com,
xfs@oss.sgi.com
Subject: Re: kernel BUG in __xfs_get_blocks at fs/xfs/linux-2.6/xfs_aops.c:1293!
Date: Fri, 01 Sep 2006 11:43:33 +0800 [thread overview]
Message-ID: <44F7AC65.5050502@cn.ibm.com> (raw)
In-Reply-To: <20060831015430.6df0d8ba.akpm@osdl.org>
Andrew Morton wrote:
>On Thu, 31 Aug 2006 18:17:26 +1000
>David Chinner <dgc@sgi.com> wrote:
>
>
>
>>>BTW, I have CONFIG_PPC_64K_PAGES enabled.
>>>
>>>
>>But that might be a good place to start. Can you see if you can
>>reproduce the problem without this config option set?
>>
>>
>
>It would be useful to compare the compiler warning output for 64k pages
>versus that for smaller-pages.
>
>Several quite worrisome-looking warnings are emitted from various parts of
>the kernel with 64k pages. Related to arithmetic on short types.
>
>
1. the config diff
blade10:/boot # diff config-2.6.18-rc5-ppc64 config-2.6.18-rc5-ppc64.64kp
4c4
< # Thu Aug 31 18:25:42 2006
---
> # Thu Aug 31 21:18:52 2006
51c51
< CONFIG_LOCALVERSION="-ppc64"
---
> CONFIG_LOCALVERSION="-ppc64.64kp"
173c173
< CONFIG_FORCE_MAX_ZONEORDER=13
---
> CONFIG_FORCE_MAX_ZONEORDER=9
204c204
< # CONFIG_PPC_64K_PAGES is not set
---
> CONFIG_PPC_64K_PAGES=y
2. the compiler warning diff
ltctest:~ # diff 4k.warning 64k.warning
0a1,5
> kernel/power/pm.c:205: warning: ‘pm_register’ is deprecated
(declared at kernel/power/pm.c:64)
> kernel/power/pm.c:205: warning: ‘pm_register’ is deprecated
(declared at kernel/power/pm.c:64)
> kernel/power/pm.c:206: warning: ‘pm_send_all’ is deprecated
(declared at kernel/power/pm.c:180)
> kernel/power/pm.c:206: warning: ‘pm_send_all’ is deprecated
(declared at kernel/power/pm.c:180)
> fs/bio.c:169: warning: ‘idx’ may be used uninitialized in this
function
8,13d12
< fs/bio.c:169: warning: ‘idx’ may be used uninitialized in this
function
< kernel/power/pm.c:205: warning: ‘pm_register’ is deprecated
(declared at kernel/power/pm.c:64)
< kernel/power/pm.c:205: warning: ‘pm_register’ is deprecated
(declared at kernel/power/pm.c:64)
< kernel/power/pm.c:206: warning: ‘pm_send_all’ is deprecated
(declared at kernel/power/pm.c:180)
< kernel/power/pm.c:206: warning: ‘pm_send_all’ is deprecated
(declared at kernel/power/pm.c:180)
< fs/eventpoll.c:500: warning: ‘fd’ may be used uninitialized in
this function
17a17,27
> fs/eventpoll.c:500: warning: ‘fd’ may be used uninitialized in
this function
> fs/fat/inode.c:1227: warning: comparison is always false due to
limited range of data type
> fs/hfs/btree.c:243: warning: comparison is always false due to
limited range of data type
> fs/hfsplus/btree.c:235: warning: comparison is always false due to
limited range of data type
> fs/ocfs2/vote.c:774: warning: ‘response’ may be used
uninitialized in this function
> fs/ocfs2/dlm/dlmdomain.c:70: warning: format ‘%lu’ expects type
‘long unsigned int’, but argument 7 has type ‘int’
> fs/ocfs2/dlm/dlmdomain.c:70: warning: format ‘%lu’ expects type
‘long unsigned int’, but argument 7 has type ‘int’
> fs/ocfs2/dlm/dlmdomain.c:70: warning: format ‘%lu’ expects type
‘long unsigned int’, but argument 7 has type ‘int’
> fs/ocfs2/dlm/dlmdomain.c:918: warning: ‘response’ may be used
uninitialized in this function
> fs/udf/balloc.c:751: warning: ‘goal_eloc.logicalBlockNum’ may be
used uninitialized in this function
> fs/udf/super.c:1364: warning: ‘ino.partitionReferenceNum’ may be
used uninitialized in this function
56a67,68
> drivers/usb/core/devio.c:620: warning: comparison is always false due
to limited range of data type
> drivers/net/r8169.c:2131: warning: ‘txd’ may be used
uninitialized in this function
59d70
< drivers/net/r8169.c:2131: warning: ‘txd’ may be used uninitialized
in this function
70,73c81
< fs/ocfs2/vote.c:774: warning: ‘response’ may be used uninitialized
in this function
< fs/ocfs2/dlm/dlmdomain.c:918: warning: ‘response’ may be used
uninitialized in this function
< fs/udf/balloc.c:751: warning: ‘goal_eloc.logicalBlockNum’ may be
used uninitialized in this function
< fs/udf/super.c:1364: warning: ‘ino.partitionReferenceNum’ may be
used uninitialized in this function
---
> net/key/af_key.c:403: warning: comparison is always false due to
limited range of data type
next prev parent reply other threads:[~2006-09-01 4:12 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-08-31 5:48 kernel BUG in __xfs_get_blocks at fs/xfs/linux-2.6/xfs_aops.c:1293! Yao Fei Zhu
2006-08-31 7:47 ` David Chinner
2006-08-31 8:02 ` Yao Fei Zhu
2006-08-31 8:17 ` David Chinner
2006-08-31 8:54 ` Andrew Morton
2006-09-01 3:43 ` Yao Fei Zhu [this message]
2006-09-01 6:34 ` Yao Fei Zhu
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=44F7AC65.5050502@cn.ibm.com \
--to=walkinair@cn.ibm.com \
--cc=akpm@osdl.org \
--cc=dgc@sgi.com \
--cc=haveblue@us.ibm.com \
--cc=linux-kernel@vger.kernel.org \
--cc=xfs@oss.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.