Re: [PATCH -v6 13/13] futex: futex_lock_pi() vs PREEMPT_RT_FULL

From: Darren Hart
Date: Mon Apr 10 2017 - 12:05:47 EST


On Mon, Apr 10, 2017 at 11:08:18AM +0200, Peter Zijlstra wrote:
> On Fri, Apr 07, 2017 at 07:26:10PM -0700, Darren Hart wrote:
> > Peter, I presume there will be a v7 with the u32 change and hopefully a couple
> > text updates?
>
> Well, tglx already committed these here patches, so no -v7. What I can
> do however is do a follow up patch that fixes some of the in-code things
> you mentioned.
>
> Something like the below is what I had lying about from your earlier
> emails; I've not looked to see if there's anything else from your later
> emails.
>
> ---
> Subject: futex: Small misc fixes..
> From: Peter Zijlstra <peterz@xxxxxxxxxxxxx>
> Date: Fri Apr 7 09:04:07 CEST 2017
>
> Feedback from Darren's review.
>
> Reported-by: Darren Hart (VMWare) <dvhart@xxxxxxxxxxxxx>
> Signed-off-by: Peter Zijlstra (Intel) <peterz@xxxxxxxxxxxxx>


OK, this addresses 2 of the big ones as I recall. I'll go through my feedback
and if I think anything else needs updating, I'll prepare a few patches.

Reviewed-by: Darren Hart (VMware) <dvhart@xxxxxxxxxxxxx>

Thanks Peter!

--
Darren Hart
VMware Open Source Technology Center