linux-kernel - Re: [PATCH 3/3] cfq-iosched: Print per slice sectors dispatched in blktrace

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <AANLkTik_GAOXAuIXBS36RjI2tX0Kes5G7ghMgMlsx6RH@mail.gmail.com>
Date:	Mon, 19 Jul 2010 15:16:29 -0700
From:	Divyesh Shah <dpshah@...gle.com>
To:	Vivek Goyal <vgoyal@...hat.com>
Cc:	linux-kernel@...r.kernel.org, axboe@...nel.dk, nauman@...gle.com,
	guijianfeng@...fujitsu.com, jmoyer@...hat.com, czoccolo@...il.com
Subject: Re: [PATCH 3/3] cfq-iosched: Print per slice sectors dispatched in 
	blktrace

Looks good to me.

Reviewed-by: Divyesh Shah <dpshah@...gle.com>

On Mon, Jul 19, 2010 at 10:20 AM, Vivek Goyal <vgoyal@...hat.com> wrote:
> - Divyesh had gotten rid of this code in the past. I want to re-introduce it
>  back as it helps me a lot during debugging.
>
> Signed-off-by: Vivek Goyal <vgoyal@...hat.com>
> ---
>  block/cfq-iosched.c |    8 ++++++--
>  1 files changed, 6 insertions(+), 2 deletions(-)
>
> diff --git a/block/cfq-iosched.c b/block/cfq-iosched.c
> index b23d7f4..1f0d4ea 100644
> --- a/block/cfq-iosched.c
> +++ b/block/cfq-iosched.c
> @@ -148,6 +148,8 @@ struct cfq_queue {
>        struct cfq_queue *new_cfqq;
>        struct cfq_group *cfqg;
>        struct cfq_group *orig_cfqg;
> +       /* Number of sectors dispatched from queue in single dispatch round */
> +       unsigned long nr_sectors;
>  };
>
>  /*
> @@ -926,8 +928,8 @@ static inline unsigned int cfq_cfqq_slice_usage(struct cfq_queue *cfqq)
>                        slice_used = cfqq->allocated_slice;
>        }
>
> -       cfq_log_cfqq(cfqq->cfqd, cfqq, "sl_used=%u, sl_disp=%u", slice_used,
> -                       cfqq->slice_dispatch);
> +       cfq_log_cfqq(cfqq->cfqd, cfqq, "sl_used=%u, disp=%u sect=%u",
> +                       slice_used, cfqq->slice_dispatch, cfqq->nr_sectors);
>        return slice_used;
>  }
>
> @@ -1608,6 +1610,7 @@ static void __cfq_set_active_queue(struct cfq_data *cfqd,
>                cfqq->allocated_slice = 0;
>                cfqq->slice_end = 0;
>                cfqq->slice_dispatch = 0;
> +               cfqq->nr_sectors = 0;
>
>                cfq_clear_cfqq_wait_request(cfqq);
>                cfq_clear_cfqq_must_dispatch(cfqq);
> @@ -1970,6 +1973,7 @@ static void cfq_dispatch_insert(struct request_queue *q, struct request *rq)
>        elv_dispatch_sort(q, rq);
>
>        cfqd->rq_in_flight[cfq_cfqq_sync(cfqq)]++;
> +       cfqq->nr_sectors += blk_rq_sectors(rq);
>        cfq_blkiocg_update_dispatch_stats(&cfqq->cfqg->blkg, blk_rq_bytes(rq),
>                                        rq_data_dir(rq), rq_is_sync(rq));
>  }
> --
> 1.7.1.1
>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/