Re: [Qemu-devel] [PULL 2/2] Delete AF_UNIX socket after close

[Eric Blake]

On 06/28/2018 06:02 AM, Daniel P. Berrangé wrote:
> From: Pavel Balaev <address@hidden>
>
> This is a second attempt at sending this patch:
>
> http://lists.nongnu.org/archive/html/qemu-devel/2018-05/msg04697.html

I'm not stopping the pull request, but this particular commit message is 
not very useful. A year from now, looking through 'git log' will tell us 
nothing about the "why" for this patch (the subject line only covers the 
"what").  And at that time, no one will care how many failed attempts 
went through the list, but only what actually got committed.  Better 
would have been just directly using the message from that mail:

> > Since version 2.12.0 AF_UNIX socket created for QMP exchange is not
> > deleted on instance shutdown.
> >
> > This is due to the fact that function qio_channel_socket_finalize() is
> > called after qio_channel_socket_close().

As a hint for future patches, mentioning that a post is a second version 
and replaces an earlier post to the list is best done after the --- line 
(where it is still readable on list as an aid to reviewers, but dropped 
by the maintainer using 'git am' as unnecessary fluff for the git log). 
More patch submission tips at: https://wiki.qemu.org/Contribute/SubmitAPatch

--
Eric Blake, Principal Software Engineer
Red Hat, Inc.           +1-919-301-3266
Virtualization:  qemu.org | libvirt.org