Message-ID: <1436780510.27924.71.camel@stgolabs.net>
Subject: Re: [PATCH -tip 0/2] futex: Fault/error injection capabilities
From: Davidlohr Bueso <dave@stgolabs.net>
To: Ingo Molnar <mingo@kernel.org>
Cc: Peter Zijlstra <peterz@infradead.org>,
        Thomas Gleixner <tglx@linutronix.de>, Darren Hart <darren@dvhart.com>,
        linux-kernel@vger.kernel.org
Date: Mon, 13 Jul 2015 02:41:50 -0700
In-Reply-To: <1435645562-975-1-git-send-email-dave@stgolabs.net>
References: <1435645562-975-1-git-send-email-dave@stgolabs.net>
Content-Type: text/plain; charset="UTF-8"
Mime-Version: 1.0
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1465
Lines: 40

Ingo, any thoughts about this?

Thanks.

On Mon, 2015-06-29 at 23:26 -0700, Davidlohr Bueso wrote:
> Hello,
> 
> I've been wanting this for a while to improve overall futex
> testing. I must say that I checked rather late when I was
> nearly finished with patch 2 if something similar for futexes
> had already been proposed. Sure enough, in 2009 this was discussed[1].
> 
> Coincidently, I also took the natural approach of making use of
> our fault-injection machinery. I have no idea if perf nowadays does
> such things, and if it does not, I honestly don't have the bandwidth
> to do it Ingo's preferred way -- when there is nothing wrong with
> this approach, imho (0 overhead). Anyway, here's a working patch.
> 
> Patch 1 is merely a trivial add-on.
> 
> [1]: https://lwn.net/Articles/364742/
> 
> Thanks!
> 
> Davidlohr Bueso (2):
>   futex: Enhance comments in futex_lock_pi() for blocking paths
>   futex: Fault/error injection capabilities
> 
>  Documentation/fault-injection/fault-injection.txt |  11 +++
>  kernel/futex.c                                    | 100 +++++++++++++++++++++-
>  lib/Kconfig.debug                                 |   7 ++
>  3 files changed, 114 insertions(+), 4 deletions(-)
> 


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/