From mboxrd@z Thu Jan  1 00:00:00 1970
From: Dario Faggioli <dario.faggioli@citrix.com>
Subject: Re: [PATCH v2 2/5] libxl: make GC_FREE reachable in
 libxl_get_scheduler()
Date: Tue, 19 Jan 2016 10:08:41 +0100
Message-ID: <1453194521.11427.112.camel@citrix.com>
References: <1451992597.13361.231.camel@citrix.com>
	<1453183061-50484-1-git-send-email-czylin@uwaterloo.ca>
Mime-Version: 1.0
Content-Type: multipart/mixed; boundary="===============1697141763595687860=="
Return-path: <xen-devel-bounces@lists.xen.org>
In-Reply-To: <1453183061-50484-1-git-send-email-czylin@uwaterloo.ca>
List-Unsubscribe: <http://lists.xen.org/cgi-bin/mailman/options/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=unsubscribe>
List-Post: <mailto:xen-devel@lists.xen.org>
List-Help: <mailto:xen-devel-request@lists.xen.org?subject=help>
List-Subscribe: <http://lists.xen.org/cgi-bin/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=subscribe>
Sender: xen-devel-bounces@lists.xen.org
Errors-To: xen-devel-bounces@lists.xen.org
To: Chester Lin <czylin@uwaterloo.ca>, xen-devel@lists.xen.org
Cc: wei.liu2@citrix.com, ian.campbell@citrix.com, stefano.stabellini@eu.citrix.com, ian.jackson@eu.citrix.com, jtotto@uwaterloo.ca, hjarmstr@uwaterloo.ca
List-Id: xen-devel@lists.xenproject.org

--===============1697141763595687860==
Content-Type: multipart/signed; micalg=pgp-sha1;
	protocol="application/pgp-signature"; boundary="=-2ymWy+ATd8iOZYhb7Tbi"

--=-2ymWy+ATd8iOZYhb7Tbi
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

On Tue, 2016-01-19 at 00:57 -0500, Chester Lin wrote:
> Coverity CID 1343309
>=20
> Make GC_FREE reachable in all cases in libxl_get_scheduler() by
> eliminating the error-path return and instead storing the error code
> in
> the returned variable.
>=20
> To make this semantically consistent, change the return type of
> libxl_get_scheduler() from libxl_scheduler to int, and make a note of
> the interpretation of the return value in libxl.h.=C2=A0=C2=A0N.B. This c=
hange
> breaks neither the API nor the ABI of libxl.
>
Not that I feel too strong about this, but I would reword this last
sentence a bit. In fact, ABI, AFAIK, we don't care. API, someone could
argue that it does actually break it, it's just the case that we don't
think it breaks it in any ways that we should care.

And maybe we should also add a note about the libxl_scheduler enum
being (and needing to continue to do so) consistent with what
xc_sched_id returns, like it's been done in another patch of this
series?

Anyway, that's all up for the tools maintainers to judge... The patch
seems to me to do what was asked during v1 review, so:

> Suggested-by: Ian Campbell <ian.campbell@citrix.com>
> Signed-off-by: Chester Lin <czylin@uwaterloo.ca>
>=20
Reviewed-by: Dario Faggioli <dario.faggioli@citrix.com>

Regards,
Dario
--=20
<<This happens because I choose it to happen!>> (Raistlin Majere)
-----------------------------------------------------------------
Dario Faggioli, Ph.D, http://about.me/dario.faggioli
Senior Software Engineer, Citrix Systems R&D Ltd., Cambridge (UK)


--=-2ymWy+ATd8iOZYhb7Tbi
Content-Type: application/pgp-signature; name="signature.asc"
Content-Description: This is a digitally signed message part
Content-Transfer-Encoding: 7bit

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1

iEYEABECAAYFAlad/RoACgkQk4XaBE3IOsTRSACeOxkqKCKZ2q43Sbi3j7tJehPb
8XQAn2jyA3tLmdAJ5URYX4PRpElMIT59
=s6/x
-----END PGP SIGNATURE-----

--=-2ymWy+ATd8iOZYhb7Tbi--


--===============1697141763595687860==
Content-Type: text/plain; charset="us-ascii"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Content-Disposition: inline

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

--===============1697141763595687860==--