linux-kernel - Re: x86-tip: cpufreq breakage - bisection fingers 89c9215165ca609096e845926d9a18f1306176a4

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]

Date:	Wed, 04 Mar 2009 15:31:04 +0900
From:	Tejun Heo <tj@...nel.org>
To:	Mike Galbraith <efault@....de>
CC:	Ingo Molnar <mingo@...e.hu>, LKML <linux-kernel@...r.kernel.org>
Subject: Re: x86-tip: cpufreq breakage - bisection	fingers	89c9215165ca609096e845926d9a18f1306176a4

Hello, Mike.

Mike Galbraith wrote:
> On Wed, 2009-03-04 at 04:43 +0100, Mike Galbraith wrote:
>> On Wed, 2009-03-04 at 10:59 +0900, Tejun Heo wrote:
>>
>>> I can't reproduce the problem here.
>> Guess I'll have to sharpen a stick or two then.
> 
> With monkey-see-monkey-do-stick, box becomes happy camper again.
> 
> diff --git a/arch/x86/kernel/setup_percpu.c b/arch/x86/kernel/setup_percpu.c
> index c29f301..bc9972a 100644
> --- a/arch/x86/kernel/setup_percpu.c
> +++ b/arch/x86/kernel/setup_percpu.c
> @@ -256,6 +256,11 @@ static struct page * __init pcpue_get_page(unsigned int cpu, int pageno)
>  			    + ((size_t)pageno << PAGE_SHIFT));
>  }
>  
> +static void __init embed_populate_pte(unsigned long addr)
> +{
> +	populate_extra_pte(addr);
> +}
> +
>  static ssize_t __init setup_pcpu_embed(size_t static_size)
>  {
>  	unsigned int cpu;
> @@ -284,10 +289,15 @@ static ssize_t __init setup_pcpu_embed(size_t static_size)
>  	pr_info("PERCPU: Embedded %zu pages at %p, static data %zu bytes\n",
>  		pcpue_unit_size >> PAGE_SHIFT, pcpue_ptr, static_size);
>  
> +#if 0
>  	return pcpu_setup_first_chunk(pcpue_get_page, static_size,
>  				      pcpue_unit_size,
>  				      pcpue_unit_size - static_size, pcpue_ptr,
>  				      NULL);
> +#else
> +	return pcpu_setup_first_chunk(pcpue_get_page, static_size,
> +				      0, 0, NULL, embed_populate_pte);
> +#endif
>  }

That isn't quite correct but is still very interesting that it fixes
the build issue.  Super weird.  I'm almost done setting up 11.0 system.
I'll dig into it as soon as I'm done with sata_nv -stable issue.

Thanks for drilling it down.

-- 
tejun
--
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/