All of lore.kernel.org
 help / color / mirror / Atom feed
From: Kiyoshi Ueda <k-ueda@ct.jp.nec.com>
To: Alasdair Kergon <agk@redhat.com>
Cc: dm-devel@redhat.com, stefan.bader@canonical.com
Subject: Re: Re: [PATCH 0/0] dm-mpath: queue-length oriented dynamic load balancer
Date: Fri, 30 Jan 2009 17:10:38 +0900	[thread overview]
Message-ID: <4982B5FE.5040807@ct.jp.nec.com> (raw)
In-Reply-To: <20090129140335.GA27390@agk-dp.fab.redhat.com>

Hi Alasdair,

Sorry for confusing you.

On 01/29/2009 11:03 PM +0900, Alasdair G Kergon wrote:
> On Thu, Jan 29, 2009 at 03:59:10PM +0900, Kiyoshi Ueda wrote:
>> This patch-set can be applied on top of 2.6.29-rc2.
> 
> It's not clear: are you reordering the dependencies now?
> 
> Previously there were three sets of patches taken in this order:
>   1 change to request-based
>   2 add queue length ps
>   3 add service time ps
> i.e. 2 and 3 depend on 1
> 
> Are you now changing that so that 2 and 3 could go upstream before 1?

I had sent those three sets in the order above since 1 enhances 2 and 3.
But 2 and 3 don't functionally depend on 1 (although 1 and 3 have
a few cosmetic dependencies), so we can reorder that as we want.

2 and 3 are simpler patches than 1, and I found some problems in 1,
so I think reordering might be a good idea if you are OK.
As for 3, I still want to get feedback from others as it'll involve
discussion about the optimal interface for load balancer.

Thanks,
Kiyoshi Ueda

  reply	other threads:[~2009-01-30  8:10 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-01-29  6:59 [PATCH 0/0] dm-mpath: queue-length oriented dynamic load balancer Kiyoshi Ueda
2009-01-29  7:02 ` [PATCH 1/2] dm-mpath: add a path selector interface Kiyoshi Ueda
2009-01-29  7:04 ` [PATCH 2/2] dm-mpath: add queue-length oriented dynamic load balancer Kiyoshi Ueda
2009-01-29 14:03 ` [PATCH 0/0] dm-mpath: " Alasdair G Kergon
2009-01-30  8:10   ` Kiyoshi Ueda [this message]
2009-01-30 13:08     ` Alasdair G Kergon

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=4982B5FE.5040807@ct.jp.nec.com \
    --to=k-ueda@ct.jp.nec.com \
    --cc=agk@redhat.com \
    --cc=dm-devel@redhat.com \
    --cc=stefan.bader@canonical.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.