From mboxrd@z Thu Jan 1 00:00:00 1970 From: Olaf Hering Subject: Re: Regression in xen-unstable due to commit 3802ecbaa9eb36 Date: Thu, 16 May 2019 14:18:05 +0200 Message-ID: <20190516141805.281f972e.ohering@suse.com> References: <1ae5e201-04d0-2f78-878f-2e3a3e213b62@suse.com> <20190516110735.67e19d15.ohering@suse.com> <20190516104540.7hiqsn6qjeaam2c6@Air-de-Roger> <20190516125735.6c1bf1ca.ohering@suse.com> <20190516112450.GW2798@zion.uk.xensource.com> <20190516133857.742e4f13.ohering@suse.com> <20190516115043.GB2798@zion.uk.xensource.com> <20190516140451.06cfcbfe.ohering@suse.com> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============5327555863726489374==" Return-path: Received: from all-amaz-eas1.inumbo.com ([34.197.232.57] helo=us1-amaz-eas2.inumbo.com) by lists.xenproject.org with esmtp (Exim 4.89) (envelope-from ) id 1hRFKy-0007dL-0F for xen-devel@lists.xenproject.org; Thu, 16 May 2019 12:18:16 +0000 In-Reply-To: <20190516140451.06cfcbfe.ohering@suse.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xenproject.org Sender: "Xen-devel" To: Wei Liu Cc: Juergen Gross , xen-devel , Ian Jackson , Roger Pau =?UTF-8?B?TW9ubsOp?= List-Id: xen-devel@lists.xenproject.org --===============5327555863726489374== Content-Type: multipart/signed; micalg=pgp-sha1; boundary="Sig_/s4/ms4yeVkDLgI369EHZpr8"; protocol="application/pgp-signature" --Sig_/s4/ms4yeVkDLgI369EHZpr8 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: quoted-printable Am Thu, 16 May 2019 14:04:51 +0200 schrieb Olaf Hering : > There are quite a few checks for device_model_version, they would be all > wrong if the assert is removed, or changed back to QEMU_XEN. Perhaps we > can continue to live with that error. device_model_version could become > a local variable. If it is not set, assume the caller just wants the > memory size and enforce QEMU_XEN again within that function. I think that is what should be done, just for the sake of libxl_domain_need_memory. If an incomplete b_info is provided, assume device_model_version=3DQEMU_XEN, maintain that state in a local variable. If we want to provide another new public API to fill missing defaults, that could still be done in a separate patch. Olaf --Sig_/s4/ms4yeVkDLgI369EHZpr8 Content-Type: application/pgp-signature Content-Description: Digitale Signatur von OpenPGP -----BEGIN PGP SIGNATURE----- iF0EARECAB0WIQSkRyP6Rn//f03pRUBdQqD6ppg2fgUCXN1U/QAKCRBdQqD6ppg2 fkffAKCNYh5INArE9YmuocvLyPFhjJvH1gCggi6fEGZWumStTgpQoLk95ed53mM= =9o5g -----END PGP SIGNATURE----- --Sig_/s4/ms4yeVkDLgI369EHZpr8-- --===============5327555863726489374== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: inline X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KWGVuLWRldmVs IG1haWxpbmcgbGlzdApYZW4tZGV2ZWxAbGlzdHMueGVucHJvamVjdC5vcmcKaHR0cHM6Ly9saXN0 cy54ZW5wcm9qZWN0Lm9yZy9tYWlsbWFuL2xpc3RpbmZvL3hlbi1kZXZlbA== --===============5327555863726489374==-- From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-1.0 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 3F4A1C46470 for ; Thu, 16 May 2019 12:18:30 +0000 (UTC) Received: from lists.xenproject.org (lists.xenproject.org [192.237.175.120]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 1584920848 for ; Thu, 16 May 2019 12:18:30 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 1584920848 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=suse.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=xen-devel-bounces@lists.xenproject.org Received: from localhost ([127.0.0.1] helo=lists.xenproject.org) by lists.xenproject.org with esmtp (Exim 4.89) (envelope-from ) id 1hRFKz-0007dQ-FE; Thu, 16 May 2019 12:18:17 +0000 Received: from all-amaz-eas1.inumbo.com ([34.197.232.57] helo=us1-amaz-eas2.inumbo.com) by lists.xenproject.org with esmtp (Exim 4.89) (envelope-from ) id 1hRFKy-0007dL-0F for xen-devel@lists.xenproject.org; Thu, 16 May 2019 12:18:16 +0000 X-Inumbo-ID: ace13000-77d4-11e9-8af8-d333dc6c8213 Received: from mx1.suse.de (unknown [195.135.220.15]) by us1-amaz-eas2.inumbo.com (Halon) with ESMTPS id ace13000-77d4-11e9-8af8-d333dc6c8213; Thu, 16 May 2019 12:18:13 +0000 (UTC) X-Virus-Scanned: by amavisd-new at test-mx.suse.de Received: from relay2.suse.de (unknown [195.135.220.254]) by mx1.suse.de (Postfix) with ESMTP id 66CC5AD57; Thu, 16 May 2019 12:18:12 +0000 (UTC) Date: Thu, 16 May 2019 14:18:05 +0200 From: Olaf Hering To: Wei Liu Message-ID: <20190516141805.281f972e.ohering@suse.com> In-Reply-To: <20190516140451.06cfcbfe.ohering@suse.com> References: <1ae5e201-04d0-2f78-878f-2e3a3e213b62@suse.com> <20190516110735.67e19d15.ohering@suse.com> <20190516104540.7hiqsn6qjeaam2c6@Air-de-Roger> <20190516125735.6c1bf1ca.ohering@suse.com> <20190516112450.GW2798@zion.uk.xensource.com> <20190516133857.742e4f13.ohering@suse.com> <20190516115043.GB2798@zion.uk.xensource.com> <20190516140451.06cfcbfe.ohering@suse.com> X-Mailer: Claws Mail 2019.04.26 (GTK+ 2.24.32; x86_64-suse-linux-gnu) MIME-Version: 1.0 Subject: Re: [Xen-devel] Regression in xen-unstable due to commit 3802ecbaa9eb36 X-BeenThere: xen-devel@lists.xenproject.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: Xen developer discussion List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Cc: Juergen Gross , xen-devel , Ian Jackson , Roger Pau =?UTF-8?B?TW9ubsOp?= Content-Type: multipart/mixed; boundary="===============5327555863726489374==" Errors-To: xen-devel-bounces@lists.xenproject.org Sender: "Xen-devel" Message-ID: <20190516121805.esHkg6GTTaPde7G4jZcQyuyCQkbYmPpDHCaNE7WDhiM@z> --===============5327555863726489374== Content-Type: multipart/signed; micalg=pgp-sha1; boundary="Sig_/s4/ms4yeVkDLgI369EHZpr8"; protocol="application/pgp-signature" --Sig_/s4/ms4yeVkDLgI369EHZpr8 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: quoted-printable Am Thu, 16 May 2019 14:04:51 +0200 schrieb Olaf Hering : > There are quite a few checks for device_model_version, they would be all > wrong if the assert is removed, or changed back to QEMU_XEN. Perhaps we > can continue to live with that error. device_model_version could become > a local variable. If it is not set, assume the caller just wants the > memory size and enforce QEMU_XEN again within that function. I think that is what should be done, just for the sake of libxl_domain_need_memory. If an incomplete b_info is provided, assume device_model_version=3DQEMU_XEN, maintain that state in a local variable. If we want to provide another new public API to fill missing defaults, that could still be done in a separate patch. Olaf --Sig_/s4/ms4yeVkDLgI369EHZpr8 Content-Type: application/pgp-signature Content-Description: Digitale Signatur von OpenPGP -----BEGIN PGP SIGNATURE----- iF0EARECAB0WIQSkRyP6Rn//f03pRUBdQqD6ppg2fgUCXN1U/QAKCRBdQqD6ppg2 fkffAKCNYh5INArE9YmuocvLyPFhjJvH1gCggi6fEGZWumStTgpQoLk95ed53mM= =9o5g -----END PGP SIGNATURE----- --Sig_/s4/ms4yeVkDLgI369EHZpr8-- --===============5327555863726489374== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: inline X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KWGVuLWRldmVs IG1haWxpbmcgbGlzdApYZW4tZGV2ZWxAbGlzdHMueGVucHJvamVjdC5vcmcKaHR0cHM6Ly9saXN0 cy54ZW5wcm9qZWN0Lm9yZy9tYWlsbWFuL2xpc3RpbmZvL3hlbi1kZXZlbA== --===============5327555863726489374==--