All of lore.kernel.org
 help / color / mirror / Atom feed
From: Martin Wilck <mwilck@suse.com>
To: Yang Feng <philip.yang@huawei.com>,
	Hannes Reinecke <hare@suse.de>,
	dm-devel@redhat.com, christophe.varoqui@opensvc.com,
	bmarzins@redhat.com, xose.vazquez@gmail.com
Cc: zouming.zouming@huawei.com, hege09@huawei.com,
	guanjunxiong@huawei.com, shenhong09@huawei.com,
	qiuxin@huawei.com
Subject: Re: [PATCH v4 0/1] multipath-tools: Prioritizer based on a latency algorithm
Date: Fri, 09 Jun 2017 10:05:52 +0200	[thread overview]
Message-ID: <1496995552.4654.16.camel@suse.com> (raw)
In-Reply-To: <197737a7-c0bd-adf7-934c-b198adac82f3@huawei.com>

Hello Yang,

> > Actually, you're not alone here; several other storage array setups
> > suffer from the same problem.
> > 
> > Eg if you have a site-failover setup with two storage arrays at
> > different locations the problem is more-or-less the same;
> > both arrays potentially will be displaying identical priority
> > information, despite one array being remote.
> > 
> 
> It's up to the value set of the argument "latency_interval".For
> example,
> If latency_interval=10ms, the paths will be grouped in priority
> groups
> with path latency 0-10ms, 10-20ms, 20-30ms, etc. If the argument
> "latency_interval" is set to appropriate value and the distance
> between
> two arrays is not enough far, two priorities may be the same, But
> it's
> OK, because between two arrays, the gap of average path latency is
> very
> small and tolerable.

I wonder if it would make sense to use "logarithmically" scaled latency
intervals here. It wouldn't make a large difference whether the latency
is 1ms or 2ms, but if we have paths where the latencies differ by order
of magnitude, it would be very important to make a distinction. With
the current linear intervals, it would be hard to get this right (us
interval size would result in too many intervals, and sec interval size
wouldn't allow a distinction between us and ms latencies).

By using logarithmic scale, you could setup the latency intevals e.g
like this: 

  < 10us, 10us-100us, 100us-1ms, 1ms-10ms, 10ms-100ms, > 100ms

IMO that would be better, in particular if the latencies differ
strongly between paths.

Regards
Martin


-- 
Dr. Martin Wilck <mwilck@suse.com>, Tel. +49 (0)911 74053 2107
SUSE Linux GmbH, GF: Felix Imendörffer, Jane Smithard, Graham Norton
HRB 21284 (AG Nürnberg)

--
dm-devel mailing list
dm-devel@redhat.com
https://www.redhat.com/mailman/listinfo/dm-devel

  reply	other threads:[~2017-06-09  8:05 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-06-06  2:43 [PATCH v4 0/1] multipath-tools: Prioritizer based on a latency algorithm Yang Feng
2017-06-06  2:43 ` [PATCH v4 1/1] " Yang Feng
2017-06-08 15:37 ` [PATCH v4 0/1] " Hannes Reinecke
2017-06-09  7:20   ` Yang Feng
2017-06-09  8:05     ` Martin Wilck [this message]
2017-06-09  8:26       ` Yang Feng

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=1496995552.4654.16.camel@suse.com \
    --to=mwilck@suse.com \
    --cc=bmarzins@redhat.com \
    --cc=christophe.varoqui@opensvc.com \
    --cc=dm-devel@redhat.com \
    --cc=guanjunxiong@huawei.com \
    --cc=hare@suse.de \
    --cc=hege09@huawei.com \
    --cc=philip.yang@huawei.com \
    --cc=qiuxin@huawei.com \
    --cc=shenhong09@huawei.com \
    --cc=xose.vazquez@gmail.com \
    --cc=zouming.zouming@huawei.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.