From: Con Kolivas <kernel@kolivas.org>
To: linux-kernel@vger.kernel.org
Subject: Re: [patch] cfq: ioprio inherit rt class
Date: Sat, 27 May 2006 12:08:32 +1000
User-Agent: KMail/1.9.1
Cc: Jens Axboe <axboe@suse.de>, ck list <ck@vds.kolivas.org>
References: <200605271150.41924.kernel@kolivas.org>
In-Reply-To: <200605271150.41924.kernel@kolivas.org>
MIME-Version: 1.0
Content-Type: text/plain;
  charset="iso-8859-1"
Content-Transfer-Encoding: 7bit
Content-Disposition: inline
Message-Id: <200605271208.33037.kernel@kolivas.org>
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 890
Lines: 25

On Saturday 27 May 2006 11:50, Con Kolivas wrote:
> Jens, ml
>
> I was wondering if cfq io priorities should be explicitly set to the
> realtime class when no io priority is specified from realtime tasks as in
> the following patch? (rt_task() will need to be modified to suit the PI
> changes in -mm)
>
> ---
> Set cfq io priority class to realtime and scale the priority according to
> the rt priority when no io priority is explicitly set in realtime tasks.

sorry, rather than this:
> +		return (task->rt_priority * IOPRIO_BE_NR / MAX_RT_PRIO);

it should be:
return ((MAX_RT_PRIO - task->rt_priority) * IOPRIO_BE_NR / MAX_RT_PRIO);

-- 
-ck
-
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/