From nobody Sat May 18 23:14:49 2024 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=fail; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1603125571; cv=none; d=zohomail.com; s=zohoarc; b=VE+zgqlwDnbTC1yDpT04XDjMDFXAyKpg+FP2PzKg5y4/PLFrCIpUOVv9j5c8UGZxg1KAQ2QOM745dHxWPdvEFirwuEMJCIoVx++OGZGJR973GavKWsYsLezQmLD0T4T42U8zfi/wagPX4z3IEIq176ecTbrtq29OVYjgUmexQvY= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1603125571; h=Content-Type:Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To; bh=CAVIJyu9OkU7Q70Y2qGkvrCnYMEjGToAce8PrSfWBvQ=; b=NdswMLFsr/sAhFlp/Szak1aIhMaLMcbsRjOBS8qjgVsNUcdMEhG6SgslEGtDBBGeNxdXDf9olJm2SGVebfB/5TNRHfN+S9KPFV4pWaWA44By+AFO2GfjylGckCj6geXaD6u0DQn9o1eRLKhqjDVP03fFUx2673C3fN/GbpMTi1U= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=fail; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail header.from= (p=none dis=none) header.from= Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1603125570935750.0764973834184; Mon, 19 Oct 2020 09:39:30 -0700 (PDT) Received: from localhost ([::1]:44188 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1kUYBy-0003Gz-Uc for importer@patchew.org; Mon, 19 Oct 2020 12:39:26 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:34490) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1kUY9w-0001im-CO for qemu-devel@nongnu.org; Mon, 19 Oct 2020 12:37:20 -0400 Received: from us-smtp-delivery-124.mimecast.com ([216.205.24.124]:46366) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA1:256) (Exim 4.90_1) (envelope-from ) id 1kUY9u-0006EV-5Q for qemu-devel@nongnu.org; Mon, 19 Oct 2020 12:37:19 -0400 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-471-ElNDdu6mPNarTwmrfz-cig-1; Mon, 19 Oct 2020 12:37:15 -0400 Received: from smtp.corp.redhat.com (int-mx01.intmail.prod.int.phx2.redhat.com [10.5.11.11]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id 1A2521007463; Mon, 19 Oct 2020 16:37:14 +0000 (UTC) Received: from localhost.localdomain (unknown [10.35.206.118]) by smtp.corp.redhat.com (Postfix) with ESMTP id 5508A5B4AD; Mon, 19 Oct 2020 16:37:11 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1603125437; 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=CAVIJyu9OkU7Q70Y2qGkvrCnYMEjGToAce8PrSfWBvQ=; b=G3WhTSXJp8tku0Q2MOip4NIzEi5BN50GtL4cQ16GMKznjysi3pUaHMbn0bICPTgPvMtyfW 4YtiCGi24jOuMwk6ePWbyi7b++9i26lqQSzfEuE/HSrnPDPOosDKtN3bQMIRBsbsuVicC3 Jnw8oJZFmn9X3cT8keTXIYgI6Qjc64Q= X-MC-Unique: ElNDdu6mPNarTwmrfz-cig-1 From: Maxim Levitsky To: qemu-devel@nongnu.org Subject: [PATCH 1/4] qdev: Fix two typos Date: Mon, 19 Oct 2020 19:36:59 +0300 Message-Id: <20201019163702.471239-2-mlevitsk@redhat.com> In-Reply-To: <20201019163702.471239-1-mlevitsk@redhat.com> References: <20201019163702.471239-1-mlevitsk@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.11 Authentication-Results: relay.mimecast.com; auth=pass smtp.auth=CUSA124A263 smtp.mailfrom=mlevitsk@redhat.com X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Transfer-Encoding: quoted-printable Received-SPF: pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Received-SPF: pass client-ip=216.205.24.124; envelope-from=mlevitsk@redhat.com; helo=us-smtp-delivery-124.mimecast.com X-detected-operating-system: by eggs.gnu.org: First seen = 2020/10/19 01:44:30 X-ACL-Warn: Detected OS = Linux 2.2.x-3.x [generic] [fuzzy] X-Spam_score_int: -20 X-Spam_score: -2.1 X-Spam_bar: -- X-Spam_report: (-2.1 / 5.0 requ) BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.001, 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_H4=0.001, RCVD_IN_MSPIKE_WL=0.001, 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: Kevin Wolf , Laurent Vivier , Thomas Huth , =?UTF-8?q?Daniel=20P=2E=20Berrang=C3=A9?= , Eduardo Habkost , qemu-block@nongnu.org, Max Reitz , Paolo Bonzini , Maxim Levitsky Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail-DKIM: fail (Header signature does not verify) Content-Type: text/plain; charset="utf-8" Signed-off-by: Maxim Levitsky Reviewed-by: Philippe Mathieu-Daud=C3=A9 Reviewed-by: Thomas Huth --- include/hw/qdev-core.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/include/hw/qdev-core.h b/include/hw/qdev-core.h index 868973319e..3761186804 100644 --- a/include/hw/qdev-core.h +++ b/include/hw/qdev-core.h @@ -163,8 +163,8 @@ struct NamedClockList { /** * DeviceState: * @realized: Indicates whether the device has been fully constructed. - * When accessed outsize big qemu lock, must be accessed with - * atomic_load_acquire() + * When accessed outside big qemu lock, must be accessed with + * qatomic_load_acquire() * @reset: ResettableState for the device; handled by Resettable interface. * * This structure should not be accessed directly. We declare it here --=20 2.26.2 From nobody Sat May 18 23:14:49 2024 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=fail; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1603125648; cv=none; d=zohomail.com; s=zohoarc; b=lgyNmdyUCeN643sNhvc4L0Gdq9P5bzpWWecbUxjcrMYiwiGly/LIIGqy975Dh1I2vO9u7uoGHcfsHEPJ7g4dYxZm+Fn/cM24fuiPlBYgbDLu6KLcc8xnDGBtC1DdNo7fioUOr0mfIrH4QL5GU6agLO8YymeQPLvu43qfhAJ1mLQ= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1603125648; h=Content-Type:Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To; bh=P0KZpg6Zs44MsxN8be3B2eLRzy7kH87QHxHii2Z6l14=; b=j7/V8KnEhK/Wl1vaP89S76xPsxFgmKTxCCe8z8OGWEzM9cklIkjzGnuiHr5pFIzyaDS6+Ov7r0IxAyfW/pMmp+estlh7d5bsnDQabQw16k1Azncn/lo+HvLF/AC5bH+7f9ZOXlzBldcxJmji0MeoGRSgoWUn9sJy7+rT2bqzkcI= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=fail; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail header.from= (p=none dis=none) header.from= Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1603125648564863.7399376149326; Mon, 19 Oct 2020 09:40:48 -0700 (PDT) Received: from localhost ([::1]:48634 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1kUYDH-0005RW-Ej for importer@patchew.org; Mon, 19 Oct 2020 12:40:47 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:34524) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1kUY9z-0001oH-E8 for qemu-devel@nongnu.org; Mon, 19 Oct 2020 12:37:23 -0400 Received: from us-smtp-delivery-124.mimecast.com ([216.205.24.124]:40839) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA1:256) (Exim 4.90_1) (envelope-from ) id 1kUY9x-0006FE-QB for qemu-devel@nongnu.org; Mon, 19 Oct 2020 12:37:23 -0400 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-171-f-fa8lPIORGJO9vTnDgL8w-1; Mon, 19 Oct 2020 12:37:18 -0400 Received: from smtp.corp.redhat.com (int-mx01.intmail.prod.int.phx2.redhat.com [10.5.11.11]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id 4A4941868403; Mon, 19 Oct 2020 16:37:17 +0000 (UTC) Received: from localhost.localdomain (unknown [10.35.206.118]) by smtp.corp.redhat.com (Postfix) with ESMTP id 853875B4AD; Mon, 19 Oct 2020 16:37:14 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1603125441; 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=P0KZpg6Zs44MsxN8be3B2eLRzy7kH87QHxHii2Z6l14=; b=h4rcVfWyr8Sr+VO/cVugTMmvDw6mVjhwae+8EV2AdOVvcAFe/6EZfgdUIVTDsnr/dx7jPh lKNioo6+QyBtgiW22jQgaic2MfZheqZbG8p5iGU3uTmhJeMJamI3rOMMEDAW1AoD4pSoJc UAwfVgp6j2L9P1QSRObzBhA7SHIGuAU= X-MC-Unique: f-fa8lPIORGJO9vTnDgL8w-1 From: Maxim Levitsky To: qemu-devel@nongnu.org Subject: [PATCH 2/4] libqtest: fix the order of buffered events Date: Mon, 19 Oct 2020 19:37:00 +0300 Message-Id: <20201019163702.471239-3-mlevitsk@redhat.com> In-Reply-To: <20201019163702.471239-1-mlevitsk@redhat.com> References: <20201019163702.471239-1-mlevitsk@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.11 Authentication-Results: relay.mimecast.com; auth=pass smtp.auth=CUSA124A263 smtp.mailfrom=mlevitsk@redhat.com X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Transfer-Encoding: quoted-printable Received-SPF: pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Received-SPF: pass client-ip=216.205.24.124; envelope-from=mlevitsk@redhat.com; helo=us-smtp-delivery-124.mimecast.com X-detected-operating-system: by eggs.gnu.org: First seen = 2020/10/19 01:44:30 X-ACL-Warn: Detected OS = Linux 2.2.x-3.x [generic] [fuzzy] X-Spam_score_int: -20 X-Spam_score: -2.1 X-Spam_bar: -- X-Spam_report: (-2.1 / 5.0 requ) BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.001, 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_H4=0.001, RCVD_IN_MSPIKE_WL=0.001, 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: Kevin Wolf , Laurent Vivier , Thomas Huth , =?UTF-8?q?Daniel=20P=2E=20Berrang=C3=A9?= , Eduardo Habkost , qemu-block@nongnu.org, Max Reitz , Paolo Bonzini , Maxim Levitsky Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail-DKIM: fail (Header signature does not verify) Content-Type: text/plain; charset="utf-8" By a mistake I added the pending events in a wrong order. Fix this by using g_list_append. Signed-off-by: Maxim Levitsky --- tests/qtest/libqtest.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/qtest/libqtest.c b/tests/qtest/libqtest.c index 08929f5ff6..bd96cb6fdd 100644 --- a/tests/qtest/libqtest.c +++ b/tests/qtest/libqtest.c @@ -621,7 +621,7 @@ QDict *qtest_qmp_receive(QTestState *s) return response; } /* Stash the event for a later consumption */ - s->pending_events =3D g_list_prepend(s->pending_events, response); + s->pending_events =3D g_list_append(s->pending_events, response); } } =20 --=20 2.26.2 From nobody Sat May 18 23:14:49 2024 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=fail; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1603125763; cv=none; d=zohomail.com; s=zohoarc; b=l3l9co6NwZbMME2llQW/E0BvRT/NpYGIYF6foC84Zbc6AofTXQ2rd/HJBYhUyzZNWCSoq8NufJo4lUIsmrwyOKJmEN+NCrhLBTGNVNmJuL2yYfcpGpBn3OncgiRRm5aSNgW95bM/Vh7SvsAEVx7jGmFQDTUJq5VvWF8UaBH+QYM= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1603125763; h=Content-Type:Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To; bh=1xyKHqoXsMaWLGNeEEfI9VERvN/PBwd2ItxKq+ji5ns=; b=jRBhoA6vzuf6NFEF1J1uwrv4M6pLrErz/k9xtLzwG6+tsaZsrd4v6Xjn/Evg7Dd9Kbzxcy3Av8/d6i0KZinvjPVCEGFGLEocVDH9xahvTsHaKI+/TfjQtka2h7w9fClk8VlRH614IrvAs560ABXtvL6C/F4ehUT+Nf+aUm+8Dd8= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=fail; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail header.from= (p=none dis=none) header.from= Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1603125763875456.2844555480044; Mon, 19 Oct 2020 09:42:43 -0700 (PDT) Received: from localhost ([::1]:53314 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1kUYF8-00080C-SW for importer@patchew.org; Mon, 19 Oct 2020 12:42:42 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:34558) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1kUYA1-0001uA-ST for qemu-devel@nongnu.org; Mon, 19 Oct 2020 12:37:25 -0400 Received: from us-smtp-delivery-124.mimecast.com ([216.205.24.124]:47223) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA1:256) (Exim 4.90_1) (envelope-from ) id 1kUYA0-0006Fn-5D for qemu-devel@nongnu.org; Mon, 19 Oct 2020 12:37:25 -0400 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-497-wWUo6pk8PfKxtTc5V1ySFg-1; Mon, 19 Oct 2020 12:37:21 -0400 Received: from smtp.corp.redhat.com (int-mx01.intmail.prod.int.phx2.redhat.com [10.5.11.11]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id 7B2D38049C2; Mon, 19 Oct 2020 16:37:20 +0000 (UTC) Received: from localhost.localdomain (unknown [10.35.206.118]) by smtp.corp.redhat.com (Postfix) with ESMTP id B5F565B4AD; Mon, 19 Oct 2020 16:37:17 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1603125443; 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=1xyKHqoXsMaWLGNeEEfI9VERvN/PBwd2ItxKq+ji5ns=; b=YbXEYw3aYoJhOzTaCEFxHCtG8C7pnV21XS23ELzTXrQ3gGFmRjJ+mulNn2uQFMHlAfErvi 4k8bYFiTBNH6lTSoyTkB/YZD3DYbtclPp6/cVMTxseHmd2C5kl+3hBvhxU60xOozG0k8+K GNmpgMDRYcJ7OE2l+zT1Gla731wQOe8= X-MC-Unique: wWUo6pk8PfKxtTc5V1ySFg-1 From: Maxim Levitsky To: qemu-devel@nongnu.org Subject: [PATCH 3/4] libqtest: fix memory leak in the qtest_qmp_event_ref Date: Mon, 19 Oct 2020 19:37:01 +0300 Message-Id: <20201019163702.471239-4-mlevitsk@redhat.com> In-Reply-To: <20201019163702.471239-1-mlevitsk@redhat.com> References: <20201019163702.471239-1-mlevitsk@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.11 Authentication-Results: relay.mimecast.com; auth=pass smtp.auth=CUSA124A263 smtp.mailfrom=mlevitsk@redhat.com X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Transfer-Encoding: quoted-printable Received-SPF: pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Received-SPF: pass client-ip=216.205.24.124; envelope-from=mlevitsk@redhat.com; helo=us-smtp-delivery-124.mimecast.com X-detected-operating-system: by eggs.gnu.org: First seen = 2020/10/19 01:44:30 X-ACL-Warn: Detected OS = Linux 2.2.x-3.x [generic] [fuzzy] X-Spam_score_int: -20 X-Spam_score: -2.1 X-Spam_bar: -- X-Spam_report: (-2.1 / 5.0 requ) BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.001, 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_H4=0.001, RCVD_IN_MSPIKE_WL=0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001 autolearn=ham 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: Kevin Wolf , Laurent Vivier , Thomas Huth , =?UTF-8?q?Daniel=20P=2E=20Berrang=C3=A9?= , Eduardo Habkost , qemu-block@nongnu.org, Max Reitz , Paolo Bonzini , Maxim Levitsky Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail-DKIM: fail (Header signature does not verify) Content-Type: text/plain; charset="utf-8" The g_list_remove_link doesn't free the link element, opposed to what I thought. Switch to g_list_delete_link that does free it. Also refactor the code a bit. Thanks for Max Reitz for helping me with this. Signed-off-by: Maxim Levitsky --- tests/qtest/libqtest.c | 11 ++++------- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/tests/qtest/libqtest.c b/tests/qtest/libqtest.c index bd96cb6fdd..9ae052d566 100644 --- a/tests/qtest/libqtest.c +++ b/tests/qtest/libqtest.c @@ -795,15 +795,12 @@ void qtest_qmp_send_raw(QTestState *s, const char *fm= t, ...) =20 QDict *qtest_qmp_event_ref(QTestState *s, const char *event) { - GList *next =3D NULL; - QDict *response; - - for (GList *it =3D s->pending_events; it !=3D NULL; it =3D next) { + while (s->pending_events) { =20 - next =3D it->next; - response =3D (QDict *)it->data; + GList *first =3D s->pending_events; + QDict *response =3D (QDict *)first->data; =20 - s->pending_events =3D g_list_remove_link(s->pending_events, it); + s->pending_events =3D g_list_delete_link(s->pending_events, first); =20 if (!strcmp(qdict_get_str(response, "event"), event)) { return response; --=20 2.26.2 From nobody Sat May 18 23:14:49 2024 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=fail; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1603125772; cv=none; d=zohomail.com; s=zohoarc; b=MRlKelULxVOqjWoAGJN6Toyu25qMbsmKrpP8E+SDiZgPD54OLnRK1i5xv6BsqZ6eGTajt6S4goMbpU8LDSJeayX0SPpgfYYHXK3v40P9UfsUQhRRGxe/P2ecLhKc9GIkqOkDHCXD0HO9sjLJ1lqs7q9EWlNOq+zfioRIFPOyaTc= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1603125772; h=Content-Type:Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To; bh=NSsBxA/vl8pmoocy+4NGXPtUMal/lnecXCnoaKlgGQw=; b=QMa1E0VLulEG3UgCJOteh3RYQQF4UEB0E0QL2Mz8XcbAFSXULoeJdFFZzdcZBMMRXpD0DkdbaRicoVhci2YFxDx2jKmLE4iYhnzW9eepGiG0YoDJD1htMLW0V0SzI0WPKiz+ANWVc8AuAm3k2eXyD5kcDBqRrnMV6GgngB1N7HI= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=fail; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail header.from= (p=none dis=none) header.from= Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1603125772677932.6570654528086; Mon, 19 Oct 2020 09:42:52 -0700 (PDT) Received: from localhost ([::1]:53802 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1kUYFH-0008D0-Jl for importer@patchew.org; Mon, 19 Oct 2020 12:42:51 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:34622) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1kUYA8-00027N-50 for qemu-devel@nongnu.org; Mon, 19 Oct 2020 12:37:32 -0400 Received: from us-smtp-delivery-124.mimecast.com ([216.205.24.124]:45157) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA1:256) (Exim 4.90_1) (envelope-from ) id 1kUYA4-0006Gd-PY for qemu-devel@nongnu.org; Mon, 19 Oct 2020 12:37:31 -0400 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-443-IhzG72XIM_yzNx0WEglxUA-1; Mon, 19 Oct 2020 12:37:25 -0400 Received: from smtp.corp.redhat.com (int-mx01.intmail.prod.int.phx2.redhat.com [10.5.11.11]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id 6CCD8425CF; Mon, 19 Oct 2020 16:37:24 +0000 (UTC) Received: from localhost.localdomain (unknown [10.35.206.118]) by smtp.corp.redhat.com (Postfix) with ESMTP id E54285B4AD; Mon, 19 Oct 2020 16:37:20 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1603125448; 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=NSsBxA/vl8pmoocy+4NGXPtUMal/lnecXCnoaKlgGQw=; b=ScmIBOPBT6nb9IY/1iRIU0A4KXDDJw6e0XJtOgFZBq2mI8DEAbzToAslKWN1gKeKmnyWJ1 OwR8tb0bSlGqU5veALvgzi1U6+X356JHy7tVCe7r8APs2e6UpS86O9wtu/Ae1O9zsqYX7J 1h/U5OK3lohpE54A/oo1Q3vmJRfLIO4= X-MC-Unique: IhzG72XIM_yzNx0WEglxUA-1 From: Maxim Levitsky To: qemu-devel@nongnu.org Subject: [PATCH 4/4] iotests: rewrite iotest 240 in python Date: Mon, 19 Oct 2020 19:37:02 +0300 Message-Id: <20201019163702.471239-5-mlevitsk@redhat.com> In-Reply-To: <20201019163702.471239-1-mlevitsk@redhat.com> References: <20201019163702.471239-1-mlevitsk@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.11 Authentication-Results: relay.mimecast.com; auth=pass smtp.auth=CUSA124A263 smtp.mailfrom=mlevitsk@redhat.com X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Transfer-Encoding: quoted-printable Received-SPF: pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Received-SPF: pass client-ip=216.205.24.124; envelope-from=mlevitsk@redhat.com; helo=us-smtp-delivery-124.mimecast.com X-detected-operating-system: by eggs.gnu.org: First seen = 2020/10/19 01:44:30 X-ACL-Warn: Detected OS = Linux 2.2.x-3.x [generic] [fuzzy] X-Spam_score_int: -20 X-Spam_score: -2.1 X-Spam_bar: -- X-Spam_report: (-2.1 / 5.0 requ) BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.001, 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_H4=0.001, RCVD_IN_MSPIKE_WL=0.001, 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: Kevin Wolf , Laurent Vivier , Thomas Huth , =?UTF-8?q?Daniel=20P=2E=20Berrang=C3=A9?= , Eduardo Habkost , qemu-block@nongnu.org, Max Reitz , Paolo Bonzini , Maxim Levitsky Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail-DKIM: fail (Header signature does not verify) Content-Type: text/plain; charset="utf-8" The recent changes that brought RCU delayed device deletion, broke few tests and this test breakage went unnoticed. Fix this test by rewriting it in python (which allows to wait for DEVICE_DELETED events before continuing). Signed-off-by: Maxim Levitsky --- tests/qemu-iotests/240 | 228 ++++++++++++++++--------------------- tests/qemu-iotests/240.out | 76 ++++++++----- 2 files changed, 143 insertions(+), 161 deletions(-) diff --git a/tests/qemu-iotests/240 b/tests/qemu-iotests/240 index 8b4337b58d..a739de6769 100755 --- a/tests/qemu-iotests/240 +++ b/tests/qemu-iotests/240 @@ -1,5 +1,5 @@ -#!/usr/bin/env bash -# +#!/usr/bin/env python3 + # Test hot plugging and unplugging with iothreads # # Copyright (C) 2019 Igalia, S.L. @@ -17,133 +17,99 @@ # # You should have received a copy of the GNU General Public License # along with this program. If not, see . -# =20 -# creator -owner=3Dberto@igalia.com - -seq=3D`basename $0` -echo "QA output created by $seq" - -status=3D1 # failure is the default! - -_cleanup() -{ - rm -f "$SOCK_DIR/nbd" -} -trap "_cleanup; exit \$status" 0 1 2 3 15 - -# get standard environment, filters and checks -. ./common.rc -. ./common.filter - -_supported_fmt generic -_supported_proto generic - -do_run_qemu() -{ - echo Testing: "$@" - $QEMU -nographic -qmp stdio -serial none "$@" - echo -} - -# Remove QMP events from (pretty-printed) output. Doesn't handle -# nested dicts correctly, but we don't get any of those in this test. -_filter_qmp_events() -{ - tr '\n' '\t' | sed -e \ - 's/{\s*"timestamp":\s*{[^}]*},\s*"event":[^,}]*\(,\s*"data":\s*{[^}]*}\)\= ?\s*}\s*//g' \ - | tr '\t' '\n' -} - -run_qemu() -{ - do_run_qemu "$@" 2>&1 | _filter_qmp | _filter_qmp_events -} - -case "$QEMU_DEFAULT_MACHINE" in - s390-ccw-virtio) - virtio_scsi=3Dvirtio-scsi-ccw - ;; - *) - virtio_scsi=3Dvirtio-scsi-pci - ;; -esac - -echo -echo =3D=3D=3D Unplug a SCSI disk and then plug it again =3D=3D=3D -echo - -run_qemu <