From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758748AbcEFR15 (ORCPT ); Fri, 6 May 2016 13:27:57 -0400 Received: from mx0a-00082601.pphosted.com ([67.231.145.42]:58039 "EHLO mx0a-00082601.pphosted.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757735AbcEFR1z (ORCPT ); Fri, 6 May 2016 13:27:55 -0400 Date: Fri, 6 May 2016 13:27:20 -0400 From: Chris Mason To: Peter Zijlstra CC: Mike Galbraith , Ingo Molnar , Matt Fleming , Subject: Re: sched: tweak select_idle_sibling to look for idle threads Message-ID: <20160506172720.a27jggrbs3deria5@floor.thefacebook.com> Mail-Followup-To: Chris Mason , Peter Zijlstra , Mike Galbraith , Ingo Molnar , Matt Fleming , linux-kernel@vger.kernel.org References: <1462200604.3736.42.camel@suse.de> <20160502145817.GW3408@twins.programming.kicks-ass.net> <20160502154725.ckiewczbdubudyc7@floor.masoncoding.com> <20160503143225.GG3448@twins.programming.kicks-ass.net> <20160503151153.wp6jcnjadmw2ypmx@floor.masoncoding.com> <20160504154510.GH3448@twins.programming.kicks-ass.net> <20160504174616.k4xud2w232b6m4by@floor.masoncoding.com> <20160505093338.GK3448@twins.programming.kicks-ass.net> <20160505135844.nfb5czbmgxlhyiw6@floor.thefacebook.com> <20160506071251.GM3448@twins.programming.kicks-ass.net> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: <20160506071251.GM3448@twins.programming.kicks-ass.net> User-Agent: Mutt/1.5.23.1 (2014-03-12) X-Originating-IP: [192.168.52.123] X-Proofpoint-Spam-Reason: safe X-FB-Internal: Safe X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10432:,, definitions=2016-05-06_08:,, signatures=0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, May 06, 2016 at 09:12:51AM +0200, Peter Zijlstra wrote: > ontent-Length: 973 > > On Thu, May 05, 2016 at 09:58:44AM -0400, Chris Mason wrote: > > > I'll try and have a prod at the program itself if you have no pending > > > changes on your end. > > > > Sorry, I don't. Look at sleep_for_runtime() and how I test/set the > > global stopping variable in different places. I've almost certainly got > > someone waiting on a wakeup that'll never come. > > The below makes it go.. Thanks Peter, pushed to git. -chris