Bug#949863: [Vasudev Kamath] Re: Bug#949863: Info received (#949863: please enable CONFIG_NET_SWITCHDEV)

2020-07-02 Thread Vasudev Kamath

Sending it to BTS to track in bug log.

--- Begin Message ---

[Sending reply to bug so we can have a record of discussion]

[Please keep Ashruth and my colleague in CC while replying]

Tzafrir Cohen  writes:
[snip]
>> 
>> I don't think we should make a config change that hurts users of stock
>> Debian, for the benefit of those using out-of-tree drivers.  And in a
>> stable point release, a known regression like that is unacceptable.
>
> Could you please clarify: is that a regression? That is: in the current
> Buster kernel the problem does not exist?

Yes it looks like regression. Had a discussion with Mellanox (Ashruth
Cced) and looks like its fixed upstream [1]. Currently we found a work
around for this that is to make bonding enabled first and then enable
SR-IOV. This bug happens only if we have enabled SR-IOV on slave
interface  before enabling bond.


Ashruth is [1] the right patch?. If not can you point to right patch?.


Ben if a patch is available in main-stream can that be cherry picked to
stable point release to enable this config option?.

[1] 
https://github.com/torvalds/linux/commit/7ff40a46dd188b83311203e72cedf42eb264fdf1


Cheers,
Vasudev
--- End Message ---


Bug#949863: Info received (#949863: please enable CONFIG_NET_SWITCHDEV)

2020-04-01 Thread Vasudev Kamath
Ben Hutchings  writes:

> On Mon, 9 Mar 2020 13:58:38 +0200 Tzafrir Cohen  wrote:
>> Hi,
>> 
>> > Also: please consider this change for inclusion in a stable update, if
>> > possible.
>> 
>> I see that this was merged into git. Thanks. What are the chances of
>> this fix getting included into Debian 10.4 to allow OVS off-loading there?
>
> I think it's reasonable, but will need to consult with the release
> team.  In case this should only happen after the change has gone into
> unstable and testing.

Wanted to update my findings on Buster with this patch. We use Mellanox
Connect-X5 series nic (dual port) and have a bonded setup.

Both ports are bonded with 802.3ad and the bond device is put on a
bridge (br0) and br0 gets the IP address.

In this setup with CONFIG_NET_SWITCHDEV enabled bond0 is not placed on
br0 and I see a message:

ifup: can't add bond0 to bridge br0: No data available

This happens with Mellanox inbox driver in Kernel but if I install
latest OFED there is no issue. So I'm not sure how other NIC's behave
with this option enabled.

Cheers,



Bug#949863: Info received (#949863: please enable CONFIG_NET_SWITCHDEV)

2020-03-19 Thread Ben Hutchings
On Mon, 9 Mar 2020 13:58:38 +0200 Tzafrir Cohen  wrote:
> Hi,
> 
> > Also: please consider this change for inclusion in a stable update, if
> > possible.
> 
> I see that this was merged into git. Thanks. What are the chances of
> this fix getting included into Debian 10.4 to allow OVS off-loading there?

I think it's reasonable, but will need to consult with the release
team.  In case this should only happen after the change has gone into
unstable and testing.

Ben.
 
-- 
Ben Hutchings
We get into the habit of living before acquiring the habit of thinking.
 - Albert Camus




signature.asc
Description: This is a digitally signed message part


Bug#949863: Info received (#949863: please enable CONFIG_NET_SWITCHDEV)

2020-03-09 Thread Tzafrir Cohen
Hi,

> Also: please consider this change for inclusion in a stable update, if
> possible.

I see that this was merged into git. Thanks. What are the chances of
this fix getting included into Debian 10.4 to allow OVS off-loading there?

Thanks,

-- Tzafrir