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=-6.8 required=3.0 tests=BAYES_00,DKIM_INVALID, DKIM_SIGNED,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_HELO_NONE,SPF_PASS autolearn=no 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 13DE5C433DF for ; Tue, 11 Aug 2020 11:07:55 +0000 (UTC) Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) (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 CEABB2075D for ; Tue, 11 Aug 2020 11:07:54 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=fail reason="signature verification failed" (1024-bit key) header.d=redhat.com header.i=@redhat.com header.b="Wvk9EXTX" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org CEABB2075D Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=redhat.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org Received: from localhost ([::1]:47238 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1k5S8I-0006dJ-3W for qemu-devel@archiver.kernel.org; Tue, 11 Aug 2020 07:07:54 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:46988) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1k5S5W-00025C-98 for qemu-devel@nongnu.org; Tue, 11 Aug 2020 07:05:02 -0400 Received: from us-smtp-2.mimecast.com ([207.211.31.81]:37074 helo=us-smtp-delivery-1.mimecast.com) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA1:256) (Exim 4.90_1) (envelope-from ) id 1k5S5P-0003bE-Qq for qemu-devel@nongnu.org; Tue, 11 Aug 2020 07:04:58 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1597143894; 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=LB0f2MRmQoQo40Yq2DAO9ZoMYhBNsLZuIU/pXNlvtX4=; b=Wvk9EXTX3wCriYj5K5hXyOO1BDppjByRcl05AoouK9CbZ3FP66F9GphVxl7Lrs9AmBDWp/ M9g9z+F2qMVL+B6xMTKlRmMFHadIbclmczS3F/M4Z4sbkrDm9vzkHaDmLksdmW5JXxhiIn PejQyNeRkZksebn1jr1lkPJ9beZ1AhY= Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-521-i4__DAIrM7uPuV6zPkm9Yw-1; Tue, 11 Aug 2020 07:04:50 -0400 X-MC-Unique: i4__DAIrM7uPuV6zPkm9Yw-1 Received: from smtp.corp.redhat.com (int-mx05.intmail.prod.int.phx2.redhat.com [10.5.11.15]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id 62654107ACCA; Tue, 11 Aug 2020 11:04:48 +0000 (UTC) Received: from gondolin (ovpn-113-33.ams2.redhat.com [10.36.113.33]) by smtp.corp.redhat.com (Postfix) with ESMTP id 4C40E5F1E9; Tue, 11 Aug 2020 11:04:37 +0000 (UTC) Date: Tue, 11 Aug 2020 13:04:34 +0200 From: Cornelia Huck To: Thiago Jung Bauermann Subject: Re: [RFC PATCH v3 8/8] target/s390x: Use start-powered-off CPUState property Message-ID: <20200811130434.7388cfa2.cohuck@redhat.com> In-Reply-To: <20200730114541.4dbdd15e.cohuck@redhat.com> References: <20200723025657.644724-1-bauerman@linux.ibm.com> <20200723025657.644724-9-bauerman@linux.ibm.com> <20200727144350.47abd7e7.cohuck@redhat.com> <87a6zjuoru.fsf@morokweng.localdomain> <20200730114541.4dbdd15e.cohuck@redhat.com> Organization: Red Hat GmbH MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.15 Authentication-Results: relay.mimecast.com; auth=pass smtp.auth=CUSA124A263 smtp.mailfrom=cohuck@redhat.com X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Received-SPF: pass client-ip=207.211.31.81; envelope-from=cohuck@redhat.com; helo=us-smtp-delivery-1.mimecast.com X-detected-operating-system: by eggs.gnu.org: First seen = 2020/08/11 04:41:55 X-ACL-Warn: Detected OS = Linux 2.2.x-3.x [generic] [fuzzy] X-Spam_score_int: -40 X-Spam_score: -4.1 X-Spam_bar: ---- X-Spam_report: (-4.1 / 5.0 requ) BAYES_00=-1.9, DKIMWL_WL_HIGH=-1, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H2=-1, SPF_HELO_NONE=0.001, SPF_PASS=-0.001 autolearn=unavailable autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Peter Maydell , Aleksandar Rikalo , Eduardo Habkost , Aleksandar Markovic , Alex =?UTF-8?B?QmVubsOpZQ==?= , David Hildenbrand , Mark Cave-Ayland , qemu-devel@nongnu.org, Greg Kurz , qemu-s390x@nongnu.org, qemu-arm@nongnu.org, qemu-ppc@nongnu.org, Artyom Tarasenko , Thomas Huth , Paolo Bonzini , Richard Henderson , Philippe =?UTF-8?B?TWF0aGlldS1EYXVkw6k=?= , Aurelien Jarno , David Gibson Errors-To: qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org Sender: "Qemu-devel" On Thu, 30 Jul 2020 11:45:41 +0200 Cornelia Huck wrote: > On Tue, 28 Jul 2020 21:51:33 -0300 > Thiago Jung Bauermann wrote: > > > Hi, > > > > Cornelia Huck writes: > > > > > On Wed, 22 Jul 2020 23:56:57 -0300 > > > Thiago Jung Bauermann wrote: > > > > > >> Instead of setting CPUState::halted to 1 in s390_cpu_initfn(), use the > > >> start-powered-off property which makes cpu_common_reset() initialize it > > >> to 1 in common code. > > >> > > >> Note that this changes behavior by setting cs->halted to 1 on reset, which > > >> didn't happen before. > > > > > > I think that should be fine, as we change the cpu state to STOPPED in > > > the reset function, which sets halted to 1. > > > > Nice, thanks for checking. > > > > >> > > >> Signed-off-by: Thiago Jung Bauermann > > >> --- > > >> target/s390x/cpu.c | 2 +- > > >> 1 file changed, 1 insertion(+), 1 deletion(-) > > >> > > >> NB: I was only able to test that this patch builds. I wasn't able to > > >> run it. > > > > > > No noticeable difference under kvm, but running under tcg seems a bit > > > more sluggish than usual, and I saw some pausing on reboot (after the > > > bios handover to the kernel). Not sure if it were just flukes on my > > > laptop, would appreciate if someone else could give it a go. > > Experimented a bit with it again. There's a pause when switching from > the bios to the kernel (after the load reset normal has been done, I > guess), which is always there, but seems to get more noticeable with > this patch (varying wildly, but seems longer on average.) Hard to pin > down, and I don't really see a reason why that should happen, as we > should end up with halted == 1 in any case. Might still be a fluke, > even though I see it both on my laptop and on an LPAR (when running > under tcg; not seen under kvm, which is much faster anyway.) Tried again, the pause now seems comparable to the pause prior to this series. Might depend on the phase of the moon. I ran kvm unit tests on it, and it looks good. So, I'm reasonable confident that this is fine, really just seems to be a fluke. Acked-by: Cornelia Huck