<br><br><div class="gmail_quote">On Thu, Oct 15, 2009 at 11:30 PM, Andrew Beekhof <span dir="ltr"><<a href="mailto:andrew@beekhof.net">andrew@beekhof.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div><div></div><div class="h5">On Thu, Oct 15, 2009 at 4:51 PM, David Teigland <<a href="mailto:teigland@redhat.com">teigland@redhat.com</a>> wrote:<br>
> On Thu, Oct 15, 2009 at 09:43:56AM +0200, Andrew Beekhof wrote:<br>
>> Looks good to me :-)<br>
>> Although it might be nice to have the automated detection in place too.<br>
>><br>
>> Less things for the admin to get wrong.  What do you think Dave?<br>
><br>
> I don't mind, now that I've taken a moment to think how it should work...<br>
> There should be three protocol options, "tcp", "sctp", "detect".  Detect<br>
> should work by reading the totem/rrp_mode value in setup_ccs().<br>
<br>
</div></div>Agreed.<br>
Now we just need someone to volunteer to write the patch :-)<br></blockquote><div><br>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.<br>
<br>Many thanks for your comments :)<br>Jiaju<br></div></div><br>