From: Ingo Molnar <mingo@kernel.org>
To: Peter Zijlstra <peterz@infradead.org>
Cc: Davidlohr Bueso <davidlohr@hp.com>,
Linus Torvalds <torvalds@linux-foundation.org>,
linux-kernel@vger.kernel.org,
"Paul E. McKenney" <paulmck@us.ibm.com>,
Thomas Gleixner <tglx@linutronix.de>,
Andrew Morton <akpm@linux-foundation.org>
Subject: Re: [GIT PULL] locking tree changes for v3.16
Date: Thu, 5 Jun 2014 14:35:12 +0200 [thread overview]
Message-ID: <20140605123512.GB1392@gmail.com> (raw)
In-Reply-To: <20140605122715.GD6758@twins.programming.kicks-ass.net>
* Peter Zijlstra <peterz@infradead.org> wrote:
> On Thu, Jun 05, 2014 at 01:11:10PM +0200, Ingo Molnar wrote:
> > Unfortunately even with the latest bits from you the breakage I
> > reported weeks ago is not resolved:
>
> crap, its the qrwlock bits, I fixed it for the config you gave me.
> Some paravirt crap header hell thing. Happen to have another
> .config?
A simple 'make defconfig' triggers the build failure for me:
kernel/locking/qrwlock.c:24:25: fatal error: asm/qrwlock.h: No such file or directory
If I apply the minimal feature-enabling patch below on top of the base
qrwlock patch.
Thanks,
Ingo
==========================>
>From a79dedeee44dadf8e033a047b7cd4409bb7ad0d8 Mon Sep 17 00:00:00 2001
From: Waiman Long <Waiman.Long@hp.com>
Date: Mon, 3 Feb 2014 13:18:57 +0100
Subject: [PATCH] x86, locking: Switch x86 to qrwlock-style rwlocks
Make x86 use the fair rwlock_t.
Don't implement a custom queue_write_unlock() yet, use the generic path.
Signed-off-by: Waiman Long <Waiman.Long@hp.com>
Signed-off-by: Peter Zijlstra <peterz@infradead.org>
Link: http://lkml.kernel.org/n/tip-r1xuzmdysvuhl3h86n5fbxi7@git.kernel.org
Cc: "Paul E.McKenney" <paulmck@linux.vnet.ibm.com>
Cc: Andi Kleen <ak@linux.intel.com>
Cc: Dave Jones <davej@redhat.com>
Cc: Jeremy Fitzhardinge <jeremy@goop.org>
Cc: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
Cc: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Oleg Nesterov <oleg@redhat.com>
Cc: Raghavendra K T <raghavendra.kt@linux.vnet.ibm.com>
[ Simplified it. ]
Signed-off-by: Ingo Molnar <mingo@kernel.org>
---
arch/x86/Kconfig | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig
index 25d2c6f..bf7626f 100644
--- a/arch/x86/Kconfig
+++ b/arch/x86/Kconfig
@@ -121,6 +121,7 @@ config X86
select MODULES_USE_ELF_RELA if X86_64
select CLONE_BACKWARDS if X86_32
select ARCH_USE_BUILTIN_BSWAP
+ select ARCH_USE_QUEUE_RWLOCK
select OLD_SIGSUSPEND3 if X86_32 || IA32_EMULATION
select OLD_SIGACTION if X86_32
select COMPAT_OLD_SIGACTION if IA32_EMULATION
next prev parent reply other threads:[~2014-06-05 12:35 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-06-03 11:48 [GIT PULL] locking tree changes for v3.16 Ingo Molnar
2014-06-03 21:50 ` Davidlohr Bueso
2014-06-03 21:55 ` Andrew Morton
2014-06-03 22:02 ` Davidlohr Bueso
2014-06-03 22:53 ` Andrew Morton
2014-06-04 10:30 ` Peter Zijlstra
2014-06-04 16:17 ` Linus Torvalds
2014-06-04 17:22 ` Peter Zijlstra
2014-06-04 21:22 ` Linus Torvalds
2014-06-04 21:35 ` Thomas Gleixner
2014-06-05 1:22 ` Davidlohr Bueso
2014-06-05 8:19 ` Ingo Molnar
2014-06-05 8:22 ` Thomas Gleixner
2014-06-05 8:29 ` Ingo Molnar
2014-06-05 8:17 ` Ingo Molnar
2014-06-05 9:21 ` Peter Zijlstra
2014-06-05 9:24 ` Ingo Molnar
2014-06-05 11:11 ` Ingo Molnar
2014-06-05 11:20 ` Ingo Molnar
2014-06-05 12:27 ` Peter Zijlstra
2014-06-05 12:35 ` Ingo Molnar [this message]
2014-06-05 12:41 ` Peter Zijlstra
2014-06-05 12:47 ` Ingo Molnar
2014-06-05 12:53 ` Peter Zijlstra
2014-06-05 13:13 ` Peter Zijlstra
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20140605123512.GB1392@gmail.com \
--to=mingo@kernel.org \
--cc=akpm@linux-foundation.org \
--cc=davidlohr@hp.com \
--cc=linux-kernel@vger.kernel.org \
--cc=paulmck@us.ibm.com \
--cc=peterz@infradead.org \
--cc=tglx@linutronix.de \
--cc=torvalds@linux-foundation.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox