Attachment 899961 Details for Bug 1001210 – totemiba: Use size of struct mcast in MTU check

[patch] totemiba: Use size of struct mcast in MTU check

2014-05-28-0001-totemiba-Use-size-of-struct-mcast-in-MTU-check.patch (text/plain), 1.52 KB, created by Jan Friesse on 2014-05-28 12:21:50 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jan Friesse

Created: 2014-05-28 12:21:50 UTC

Size: 1.52 KB

patch

obsolete

>From 2bcd96e762f5ce44417b9ea749a990caca273f59 Mon Sep 17 00:00:00 2001
>From: Jan Friesse <jfriesse@redhat.com>
>Date: Wed, 28 May 2014 14:09:12 +0200
>Subject: [PATCH] totemiba: Use size of struct mcast in MTU check
>
>Signed-off-by: Jan Friesse <jfriesse@redhat.com>
>---
> exec/totemiba.c |    8 ++++----
> 1 files changed, 4 insertions(+), 4 deletions(-)
>
>diff --git a/exec/totemiba.c b/exec/totemiba.c
>index 4e3f1cc..7ef8908 100644
>--- a/exec/totemiba.c
>+++ b/exec/totemiba.c
>@@ -1105,16 +1105,16 @@ static int recv_token_bind (struct totemiba_instance *instance)
> 	}
> 
> 	/*
>-	 * Determine active_mtu of port and compare it with the configured one (160 is aproximation of all totem
>-	 * structures.
>+	 * Determine active_mtu of port and compare it with the configured one (98 is sizeof(struct mcast) defined
>+	 * in totemsrp.c.
> 	 *
> 	 * TODO: Implement MTU discovery also for IP and handle MTU correctly for all structures inside totemsrp,
> 	 *       crypto, ...
> 	 */
> 	res = ibv_query_port (instance->listen_recv_token_cma_id->verbs, instance->listen_recv_token_cma_id->port_num, &port_attr);
>-	if ( (1 << (port_attr.active_mtu + 7)) < instance->totem_config->net_mtu + 160) {
>+	if ( (1 << (port_attr.active_mtu + 7)) < instance->totem_config->net_mtu + 98) {
> 		log_printf (LOGSYS_LEVEL_ERROR, "requested net_mtu is %d and is larger than the active port mtu %d\n",\
>-				instance->totem_config->net_mtu + 160, (1 << (port_attr.active_mtu + 7)));
>+				instance->totem_config->net_mtu + 98, (1 << (port_attr.active_mtu + 7)));
> 		return (-1);
> 	}
> 
>-- 
>1.7.1
>

Actions: View | Diff

Attachments on bug 1001210: 790643 | 790644 | 790645 | 790646 | 790647 | 793906 | 797602 | 797603 | 797604 | 797605 | 797606 | 797607 | 797619 | 797620 | 800375 | 800376 | 800377 | 892304 | 895478 | 895956 | 899961