* [PATCH] fat-fs printk arg. fix
@ 2003-06-03 23:02 Randy.Dunlap
2003-06-03 23:11 ` Andrew Morton
2003-06-04 1:14 ` OGAWA Hirofumi
0 siblings, 2 replies; 4+ messages in thread
From: Randy.Dunlap @ 2003-06-03 23:02 UTC (permalink / raw)
To: OGAWA Hirofumi, akpm; +Cc: lkml
Hi,
A recent fatfs patch for large partitions upset printk.
Here's a patch for it.
--
~Randy
patch_name: fat-printk.patch
patch_version: 2003-06-03.15:37:48
author: Randy.Dunlap <rddunlap@osdl.org>
description: printk() args are unhappy with recent change for large
partition support;
product: Linux
product_versions: 2.5.70
maintainer: OGAWA Hirofumi
diffstat: =
fs/fat/misc.c | 2 +-
1 files changed, 1 insertion(+), 1 deletion(-)
diff -Naur ./fs/fat/misc.c%PRTK ./fs/fat/misc.c
--- ./fs/fat/misc.c%PRTK 2003-06-02 14:35:15.000000000 -0700
+++ ./fs/fat/misc.c 2003-06-03 14:40:49.000000000 -0700
@@ -311,7 +311,7 @@
*bh = sb_bread(sb, phys);
if (*bh == NULL) {
printk(KERN_ERR "FAT: Directory bread(block %llu) failed\n",
- phys);
+ (u64)phys);
/* skip this block */
*pos = (iblock + 1) << sb->s_blocksize_bits;
goto next;
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH] fat-fs printk arg. fix 2003-06-03 23:02 [PATCH] fat-fs printk arg. fix Randy.Dunlap @ 2003-06-03 23:11 ` Andrew Morton 2003-06-04 1:14 ` OGAWA Hirofumi 1 sibling, 0 replies; 4+ messages in thread From: Andrew Morton @ 2003-06-03 23:11 UTC (permalink / raw) To: Randy.Dunlap; +Cc: hirofumi, linux-kernel "Randy.Dunlap" <rddunlap@osdl.org> wrote: > > printk(KERN_ERR "FAT: Directory bread(block %llu) failed\n", > - phys); > + (u64)phys); The printk control string says, precisely, "unsigned long long". So that is what we should be casting the argument to. I have made that change, thanks. ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] fat-fs printk arg. fix 2003-06-03 23:02 [PATCH] fat-fs printk arg. fix Randy.Dunlap 2003-06-03 23:11 ` Andrew Morton @ 2003-06-04 1:14 ` OGAWA Hirofumi 1 sibling, 0 replies; 4+ messages in thread From: OGAWA Hirofumi @ 2003-06-04 1:14 UTC (permalink / raw) To: Randy.Dunlap; +Cc: akpm, lkml "Randy.Dunlap" <rddunlap@osdl.org> writes: > printk(KERN_ERR "FAT: Directory bread(block %llu) failed\n", > - phys); > + (u64)phys); > /* skip this block */ > *pos = (iblock + 1) << sb->s_blocksize_bits; > goto next; I was forgeting that issue of sector_t with printk. Thanks for correcting. -- OGAWA Hirofumi <hirofumi@mail.parknet.co.jp> ^ permalink raw reply [flat|nested] 4+ messages in thread
[parent not found: <174530962@toto.iv>]
* [PATCH] fat-fs printk arg. fix [not found] <174530962@toto.iv> @ 2003-06-04 3:12 ` Peter Chubb 0 siblings, 0 replies; 4+ messages in thread From: Peter Chubb @ 2003-06-04 3:12 UTC (permalink / raw) To: Randy.Dunlap; +Cc: OGAWA Hirofumi, akpm, lkml >>>>> "Randy" == Randy Dunlap <Randy.Dunlap> writes: Randy> Hi, A recent fatfs patch for large partitions upset printk. Please cast to unsigned long long, not to u64, or the 64-bit architectures (where u64 is unsigned long, not unsigned long long) will see warnings. PeterC ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2003-06-04 2:59 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2003-06-03 23:02 [PATCH] fat-fs printk arg. fix Randy.Dunlap
2003-06-03 23:11 ` Andrew Morton
2003-06-04 1:14 ` OGAWA Hirofumi
[not found] <174530962@toto.iv>
2003-06-04 3:12 ` Peter Chubb
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox