linux-kernel - Re: [2.6.30-rc8] gcc 3.3 : _

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] [thread-next>] [day] [month] [year] [list]

Message-Id: <200906050651.n556p7Vo094549@www262.sakura.ne.jp>
Date:	Fri, 05 Jun 2009 15:51:07 +0900
From:	Tetsuo Handa <penguin-kernel@...ove.sakura.ne.jp>
To:	akpm@...ux-foundation.org
Cc:	linux-kernel@...r.kernel.org, xiyou.wangcong@...il.com,
	andreas.herrmann3@....com, mingo@...e.hu
Subject: Re: [2.6.30-rc8] gcc 3.3 : __udivdi3 undefined.

Andrew Morton wrote:
> We could do the obvious:
> 
> --- a/arch/x86/mm/memtest.c~a
> +++ a/arch/x86/mm/memtest.c
> @@ -8,6 +8,7 @@
>  #include <linux/pfn.h>
>  
>  #include <asm/e820.h>
> +#include <asm/div64.h>
>  
>  static u64 patterns[] __initdata = {
>  	0,
> @@ -48,7 +49,8 @@ static void __init memtest(u64 pattern, 
>  
>  	incr = sizeof(pattern);
>  	start_phys_aligned = ALIGN(start_phys, incr);
> -	count = (size - (start_phys_aligned - start_phys))/incr;
> +	count = size - (start_phys_aligned - start_phys);
> +	do_div(count, incr);
>  	start = __va(start_phys_aligned);
>  	start_bad = 0;
>  	last_bad = 0;

"make" of "make allmodconfig" + CONFIG_KVM=n on gcc 3.3.5 completed
successfully.

>but I wonder why all those things are u64.  They all hold virtual
>addresses, don't they?  The code doesn't test highmem.  So shouldn't
>these all be unsigned longs?
memtest() in linux-2.6.29.4/arch/x86/mm/memtest.c is using "unsigned long".
2.6.30 changed to use "u64" by some reason.

Thank you.
--
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/