From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:35695)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <yuanhan.liu@linux.intel.com>) id 1aqAsE-0004ms-W4
	for qemu-devel@nongnu.org; Tue, 12 Apr 2016 22:49:47 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <yuanhan.liu@linux.intel.com>) id 1aqAs9-0003cM-WF
	for qemu-devel@nongnu.org; Tue, 12 Apr 2016 22:49:46 -0400
Received: from mga03.intel.com ([134.134.136.65]:33756)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <yuanhan.liu@linux.intel.com>) id 1aqAs9-0003cD-ND
	for qemu-devel@nongnu.org; Tue, 12 Apr 2016 22:49:41 -0400
Date: Wed, 13 Apr 2016 10:52:29 +0800
From: Yuanhan Liu <yuanhan.liu@linux.intel.com>
Message-ID: <20160413025229.GN3080@yliu-dev.sh.intel.com>
References: <1459509388-6185-1-git-send-email-marcandre.lureau@redhat.com>
	<1459509388-6185-19-git-send-email-marcandre.lureau@redhat.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <1459509388-6185-19-git-send-email-marcandre.lureau@redhat.com>
Subject: Re: [Qemu-devel] [PATCH 18/18] test: add shutdown support vubr test
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel/>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: marcandre.lureau@redhat.com
Cc: qemu-devel@nongnu.org, "Michael S. Tsirkin" <mst@redhat.com>, Tetsuya Mukawa <mukawa@igel.co.jp>, jonshin@cisco.com, Ilya Maximets <i.maximets@samsung.com>

On Fri, Apr 01, 2016 at 01:16:28PM +0200, marcandre.lureau@redhat.com wrote:
> +static void
> +vubr_handle_slave_reply(VhostUserMsg *vmsg)
> +{
> +    DPRINT(
> +        "==================   Vhost slave reply from QEMU   ==================\n");
> +    DPRINT("Request: %s (%d)\n", vubr_slave_request_str[vmsg->request],
> +           vmsg->request);
> +    DPRINT("Flags:   0x%x\n", vmsg->flags);
> +    DPRINT("Size:    %d\n", vmsg->size);
> +
> +    switch (vmsg->request) {
> +    case VHOST_USER_SLAVE_SHUTDOWN:
> +        DPRINT("Shutdown success: 0x%016"PRIx64"\n", vmsg->payload.u64);
> +        if (vmsg->payload.u64 == 0) {
> +            exit(0);
> +        }
> +    default:
> +        DPRINT("Invalid slave reply");
> +    };
       ^^

Minor nit: redundant ';'.

	--yliu