linux-kernel - Re: [patch 00/11] tip/x86/nmi nmi-32/64.c merge v3

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]

Date:	Sun, 25 May 2008 19:07:49 +0200 (CEST)
From:	Thomas Gleixner <tglx@...utronix.de>
To:	Cyrill Gorcunov <gorcunov@...il.com>
cc:	hpa@...or.com, mingo@...hat.com, linux-kernel@...r.kernel.org
Subject: Re: [patch 00/11] tip/x86/nmi nmi-32/64.c merge v3

On Sun, 25 May 2008, Cyrill Gorcunov wrote:

> [Thomas Gleixner - Sun, May 25, 2008 at 06:44:28PM +0200]
> | On Sat, 24 May 2008, Cyrill Gorcunov wrote:
> | 
> | > This series tries to merge down nmi_32/64.c to nmi.c. It is done by small
> | > steps in hope it would help to bisect problems (which will appear anyway ;)
> | > Please review *carefully* since it is NMI what I've touched (and I'm not
> | > nmi specialist). So I really apreciate _ANY_ comments on this. The series
> | > is produced over today 'tip' tree branch 'x86/nmi'.
> | 
> | Cyrill,
> | 
> | very good job. I did a short review and found no obvious problems
> | yet. Will have a closer look when I apply it.
> | 
> | Thanks,
> | 	tglx
> | 
> | 
> | 
> 
> thanks Thomas, unfortunelly I think the problems will arise soon,
> I've just noticed the commit 
> 
> commit 57b2b8947a5f68e59d5f50ab749efdc79d9ba040
> Author: Mike Travis <travis@....com>
> Date:   Mon May 12 21:21:12 2008 +0200
> 
>     x86: fix remove cpu_pda table patch
> 
> in tip/testing tree
> 
> so we will need some workaround for check_nmi_watchdog() 'cause of  changing
> 
> ...
> -	prev_nmi_count = kmalloc(NR_CPUS * sizeof(int), GFP_KERNEL);
> +	prev_nmi_count = kmalloc(nr_cpu_ids * sizeof(int), GFP_KERNEL);

Nothing to worry for you. Your patch applies fine on the x86/nmi
branch and the merge conflict is easy to resolve.

Thanks for noticing.

       tglx

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