]> git.proxmox.com Git - mirror_corosync.git/commit
knet: Fix a couple of errors when adding a new link
authorChristine Caulfield <ccaulfie@redhat.com>
Thu, 2 May 2019 13:22:47 +0000 (14:22 +0100)
committerJan Friesse <jfriesse@redhat.com>
Thu, 2 May 2019 14:42:03 +0000 (16:42 +0200)
commit01ce5a96ef9dea2d517d84abec7aecc6af99e7aa
tree7db166c2580ab7c7d70890ecf220cc3dcb3f9d8a
parent70cda5d55f57b99c564be5e7871d291118553a74
knet: Fix a couple of errors when adding a new link

When adding a new link for the first time you will often see:
1) knet_link_set_ping_timers for nodeid 1, link 1 failed: Invalid
argument (22)
2) New config has different knet transport for link 1. Internal value
was NOT changed. To reconfigure an interface it must be deleted and
recreated. A working interface needs to be available to corosync at all
times

1) is caused by setting the ping timers twice, once in
totemknet_member_add() and once in totemknet_refresh_config().
The first time we don't know the value
so it's zero and thus display an error. For this we simply check
for the zero and skip the knet API call. It's not ideal, but
totemconfig needs a lot of reconfiguring itself before we can
make this more sane.

2) was caused by simply comparing an unconfigured link with
a configured one, so OF COURSE, they are going to be different!

Signed-off-by: Christine Caulfield <ccaulfie@redhat.com>
Reviewed-by: Jan Friesse <jfriesse@redhat.com>
exec/totemconfig.c
exec/totemknet.c