From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Vrabel Subject: Re: [Xen-devel] [PATCH] xen-netback: correct return value checks on xenbus_scanf() Date: Thu, 7 Jul 2016 11:45:19 +0100 Message-ID: <577E32BF.9050007@citrix.com> References: <577E277C02000078000FBEBB@prv-mh.provo.novell.com> <577E27B8.2050603@citrix.com> <20160707103513.GB416@citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit Cc: , Jan Beulich , To: Wei Liu , David Vrabel Return-path: Received: from smtp.citrix.com ([66.165.176.89]:13191 "EHLO SMTP.CITRIX.COM" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751134AbcGGKpW (ORCPT ); Thu, 7 Jul 2016 06:45:22 -0400 In-Reply-To: <20160707103513.GB416@citrix.com> Sender: netdev-owner@vger.kernel.org List-ID: On 07/07/16 11:35, Wei Liu wrote: > On Thu, Jul 07, 2016 at 10:58:16AM +0100, David Vrabel wrote: >> On 07/07/16 08:57, Jan Beulich wrote: >>> Only a positive return value indicates success. >> >> This is not correct. >> > > Do you mean the commit message is not correct or the code is not > correct? If it is the formal, do you have any suggestion to fix it? This code is correct as-is, thus the commit message is wrong or misleading. David