All of lore.kernel.org
 help / color / mirror / Atom feed
From: Xose Vazquez Perez <xose.vazquez@gmail.com>
To: linux-s390@vger.kernel.org
Subject: Re: [PATCH 1/1] dasd: do not force use of deadline IO scheduler
Date: Fri, 30 Apr 2010 20:33:21 +0000	[thread overview]
Message-ID: <4BDB3E91.8070505@gmail.com> (raw)

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain, Size: 1860 bytes --]

On 2008-03-04 8:52:10, Martin Schwidefsky wrote:

a bit old.

> On Mon, 2008-03-03 at 15:21 -0500, Josef 'Jeff' Sipek wrote:
>> Signed-off-by: Josef 'Jeff' Sipek <jeffpc@josefsipek.net>
>> ---
>>  drivers/s390/block/dasd.c |    9 ---------
>>  1 files changed, 0 insertions(+), 9 deletions(-)
>> 
>> diff --git a/drivers/s390/block/dasd.c b/drivers/s390/block/dasd.c
>> index ccf46c9..4d4f6ff 100644
>> --- a/drivers/s390/block/dasd.c
>> +++ b/drivers/s390/block/dasd.c
>> @@ -1946,21 +1946,12 @@ static void do_dasd_request(struct request_queue *queue)
>>   */
>>  static int dasd_alloc_queue(struct dasd_block *block)
>>  {
>> -	int rc;
>> -
>>  	block->request_queue = blk_init_queue(do_dasd_request,
>>  					       &block->request_queue_lock);
>>  	if (block->request_queue == NULL)
>>  		return -ENOMEM;
>> 
>>  	block->request_queue->queuedata = block;
>> -
>> -	elevator_exit(block->request_queue->elevator);
>> -	rc = elevator_init(block->request_queue, "deadline");
>> -	if (rc) {
>> -		blk_cleanup_queue(block->request_queue);
>> -		return rc;
>> -	}
>>  	return 0;
>>  }
> 
> Um, why? We have a reason to force the deadline scheduler, our
> performance test have shown that the deadline scheduler get the highest
> throughout out of the storage subsystems which are used with the
> machines.

Are there recent tests, cfq vs. deadline, with current kernels?

This FCP/SCSI test is a bit old(2005-03-16), and cfq vs. deadline shows no differences:
http://www.ibm.com/developerworks/linux/linux390/perf/tuning_res_dasd_ioScheduler.html

Any DASD test out there ?

-thanks-

-- 
�All� muevan feroz guerra, ciegos reyes por un palmo m�s de tierra;
que yo aqu� tengo por m�o cuanto abarca el mar brav�o, a quien nadie
impuso leyes. Y no hay playa, sea cualquiera, ni bandera de esplendor,
que no sienta mi derecho y d� pecho a mi valor.�

             reply	other threads:[~2010-04-30 20:33 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-04-30 20:33 Xose Vazquez Perez [this message]
     [not found] <20080304192601.GK6664@josefsipek.net>
2008-03-05  8:58 ` [PATCH 1/1] dasd: do not force use of deadline IO scheduler Heiko Carstens

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=4BDB3E91.8070505@gmail.com \
    --to=xose.vazquez@gmail.com \
    --cc=linux-s390@vger.kernel.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.