Re: [PATCH] nilfs2: fix format string compile warning (ino_t)

[Ryusuke Konishi <konishi.ryusuke@lab.ntt.co.jp>]

On Thu, 3 Sep 2009 17:42:48 +0200, Heiko Carstens <heiko.carstens@de.ibm.com> wrote:
> From: Heiko Carstens <heiko.carstens@de.ibm.com>
> 
> Unlike on most other architectures ino_t is an unsigned int on s390.
> So add an explicit cast to avoid this compile warning:
> 
> fs/nilfs2/recovery.c: In function 'recover_dsync_blocks':
> fs/nilfs2/recovery.c:555: warning: format '%lu' expects type 'long unsigned int', but argument 3 has type 'ino_t'
> 
> Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com>
> ---
> 
> Patch applies on top of linux-next.

Applied, thanks!

Ryusuke Konishi

>  fs/nilfs2/recovery.c |    3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> Index: linux-next/fs/nilfs2/recovery.c
> ===================================================================
> --- linux-next.orig/fs/nilfs2/recovery.c
> +++ linux-next/fs/nilfs2/recovery.c
> @@ -552,7 +552,8 @@ static int recover_dsync_blocks(struct n
>  		printk(KERN_WARNING
>  		       "NILFS warning: error recovering data block "
>  		       "(err=%d, ino=%lu, block-offset=%llu)\n",
> -		       err, rb->ino, (unsigned long long)rb->blkoff);
> +		       err, (unsigned long)rb->ino,
> +		       (unsigned long long)rb->blkoff);
>  		if (!err2)
>  			err2 = err;
>   next:
> --