Re: [PATCH net-next] ipvlan: minor optimization for ipvlan outbound process

From: Yunsheng Lin
Date: Tue Nov 01 2022 - 23:48:13 EST


On 2022/11/2 11:23, Eric Dumazet wrote:
> On Tue, Nov 1, 2022 at 7:15 PM Yunsheng Lin <linyunsheng@xxxxxxxxxx> wrote:
>>
>> Avoid some local variable initialization and remove some
>> redundant assignment in ipvlan outbound process.
>>
>> Signed-off-by: Yunsheng Lin <linyunsheng@xxxxxxxxxx>
>
> Really I do not see the point of such a patch, making future backports
> more difficult.

As the ipvlan outbound process is in the fast path, avoiding the
unnecessary steps might be worth the backport cost.

Anyway, it is more of judgment call, not a rule, right?

>
> Changing old code like that should only be done if this is really necessary,
> for instance before adding a new functionality.
> .
>