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 Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 6A009C4332F for ; Wed, 19 Oct 2022 13:44:42 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232424AbiJSNok (ORCPT ); Wed, 19 Oct 2022 09:44:40 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:44154 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233463AbiJSNoQ (ORCPT ); Wed, 19 Oct 2022 09:44:16 -0400 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id E84751AAE5C for ; Wed, 19 Oct 2022 06:31:07 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1666186201; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=G5vLQNMn7hVRRf1id7QHDGitVEbk5oVSMTQ+LDM09N4=; b=cHUtLgRYrgxAkeYYiGWGApBI42TZOLbV3H6xK9UCFpmS6vhv34oNF4S6KFo0kbh0KXcZwa ZaDDY5FIvvNoFviAnnLAHV5Z4tO4d2/AJ9nB59c1urgnWU980TFUdrYR7XulglzAt5R+MG zzdZr9aYC7fRMqzq+ylh25hMah5yjHQ= Received: from mimecast-mx02.redhat.com (mimecast-mx02.redhat.com [66.187.233.88]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-27-dFhsFNxkOPClW0zmlqRW8A-1; Wed, 19 Oct 2022 05:04:16 -0400 X-MC-Unique: dFhsFNxkOPClW0zmlqRW8A-1 Received: from smtp.corp.redhat.com (int-mx06.intmail.prod.int.rdu2.redhat.com [10.11.54.6]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx02.redhat.com (Postfix) with ESMTPS id 9AB31185A794; Wed, 19 Oct 2022 09:04:00 +0000 (UTC) Received: from localhost (unknown [10.39.195.35]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 14FBE2166B41; Wed, 19 Oct 2022 09:03:50 +0000 (UTC) From: Cornelia Huck To: =?utf-8?Q?C=C3=A9dric?= Le Goater , Pierre Morel , qemu-s390x@nongnu.org Cc: qemu-devel@nongnu.org, borntraeger@de.ibm.com, pasic@linux.ibm.com, richard.henderson@linaro.org, david@redhat.com, thuth@redhat.com, mst@redhat.com, pbonzini@redhat.com, kvm@vger.kernel.org, ehabkost@redhat.com, marcel.apfelbaum@gmail.com, eblake@redhat.com, armbru@redhat.com, seiden@linux.ibm.com, nrb@linux.ibm.com, frankja@linux.ibm.com, berrange@redhat.com Subject: Re: [PATCH v10 6/9] s390x/cpu topology: add topology-disable machine property In-Reply-To: <08bbd6f8-6ae3-4a28-66ed-d5a290c1a30d@kaod.org> Organization: Red Hat GmbH References: <20221012162107.91734-1-pmorel@linux.ibm.com> <20221012162107.91734-7-pmorel@linux.ibm.com> <08bbd6f8-6ae3-4a28-66ed-d5a290c1a30d@kaod.org> User-Agent: Notmuch/0.37 (https://notmuchmail.org) Date: Wed, 19 Oct 2022 11:03:47 +0200 Message-ID: <87y1tcjibw.fsf@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable X-Scanned-By: MIMEDefang 3.1 on 10.11.54.6 Precedence: bulk List-ID: X-Mailing-List: kvm@vger.kernel.org On Tue, Oct 18 2022, C=C3=A9dric Le Goater wrote: > On 10/12/22 18:21, Pierre Morel wrote: >> S390 CPU topology is only allowed for s390-virtio-ccw-7.3 and >> newer S390 machines. >> We keep the possibility to disable the topology on these newer >> machines with the property topology-disable. >>=20 >> Signed-off-by: Pierre Morel >> --- >> include/hw/boards.h | 3 ++ >> include/hw/s390x/cpu-topology.h | 18 +++++++++- >> include/hw/s390x/s390-virtio-ccw.h | 2 ++ >> hw/core/machine.c | 5 +++ >> hw/s390x/s390-virtio-ccw.c | 53 +++++++++++++++++++++++++++++- >> util/qemu-config.c | 4 +++ >> qemu-options.hx | 6 +++- >> 7 files changed, 88 insertions(+), 3 deletions(-) >>=20 >> diff --git a/include/hw/boards.h b/include/hw/boards.h >> index 311ed17e18..67147c47bf 100644 >> --- a/include/hw/boards.h >> +++ b/include/hw/boards.h >> @@ -379,6 +379,9 @@ struct MachineState { >> } \ >> type_init(machine_initfn##_register_types) >>=20=20=20 >> +extern GlobalProperty hw_compat_7_2[]; >> +extern const size_t hw_compat_7_2_len; > > QEMU 7.2 is not out yet. Yes, and the introduction of the new compat machines needs to go into a separate patch. I'm usually preparing that patch while QEMU is in freeze, but feel free to cook up a patch earlier if you need it. (...) >> +static void ccw_machine_7_3_instance_options(MachineState *machine) >> +{ >> +} >> + >> +static void ccw_machine_7_3_class_options(MachineClass *mc) >> +{ >> +} >> +DEFINE_CCW_MACHINE(7_3, "7.3", true); > > That's too early. Also, the next QEMU version will be 8.0, not 7.3 :)