linux-kernel - Re: [PATCH 04/11] x86: cleanup remaining cpumask

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

Message-Id: <1231223480.3235.22.camel@jaswinder.satnam>
Date:	Tue, 06 Jan 2009 12:01:20 +0530
From:	Jaswinder Singh Rajput <jaswinder@...radead.org>
To:	Mike Travis <travis@....com>, Ingo Molnar <mingo@...hat.com>,
	Rusty Russell <rusty@...tcorp.com.au>,
	"H. Peter Anvin" <hpa@...or.com>,
	Thomas Gleixner <tglx@...utronix.de>,
	Linus Torvalds <torvalds@...ux-foundation.org>,
	Jack Steiner <steiner@....com>,
	LKML <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 04/11] x86: cleanup remaining cpumask_t ops in smpboot
	code

Hello Mike,

On Sun, Jan 4, 2009 at 6:48 PM, Mike Travis <travis@....com> wrote:
> --- linux-2.6-for-ingo.orig/arch/x86/include/asm/smp.h
> +++ linux-2.6-for-ingo/arch/x86/include/asm/smp.h
> @@ -18,9 +18,26 @@
>  #include <asm/pda.h>
>  #include <asm/thread_info.h>
>  
> +#ifdef CONFIG_X86_64
> +
> +extern cpumask_var_t cpu_callin_mask;
> +extern cpumask_var_t cpu_callout_mask;
> +extern cpumask_var_t cpu_initialized_mask;
> +extern cpumask_var_t cpu_sibling_setup_mask;
> +
> +#else /* CONFIG_X86_32 */
> +
> +extern cpumask_t cpu_callin_map;


I am not able to understand, why you choose asm/smp.h

You know that asm/smp.h should only be included for CONFIG_SMP

----- linux/smp.h -----
#ifdef CONFIG_SMP

#include <asm/smp.h>

#endif /* !SMP */

-----------------------

If someday this will happen:

----- asm/smp.h -----
#ifndef _ASM_X86_SMP_H
#define _ASM_X86_SMP_H
#ifndef __ASSEMBLY__
#ifdef CONFIG_SMP
...
#else /* CONFIG_SMP */
#error "Including wrong file, why you need smp.h for uniprocesor"
#endif /* CONFIG_SMP */
#endif /* __ASSEMBLY__ */
#endif /* _ASM_X86_SMP_H */

-----------------------

Then ;-)

Are you planning to make some new file for this or you think some other file will be suitable for this.

If you have some future plans, please let me know.

Thanks
--
JSR

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