lkml.org 
[lkml]   [2009]   [Nov]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH 05/16] blkio: Implement per cfq group latency target and busy queue avg
From
On Fri, Nov 13, 2009 at 12:32 AM, Vivek Goyal <vgoyal@redhat.com> wrote:
>  static inline void
> @@ -441,10 +445,13 @@ cfq_set_prio_slice(struct cfq_data *cfqd, struct cfq_queue *cfqq)
>        if (cfqd->cfq_latency) {
>                /* interested queues (we consider only the ones with the same
>                 * priority class) */
This comment needs to be updated
> -               unsigned iq = cfq_get_avg_queues(cfqd, cfq_class_rt(cfqq));
> +               unsigned iq = cfq_group_get_avg_queues(cfqd, cfqq->cfqg,
> +                                               cfq_class_rt(cfqq));
>                unsigned sync_slice = cfqd->cfq_slice[1];
>                unsigned expect_latency = sync_slice * iq;
> -               if (expect_latency > cfq_target_latency) {
> +               unsigned group_target_lat = cfq_target_latency/cfqd->nr_groups;

I'm not sure that we should divide the target latency evenly among groups.
Groups with different weights will have different percentage of time
in each 300ms round, so probably we should consider it here.

> +
> +               if (expect_latency > group_target_lat) {
>                        unsigned base_low_slice = 2 * cfqd->cfq_slice_idle;
>                        /* scale low_slice according to IO priority
>                         * and sync vs async */
> @@ -452,7 +459,7 @@ cfq_set_prio_slice(struct cfq_data *cfqd, struct cfq_queue *cfqq)
>                                min(slice, base_low_slice * slice / sync_slice);
>                        /* the adapted slice value is scaled to fit all iqs
>                         * into the target latency */
> -                       slice = max(slice * cfq_target_latency / expect_latency,
> +                       slice = max(slice * group_target_lat / expect_latency,
>                                    low_slice);
>                }
>        }

Thanks,
Corrado
\
 
 \ /
  Last update: 2009-11-13 11:49    [from the cache]
©2003-2014 Jasper Spaans. Advertise on this site