Subject: Re: [PATCH v3 2/4] i40e: geneve tunnel offload support

From: "John W. Linville" <[email-masked]>
Date: Tue, 8 Dec 2015 13:22:40 -0500

On Tue, Dec 08, 2015 at 10:12:12AM -0800, Anjali Singhai Jain wrote:
> This patch adds driver hooks to implement ndo_ops to add/del
> udp port in the HW to identify GENEVE tunnels.
> 
> Signed-off-by: Anjali Singhai Jain <[email-masked]>
> Signed-off-by: Kiran Patil <[email-masked]>

> @@ -10406,6 +10508,9 @@ static void i40e_print_features(struct i40e_pf *pf)
>  #if IS_ENABLED(CONFIG_VXLAN)
>  	i += snprintf(&buf[i], REMAIN(i), " VxLAN");
>  #endif
> +#if IS_ENABLED(CONFIG_GENEVE)
> +	i += snprintf(&buf[i], REMAIN(i), "Geneve ");
> +#endif
>  	if (pf->flags & I40E_FLAG_PTP)
>  		i += snprintf(&buf[i], REMAIN(i), " PTP");
>  #ifdef I40E_FCOE

It seems like it should be " Geneve" rather than "Geneve "?

-- 
John W. Linville		Someday the world will need a hero, and you
[email-masked]			might be all we have.  Be ready.