Re: LRO: creating vlan subports affects parent port's LRO settings

2020-12-06 Thread Limin Wang
I might be wrong. One potential issue I found in netdev_sync_upper_features() is that it depends on the wanted_feature of upper_dev if (!(upper->wanted_features & feature) && (features & feature)) { netdev_dbg(lower, "Dropping feature %pNF, upper dev %s has it off.\n", &feature, upper->name);

Re: LRO: creating vlan subports affects parent port's LRO settings

2020-12-06 Thread Limin Wang
Thanks to Jakub, Michal and Jarod for the time and consideration. I have been caught up with other stuff, and not spent much time on this since. I am not that familiar with the upper/lower sync logic and implications. But I agree with Michal, it may not be necessary to have a blanket exclusion of

Re: LRO: creating vlan subports affects parent port's LRO settings

2020-12-06 Thread Jarod Wilson
On Sun, Dec 6, 2020 at 11:49 AM Michal Kubecek wrote: > > On Sat, Dec 05, 2020 at 07:04:06PM -0500, Jarod Wilson wrote: > > On Mon, Nov 23, 2020 at 7:27 PM Jakub Kicinski wrote: > > > > > > On Thu, 19 Nov 2020 20:37:27 -0500 Limin Wang wrote: > > > > Under relatively recent kernels (v4.4+), creat

Re: LRO: creating vlan subports affects parent port's LRO settings

2020-12-06 Thread Michal Kubecek
On Sat, Dec 05, 2020 at 07:04:06PM -0500, Jarod Wilson wrote: > On Mon, Nov 23, 2020 at 7:27 PM Jakub Kicinski wrote: > > > > On Thu, 19 Nov 2020 20:37:27 -0500 Limin Wang wrote: > > > Under relatively recent kernels (v4.4+), creating a vlan subport on a > > > LRO supported parent NIC may turn LRO

Re: LRO: creating vlan subports affects parent port's LRO settings

2020-12-05 Thread Jarod Wilson
On Mon, Nov 23, 2020 at 7:27 PM Jakub Kicinski wrote: > > On Thu, 19 Nov 2020 20:37:27 -0500 Limin Wang wrote: > > Under relatively recent kernels (v4.4+), creating a vlan subport on a > > LRO supported parent NIC may turn LRO off on the parent port and > > further render its LRO feature practical

Re: LRO: creating vlan subports affects parent port's LRO settings

2020-11-23 Thread Jakub Kicinski
On Thu, 19 Nov 2020 20:37:27 -0500 Limin Wang wrote: > Under relatively recent kernels (v4.4+), creating a vlan subport on a > LRO supported parent NIC may turn LRO off on the parent port and > further render its LRO feature practically unchangeable. That does sound like an oversight in commit fd8

LRO: creating vlan subports affects parent port's LRO settings

2020-11-19 Thread Limin Wang
Under relatively recent kernels (v4.4+), creating a vlan subport on a LRO supported parent NIC may turn LRO off on the parent port and further render its LRO feature practically unchangeable. This can be easily reproduced on different distros, and independent of NIC vendors. Hopefully, this is not