From: Tejun Heo <tj-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
To: axboe-tSWWG44O7X1aa/9Udqfwiw@public.gmane.org
Cc: cgroups-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
ctalbott-hpIqsD4AKlfQT0dZR+AlfA@public.gmane.org,
containers-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org,
rni-hpIqsD4AKlfQT0dZR+AlfA@public.gmane.org,
vgoyal-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org
Subject: [PATCHSET] block: cosmetic updates to blkcg API
Date: Mon, 16 Apr 2012 13:57:21 -0700 [thread overview]
Message-ID: <1334609847-12747-1-git-send-email-tj@kernel.org> (raw)
Hello,
With recent blkcg updates, any out-of-tree change is already way
broken and it's a good time to clean up blkcg API for more
consistency. This patchset makes a series of mostly cosmetic changes
in an attempt to make the blkcg API easier on the eyes.
This is finally the last patchset of the long blkcg API update effort.
I think that we now have something which at least is sane & logical
and can be improved incrementally without having to be turned upside
down. My thanks to everyone involved for enduring the boring and
rather painful process.
This patchset contains the following six patches.
0001-blkcg-blkg_rwstat_read-was-missing-inline.patch
0002-blkcg-remove-blkio_group-path.patch
0003-blkcg-style-cleanups-for-blk-cgroup.h.patch
0004-blkcg-mass-rename-of-blkcg-API.patch
0005-blkcg-embed-struct-blkg_policy_data-in-policy-specif.patch
0006-blkcg-collapse-blkcg_policy_ops-into-blkcg_policy.patch
Other than 0002 which removes blkg->path[], none of the patches makes
any functional difference.
This patchset is on top of
block/for-3.5/core 5bc4afb1ec "blkcg: drop BLKCG_STAT_{PRIV|POL|OFF} macros"
+ [1] [PATCHSET] block: per-queue policy activation, take#2
and available in the following git branch.
git://git.kernel.org/pub/scm/linux/kernel/git/tj/misc.git blkcg-rename
diffstat follows.
block/blk-cgroup.c | 253 ++++++++++++++++++++++++-------------------------
block/blk-cgroup.h | 240 ++++++++++++++++++++++++----------------------
block/blk-throttle.c | 121 +++++++++++++----------
block/cfq-iosched.c | 138 ++++++++++++++------------
include/linux/blkdev.h | 4
5 files changed, 397 insertions(+), 359 deletions(-)
Thanks.
--
tejun
[1] https://lkml.org/lkml/2012/4/13/380
next reply other threads:[~2012-04-16 20:57 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-04-16 20:57 Tejun Heo [this message]
[not found] ` <1334609847-12747-1-git-send-email-tj-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
2012-04-16 20:57 ` [PATCH 1/6] blkcg: blkg_rwstat_read() was missing inline Tejun Heo
2012-04-16 20:57 ` [PATCH 2/6] blkcg: remove blkio_group->path[] Tejun Heo
2012-04-16 20:57 ` [PATCH 3/6] blkcg: style cleanups for blk-cgroup.h Tejun Heo
2012-04-16 20:57 ` [PATCH 4/6] blkcg: mass rename of blkcg API Tejun Heo
2012-04-16 20:57 ` [PATCH 5/6] blkcg: embed struct blkg_policy_data in policy specific data Tejun Heo
2012-04-16 20:57 ` [PATCH 6/6] blkcg: collapse blkcg_policy_ops into blkcg_policy Tejun Heo
2012-04-20 8:10 ` [PATCHSET] block: cosmetic updates to blkcg API Jens Axboe
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1334609847-12747-1-git-send-email-tj@kernel.org \
--to=tj-dgejt+ai2ygdnm+yrofe0a@public.gmane.org \
--cc=axboe-tSWWG44O7X1aa/9Udqfwiw@public.gmane.org \
--cc=cgroups-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=containers-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org \
--cc=ctalbott-hpIqsD4AKlfQT0dZR+AlfA@public.gmane.org \
--cc=rni-hpIqsD4AKlfQT0dZR+AlfA@public.gmane.org \
--cc=vgoyal-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).