Re: [ovs-dev] [PATCH v2 3/5] netdev-linux: Rename sturct nedev_linux field as is_lag_primary

2024-03-06 Thread Simon Horman
On Fri, Mar 01, 2024 at 05:41:44PM +0100, Eelco Chaudron wrote: > > > On 1 Mar 2024, at 15:50, Simon Horman wrote: > > > Recently OVS adopted a policy of using the inclusive naming word list v1 > > [1, 2]. > > > > This patch partially addresses the use of the term master in the > > context of LA

Re: [ovs-dev] [PATCH v2 3/5] netdev-linux: Rename sturct nedev_linux field as is_lag_primary

2024-03-01 Thread Eelco Chaudron
On 1 Mar 2024, at 15:50, Simon Horman wrote: > Recently OVS adopted a policy of using the inclusive naming word list v1 > [1, 2]. > > This patch partially addresses the use of the term master in the > context of LAG devices by using the term primary instead: the > is_lag_master field of struct

Re: [ovs-dev] [PATCH v2 3/5] netdev-linux: Rename sturct nedev_linux field as is_lag_primary

2024-03-01 Thread 0-day Robot
Bleep bloop. Greetings Simon Horman, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. checkpatch: WARNING: The subject summary should end with a dot. Subject: netdev-linux: Rename sturct nedev_

[ovs-dev] [PATCH v2 3/5] netdev-linux: Rename sturct nedev_linux field as is_lag_primary

2024-03-01 Thread Simon Horman
Recently OVS adopted a policy of using the inclusive naming word list v1 [1, 2]. This patch partially addresses the use of the term master in the context of LAG devices by using the term primary instead: the is_lag_master field of struct netdev_linux is renamed is_lag_primary. A related comment i