public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
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



  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