cluster-devel.redhat.com archive mirror
 help / color / mirror / Atom feed
From: Jiaju Zhang <jjzhang.linux@gmail.com>
To: cluster-devel.redhat.com
Subject: [Cluster-devel] Re: [RFC PATCH] dlm: enhancing dlm_controld (pcmk) to be able to handle redundant rings
Date: Fri, 16 Oct 2009 00:07:33 +0800	[thread overview]
Message-ID: <f8c992a30910150907r518cc80cx57629cba0011b03d@mail.gmail.com> (raw)
In-Reply-To: <b80f82d20910150830j14a3e92fhca05fa79bf6dfcc2@mail.gmail.com>

On Thu, Oct 15, 2009 at 11:30 PM, Andrew Beekhof <andrew@beekhof.net> wrote:

> On Thu, Oct 15, 2009 at 4:51 PM, David Teigland <teigland@redhat.com>
> wrote:
> > On Thu, Oct 15, 2009 at 09:43:56AM +0200, Andrew Beekhof wrote:
> >> Looks good to me :-)
> >> Although it might be nice to have the automated detection in place too.
> >>
> >> Less things for the admin to get wrong.  What do you think Dave?
> >
> > I don't mind, now that I've taken a moment to think how it should work...
> > There should be three protocol options, "tcp", "sctp", "detect".  Detect
> > should work by reading the totem/rrp_mode value in setup_ccs().
>
> Agreed.
> Now we just need someone to volunteer to write the patch :-)
>

I would like to write the patch. Adding the detection function in
setup_ccs() is more reasonable. In fact, I have ever been thinking about
adding this function in setup_ccs(). But when I looked into the code, I
found another way could make less modification in code, so I wrote that
patch and sent out for comments.

Many thanks for your comments :)
Jiaju
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/cluster-devel/attachments/20091016/8aa5531a/attachment.htm>

  reply	other threads:[~2009-10-15 16:07 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-10-14 10:50 [Cluster-devel] [RFC PATCH] dlm: enhancing dlm_controld (pcmk) to be able to handle redundant rings Jiaju Zhang
2009-10-14 13:28 ` [Cluster-devel] " Jiaju Zhang
2009-10-14 18:17 ` David Teigland
2009-10-15  5:34   ` Jiaju Zhang
2009-10-15  7:43     ` Andrew Beekhof
2009-10-15 14:51       ` David Teigland
2009-10-15 15:30         ` Andrew Beekhof
2009-10-15 16:07           ` Jiaju Zhang [this message]
2009-10-16 10:17         ` Lars Marowsky-Bree
  -- strict thread matches above, loose matches on Subject: below --
2009-10-23 10:38 jjzhang
2009-10-23 13:05 ` Andrew Beekhof
2009-10-23 13:23   ` Jiaju Zhang
2009-10-23 13:34     ` Lars Marowsky-Bree
2009-10-23 17:55     ` David Teigland
2009-10-23 19:18       ` Steven Dake
2009-10-23 20:51         ` David Teigland
2009-10-26 10:44           ` Jiaju Zhang

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=f8c992a30910150907r518cc80cx57629cba0011b03d@mail.gmail.com \
    --to=jjzhang.linux@gmail.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).