Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754904AbbERV7Q (ORCPT ); Mon, 18 May 2015 17:59:16 -0400 Received: from mail-qg0-f50.google.com ([209.85.192.50]:34803 "EHLO mail-qg0-f50.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752437AbbERV7M (ORCPT ); Mon, 18 May 2015 17:59:12 -0400 Date: Mon, 18 May 2015 17:59:07 -0400 From: Tejun Heo To: Aleksa Sarai Cc: lizefan@huawei.com, mingo@redhat.com, peterz@infradead.org, richard@nod.at, fweisbec@gmail.com, linux-kernel@vger.kernel.org, cgroups@vger.kernel.org, Jens Axboe , Vivek Goyal Subject: Re: [PATCH v12 4/8] cgroup, block: implement task_get_css() and use it in bio_associate_current() Message-ID: <20150518215907.GJ24861@htj.duckdns.org> References: <1431960667-26593-1-git-send-email-cyphar@cyphar.com> <1431960667-26593-5-git-send-email-cyphar@cyphar.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1431960667-26593-5-git-send-email-cyphar@cyphar.com> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 943 Lines: 24 On Tue, May 19, 2015 at 12:51:03AM +1000, Aleksa Sarai wrote: > From: Tejun Heo > > bio_associate_current() currently open codes task_css() and > css_tryget_online() to find and pin $current's blkcg css. Abstract it > into task_get_css() which is implemented from cgroup side. As a task > is always associated with an online css for every subsystem except > while the css_set update is propagating, task_get_css() retries till > css_tryget_online() succeeds. > > This is a cleanup and shouldn't lead to noticeable behavior changes. Applied 1-4 but skipped bio_associate_current() changes from this patch. I'll route that part through a different patchset. Thanks. -- tejun -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/