All of lore.kernel.org
 help / color / mirror / Atom feed
From: Dongmao Zhang <dmzhang@suse.com>
To: Brassow Jonathan <jbrassow@redhat.com>
Cc: dm-devel@redhat.com
Subject: Re: [PATCH 0/2] patches to improve cluster raid1 performance [V2]
Date: Tue, 15 Oct 2013 14:32:59 +0800	[thread overview]
Message-ID: <525CE19B.6040703@suse.com> (raw)
In-Reply-To: <66B8343F-84A1-4F8C-B41D-3204EF16651A@redhat.com>

于 2013年10月11日 00:29, Brassow Jonathan 写道:
> Perhaps a better thing to do is to integrate the marks (and possibly
> clears) into the flush - DM_INTEGRATED_FLUSH.  The payload for the
> DM_ULOG_FLUSH flush communication is empty.  It seems to me now that
> it would make more sense to make use of that empty space and fill it
> with mark/clear requests.  What do you think?
>

DM_INTEGRATED_FLUSH is a good idea. It just simplify the confusion of
DM_FLUSH_WITH_MARK thing. My plan is to add (mark and clear) payload to
flush. It is like this:

'X' means we have this kind of request, while '0' means none.

mark request :     X               X               0

clear request:     X               0               X

flush method:    payload_flush   payload_flush   delayed flush.

If there is only clear requests, we can send the clear request first, 
and send normal flush later.  how do you like this strategy?

As for the testing, I only have ocfs2 available.(not GFS2). I will
test on that;

Dongmao Zhang.
















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

  reply	other threads:[~2013-10-15  6:32 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-09-26 10:50 [PATCH 0/2] patches to improve cluster raid1 performance [V2] dongmao zhang
2013-09-26 10:50 ` [PATCH 1/2] improve the performance of dm-log-userspace dongmao zhang
2013-09-26 10:50 ` [PATCH 2/2] change API of dm-log-userspace to support delay flush dongmao zhang
2013-09-26 17:17 ` [PATCH 0/2] patches to improve cluster raid1 performance [V2] Brassow Jonathan
2013-09-27  7:59   ` Dongmao Zhang
2013-09-30 14:24 ` Brassow Jonathan
2013-10-08  8:33   ` Dong Mao Zhang
2013-10-10 16:29     ` Brassow Jonathan
2013-10-15  6:32       ` Dongmao Zhang [this message]
2013-10-18 21:04         ` Brassow Jonathan

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=525CE19B.6040703@suse.com \
    --to=dmzhang@suse.com \
    --cc=dm-devel@redhat.com \
    --cc=jbrassow@redhat.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.