Re: New filesystem for Linux
- From: Adrian Bunk <bunk@xxxxxxxxx>
- Date: Fri, 3 Nov 2006 20:08:24 +0100
On Fri, Nov 03, 2006 at 06:36:09PM +0100, Oleg Verych wrote:
On Fri, Nov 03, 2006 at 06:09:39PM +0100, Mikulas Patocka wrote:
,--In gmane.linux.kernel, you wrote:
[]
From: Andrew Morton <akpm@xxxxxxxx>
As Mikulas points out, (1 << anything) won't be evaluating to zero.
How about integer overflow ?
C standard defines that shifts by more bits than size of a type are
undefined (in fact 1<<32 produces 1 on i386, because processor uses only 5
bits of a count).
|#include <stdio.h>
|int main(void) {
| unsigned int b = 1;
|
| printf("%u\n", (1 << 33));
| printf("%u\n", (b << 33));
| return 0;
|}
|$ gcc bit.c && ./a.out
`--
There *is* difference, isn't it?
It's undefined, and the results with your example depend on the gcc
version and optimization level.
E.g. with gcc 4.1, there is *no* difference any more if you turn on
optimization.
cu
Adrian
--
"Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
"Only a promise," Lao Er said.
Pearl S. Buck - Dragon Seed
-
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/
- Follow-Ups:
- Re: New filesystem for Linux
- From: Oleg Verych
- Re: New filesystem for Linux
- References:
- New filesystem for Linux
- From: Mikulas Patocka
- Re: New filesystem for Linux
- From: Gabriel C
- Re: New filesystem for Linux
- From: Mikulas Patocka
- Re: New filesystem for Linux
- From: Andrew Morton
- Re: New filesystem for Linux
- From: Oleg Verych
- Re: New filesystem for Linux
- From: Mikulas Patocka
- Re: New filesystem for Linux
- From: Oleg Verych
- New filesystem for Linux
- Prev by Date: Re: [ANNOUNCE] pahole and other DWARF2 utilities
- Next by Date: Re: New filesystem for Linux
- Previous by thread: Re: New filesystem for Linux
- Next by thread: Re: New filesystem for Linux
- Index(es):
Relevant Pages
|
|