[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20090112092735.GB1822@elte.hu>
Date: Mon, 12 Jan 2009 10:27:35 +0100
From: Ingo Molnar <mingo@...e.hu>
To: Yinghai Lu <yinghai@...nel.org>
Cc: Mike Travis <travis@....com>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] x86: include apicnum.h in acpidef.h
* Ingo Molnar <mingo@...e.hu> wrote:
> like:
>
> In file included from /home/mingo/tip/arch/x86/include/asm/i387.h:16,
> from
> /home/mingo/tip/arch/x86/include/asm/suspend_32.h:10,
> from /home/mingo/tip/arch/x86/include/asm/suspend.h:2,
> from include/linux/suspend.h:5,
> from arch/x86/kernel/asm-offsets_32.c:11,
> from arch/x86/kernel/asm-offsets.c:2:
> include/linux/hardirq.h:41:27: error: ':' without preceding '?'
> In file included from arch/x86/kernel/asm-offsets_32.c:24,
> from arch/x86/kernel/asm-offsets.c:2:
> include/linux/lguest.h:60:1: error: unterminated argument list invoking
> macro "BITS_TO_LONGS"
> In file included from arch/x86/kernel/asm-offsets_32.c:25,
> from arch/x86/kernel/asm-offsets.c:2:
> arch/x86/kernel/../../../drivers/lguest/lg.h:236:1: error: unterminated
> argument list invoking macro "BITS_TO_LONGS"
>
> so i've reverted it again. config attached.
(Below are the two commits that i reverted - please reuse the improved
changelogs when resubmitting it.)
Ingo
----------->
>From af95e2305842d3192ab782abb8ceedc1a2b82d39 Mon Sep 17 00:00:00 2001
From: Yinghai Lu <yinghai@...nel.org>
Date: Sun, 11 Jan 2009 21:42:13 -0800
Subject: [PATCH] x86: include apicnum.h in acpidef.h
Impact: clean up
MAX_IO_APICS and MAX_LOCAL_APIC is already defined in apicnum.h, no
need to duplicate it in apicdef.h.
Signed-off-by: Yinghai Lu <yinghai@...nel.org>
Signed-off-by: Ingo Molnar <mingo@...e.hu>
---
arch/x86/include/asm/apicdef.h | 7 +------
1 files changed, 1 insertions(+), 6 deletions(-)
diff --git a/arch/x86/include/asm/apicdef.h b/arch/x86/include/asm/apicdef.h
index 63134e3..85b50a0 100644
--- a/arch/x86/include/asm/apicdef.h
+++ b/arch/x86/include/asm/apicdef.h
@@ -132,12 +132,7 @@
#define APIC_BASE_MSR 0x800
#define X2APIC_ENABLE (1UL << 10)
-#ifdef CONFIG_X86_32
-# define MAX_IO_APICS 64
-#else
-# define MAX_IO_APICS 128
-# define MAX_LOCAL_APIC 32768
-#endif
+#include <asm/apicnum.h>
/*
* All x86-64 systems are xAPIC compatible.
>From adf320a657d774d8bdf58b8ffc81670238824c9c Mon Sep 17 00:00:00 2001
From: Yinghai Lu <yinghai@...nel.org>
Date: Sun, 11 Jan 2009 21:43:49 -0800
Subject: [PATCH] x86: use nr_cpus in max_nr_irqs()
Impact: save RAM with large NR_CPUS, get smaller nr_irqs
Signed-off-by: Yinghai Lu <yinghai@...nel.org>
Cc: Mike Travis <travis@....com>
Signed-off-by: Ingo Molnar <mingo@...e.hu>
---
arch/x86/include/asm/irq_vectors.h | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/x86/include/asm/irq_vectors.h b/arch/x86/include/asm/irq_vectors.h
index 602361a..5b285ca 100644
--- a/arch/x86/include/asm/irq_vectors.h
+++ b/arch/x86/include/asm/irq_vectors.h
@@ -117,8 +117,8 @@
/* defined as a macro so nr_irqs = max_nr_irqs(nr_cpu_ids) can be used */
# define max_nr_irqs(nr_cpus) \
- ((8 * nr_cpus) > (32 * MAX_IO_APICS) ? \
- (NR_VECTORS + (8 * NR_CPUS)) : \
+ ((8 * (nr_cpus)) > (32 * MAX_IO_APICS) ? \
+ (NR_VECTORS + (8 * (nr_cpus)) : \
(NR_VECTORS + (32 * MAX_IO_APICS))) \
# define NR_IRQS max_nr_irqs(NR_CPUS)
--
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/
Powered by blists - more mailing lists