mbox series

[ovs-dev,v2,0/5] DEI: Address some instances of master and slave

Message ID 20240301-dei-t1-v2-0-10ee89c17b89@ovn.org
Headers show
Series DEI: Address some instances of master and slave | expand

Message

Simon Horman March 1, 2024, 2:50 p.m. UTC
Recently OVS adopted a policy of using the inclusive naming word list v1
[1, 2].

This patch-set addresses some uses of the terms master and slave
by using alternate terms.

These changes are not intended to have any runtime effect.

[1] df5e5cf ("Documentation: Add section on inclusive language.")
[2] https://inclusivenaming.org/word-lists/

---
Simon Horman (5):
      vswitch.xml: Use member wording for bonds.
      Documentation: Update to refer to main repository.
      netdev-linux: Rename sturct nedev_linux field as is_lag_primary
      netdev-linux: Rename local variables as primary_*.
      utilities: Use localhost as sample hostname.

 .../internals/committer-grant-revocation.rst       |  2 +-
 Documentation/intro/install/dpdk.rst               |  2 +-
 lib/netdev-linux-private.h                         |  2 +-
 lib/netdev-linux.c                                 | 35 +++++++++++-----------
 utilities/usdt-scripts/kernel_delay.rst            | 10 +++----
 vswitchd/vswitch.xml                               |  4 +--
 6 files changed, 28 insertions(+), 27 deletions(-)

base-commit: cc0e7951818a48fbbd11664e1ad0a509a180b558

Comments

Kevin Traynor March 5, 2024, 4:08 p.m. UTC | #1
On 01/03/2024 14:50, Simon Horman wrote:
> Recently OVS adopted a policy of using the inclusive naming word list v1
> [1, 2].
> 
> This patch-set addresses some uses of the terms master and slave
> by using alternate terms.
> 
> These changes are not intended to have any runtime effect.
> 
> [1] df5e5cf ("Documentation: Add section on inclusive language.")
> [2] https://inclusivenaming.org/word-lists/
> 
> ---
> Simon Horman (5):
>       vswitch.xml: Use member wording for bonds.
>       Documentation: Update to refer to main repository.
>       netdev-linux: Rename sturct nedev_linux field as is_lag_primary
>       netdev-linux: Rename local variables as primary_*.
>       utilities: Use localhost as sample hostname.
> 
>  .../internals/committer-grant-revocation.rst       |  2 +-
>  Documentation/intro/install/dpdk.rst               |  2 +-
>  lib/netdev-linux-private.h                         |  2 +-
>  lib/netdev-linux.c                                 | 35 +++++++++++-----------
>  utilities/usdt-scripts/kernel_delay.rst            | 10 +++----
>  vswitchd/vswitch.xml                               |  4 +--
>  6 files changed, 28 insertions(+), 27 deletions(-)
> 
> base-commit: cc0e7951818a48fbbd11664e1ad0a509a180b558
> 
> _______________________________________________
> dev mailing list
> dev@openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
> 

Series LGTM (nit: typo sturct in one of the titles). Thanks Simon.

Acked-by: Kevin Traynor <ktraynor@redhat.com>
Simon Horman March 6, 2024, 10:08 a.m. UTC | #2
On Tue, Mar 05, 2024 at 04:08:19PM +0000, Kevin Traynor wrote:
> On 01/03/2024 14:50, Simon Horman wrote:
> > Recently OVS adopted a policy of using the inclusive naming word list v1
> > [1, 2].
> > 
> > This patch-set addresses some uses of the terms master and slave
> > by using alternate terms.
> > 
> > These changes are not intended to have any runtime effect.
> > 
> > [1] df5e5cf ("Documentation: Add section on inclusive language.")
> > [2] https://inclusivenaming.org/word-lists/
> > 
> > ---
> > Simon Horman (5):
> >       vswitch.xml: Use member wording for bonds.
> >       Documentation: Update to refer to main repository.
> >       netdev-linux: Rename sturct nedev_linux field as is_lag_primary
> >       netdev-linux: Rename local variables as primary_*.
> >       utilities: Use localhost as sample hostname.
> > 
> >  .../internals/committer-grant-revocation.rst       |  2 +-
> >  Documentation/intro/install/dpdk.rst               |  2 +-
> >  lib/netdev-linux-private.h                         |  2 +-
> >  lib/netdev-linux.c                                 | 35 +++++++++++-----------
> >  utilities/usdt-scripts/kernel_delay.rst            | 10 +++----
> >  vswitchd/vswitch.xml                               |  4 +--
> >  6 files changed, 28 insertions(+), 27 deletions(-)
> > 
> > base-commit: cc0e7951818a48fbbd11664e1ad0a509a180b558
> > 
> > _______________________________________________
> > dev mailing list
> > dev@openvswitch.org
> > https://mail.openvswitch.org/mailman/listinfo/ovs-dev
> > 
> 
> Series LGTM (nit: typo sturct in one of the titles). Thanks Simon.

Thanks,

spelling of struct corrected and trailing '.' added to title of same patch.

> Acked-by: Kevin Traynor <ktraynor@redhat.com>
>
Simon Horman March 6, 2024, 10:45 a.m. UTC | #3
On Fri, Mar 01, 2024 at 02:50:29PM +0000, Simon Horman wrote:
> Recently OVS adopted a policy of using the inclusive naming word list v1
> [1, 2].
> 
> This patch-set addresses some uses of the terms master and slave
> by using alternate terms.
> 
> These changes are not intended to have any runtime effect.
> 
> [1] df5e5cf ("Documentation: Add section on inclusive language.")
> [2] https://inclusivenaming.org/word-lists/

Thanks Eelco and Kevin for your review,
I have applied this series.

- utilities: Use localhost as sample hostname.
  https://github.com/openvswitch/ovs/commit/e0aa15f897f1
- netdev-linux: Rename local variables as primary_*.
  https://github.com/openvswitch/ovs/commit/b3ebc34a065e
- netdev-linux: Rename struct nedev_linux field as is_lag_primary.
  https://github.com/openvswitch/ovs/commit/f92b30a0ff88
- Documentation: Update to refer to main repository.
  https://github.com/openvswitch/ovs/commit/0c255bf763cc
- vswitch.xml: Use member wording for bonds.
  https://github.com/openvswitch/ovs/commit/29e09c80916c