All of lore.kernel.org
 help / color / mirror / Atom feed
From: NeilBrown <neilb@suse.de>
To: Sebastian Parschauer <sebastian.riemer@profitbricks.com>
Cc: "Linux RAID" <linux-raid@vger.kernel.org>,
	"Florian-Ewald Müller" <florian-ewald.mueller@profitbricks.com>
Subject: Re: [RFC] Process requests instead of bios to use a scheduler
Date: Mon, 2 Jun 2014 20:20:50 +1000	[thread overview]
Message-ID: <20140602202050.14903534@notabene.brown> (raw)
In-Reply-To: <538C4938.6010704@profitbricks.com>

[-- Attachment #1: Type: text/plain, Size: 1743 bytes --]

On Mon, 02 Jun 2014 11:51:52 +0200 Sebastian Parschauer
<sebastian.riemer@profitbricks.com> wrote:


> > Having a scheduler for RAID0 doesn't make any sense to me.
> > RAID0 simply passes each request down to the appropriate underlying device.
> > That device then does its own scheduling.
> > 
> > Adding a scheduler may well make sense for RAID1 (the current "scheduler"
> > only does some read balancing and is rather simplistic) and for RAID4/5/6/10.
> > 
> > But not for RAID0 .... was that a typo?
> 
> Nope, we have our RAID-1+0. So it is more or less a RAID-10 and putting
> the scheduler to this RAID-0 layer makes sense for us.

I still cannot imagine how this would work.  RAID-0 has no decisions to make,
so no where for a scheduler to fit.

Just to clarify: is this md/raid0 over md/raid1 or md/raid0 over
hardware/raid1?


> > Could you do a graph?  I like graphs :-)
> > I can certainly seem something has changed here...
> 
> Sure, please find the graphs attached. I've converted it into percentage
> so that number of bios can be compared to number of requests.

Thanks.

> > 
> > Show me the code and I might be able to provide a more detailed opinion.
> 
> I would say let the user decide whether an MD device should be equipped
> with a scheduler or not. We can port our code to latest kernel + latest
> mdadm and send you a patch set for testing. Just give me some time to do it.

In the first instance, I just want to get a concrete idea of what you have
done because what you have said doesn't make sense to me.  I'm happy to look
at code against a not-quite-current kernel to get that idea.  But I'm also
happy for it to be against the latest, whatever suits you.

NeilBrown

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 828 bytes --]

  reply	other threads:[~2014-06-02 10:20 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-05-28 13:04 [RFC] Process requests instead of bios to use a scheduler Sebastian Parschauer
2014-06-01 23:32 ` NeilBrown
2014-06-02  9:51   ` Sebastian Parschauer
2014-06-02 10:20     ` NeilBrown [this message]
2014-06-02 11:12       ` Sebastian Parschauer
2014-06-04 17:09       ` [RFC PATCH 0/4] md/mdadm: introduce request function mode support Sebastian Parschauer
2014-06-04 17:09         ` [RFC PATCH 1/4] md: complete bio accounting and add io_latency extension Sebastian Parschauer
2014-06-04 17:10         ` [RFC PATCH 2/4] md: introduce request function mode support Sebastian Parschauer
2014-06-04 17:10         ` [RFC PATCH 3/4] md: handle IO latency accounting in rqfn mode Sebastian Parschauer
2014-06-04 17:10         ` [RFC PATCH 4/4] mdadm: introduce '--use-requestfn' create/assembly option Sebastian Parschauer
2014-06-17 13:20         ` [RFC PATCH 0/4] md/mdadm: introduce request function mode support Sebastian Parschauer
     [not found]           ` <CAH3kUhEK26+4KryoReosMt654-vcrkkgkxaW5tKkFRDBqgX82w@mail.gmail.com>
     [not found]             ` <53A14513.20902@profitbricks.com>
2014-06-18 13:57               ` Roberto Spadim
2014-06-18 14:43                 ` Sebastian Parschauer
2014-06-24  7:09           ` NeilBrown

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=20140602202050.14903534@notabene.brown \
    --to=neilb@suse.de \
    --cc=florian-ewald.mueller@profitbricks.com \
    --cc=linux-raid@vger.kernel.org \
    --cc=sebastian.riemer@profitbricks.com \
    /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.