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 > > >> 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. >> */ >> >