linux-kernel - Re: [PATCHSET linux-2.6-x86:core/percpu] x86: misc clean up and unify 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] [thread-next>] [day] [month] [year] [list]

Date:	Wed, 21 Jan 2009 10:26:33 +0100
From:	Ingo Molnar <mingo@...e.hu>
To:	Tejun Heo <tj@...nel.org>
Cc:	linux-kernel@...r.kernel.org, brgerst@...il.com,
	"H. Peter Anvin" <hpa@...or.com>,
	Thomas Gleixner <tglx@...utronix.de>
Subject: Re: [PATCHSET linux-2.6-x86:core/percpu] x86: misc clean up and
	unify x86_32/64 code paths


* Tejun Heo <tj@...nel.org> wrote:

> Hello,
> 
> This patchset contains the following 10 patches mostly unifying 32 and
> 64bit code paths.  0001 and 0007-0010 are from me and 0002-0006 are
> from Brian Gerst.
> 
>   0001-x86-update-canary-handling-during-switch.patch
>   0002-x86-clean-up-gdt_page-definition.patch
>   0003-x86-fix-percpu_write-with-64-bit-constants.patch
>   0004-x86-set-fs-to-__KERNEL_PERCPU-unconditionally-for.patch
>   0005-x86-merge-mmu_context.h.patch
>   0006-x86-merge-irq_regs.h.patch
>   0007-x86-uv-cleanup.patch
>   0008-x86-prepare-for-tlb-merge.patch
>   0009-x86-make-x86_32-use-tlb_64.c.patch
>   0010-x86-rename-tlb_64.c-to-tlb.c.patch
> 
> 0001-0002 clean up after previous percpu changes.  0003 improves
> percpu op slightly.  0004 simplifies x86_32 init path.  0005-0010
> unifies mmu_context.h, irq_regs.h and tlb.c, which not only simplifies
> the source code but also improves performance by applying
> optimizations implemented for either bitness to the other one while
> unifying.
> 
> The following git tree contains the above changes on top of
> core/percpu[1].  Please pull from it if there's no objection.
> 
>   git://git.kernel.org/pub/scm/linux/kernel/git/tj/misc.git tj-percpu

Pulled into tip/core/percpu, thanks Tejun.

Very nice stuff! The SMP TLB flush code unification is a gem - i never 
thought we'd get so far this quick ;-)

Note, i also merged the latest cpumask work into tip/core/percpu, to 
resolve tlb_32.c conflicts. I also moved tlb.c from arch/x86/kernel/ to 
arch/x86/mm/, where it really belongs conceptually.

-tip testing also stumbled upon a build error caused by the UV cleanups - 
see the fix below.

	Ingo

-------------->
>From 4ec71fa2d2c3f1040348f2604f4b8ccc833d1c2e Mon Sep 17 00:00:00 2001
From: Ingo Molnar <mingo@...e.hu>
Date: Wed, 21 Jan 2009 10:24:27 +0100
Subject: [PATCH] x86: uv cleanup, build fix
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 8bit

Fix:

 arch/x86/mm/srat_64.c: In function ‘acpi_numa_processor_affinity_init’:
 arch/x86/mm/srat_64.c:141: error: implicit declaration of function ‘get_uv_system_type’
 arch/x86/mm/srat_64.c:141: error: ‘UV_X2APIC’ undeclared (first use in this function)
 arch/x86/mm/srat_64.c:141: error: (Each undeclared identifier is reported only once
 arch/x86/mm/srat_64.c:141: error: for each function it appears in.)

A couple of UV definitions were moved to asm/uv/uv.h, but srat_64.c did
not include that header. Add it.

Signed-off-by: Ingo Molnar <mingo@...e.hu>
---
 arch/x86/mm/srat_64.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/arch/x86/mm/srat_64.c b/arch/x86/mm/srat_64.c
index 09737c8..15df1ba 100644
--- a/arch/x86/mm/srat_64.c
+++ b/arch/x86/mm/srat_64.c
@@ -21,6 +21,7 @@
 #include <asm/numa.h>
 #include <asm/e820.h>
 #include <asm/genapic.h>
+#include <asm/uv/uv.h>
 
 int acpi_numa __initdata;
 
--
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/