[ofa-general] Re: [PATCH][TRIVIAL] ibsim: Some cosmetic changes

Sasha Khapyorsky sashak at voltaire.com
Tue Mar 10 02:43:53 PDT 2009


On 15:50 Fri 06 Mar     , Hal Rosenstock wrote:
> 
> Signed-off-by: Hal Rosenstock <hal.rosenstock at gmail.com>

Applied with modification noted below. Thanks.

> diff --git a/ibsim/sim_cmd.c b/ibsim/sim_cmd.c
> index 94e0a14..bbd51b9 100644
> --- a/ibsim/sim_cmd.c
> +++ b/ibsim/sim_cmd.c
> @@ -274,10 +274,9 @@ static void port_change_lid(Port * port, int lid, int lmc)
>  	if (lmc > 0)
>  		port->lmc = lmc;
>  
> -	if (port->node->type == SWITCH_NODE) {
> -		if (port->node->sw)
> -			port->node->sw->portchange = 1;
> -	} else if (port->remotenode && port->remotenode->sw)
> +	if (port->node->type == SWITCH_NODE && port->node->sw)
> +		port->node->sw->portchange = 1;
> +	else if (port->remotenode && port->remotenode->sw)
>  		port->remotenode->sw->portchange = 1;
>  }

This is not equivalent. So I'm removing this chunk.

Sasha



More information about the general mailing list