blkcg: drop blkiocg_file_write_u64()
authorTejun Heo <tj@kernel.org>
Sun, 1 Apr 2012 21:38:43 +0000 (14:38 -0700)
committerTejun Heo <tj@kernel.org>
Sun, 1 Apr 2012 21:38:43 +0000 (14:38 -0700)
blkiocg_file_write_u64() has single switch case.  Drop
blkiocg_file_write_u64(), rename blkio_weight_write() to
blkcg_set_weight() and use it directly for .write_u64 callback.

Signed-off-by: Tejun Heo <tj@kernel.org>
block/blk-cgroup.c

index 951e7f3a8c899204ab090eb55683519e3dde6d6c..13cae77e8bf6ede42bb339ecf4e6663d6e28bac9 100644 (file)
@@ -1207,8 +1207,9 @@ static int blkiocg_file_read(struct cgroup *cgrp, struct cftype *cft,
        return 0;
 }
 
-static int blkio_weight_write(struct blkio_cgroup *blkcg, int plid, u64 val)
+static int blkcg_set_weight(struct cgroup *cgrp, struct cftype *cft, u64 val)
 {
+       struct blkio_cgroup *blkcg = cgroup_to_blkio_cgroup(cgrp);
        struct blkio_group *blkg;
        struct hlist_node *n;
 
@@ -1220,10 +1221,11 @@ static int blkio_weight_write(struct blkio_cgroup *blkcg, int plid, u64 val)
        blkcg->weight = (unsigned int)val;
 
        hlist_for_each_entry(blkg, n, &blkcg->blkg_list, blkcg_node) {
-               struct blkg_policy_data *pd = blkg->pd[plid];
+               struct blkg_policy_data *pd = blkg->pd[BLKIO_POLICY_PROP];
 
-               if (!pd->conf.weight)
-                       blkio_update_group_weight(blkg, plid, blkcg->weight);
+               if (pd && !pd->conf.weight)
+                       blkio_update_group_weight(blkg, BLKIO_POLICY_PROP,
+                                                 blkcg->weight);
        }
 
        spin_unlock_irq(&blkcg->lock);
@@ -1251,29 +1253,6 @@ static u64 blkiocg_file_read_u64 (struct cgroup *cgrp, struct cftype *cft) {
        return 0;
 }
 
-static int
-blkiocg_file_write_u64(struct cgroup *cgrp, struct cftype *cft, u64 val)
-{
-       struct blkio_cgroup *blkcg;
-       enum blkio_policy_id plid = BLKIOFILE_POLICY(cft->private);
-       int name = BLKIOFILE_ATTR(cft->private);
-
-       blkcg = cgroup_to_blkio_cgroup(cgrp);
-
-       switch(plid) {
-       case BLKIO_POLICY_PROP:
-               switch(name) {
-               case BLKIO_PROP_weight:
-                       return blkio_weight_write(blkcg, plid, val);
-               }
-               break;
-       default:
-               BUG();
-       }
-
-       return 0;
-}
-
 struct cftype blkio_files[] = {
        {
                .name = "weight_device",
@@ -1288,7 +1267,7 @@ struct cftype blkio_files[] = {
                .private = BLKIOFILE_PRIVATE(BLKIO_POLICY_PROP,
                                BLKIO_PROP_weight),
                .read_u64 = blkiocg_file_read_u64,
-               .write_u64 = blkiocg_file_write_u64,
+               .write_u64 = blkcg_set_weight,
        },
        {
                .name = "time",