From mboxrd@z Thu Jan 1 00:00:00 1970 From: Li Zefan Subject: Re: [PATCH v4 0/2] cgroups: implement moving a threadgroup's threads atomically with cgroup.procs Date: Wed, 04 Aug 2010 10:00:41 +0800 Message-ID: <4C58C9C9.3040008@cn.fujitsu.com> References: <20100730235649.GA22644@ghc17.ghc.andrew.cmu.edu> <20100803125827.0822e6ab.akpm@linux-foundation.org> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20100803125827.0822e6ab.akpm-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: containers-bounces-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org Errors-To: containers-bounces-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org To: Andrew Morton Cc: Ben Blum , containers-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, oleg-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org, menage-hpIqsD4AKlfQT0dZR+AlfA@public.gmane.org, ebiederm-aS9lmoZGLiVWk0Htik3J/w@public.gmane.org List-Id: containers.vger.kernel.org Andrew Morton wrote: > On Fri, 30 Jul 2010 19:56:49 -0400 > Ben Blum wrote: > >> This patch series implements a write function for the 'cgroup.procs' >> per-cgroup file, which enables atomic movement of multithreaded >> applications between cgroups. Writing the thread-ID of any thread in a >> threadgroup to a cgroup's procs file causes all threads in the group to >> be moved to that cgroup safely with respect to threads forking/exiting. >> (Possible usage scenario: If running a multithreaded build system that >> sucks up system resources, this lets you restrict it all at once into a >> new cgroup to keep it under control.) > > I can see how that would be useful. No comments from anyone else? > Oleg had been commenting on this patchset, so it would be nice to know if he's comfortable with the changes in this version. From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758134Ab0HDBzz (ORCPT ); Tue, 3 Aug 2010 21:55:55 -0400 Received: from cn.fujitsu.com ([222.73.24.84]:53524 "EHLO song.cn.fujitsu.com" rhost-flags-OK-FAIL-OK-OK) by vger.kernel.org with ESMTP id S1758008Ab0HDBzx (ORCPT ); Tue, 3 Aug 2010 21:55:53 -0400 Message-ID: <4C58C9C9.3040008@cn.fujitsu.com> Date: Wed, 04 Aug 2010 10:00:41 +0800 From: Li Zefan User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.1b3pre) Gecko/20090513 Fedora/3.0-2.3.beta2.fc11 Thunderbird/3.0b2 MIME-Version: 1.0 To: Andrew Morton CC: Ben Blum , linux-kernel@vger.kernel.org, containers@lists.linux-foundation.org, ebiederm@xmission.com, matthltc@us.ibm.com, menage@google.com, oleg@redhat.com Subject: Re: [PATCH v4 0/2] cgroups: implement moving a threadgroup's threads atomically with cgroup.procs References: <20100730235649.GA22644@ghc17.ghc.andrew.cmu.edu> <20100803125827.0822e6ab.akpm@linux-foundation.org> In-Reply-To: <20100803125827.0822e6ab.akpm@linux-foundation.org> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Andrew Morton wrote: > On Fri, 30 Jul 2010 19:56:49 -0400 > Ben Blum wrote: > >> This patch series implements a write function for the 'cgroup.procs' >> per-cgroup file, which enables atomic movement of multithreaded >> applications between cgroups. Writing the thread-ID of any thread in a >> threadgroup to a cgroup's procs file causes all threads in the group to >> be moved to that cgroup safely with respect to threads forking/exiting. >> (Possible usage scenario: If running a multithreaded build system that >> sucks up system resources, this lets you restrict it all at once into a >> new cgroup to keep it under control.) > > I can see how that would be useful. No comments from anyone else? > Oleg had been commenting on this patchset, so it would be nice to know if he's comfortable with the changes in this version.