linux-next: ext4 tree build warning

linux-next: ext4 tree build warning

[Stephen Rothwell]

[-- Attachment #1: Type: text/plain, Size: 648 bytes --]

Hi Ted,

The linux-next builds (ppc64 allyesconfig) produce this warning:
(I built with CONFIG_DEBUG_SECTION_MISMATCH=y)

WARNING: fs/ext4/ext4.o(.text+0x5b6c8): Section mismatch in reference from the function .exit_ext4_mballoc() to the function .exit.text:.ext4_remove_debugfs_entry()
The function .exit_ext4_mballoc() references a function in an exit section.
Often the function .ext4_remove_debugfs_entry() has valid usage outside the exit section
and the fix is to remove the __exit annotation of .ext4_remove_debugfs_entry.

-- 
Cheers,
Stephen Rothwell                    sfr@canb.auug.org.au
http://www.canb.auug.org.au/~sfr/

[-- Attachment #2: Type: application/pgp-signature, Size: 197 bytes --]

Re: linux-next: ext4 tree build warning

[Theodore Tso]

On Thu, Aug 27, 2009 at 05:17:04PM +1000, Stephen Rothwell wrote:
> Hi Ted,
> 
> The linux-next builds (ppc64 allyesconfig) produce this warning:
> (I built with CONFIG_DEBUG_SECTION_MISMATCH=y)
> 
> WARNING: fs/ext4/ext4.o(.text+0x5b6c8): Section mismatch in reference from the function .exit_ext4_mballoc() to the function .exit.text:.ext4_remove_debugfs_entry()
> The function .exit_ext4_mballoc() references a function in an exit section.
> Often the function .ext4_remove_debugfs_entry() has valid usage outside the exit section
> and the fix is to remove the __exit annotation of .ext4_remove_debugfs_entry.

Thanks for pointing that out, I'll get that fixed.

						- Ted

linux-next: ext4 tree build warning

[Stephen Rothwell]

[-- Attachment #1: Type: text/plain, Size: 511 bytes --]

Hi Ted,

Today's linux-next build (powerpc ppc64_defconfig) produced this warning:

fs/ext4/inode.c: In function 'ext4_dirty_inode':
fs/ext4/inode.c:5615: warning: unused variable 'current_handle'

Introduced by commit f3dc272fd5e2ae08244796bb39e7e1ce4b25d3b3 ("ext4:
Make sure ext4_dirty_inode() updates the inode in no journal mode"). This
variable is now only referenced by a jbd_debug().

-- 
Cheers,
Stephen Rothwell                    sfr@canb.auug.org.au
http://www.canb.auug.org.au/~sfr/

[-- Attachment #2: Type: application/pgp-signature, Size: 198 bytes --]

Re: linux-next: ext4 tree build warning

[Theodore Tso]

On Fri, Oct 02, 2009 at 11:26:37AM +1000, Stephen Rothwell wrote:
> Hi Ted,
> 
> Today's linux-next build (powerpc ppc64_defconfig) produced this warning:
> 
> fs/ext4/inode.c: In function 'ext4_dirty_inode':
> fs/ext4/inode.c:5615: warning: unused variable 'current_handle'
> 
> Introduced by commit f3dc272fd5e2ae08244796bb39e7e1ce4b25d3b3 ("ext4:
> Make sure ext4_dirty_inode() updates the inode in no journal mode"). This
> variable is now only referenced by a jbd_debug().

Thanks for pointing this out.   I'll get this fixed.

       	   	    	 	     	      - Ted

linux-next: ext4 tree build warning

[Stephen Rothwell]

[-- Attachment #1: Type: text/plain, Size: 741 bytes --]

Hi Ted,

Today's linux-next build (powerpc ppc64_defconfig) produced this warning:

In file included from include/trace/ftrace.h:262,
                 from include/trace/define_trace.h:57,
                 from include/trace/events/ext4.h:720,
                 from fs/ext4/super.c:51:
include/trace/events/ext4.h: In function 'ftrace_raw_output_ext4_free_inode':
include/trace/events/ext4.h:12: warning: format '%llu' expects type 'long long unsigned int', but argument 8 has type 'blkcnt_t'

Introduced by commit a6e130e94db16b324fbf2eeed2b310273625acf7 ("ext4:
convert instrumentation from markers to tracepoints").

-- 
Cheers,
Stephen Rothwell                    sfr@canb.auug.org.au
http://www.canb.auug.org.au/~sfr/

[-- Attachment #2: Type: application/pgp-signature, Size: 197 bytes --]

linux-next: ext4 tree build warning

[Stephen Rothwell]

[-- Attachment #1: Type: text/plain, Size: 497 bytes --]

Hi Ted,

Today's linux-next build (powerpc ppc64_defconfig) produced this warning:

fs/ext4/inode.c: In function 'ext4_ind_get_blocks':
fs/ext4/inode.c:939: warning: unused variable 'disksize'
fs/ext4/inode.c:936: warning: unused variable 'ei'

Introduced by commit fb8c2ba20b8530d74239c2420d335df81aa18e13 ("ext4: Get
rid of EXTEND_DISKSIZE flag of ext4_get_blocks_handle()").

-- 
Cheers,
Stephen Rothwell                    sfr@canb.auug.org.au
http://www.canb.auug.org.au/~sfr/

[-- Attachment #2: Type: application/pgp-signature, Size: 197 bytes --]

Re: linux-next: ext4 tree build warning

[Theodore Tso]

On Tue, Jun 09, 2009 at 10:54:58AM +1000, Stephen Rothwell wrote:
> Hi Ted,
> 
> Today's linux-next build (powerpc ppc64_defconfig) produced this warning:
> 
> fs/ext4/inode.c: In function 'ext4_ind_get_blocks':
> fs/ext4/inode.c:939: warning: unused variable 'disksize'
> fs/ext4/inode.c:936: warning: unused variable 'ei'
> 
> Introduced by commit fb8c2ba20b8530d74239c2420d335df81aa18e13 ("ext4: Get
> rid of EXTEND_DISKSIZE flag of ext4_get_blocks_handle()").

Thanks for pointing that out.  I've fixed that up in my tree.

       	   	    	       	    	       - Ted

Re: linux-next: ext4 tree build warning

[Stephen Rothwell]

[-- Attachment #1: Type: text/plain, Size: 270 bytes --]

Hi Ted,

On Tue, 9 Jun 2009 00:23:10 -0400 Theodore Tso <tytso@mit.edu> wrote:
>
> Thanks for pointing that out.  I've fixed that up in my tree.

Thanks.

-- 
Cheers,
Stephen Rothwell                    sfr@canb.auug.org.au
http://www.canb.auug.org.au/~sfr/

[-- Attachment #2: Type: application/pgp-signature, Size: 197 bytes --]

linux-next: ext4 tree build warning

[Stephen Rothwell]

[-- Attachment #1: Type: text/plain, Size: 489 bytes --]

Hi Ted,

Today's linux-next build (powerpc ppc64_defconfig) produced this warning:

fs/ext4/inode.c: In function '__ext4_check_blockref':
fs/ext4/inode.c:385: warning: format '%u' expects type 'unsigned int', but argument 7 has type 'long int'

Introduced by commit 768df0e2c1d53ee0384e3021e2f1c90882804548 ("ext4: add
checks of block references for non-extent inodes").

-- 
Cheers,
Stephen Rothwell                    sfr@canb.auug.org.au
http://www.canb.auug.org.au/~sfr/

[-- Attachment #2: Type: application/pgp-signature, Size: 197 bytes --]