Re: [ethtool] ethtool: Remove UDP Fragmentation Offload use from ethtool

On 28/08/2017 9:22 PM, John W. Linville wrote:

On Mon, Aug 28, 2017 at 08:00:11AM -0700, Eric Dumazet wrote:

quoted

On Mon, 2017-08-28 at 15:38 +0300, Tariq Toukan wrote:

quoted

From: Shaker Daibes <redacted>

UFO was removed in kernel, here we remove it in ethtool app.

Fixes the following issue:
Features for ens8:
Cannot get device udp-fragmentation-offload settings: Operation not supported

Tested with "make check"

Signed-off-by: Shaker Daibes <redacted>
Signed-off-by: Tariq Toukan <redacted>
---

Hi guys

I would rather remove the warning, but leave the ability to switch UFO
on machines running old kernel but a recent ethtool.

ethtool does not need to be downgraded every time we boot an old
kernel ;)

Thanks all for your quick replies.

We thought about the backward compatibility issue before getting to writing this patch.
But, as the feature has very few device support, and is not that useful,
we thought it would be best to just totally remove it from ethtool.

We can re-work this so the feature would still be available on old kernels.

But I wonder how the warning removal should be done??

I have some suggestions in mind:
1) Have a special condition that does not print a warning only in the case of UFO?
2) Remove the warning totally? I don't like this option.
3) Add a max_kernel_ver field in struct off_flag_def, and use it to not print the warning, or to mark the feature 'off [fixed]'.

Please let me know what you think.

No, definitely not.
  

quoted

Thanks !

Tariq, will you be reworking this as Eric suggests?

Yes. Once we decide what is the correct way to keep it backward compatible.

John

Regards,
Tariq Toukan