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 3:43 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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox