From mboxrd@z Thu Jan 1 00:00:00 1970 From: Martin Pohlack Subject: Re: [PATCH v1 1/4] xsm/libxl/xen_version: Add XSM for some of the xen_version commands. Date: Fri, 30 Oct 2015 11:24:36 +0100 Message-ID: <56334564.30501@amazon.com> References: <1444359390-14153-1-git-send-email-konrad.wilk@oracle.com> <1444359390-14153-2-git-send-email-konrad.wilk@oracle.com> <1444383111.1410.314.camel@citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail6.bemta14.messagelabs.com ([193.109.254.103]) by lists.xen.org with esmtp (Exim 4.72) (envelope-from ) id 1Zs6rt-0005E7-Qo for xen-devel@lists.xenproject.org; Fri, 30 Oct 2015 10:25:09 +0000 In-Reply-To: <1444383111.1410.314.camel@citrix.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Ian Campbell , Konrad Rzeszutek Wilk , xen-devel@lists.xenproject.org, wei.liu2@citrix.com, ian.jackson@eu.citrix.com, jbeulich@suse.com, andrew.cooper3@citrix.com, mpohlack@amazon.de, dgdegra@tycho.nsa.gov List-Id: xen-devel@lists.xenproject.org On 09.10.2015 11:31, Ian Campbell wrote: > On Thu, 2015-10-08 at 22:56 -0400, Konrad Rzeszutek Wilk wrote: >> The XENVER_[compile_info|changeset|commandline] are now >> guarded by an XSM check. > > I can guess, but please explain/justify why this is the case for these > here. > >> The rest: XENVER_[version|extraversion|capabilities| >> parameters|get_features|page_size|guest_handle] behave >> as before (no XSM check). > > and correspondingly why these ones to not warrant such a change. > >> As such we also modify the toolstack such that if we fail >> to get any data instead of printing (null) we just print "". > > Perhaps the hypervisor should instead return "" or some suitable > string indicating why ()? Yes, that makes it more obvious to userland what is going on and when debugging and might limit userland breakage where at least some string is required. Martin Amazon Development Center Germany GmbH Krausenstr. 38 10117 Berlin Geschaeftsfuehrer: Dr. Ralf Herbrich, Christian Schlaeger Ust-ID: DE289237879 Eingetragen am Amtsgericht Charlottenburg HRB 149173 B