public inbox for netdev@vger.kernel.org 
 help / color / mirror / Atom feed
From: "John W. Linville" <linville@tuxdriver•com>
To: Anjali Singhai Jain <anjali.singhai@intel•com>
Cc: netdev@vger•kernel.org, jeffrey.t.kirsher@intel•org,
	Kiran Patil <kiran.patil@intel•com>
Subject: Re: [PATCH v3 2/4] i40e: geneve tunnel offload support
Date: Tue, 8 Dec 2015 13:22:40 -0500	[thread overview]
Message-ID: <20151208182239.GF3598@tuxdriver.com> (raw)
In-Reply-To: <1449598334-97674-3-git-send-email-anjali.singhai@intel.com>

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 <anjali.singhai@intel•com>
> Signed-off-by: Kiran Patil <kiran.patil@intel•com>

> @@ -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
linville@tuxdriver•com			might be all we have.  Be ready.

  parent reply	other threads:[~2015-12-08 18:30 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-12-08 18:12 Add support for Geneve udp port offload Anjali Singhai Jain
2015-12-08 18:12 ` [PATCH v3 1/4] geneve: Add geneve udp port offload for ethernet devices Anjali Singhai Jain
2015-12-12  0:58   ` David Miller
2015-12-12  3:11   ` Tom Herbert
2015-12-14 16:03     ` Singhai, Anjali
2015-12-08 18:12 ` [PATCH v3 2/4] i40e: geneve tunnel offload support Anjali Singhai Jain
2015-12-08 18:20   ` Alexei Starovoitov
2015-12-08 18:36     ` Jesse Gross
2015-12-08 23:08       ` Singhai, Anjali
2015-12-08 18:22   ` John W. Linville [this message]
2015-12-11  5:44   ` Jeff Kirsher
2015-12-08 18:12 ` [PATCH v3 3/4] i40e: Kernel dependency update for i40e to support geneve offload Anjali Singhai Jain
2015-12-11  5:45   ` Jeff Kirsher
2015-12-08 18:12 ` [PATCH v3 4/4] geneve: Add geneve_get_rx_port support Anjali Singhai Jain
2015-12-12  1:00   ` David Miller

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20151208182239.GF3598@tuxdriver.com \
    --to=linville@tuxdriver$(echo .)com \
    --cc=anjali.singhai@intel$(echo .)com \
    --cc=jeffrey.t.kirsher@intel$(echo .)org \
    --cc=kiran.patil@intel$(echo .)com \
    --cc=netdev@vger$(echo .)kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox