All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jens Axboe <axboe@suse.de>
To: Andrew Morton <akpm@osdl.org>
Cc: David Miller <davem@davemloft.net>, linux-kernel@vger.kernel.org
Subject: Re: softirq considered harmful
Date: Mon, 14 Aug 2006 09:37:25 +0200	[thread overview]
Message-ID: <20060814073724.GJ4231@suse.de> (raw)
In-Reply-To: <20060812182234.605b4fb4.akpm@osdl.org>

On Sat, Aug 12 2006, Andrew Morton wrote:
> On Sat, 12 Aug 2006 18:09:44 -0700 (PDT)
> David Miller <davem@davemloft.net> wrote:
> 
> > From: Andrew Morton <akpm@osdl.org>
> > Date: Sat, 12 Aug 2006 17:45:49 -0700
> > 
> > > Is that also adding 150 usecs to each IO operation?
> > 
> > I have no idea, Jens hasn't done enough to narrow down the true cause
> > of the latencies he is seeing.  So pinpointing it on anything specific
> > is highly premature at this stage.
> 
> Determining whether pre-conversion scsi was impacted in the same manner
> would be part of that pinpointing process.
> 
> Deferring to softirq _has_ to add latency and any latency addition in
> synchronous disk IO is very bad.  That being said, 150 usecs per request is
> so bad that I'd be suspecting that it's not affecting most people, else
> we'd have heard.

Hopefully you often end up doing > 1 request for a busy IO sub system,
otherwise the softirq stuff is pointless. But it's still pretty bad for
single requests.

> > My point was merely to encourage you to find out the facts before
> > tossing accusations around. :-)
> 
> No, your point was that slotting this change into mainline without telling
> anyone was OK because SCSI has been doing something similar.

Not similar, identical. Andrew, there was _no_ real change there!

-- 
Jens Axboe


  reply	other threads:[~2006-08-14  7:35 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-08-10 11:06 softirq considered harmful Jens Axboe
2006-08-10 11:41 ` David Miller
2006-08-10 12:26   ` Jens Axboe
2006-08-12 23:28 ` Andrew Morton
2006-08-13  0:43   ` David Miller
2006-08-13  0:45     ` Andrew Morton
2006-08-13  1:09       ` David Miller
2006-08-13  1:22         ` Andrew Morton
2006-08-14  7:37           ` Jens Axboe [this message]
2006-08-14  8:44             ` David Miller
2006-08-14  8:51               ` Jens Axboe
2006-08-14  7:34       ` Jens Axboe
2006-08-13 14:31   ` Jens Axboe

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=20060814073724.GJ4231@suse.de \
    --to=axboe@suse.de \
    --cc=akpm@osdl.org \
    --cc=davem@davemloft.net \
    --cc=linux-kernel@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.