linux-kernel - Re: patch for linux-2.6.30/arch/x86

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]

Message-ID: <86802c440906211327o7b6d1532x2fac9d85fec7148@mail.gmail.com>
Date:	Sun, 21 Jun 2009 13:27:13 -0700
From:	Yinghai Lu <yhlu.kernel@...il.com>
To:	jing zhang <zj.barak@...il.com>, Andi Kleen <andi@...stfloor.org>
Cc:	linux-kernel@...r.kernel.org, mingo@...hat.com
Subject: Re: patch for linux-2.6.30/arch/x86_64/mm/init.c

On Sat, Jun 20, 2009 at 11:26 PM, jing zhang<zj.barak@...il.com> wrote:
> hi,
>
> I have no way sending to Andi Kleen, then try Mr. linux-kernel and Mr.
> Ingo Molnar.
>
> btw, AC's mailbox @ suse closed, and why?

he is not working for them.

>
> thank you
>
> ---------- Forwarded message ----------
> From: jing zhang <zj.barak@...il.com>
> Date: Sun, 21 Jun 2009 14:15:09 +0800
> Subject: patch for linux-2.6.30/arch/x86_64/mm/init.c
> To: ak@...e.de
>
> Good Sunday Mr. Andi Kleen,
>
> It seems that in function
>    unsigned long __init_refok
>    init_memory_mapping(unsigned long start, unsigned long end)
>
> CONFIG_DEBUG_PAGEALLOC conflicts with use_pse and use_gbpages when
> CONFIG_X86_32 also defined.
>
> The following is my patch to you,
>
> best
>
> look forwar to your opion.
> zj
>
> //<<<<<<<<<<<<<<<<<<<<<
>
> #ifdef CONFIG_DEBUG_PAGEALLOC
>        /*
>         * For CONFIG_DEBUG_PAGEALLOC, identity mapping will use small pages.
>         * This will simplify cpa(), which otherwise needs to support splitting
>         * large pages into small in interrupt context, etc.
>         */
>        use_pse = use_gbpages = 0;
> #else
>        use_pse = cpu_has_pse;
>        use_gbpages = direct_gbpages;
> #endif
>
> #ifdef CONFIG_X86_32
> #ifdef CONFIG_X86_PAE
>        set_nx();
>        if (nx_enabled)
>                printk(KERN_INFO "NX (Execute Disable) protection: active\n");
> #endif
>        /* Enable PSE if available */
> -       if (cpu_has_pse)
> +       /* and #ifdef CONFIG_DEBUG_PAGEALLOC */
> +       if (cpu_has_pse && use_pse)
>                set_in_cr4(X86_CR4_PSE);
>
>        /* Enable PGE if available */
> -       if (cpu_has_pge) {
> +       /* and #ifdef CONFIG_DEBUG_PAGEALLOC */
> +       if (cpu_has_pge && use_gbpages) {
>                set_in_cr4(X86_CR4_PGE);
>                __supported_pte_mask |= _PAGE_GLOBAL;

pge and gbpages is not same ...

also wonder what is code based for?

please check tip/master
http://people.redhat.com/mingo/tip.git/readme.txt

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