Re: [PATCH v2] docs: Update vhost-user spec regarding backend program conventions

From: Stefan Hajnoczi <stefanha@gmail.com>
To: Sebastien Boeuf <sebastien.boeuf@intel.com>
Cc: marcandre.lureau@redhat.com, berrange@redhat.com,
	qemu-devel@nongnu.org, stefanha@redhat.com, mst@redhat.com
Subject: Re: [PATCH v2] docs: Update vhost-user spec regarding backend program conventions
Date: Fri, 6 Mar 2020 12:34:53 +0000	[thread overview]
Message-ID: <20200306123453.GI1335569@stefanha-x1.localdomain> (raw)
In-Reply-To: <20200224182454.24610-1-sebastien.boeuf@intel.com>

[-- Attachment #1: Type: text/plain, Size: 888 bytes --]

On Mon, Feb 24, 2020 at 07:24:54PM +0100, Sebastien Boeuf wrote:
> The vhost-user specification is not clearly stating the expected
> behavior from a backend program whenever the client disconnects.
> 
> This patch addresses the issue by defining the default behavior and
> proposing an alternative through a command line option.
> 
> By default, a backend program will terminate whenever the client
> disconnects, unless told otherwise through the newly introduced
> option --keep-listening.
> 
> Signed-off-by: Sebastien Boeuf <sebastien.boeuf@intel.com>
> Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
> ---
>  docs/interop/vhost-user.rst | 12 ++++++++++++
>  1 file changed, 12 insertions(+)

Perhaps --keep-listening should be optional?

If the maintainers are happy with the patch as is then so am I:

Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]