[dpdk-dev] [PATCH 1/2] net/tap: remove useless offload capa functions
Wiles, Keith
keith.wiles at intel.com
Thu Jul 1 21:19:52 CEST 2021
> On Jul 1, 2021, at 12:18 PM, Stephen Hemminger <stephen at networkplumber.org> wrote:
>
> On Thu, 1 Jul 2021 17:16:21 +0300
> Andrew Rybchenko <andrew.rybchenko at oktetlabs.ru> wrote:
>
>> On 6/16/21 7:15 AM, Stephen Hemminger wrote:
>>> Since these always return 0, they were doing nothing useful.
>>>
>>> Signed-off-by: Stephen Hemminger <stephen at networkplumber.org>
>>
>> I have no strong opinion on the patch, but sometimes even
>> empty functions with comments add value. So, I see no point
>> to touch it. So, I'll wait for maintainer reply.
>
> There are always many ways to write more obscure and technically correct
> code. It is best if the code is the smallest correct way.
>
> “I apologize for such a long letter - I didn't have time to write a short one.”
> ― Mark Twain
I agree with Stephen, the extra functions provide no useful feature or even the comments are not very useful IMO.
More information about the dev
mailing list