mbox series

[0/3] Vhost-user: replace _SLAVE_ with _BACKEND_

Message ID 20230130104548.13262-1-maxime.coquelin@redhat.com
Headers show
Series Vhost-user: replace _SLAVE_ with _BACKEND_ | expand

Message

Maxime Coquelin Jan. 30, 2023, 10:45 a.m. UTC
This series continues the work done to get rid of harmful
language in the Vhost-user specification.

While the spec texts were changed to replace slave with
backend, the protocol features and messages names hadn't
been changed. 

This series renames remaining occurences in the spec and
make use of the new names in both libvhost-user and the
Vhost-user frontend code.

Maxime Coquelin (3):
  docs: vhost-user: replace _SLAVE_ with _BACKEND_
  libvhost-user: Adopt new backend naming
  vhost-user: Adopt new backend naming

 docs/interop/vhost-user.rst               | 40 +++++++++++------------
 hw/virtio/vhost-user.c                    | 30 ++++++++---------
 hw/virtio/virtio-qmp.c                    | 12 +++----
 subprojects/libvhost-user/libvhost-user.c | 20 ++++++------
 subprojects/libvhost-user/libvhost-user.h | 20 ++++++------
 5 files changed, 61 insertions(+), 61 deletions(-)

Comments

Michael S. Tsirkin Jan. 30, 2023, 11:08 a.m. UTC | #1
On Mon, Jan 30, 2023 at 11:45:45AM +0100, Maxime Coquelin wrote:
> This series continues the work done to get rid of harmful
> language in the Vhost-user specification.

I prefer a positive "switch to a more inclusive terminology".
To consider if you keep doing this work.

> While the spec texts were changed to replace slave with
> backend, the protocol features and messages names hadn't
> been changed. 
> 
> This series renames remaining occurences in the spec and
> make use of the new names in both libvhost-user and the
> Vhost-user frontend code.
> 
> Maxime Coquelin (3):
>   docs: vhost-user: replace _SLAVE_ with _BACKEND_
>   libvhost-user: Adopt new backend naming
>   vhost-user: Adopt new backend naming
> 
>  docs/interop/vhost-user.rst               | 40 +++++++++++------------
>  hw/virtio/vhost-user.c                    | 30 ++++++++---------
>  hw/virtio/virtio-qmp.c                    | 12 +++----
>  subprojects/libvhost-user/libvhost-user.c | 20 ++++++------
>  subprojects/libvhost-user/libvhost-user.h | 20 ++++++------
>  5 files changed, 61 insertions(+), 61 deletions(-)
> 
> -- 
> 2.39.1
Maxime Coquelin Jan. 30, 2023, 11:48 a.m. UTC | #2
On 1/30/23 12:08, Michael S. Tsirkin wrote:
> On Mon, Jan 30, 2023 at 11:45:45AM +0100, Maxime Coquelin wrote:
>> This series continues the work done to get rid of harmful
>> language in the Vhost-user specification.
> 
> I prefer a positive "switch to a more inclusive terminology".
> To consider if you keep doing this work.

Right, it is indeed better. I will post a new revision using positive
wording.


Thanks for the review,
Maxime

>> While the spec texts were changed to replace slave with
>> backend, the protocol features and messages names hadn't
>> been changed.
>>
>> This series renames remaining occurences in the spec and
>> make use of the new names in both libvhost-user and the
>> Vhost-user frontend code.
>>
>> Maxime Coquelin (3):
>>    docs: vhost-user: replace _SLAVE_ with _BACKEND_
>>    libvhost-user: Adopt new backend naming
>>    vhost-user: Adopt new backend naming
>>
>>   docs/interop/vhost-user.rst               | 40 +++++++++++------------
>>   hw/virtio/vhost-user.c                    | 30 ++++++++---------
>>   hw/virtio/virtio-qmp.c                    | 12 +++----
>>   subprojects/libvhost-user/libvhost-user.c | 20 ++++++------
>>   subprojects/libvhost-user/libvhost-user.h | 20 ++++++------
>>   5 files changed, 61 insertions(+), 61 deletions(-)
>>
>> -- 
>> 2.39.1
>