Re: [PATCH 2/3] net/mlx5e: make VXLAN support conditional

2016-05-12 Thread Arnd Bergmann
On Sunday 08 May 2016 14:58:38 Saeed Mahameed wrote: > On Fri, May 6, 2016 at 10:35 PM, David Miller wrote: > > From: Arnd Bergmann > > Date: Thu, 05 May 2016 20:09:19 +0200 > > > >> For reference, I've tried it out on the MLX4 driver, and it does > >> seem nicer that way, see below. > > > > Is i

Re: [PATCH 2/3] net/mlx5e: make VXLAN support conditional

2016-05-08 Thread Saeed Mahameed
On Fri, May 6, 2016 at 10:35 PM, David Miller wrote: > From: Arnd Bergmann > Date: Thu, 05 May 2016 20:09:19 +0200 > >> For reference, I've tried it out on the MLX4 driver, and it does >> seem nicer that way, see below. > > Is it possible to wind down this conversation and have someone submit > w

Re: [PATCH 2/3] net/mlx5e: make VXLAN support conditional

2016-05-06 Thread Alexander Duyck
On Thu, May 5, 2016 at 11:09 AM, Arnd Bergmann wrote: > On Thursday 05 May 2016 19:44:36 Saeed Mahameed wrote: >> On Wed, May 4, 2016 at 3:31 PM, Arnd Bergmann wrote: >> > VXLAN can be disabled at compile-time or it can be a loadable >> > module while mlx5 is built-in, which leads to a link error

Re: [PATCH 2/3] net/mlx5e: make VXLAN support conditional

2016-05-06 Thread David Miller
From: Arnd Bergmann Date: Thu, 05 May 2016 20:09:19 +0200 > For reference, I've tried it out on the MLX4 driver, and it does > seem nicer that way, see below. Is it possible to wind down this conversation and have someone submit whatever final patch everyone agrees to? Thanks.

Re: [PATCH 2/3] net/mlx5e: make VXLAN support conditional

2016-05-05 Thread Arnd Bergmann
On Thursday 05 May 2016 19:44:36 Saeed Mahameed wrote: > On Wed, May 4, 2016 at 3:31 PM, Arnd Bergmann wrote: > > VXLAN can be disabled at compile-time or it can be a loadable > > module while mlx5 is built-in, which leads to a link error: > > > > drivers/net/built-in.o: In function `mlx5e_create_

Re: [PATCH 2/3] net/mlx5e: make VXLAN support conditional

2016-05-05 Thread Saeed Mahameed
On Wed, May 4, 2016 at 3:31 PM, Arnd Bergmann wrote: > VXLAN can be disabled at compile-time or it can be a loadable > module while mlx5 is built-in, which leads to a link error: > > drivers/net/built-in.o: In function `mlx5e_create_netdev': > ntb_netdev.c:(.text+0x106de4): undefined reference to

[PATCH 2/3] net/mlx5e: make VXLAN support conditional

2016-05-04 Thread Arnd Bergmann
VXLAN can be disabled at compile-time or it can be a loadable module while mlx5 is built-in, which leads to a link error: drivers/net/built-in.o: In function `mlx5e_create_netdev': ntb_netdev.c:(.text+0x106de4): undefined reference to `vxlan_get_rx_port' This avoids the link error and makes the v