From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:34899) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dx7Jt-0001Og-1w for qemu-devel@nongnu.org; Wed, 27 Sep 2017 04:03:54 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dx7Jn-0004uB-1t for qemu-devel@nongnu.org; Wed, 27 Sep 2017 04:03:49 -0400 Received: from mx0a-001b2d01.pphosted.com ([148.163.156.1]:54898) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1dx7Jm-0004tr-OO for qemu-devel@nongnu.org; Wed, 27 Sep 2017 04:03:42 -0400 Received: from pps.filterd (m0098410.ppops.net [127.0.0.1]) by mx0a-001b2d01.pphosted.com (8.16.0.21/8.16.0.21) with SMTP id v8R82hn3111965 for ; Wed, 27 Sep 2017 04:03:39 -0400 Received: from e06smtp12.uk.ibm.com (e06smtp12.uk.ibm.com [195.75.94.108]) by mx0a-001b2d01.pphosted.com with ESMTP id 2d87p11r6j-1 (version=TLSv1.2 cipher=AES256-SHA bits=256 verify=NOT) for ; Wed, 27 Sep 2017 04:03:38 -0400 Received: from localhost by e06smtp12.uk.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Wed, 27 Sep 2017 09:03:34 +0100 From: Christian Borntraeger References: <20170926133624.155394-1-borntraeger@de.ibm.com> <20170926133624.155394-3-borntraeger@de.ibm.com> <5770bcf6-3608-11e7-7f4e-a91f5046c8c1@de.ibm.com> Date: Wed, 27 Sep 2017 10:03:31 +0200 MIME-Version: 1.0 In-Reply-To: <5770bcf6-3608-11e7-7f4e-a91f5046c8c1@de.ibm.com> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Message-Id: <2f3eb94e-c07f-8752-059c-de5900859fff@de.ibm.com> Subject: Re: [Qemu-devel] [PATCH v2 2/2] s390x/ais: disable ais for compat machines List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: David Hildenbrand , Cornelia Huck Cc: Richard Henderson , Alexander Graf , Yi Min Zhao , Halil Pasic , qemu-devel , "Jason J . Herne" , "Dr. David Alan Gilbert" On 09/27/2017 09:12 AM, Christian Borntraeger wrote: > > > On 09/26/2017 03:51 PM, David Hildenbrand wrote: >> On 26.09.2017 15:36, Christian Borntraeger wrote: >>> With newer kernels that do support the ais feature (4.13) a qemu 2.11 >>> will not only enable the ais feature for the 2.11 machine, but also >>> for a <=2.10 compat machine. As this feature is not available in >>> QEMU <=2.9 (and QEMU 2.10.1), this guest will fail to migrate >>> back to an older qemu like 2.9 with: >>> >>> _snip_ >>> error while loading state for instance 0x0 of device 's390-flic' >>> _snip_ >>> >>> making the whole compat machine dis-functional. As a permanent fix, we >>> need to fence the ais feature for machines <= 2.10 >>> >>> Due to ais being enabled on 2.10.0 (fixed in 2.10.1) this will prevent >>> migration of ais-enabled guests from 2.10.0 with >>> >>> _snip_ >>> qemu-system-s390x: Failed to load s390-flic/ais:tmp >>> qemu-system-s390x: error while loading state for instance 0x0 of device 's390-flic' >>> qemu-system-s390x: load of migration failed: Function not implemented >>> _snip_ >>> >>> Signed-off-by: Christian Borntraeger >>> Cc: Yi Min Zhao >>> Cc: Dr. David Alan Gilbert >>> --- >>> hw/intc/s390_flic_kvm.c | 4 +++- >> >> >> As discussed, I think we should use cpu_model_allowed() instead. > I think I still prefer the explicit check for ais-enabled to make managedsave > (on the same system) continue to work if the user does not specify a cpu model > at all (which will then fallback to the host model). We already fence other > things (like guarded storage) and yes it will grow over time but the > *_allowed things seem to be the smallest maintenance issue in this area. Hmm, on the other hand this fails only when we migrate to an older qemu during managedsave. So yes, maybe this is just an "will not work anyway"