lkml.org 
[lkml]   [2008]   [Aug]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: LogFS: confused with using s64 and u64
On Sat, 9 August 2008 01:12:36 +0400, Alexander Beregalov wrote:
>
> I am a bit confused with mixing signed and unsigned types here:
>
>
> static inline u64 dev_ofs(struct super_block *sb, u32 segno, u32 ofs)
> {
> return ((u64)segno << logfs_super(sb)->s_segshift) + ofs;
> }
>
> static s64 logfs_get_free_bytes(struct logfs_area *area, size_t bytes)
> {
> s32 ofs;
> ...
> ofs = area->a_used_bytes;
> ...
> return dev_ofs(area->a_sb, area->a_segno, ofs);
> }
>
> Can you please explain why do you use s64 for function which returns u64?

Sloppiness. Until the 64bit limit actually gets close, I prefer to use
negative return values to indicate errors. It would be possible to have
a seperate return value, as jffs2 does. But functions with two return
values are just icky. So many functions return s64

Functions that never return errors tend to use u64 instead. And I mix
them fairly liberally, "knowing" (as a programmer) that u64 will never
overflow into a negative s64.

Some sort of cleanup in that area surely wouldn't hurt.

Jörn

--
Don't patch bad code, rewrite it.
-- Kernigham and Pike, according to Rusty
--
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: 2008-08-09 13:53    [W:0.063 / U:0.012 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site