From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755484Ab1LFWr1 (ORCPT ); Tue, 6 Dec 2011 17:47:27 -0500 Received: from mail.linuxfoundation.org ([140.211.169.12]:43118 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753993Ab1LFWrY (ORCPT ); Tue, 6 Dec 2011 17:47:24 -0500 Date: Tue, 6 Dec 2011 14:47:21 -0800 From: Andrew Morton To: David Rientjes Cc: Mike Galbraith , Paul Menage , LKML , Tejun Heo , Li Zefan , Peter Zijlstra Subject: Re: [patch-final] Re: patch] cpusets, cgroups: disallow attaching kthreadd Message-Id: <20111206144721.7b1d473d.akpm@linux-foundation.org> In-Reply-To: References: <1316758874.7393.2.camel@marge.simson.net> <4E7C2E7F.40307@cn.fujitsu.com> <1316762345.8168.4.camel@marge.simson.net> <1316770936.6641.11.camel@marge.simson.net> <1316775204.7562.9.camel@marge.simson.net> <1316788392.6544.33.camel@marge.simson.net> <1318224892.6161.45.camel@marge.simson.net> <1318233815.6527.5.camel@marge.simson.net> <1318925436.9641.23.camel@marge.simson.net> <1319001860.6222.34.camel@marge.simson.net> X-Mailer: Sylpheed 3.0.2 (GTK+ 2.20.1; x86_64-pc-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, 6 Dec 2011 12:13:10 -0800 (PST) David Rientjes wrote: > On Thu, 10 Nov 2011, David Rientjes wrote: > > > On Wed, 26 Oct 2011, David Rientjes wrote: > > > > > On Wed, 19 Oct 2011, Mike Galbraith wrote: > > > > > > > cpusets, cgroups: disallow attaching kthreadd > > > > > > > > Allowing kthreadd to be moved to a non-root group makes no sense, it being > > > > a global resource, and needlessly leads unsuspecting users toward trouble. > > > > > > > > 1. An RT workqueue worker thread spawned in a task group with no rt_runtime > > > > allocated is not schedulable. Simple user error, but harmful to the box. > > > > > > > > 2. A worker thread which acquires PF_THREAD_BOUND can never leave a cpuset, > > > > rendering the cpuset immortal. > > > > > > > > Save the user some unexpected trouble, just say no. > > > > > > > > Signed-off-by: Mike Galbraith > > > > Acked-by: David Rientjes > > > > Acked-by: Paul Menage > > > > > > > > > > Let's add Andrew to the cc so we can get it in -mm, I haven't seen it hit > > > linux-next yet. > > > > > > > Ping? Still missing from -mm and linux-next. > > > > Ping #2? > Why am I being pinged about scheduler patches? My sole contribution to this one is to point out that "its"->possessive and "it's"->"it is". Also, Peter has said : I really think that if we want to restrain userspace from doing : something stupid we might as well do something that makes sense, and : that is mandate kthreadd stays in the root group at all times for : everybody. which appears to be what the patch already did, so I'm confused again. It's time for a fresh resend, IMO.