On 10/4/23 14:54, Michael S. Tsirkin wrote: > On Wed, Oct 04, 2023 at 12:08:52PM +0200, Laszlo Ersek wrote: >> On 10/4/23 11:06, Michael S. Tsirkin wrote: >>> On Mon, Oct 02, 2023 at 10:32:15PM +0200, Laszlo Ersek wrote: >>>> Cc: "Michael S. Tsirkin" <m...@redhat.com> (supporter:vhost) >>> >>> why the (supporter:vhost) part? not all scripts will cope >>> well with text after the mail. If you really want to keep >>> it around, I think you should add a hash tag # before that - >>> more tools know to ignore that. >> >> It looked too tiresome to strip all these comments, plus I expected >> that, if the get_maintainer.pl script output these lines, they were fit >> for inclusion in "Cc:" tags in the commit message. >> >> If they're not, then the tool should indeed insert a # in-between, or >> else provide the explanation for each name+email printed on separate >> (preceding) lines, potentially prefixed with "#". That makes for easy >> human reading and also for easy machine reading (filtering them out). >> >> Laszlo > > /me shrugs > > get_maintainer.pl doesn't output Cc tags either. Just pipe to > sed 's/(.*//' ?
Yes, I'll seek to remember that. Laszlo > >>> >>> >>>> Cc: Eugenio Perez Martin <epere...@redhat.com> >>>> Cc: German Maglione <gmagli...@redhat.com> >>>> Cc: Liu Jiang <ge...@linux.alibaba.com> >>>> Cc: Sergio Lopez Pascual <s...@redhat.com> >>>> Cc: Stefano Garzarella <sgarz...@redhat.com> >>>> Signed-off-by: Laszlo Ersek <ler...@redhat.com> >>>> Reviewed-by: Stefano Garzarella <sgarz...@redhat.com> >>>> Reviewed-by: Philippe Mathieu-Daudé <phi...@linaro.org> >>>> Tested-by: Albert Esteve <aest...@redhat.com> >>>> Reviewed-by: Eugenio Pérez <epere...@redhat.com> >>>> --- >>>> >>>> Notes: >>>> v3: >>>> >>>> - pick up R-b from Phil and Eugenio, T-b from Albert >>>> >>>> v2: >>>> >>>> - pick up Stefano's R-b >>>> >>>> hw/virtio/vhost-user.c | 2 +- >>>> 1 file changed, 1 insertion(+), 1 deletion(-) >>>> >>>> diff --git a/hw/virtio/vhost-user.c b/hw/virtio/vhost-user.c >>>> index 8dcf049d422b..b4b677c1ce66 100644 >>>> --- a/hw/virtio/vhost-user.c >>>> +++ b/hw/virtio/vhost-user.c >>>> @@ -398,7 +398,7 @@ static int vhost_user_write(struct vhost_dev *dev, >>>> VhostUserMsg *msg, >>>> * operations such as configuring device memory mappings or issuing >>>> device >>>> * resets, which affect the whole device instead of individual VQs, >>>> * vhost-user messages should only be sent once. >>>> - * >>>> + * >>>> * Devices with multiple vhost_devs are given an associated >>>> dev->vq_index >>>> * so per_device requests are only sent if vq_index is 0. >>>> */ >>>> >>> >