Re: div64.h:do_div() bug - SPARC, SH anbody?

From: Artur Klauser
Date: Tue Sep 30 2003 - 08:05:10 EST


Got confirmation that ARM and CRIS experience the problem, pending info
from folks with access to SPARC or SH architectures... anybody?

-r2r-

On Mon, 29 Sep 2003, Artur Klauser wrote:
> I've found that a bug in asm-arm/div64.h:do_div() is preventing correct
> conversion of timestamps in smbfs (and probably ntfs as well) from NT to
> Unix format. I'll post a patch that fixes the bug, but I think it is also
> present in other architectures - at least SPARC, SH, and CRIS look
> suspicious.
>
> If people with access to these architectures could run the following small
> test and let me know the outcome, I can fix it there too - thanks.
>
> //-----------------------------------------------------------------------------
> #define __KERNEL__
> #include <asm/types.h> // get kernel definition of u64, u32
> #undef __KERNEL__
> #include <asm/div64.h> // get definition of do_div()
> #include <stdio.h>
>
> main () {
> union {
> u64 n64;
> u32 n32[2];
> } in, out;
>
> in.n32[0] = 1;
> in.n32[1] = 1;
> out = in;
>
> do_div(out.n64, 1);
>
> if (in.n64 != out.n64) {
> printf("FAILURE: asm/div64.h:do_div() is broken for 64-bit dividends\n");
> exit(1);
> } else {
> printf("Congratulations: asm/div64.h:do_div() handles 64-bit dividends\n");
> }
> return 0;
> }
> //-----------------------------------------------------------------------------
-
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/