lkml.org 
[lkml]   [2002]   [May]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [patch 6/15] larger b_size, and misc fixlets
On Sun, 19 May 2002, Andrew Morton wrote:
> Andreas Dilger wrote:
> > On May 19, 2002 12:39 -0700, Andrew Morton wrote:
> > > - printk(KERN_ERR "Buffer I/O error on device %s, logical block %ld\n",
> > > - bdevname(bh->b_bdev), bh->b_blocknr);
> > > + printk(KERN_ERR "Buffer I/O error on device %s, logical block %Ld\n",
> > > + bdevname(bh->b_bdev), (u64)bh->b_blocknr);
> >
> > Not that I'm a 64-bit system user/developer, but it is my understanding
> > that u64 == long on a 64-bit platform, so your cast to u64 does not
> > actually change the type of b_blocknr as far as printk is concerned.
> > You would need to cast it to unsigned long long instead.
>
> Yes, I suppose so. That more closely matches what "%L" does.

/me can't help it: Didn't I say earlier on that one has to use (unsigned)
long long and not u64? (-; But noone would listen...

But to be fully correct, if you want unsigned long long you really ought
to write %Lu not %Ld... (-8

Cheers,
Anton
--
Anton Altaparmakov <aia21 at cantab.net> (replace at with @)
Linux NTFS maintainer / IRC: #ntfs on irc.openprojects.net
WWW: http://linux-ntfs.sf.net/ & http://www-stu.christs.cam.ac.uk/~aia21/

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

\
 
 \ /
  Last update: 2005-03-22 13:26    [W:0.038 / U:0.012 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site