Re: PATCH: fix some 32bit isms

[H. Peter Anvin]

Followup to:  <20040728135941.GA17409@xxxxxxxxxxxxxxxxxxxxxxxx>
By author:    Alan Cox <alan@xxxxxxxxxx>
In newsgroup: linux.dev.kernel
>
> Fairly self explanatory. int is not size_t.
>  
> Alan 
> 
> OSDL Developer Certificate of Origin 1.0 included herein by reference
> 
> 
> diff -u --new-file --recursive --exclude-from /usr/src/exclude
> linux.vanilla-2.6.8-rc2/drivers/message/fusion/mptbase.c linux-2.6.8-rc2/drivers/message/fusion/mptbase.c
> --- linux.vanilla-2.6.8-rc2/drivers/message/fusion/mptbase.c	2004-07-27 19:22:42.000000000 +0100
> +++ linux-2.6.8-rc2/drivers/message/fusion/mptbase.c	2004-07-28 14:27:53.603586584 +0100
> @@ -2417,7 +2417,7 @@
>  	} else {
>  		printk(MYIOC_s_ERR_FMT 
>  		     "Invalid IOC facts reply, msgLength=%d offsetof=%d!\n",
> -		     ioc->name, facts->MsgLength, (offsetof(IOCFactsReply_t,
> +		     ioc->name, facts->MsgLength, (int)(offsetof(IOCFactsReply_t,
>  		     RequestFrameSize)/sizeof(u32)));
>  		return -66;
>  	}


This is probably better fixed by changing %d to %zu.

	-hpa
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/