From mboxrd@z Thu Jan 1 00:00:00 1970 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751968AbeAJUHp (ORCPT + 1 other); Wed, 10 Jan 2018 15:07:45 -0500 Received: from mx0a-001b2d01.pphosted.com ([148.163.156.1]:40942 "EHLO mx0a-001b2d01.pphosted.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752139AbeAJUHn (ORCPT ); Wed, 10 Jan 2018 15:07:43 -0500 Date: Wed, 10 Jan 2018 12:08:21 -0800 From: "Paul E. McKenney" To: Tejun Heo Cc: Prateek Sood , Peter Zijlstra , avagin@gmail.com, mingo@kernel.org, linux-kernel@vger.kernel.org, cgroups@vger.kernel.org, sramana@codeaurora.org Subject: Re: [PATCH] cgroup/cpuset: fix circular locking dependency Reply-To: paulmck@linux.vnet.ibm.com References: <20180102180119.GA1355@linux.vnet.ibm.com> <20180108122823.GL3668920@devbig577.frc2.facebook.com> <20180108225238.GN9671@linux.vnet.ibm.com> <20180109003127.GA30224@linux.vnet.ibm.com> <20180109034211.GC3668920@devbig577.frc2.facebook.com> <20180109042016.GR9671@linux.vnet.ibm.com> <20180109134448.GE3668920@devbig577.frc2.facebook.com> <20180109152112.GT9671@linux.vnet.ibm.com> <20180109153752.GI3668920@devbig577.frc2.facebook.com> <20180109160022.GW9671@linux.vnet.ibm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20180109160022.GW9671@linux.vnet.ibm.com> User-Agent: Mutt/1.5.21 (2010-09-15) X-TM-AS-GCONF: 00 x-cbid: 18011020-0044-0000-0000-000003CC6AB6 X-IBM-SpamModules-Scores: X-IBM-SpamModules-Versions: BY=3.00008355; HX=3.00000241; KW=3.00000007; PH=3.00000004; SC=3.00000245; SDB=6.00973113; UDB=6.00493040; IPR=6.00753063; BA=6.00005772; NDR=6.00000001; ZLA=6.00000005; ZF=6.00000009; ZB=6.00000000; ZP=6.00000000; ZH=6.00000000; ZU=6.00000002; MB=3.00018967; XFM=3.00000015; UTC=2018-01-10 20:07:39 X-IBM-AV-DETECTION: SAVI=unused REMOTE=unused XFE=unused x-cbparentid: 18011020-0045-0000-0000-000007FBC314 Message-Id: <20180110200821.GA22541@linux.vnet.ibm.com> X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10432:,, definitions=2018-01-10_10:,, signatures=0 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 priorityscore=1501 malwarescore=0 suspectscore=0 phishscore=0 bulkscore=0 spamscore=0 clxscore=1015 lowpriorityscore=0 impostorscore=0 adultscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.0.1-1709140000 definitions=main-1801100277 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Return-Path: On Tue, Jan 09, 2018 at 08:00:22AM -0800, Paul E. McKenney wrote: > On Tue, Jan 09, 2018 at 07:37:52AM -0800, Tejun Heo wrote: > > Hello, Paul. > > > > On Tue, Jan 09, 2018 at 07:21:12AM -0800, Paul E. McKenney wrote: > > > > The code was previously using both system_power_efficient_wq and > > > > system_workqueue (for the expedited path). I guess the options were > > > > either using two workqueues or dropping POWER_EFFICIENT. I have no > > > > idea how big an impact this will make or whether it'd even be > > > > noticeable but maybe it'd be worthwhile to mention that in the > > > > description? > > > > > > Good point! How about if I change the last paragraph of the commit > > > log to read as follows? > > > > > > Thanx, Paul > > > > > > ------------------------------------------------------------------------ > > > > > > This commit also causes SRCU to use this new RCU-specific > > > workqueue_struct. Note that SRCU's use of workqueues never blocks them > > > waiting for readers, so this should be safe from a forward-progress > > > viewpoint. Note that this moves SRCU from system_power_efficient_wq > > > to a normal workqueue. In the unlikely event that this results in > > > measurable degradation, a separate power-efficient workqueue will be > > > creates for SRCU. > > > > Sounds good. Please feel free to add > > > > Acked-by: Tejun Heo > > Done, thank you! And one additional question... How are we pushing this upstream? By default, I would push things starting this late into the merge window following the next one (v4.17), but would be more than willing to make an exception given that this fixes a valid real-world complaint. For that matter, if you would rather push my commit along with your pair of commits, that works for me! Either way, please just let me know. Thanx, Paul