linux-kernel - Re: LogFS: confused with using s64 and u64

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Sat, 9 Aug 2008 13:16:54 +0200
From:	Jörn Engel <joern@...fs.org>
To:	Alexander Beregalov <a.beregalov@...il.com>
Cc:	linux-kernel@...r.kernel.org
Subject: Re: 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@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/