Re: [PATCH net-next 1/4] mlx5: Make building eswitch configurable

2017-01-31 Thread Mohamad Haj Yahia
>> Is it multi host configuration or bare metal ? > > > multihost > >> Do you have internal loopback traffic between different hosts ? > > > in a multihost? how can I check that? > Is there an ethtool command? > You can check that by sending traffic from one PF to another on the same port. >>>

Re: [PATCH net-next 1/4] mlx5: Make building eswitch configurable

2017-01-30 Thread Alexei Starovoitov
On 1/30/17 1:18 PM, Saeed Mahameed wrote: On Mon, Jan 30, 2017 at 6:45 PM, Alexei Starovoitov wrote: On 1/29/17 1:11 AM, Saeed Mahameed wrote: ConnectX4/5 and hopefully so on .. provide three different isolated steering layers: 3. vport layer: avaialbe for any PF/VF vport nic

Re: [PATCH net-next 1/4] mlx5: Make building eswitch configurable

2017-01-30 Thread Saeed Mahameed
On Mon, Jan 30, 2017 at 6:45 PM, Alexei Starovoitov wrote: > On 1/29/17 1:11 AM, Saeed Mahameed wrote: >> >> >> ConnectX4/5 and hopefully so on .. provide three different isolated >> steering layers: >> >> 3. vport layer: avaialbe for any PF/VF vport nic driver instance >>

Re: [PATCH net-next 1/4] mlx5: Make building eswitch configurable

2017-01-30 Thread Alexei Starovoitov
On 1/29/17 1:11 AM, Saeed Mahameed wrote: ConnectX4/5 and hopefully so on .. provide three different isolated steering layers: 3. vport layer: avaialbe for any PF/VF vport nic driver instance (netdevice), it allows vlan/mac filtering ,RSS hashing and n-tuple steering (for both encapsulated

Re: [PATCH net-next 1/4] mlx5: Make building eswitch configurable

2017-01-29 Thread Saeed Mahameed
On Sat, Jan 28, 2017 at 7:52 PM, Alexei Starovoitov wrote: > On 1/28/17 3:20 AM, Saeed Mahameed wrote: >> >> On Sat, Jan 28, 2017 at 1:23 AM, Alexei Starovoitov wrote: >>> >>> On 1/27/17 1:15 PM, Saeed Mahameed wrote: It is only mandatory for

Re: [PATCH net-next 1/4] mlx5: Make building eswitch configurable

2017-01-28 Thread Alexei Starovoitov
On 1/28/17 3:20 AM, Saeed Mahameed wrote: On Sat, Jan 28, 2017 at 1:23 AM, Alexei Starovoitov wrote: On 1/27/17 1:15 PM, Saeed Mahameed wrote: It is only mandatory for configurations that needs eswitch, where the driver has no way to know about them, for a good old bare metal

Re: [PATCH net-next 1/4] mlx5: Make building eswitch configurable

2017-01-28 Thread Saeed Mahameed
On Sat, Jan 28, 2017 at 1:23 AM, Alexei Starovoitov wrote: > On 1/27/17 1:15 PM, Saeed Mahameed wrote: >> >> It is only mandatory for configurations that needs eswitch, where the >> driver has no way to know about them, for a good old bare metal box, >> eswitch is not needed. >> >>

Re: [PATCH net-next 1/4] mlx5: Make building eswitch configurable

2017-01-27 Thread Alexei Starovoitov
On 1/27/17 1:15 PM, Saeed Mahameed wrote: It is only mandatory for configurations that needs eswitch, where the driver has no way to know about them, for a good old bare metal box, eswitch is not needed. we can do some work to strip the l2 table logic - needed for PFs to work on multi-host -

Re: [PATCH net-next 1/4] mlx5: Make building eswitch configurable

2017-01-27 Thread Saeed Mahameed
On Fri, Jan 27, 2017 at 8:42 PM, Tom Herbert wrote: > On Fri, Jan 27, 2017 at 10:28 AM, Saeed Mahameed > wrote: >> On Fri, Jan 27, 2017 at 8:16 PM, Tom Herbert wrote: >>> On Fri, Jan 27, 2017 at 10:05 AM, Saeed Mahameed >>>

Re: [PATCH net-next 1/4] mlx5: Make building eswitch configurable

2017-01-27 Thread Saeed Mahameed
On Fri, Jan 27, 2017 at 8:33 PM, Tom Herbert wrote: > On Fri, Jan 27, 2017 at 10:19 AM, Saeed Mahameed > wrote: >> On Fri, Jan 27, 2017 at 1:32 AM, Tom Herbert wrote: >>> Add a configuration option (CONFIG_MLX5_CORE_ESWITCH)

Re: [PATCH net-next 1/4] mlx5: Make building eswitch configurable

2017-01-27 Thread Tom Herbert
On Fri, Jan 27, 2017 at 10:19 AM, Saeed Mahameed wrote: > On Fri, Jan 27, 2017 at 1:32 AM, Tom Herbert wrote: >> Add a configuration option (CONFIG_MLX5_CORE_ESWITCH) for controlling >> whether the eswitch code is built. Change Kconfig and

Re: [PATCH net-next 1/4] mlx5: Make building eswitch configurable

2017-01-27 Thread Tom Herbert
On Fri, Jan 27, 2017 at 10:28 AM, Saeed Mahameed wrote: > On Fri, Jan 27, 2017 at 8:16 PM, Tom Herbert wrote: >> On Fri, Jan 27, 2017 at 10:05 AM, Saeed Mahameed >> wrote: >>> On Fri, Jan 27, 2017 at 7:50 PM, Tom

Re: [PATCH net-next 1/4] mlx5: Make building eswitch configurable

2017-01-27 Thread Saeed Mahameed
On Fri, Jan 27, 2017 at 1:32 AM, Tom Herbert wrote: > Add a configuration option (CONFIG_MLX5_CORE_ESWITCH) for controlling > whether the eswitch code is built. Change Kconfig and Makefile > accordingly. > > Signed-off-by: Tom Herbert > --- >

Re: [PATCH net-next 1/4] mlx5: Make building eswitch configurable

2017-01-27 Thread Saeed Mahameed
On Fri, Jan 27, 2017 at 7:50 PM, Tom Herbert wrote: > On Fri, Jan 27, 2017 at 9:38 AM, Saeed Mahameed > wrote: >> On Fri, Jan 27, 2017 at 7:34 AM, Or Gerlitz wrote: >>> On Fri, Jan 27, 2017 at 1:32 AM, Tom Herbert

Re: [PATCH net-next 1/4] mlx5: Make building eswitch configurable

2017-01-27 Thread Saeed Mahameed
On Fri, Jan 27, 2017 at 8:16 PM, Tom Herbert wrote: > On Fri, Jan 27, 2017 at 10:05 AM, Saeed Mahameed > wrote: >> On Fri, Jan 27, 2017 at 7:50 PM, Tom Herbert wrote: >>> On Fri, Jan 27, 2017 at 9:38 AM, Saeed Mahameed >>>

Re: [PATCH net-next 1/4] mlx5: Make building eswitch configurable

2017-01-27 Thread Tom Herbert
On Fri, Jan 27, 2017 at 10:05 AM, Saeed Mahameed wrote: > On Fri, Jan 27, 2017 at 7:50 PM, Tom Herbert wrote: >> On Fri, Jan 27, 2017 at 9:38 AM, Saeed Mahameed >> wrote: >>> On Fri, Jan 27, 2017 at 7:34 AM, Or Gerlitz

Re: [PATCH net-next 1/4] mlx5: Make building eswitch configurable

2017-01-27 Thread Tom Herbert
On Fri, Jan 27, 2017 at 9:38 AM, Saeed Mahameed wrote: > On Fri, Jan 27, 2017 at 7:34 AM, Or Gerlitz wrote: >> On Fri, Jan 27, 2017 at 1:32 AM, Tom Herbert wrote: >>> Add a configuration option (CONFIG_MLX5_CORE_ESWITCH) for

Re: [PATCH net-next 1/4] mlx5: Make building eswitch configurable

2017-01-27 Thread Saeed Mahameed
On Fri, Jan 27, 2017 at 7:34 AM, Or Gerlitz wrote: > On Fri, Jan 27, 2017 at 1:32 AM, Tom Herbert wrote: >> Add a configuration option (CONFIG_MLX5_CORE_ESWITCH) for controlling >> whether the eswitch code is built. Change Kconfig and Makefile >>

Re: [PATCH net-next 1/4] mlx5: Make building eswitch configurable

2017-01-26 Thread Or Gerlitz
On Fri, Jan 27, 2017 at 1:32 AM, Tom Herbert wrote: > Add a configuration option (CONFIG_MLX5_CORE_ESWITCH) for controlling > whether the eswitch code is built. Change Kconfig and Makefile > accordingly. Tom, FWIW, please note that the basic e-switch functionality is needed

[PATCH net-next 1/4] mlx5: Make building eswitch configurable

2017-01-26 Thread Tom Herbert
Add a configuration option (CONFIG_MLX5_CORE_ESWITCH) for controlling whether the eswitch code is built. Change Kconfig and Makefile accordingly. Signed-off-by: Tom Herbert --- drivers/net/ethernet/mellanox/mlx5/core/Kconfig | 11 +++