From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jan Friesse Date: Tue, 04 Jan 2011 10:18:11 +0100 Subject: [Cluster-devel] [PATCH] cman: Fix udpb handling in cman-preconfig In-Reply-To: <4D22E2F6.90505@redhat.com> References: <1294078742-8510-1-git-send-email-lhh@redhat.com> <4D22E191.1040606@redhat.com> <4D22E2F6.90505@redhat.com> Message-ID: <4D22E5D3.9090405@redhat.com> List-Id: To: cluster-devel.redhat.com MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Fabio M. Di Nitto wrote: > On 1/4/2011 10:00 AM, Jan Friesse wrote: >> Lon, >> even it may "logically" seemed that original code is incorrect and your >> patch make sense, it's not the case. >> >> Actually, corosync has 3 transports: >> udp, udpu and iba. >> >> Broadcast is same transport as udp but only has broadcast flag set on. >> >> In other words, this patch is not correct. >> > > So in order to configure broadcast we still need to use: > > ? > > I find that redundant tho... > > Cheers > Fabio > No. Actually. Original idea was to use transport keyword and give user ability to use udp (for multicast), udpu (for unicast), udpb (for broadcast) and rdma. But also we must keep compatibility so thats why broadcast keyword still exists. So actually, transport="udpb" is same as transport="udp" broadcast="yes". Regards, Honza >> Regards, >> Honza >> >> Lon Hohberger wrote: >>> Signed-off-by: Lon Hohberger >>> --- >>> cman/daemon/cman-preconfig.c | 2 +- >>> 1 files changed, 1 insertions(+), 1 deletions(-) >>> >>> diff --git a/cman/daemon/cman-preconfig.c b/cman/daemon/cman-preconfig.c >>> index 5239d89..5d608a7 100644 >>> --- a/cman/daemon/cman-preconfig.c >>> +++ b/cman/daemon/cman-preconfig.c >>> @@ -286,7 +286,7 @@ static int add_ifaddr(struct objdb_iface_ver0 *objdb, char *mcast, char *ifaddr, >>> int ret = 0; >>> const char *tx_mech_to_str[] = { >>> [TX_MECH_UDP] = "udp", >>> - [TX_MECH_UDPB] = "udp", >>> + [TX_MECH_UDPB] = "udpb", >>> [TX_MECH_UDPU] = "udpu", >>> [TX_MECH_RDMA] = "iba", >>> }; >