From nobody Sun May 5 13:30:27 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=web.de ARC-Seal: i=1; a=rsa-sha256; t=1616369706; cv=none; d=zohomail.com; s=zohoarc; b=bXGe+dDDcMh0MIFCaXczWiTuYn3yxnJJhcTCYGIWO2ewah467cBWllUNuF09XEd+5bq+QX01keT730Xj/TlCr2R7igiJiI3sxIWlXGCAy5EhSjxws1on3HtfA944bSpceQNuliB4fqZ/OzLbUnD/TYyc62LUn6LG+A6qzGcmgcQ= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1616369706; h=Content-Type: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=C6v1TOdliCCrxIvBovQT5z0BJq+dbMy0RP3hujHbz4Q=; b=HOxWOlvINNDIP6B+f+NMh2MIY1DQNdfPtx0psrU6dSSdc4ERDKudka9immFGVmar5UrnmcpFaUsn+Y1OuLykUMCUOUdtDxsU7M7uw/DwywLiIXlqzYi+sjy6xfd+Log5z8qGVRmjcXzaeNx6cZovluUMNcqmrUNKUO9wsVoLSv8= 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 1616369706266206.6076673115831; Sun, 21 Mar 2021 16:35:06 -0700 (PDT) Received: from localhost ([::1]:51154 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1lO7b7-00036Y-8P for importer@patchew.org; Sun, 21 Mar 2021 19:35:05 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:33198) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1lO7Y2-0007vZ-69 for qemu-devel@nongnu.org; Sun, 21 Mar 2021 19:31:54 -0400 Received: from mout.web.de ([212.227.17.12]:45101) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1lO7Y0-0000FN-Gv for qemu-devel@nongnu.org; Sun, 21 Mar 2021 19:31:53 -0400 Received: from gecko.fritz.box ([94.134.180.131]) by smtp.web.de (mrweb102 [213.165.67.124]) with ESMTPSA (Nemesis) id 0MORuj-1lIK7r4B4H-005u5o; Mon, 22 Mar 2021 00:31:47 +0100 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=web.de; s=dbaedf251592; t=1616369507; bh=tuyfOEeBolxuiT4kdpqwI8Wgqhvay43ezbANQOEuxAI=; h=X-UI-Sender-Class:Date:From:To:Cc:Subject:In-Reply-To:References; b=Woh8cQiJ52L9iC99mHeZ1PdqIzJSdUDyT24vfwyuqvhUUc4eP4nap3h+rvOJpV1xc KKtX+XAgF9aStrJX6DgxMirrM6wIM6A0sI8GnaIukHM6Q24LwPtKSBMbn1mtjqEo6I RNjnLQcvxwQCXzYQVPGvyQFFQl6Kvc5IgZ1EdK68= X-UI-Sender-Class: c548c8c5-30a9-4db5-a2e7-cb6cb037b8f9 Date: Mon, 22 Mar 2021 00:31:45 +0100 From: Lukas Straub To: qemu-devel Subject: [PATCH 1/5] tests: Use the normal yank code instead of stubs in relevant tests Message-ID: <950007e82e19e75831b29fac07ab990c213d2352.1616368879.git.lukasstraub2@web.de> In-Reply-To: References: MIME-Version: 1.0 Content-Type: multipart/signed; boundary="Sig_/z9c+D=8yEXVEulNExJJ/WB5"; protocol="application/pgp-signature"; micalg=pgp-sha256 X-Provags-ID: V03:K1:ej3DrsVhwNwtE0w9ysuxD0oWGlavD6FrHOGq1wTRfRmWhgkwff0 JYnDvKyQiZZOu6Vejvx6CQEHm17li9n6aGY/hAyg9uw/t5IQGFxiSGZsPitLUmaoJ6QMiWn IosrXShu8/d9t/mUAjQZDKHgimtzGv6c0x5MDIbQLSYuC83ppzLX23pWGYnPYYofQmQsu76 VE0sEvHbwCku7utRxqccg== X-UI-Out-Filterresults: notjunk:1;V03:K0:Sg51EiL4+wg=:Dt/Q0kx7ADKNQWYup/gE/1 VZ/bjCpqqOuI4bB+GjG9xCa0LqACzZYr1X7wkk4NWbAZ8JxoQPcAQygYh0NFPexR5uHyuQzJl un4/4aC/do5Vlcd0T4G1BurqVgogCBn54YGvW3G/yWu+FN1MllpBIPlIFAqPXenJrqP5hPuka q77Gh062SljYoOqX61TMDNXpU/wwuaY/lVRZhB803CqTvkKiFB+xe89roerjZrp682UMxCyEj fsAziPV6ni/tHwRAAKB+IIFF6zfzp5h3x394HDr9Z/8GudIgciSgUjiFKYf6FPLcDlp1s3tnp lmc0OxiOe10qnmSnQY9ITbDwDsEIg5lJxq50kcrqFdDhirs9+28ShdGqjS1Fgs4hgd0FCYLDy v0Emm8hGpI9UIE7Aop8AYaXfQ9i5mf3infThCpCcK26oBMV2mmX9NLHuMje2lPkarte+a1cMY JKNRLRNwNOdXdfzgyP3bjOXD7UVtubiOCQC9s14X+qb5xDFtpDbKmREM0cX1IxABEbUf3m6ol vAdwO8JwUBIy9BCzmTIr1xfz9L8eTIrUj85zU9KpvSrRkS7Dar+nAMnDpHgCkNO/hk8sPm39n NsgAA9GHra87WGttqr4r9UkUYC+rE+kgfdu3MNGqqv2UJOcY6LmoKRCnc9qoMIiGPh/HHXAZe vEfciSRfPLx8j959bXSXDLekSw91gq0aQ9jGK+3rmYI9VMYqSvP+2tUKtDi8Brf3quesY9Fyu 66n2GNw3OngCFIVwuhc0q4oCuUYrchQNJSu9xkBkIeDeQ77YRJ5ljWXtBf3OajwDLu7wBKRAG t/5szLcoCreklH81kMcNp8AW3UihE56UgOCtSmeOCI/j3big+RXljo8VV7Rph8Hl8WfqZ3s73 G68eQdyz3GQtMkDltKS8stTb6XDOUdUHzbZrH0PYU= 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=212.227.17.12; envelope-from=lukasstraub2@web.de; helo=mout.web.de X-Spam_score_int: 8 X-Spam_score: 0.8 X-Spam_bar: / X-Spam_report: (0.8 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_ENVFROM_END_DIGIT=0.25, FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_LOW=-0.7, RCVD_IN_MSPIKE_H3=-0.01, RCVD_IN_MSPIKE_WL=-0.01, RCVD_IN_SBL_CSS=3.335, SPF_HELO_NONE=0.001, SPF_PASS=-0.001 autolearn=no 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: Laurent Vivier , Marc-Andre Lureau , Thomas Huth , Li Zhang , Paolo Bonzini Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-Zoho-Virus-Status: 1 X-ZohoMail-DKIM: fail (Header signature does not verify) --Sig_/z9c+D=8yEXVEulNExJJ/WB5 Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Use the normal yank code instead of stubs in relevant tests to increase coverage and to ensure that registering and unregistering of yank instances and functions is done correctly. Signed-off-by: Lukas Straub --- tests/qtest/meson.build | 6 +++--- tests/unit/meson.build | 4 ++-- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/tests/qtest/meson.build b/tests/qtest/meson.build index 66ee9fbf45..40e1f495f7 100644 --- a/tests/qtest/meson.build +++ b/tests/qtest/meson.build @@ -234,9 +234,9 @@ tpmemu_files =3D ['tpm-emu.c', 'tpm-util.c', 'tpm-tests= .c'] qtests =3D { 'bios-tables-test': [io, 'boot-sector.c', 'acpi-utils.c', 'tpm-emu.c'], 'cdrom-test': files('boot-sector.c'), - 'dbus-vmstate-test': files('migration-helpers.c') + dbus_vmstate1, + 'dbus-vmstate-test': ['migration-helpers.c', dbus_vmstate1, '../../monit= or/yank.c'], 'ivshmem-test': [rt, '../../contrib/ivshmem-server/ivshmem-server.c'], - 'migration-test': files('migration-helpers.c'), + 'migration-test': ['migration-helpers.c', io, '../../monitor/yank.c'], 'pxe-test': files('boot-sector.c'), 'qos-test': [chardev, io, qos_test_ss.apply(config_host, strict: false).= sources()], 'tpm-crb-swtpm-test': [io, tpmemu_files], @@ -266,7 +266,7 @@ foreach dir : target_dirs endif qtest_env.set('G_TEST_DBUS_DAEMON', meson.source_root() / 'tests/dbus-vm= state-daemon.sh') qtest_env.set('QTEST_QEMU_BINARY', './qemu-system-' + target_base) - + foreach test : target_qtests # Executables are shared across targets, declare them only the first t= ime we # encounter them diff --git a/tests/unit/meson.build b/tests/unit/meson.build index 4bfe4627ba..8ccf60af66 100644 --- a/tests/unit/meson.build +++ b/tests/unit/meson.build @@ -123,7 +123,7 @@ if have_system 'test-util-sockets': ['socket-helpers.c'], 'test-base64': [], 'test-bufferiszero': [], - 'test-vmstate': [migration, io] + 'test-vmstate': [migration, io, '../../monitor/yank.c'] } if 'CONFIG_INOTIFY1' in config_host tests +=3D {'test-util-filemonitor': []} @@ -135,7 +135,7 @@ if have_system if 'CONFIG_TSAN' not in config_host if 'CONFIG_POSIX' in config_host tests +=3D { - 'test-char': ['socket-helpers.c', qom, io, chardev] + 'test-char': ['socket-helpers.c', qom, io, chardev, '../../monit= or/yank.c'] } endif -- 2.30.2 --Sig_/z9c+D=8yEXVEulNExJJ/WB5 Content-Type: application/pgp-signature Content-Description: OpenPGP digital signature -----BEGIN PGP SIGNATURE----- iQIzBAEBCAAdFiEEg/qxWKDZuPtyYo+kNasLKJxdslgFAmBX12EACgkQNasLKJxd sliXFQ//XqMGoefXAqqMs9wZwZmVrUdjtoryr7Zlkdx4Z/ZNwkIvpf5WqqdqGYhu 5orW/xFlEG4GgCkuy82kaaZtiFhhxQLO3byf5+/Qh68NJ//jOiskwOX6F8faMrZ/ BdB9jcBLT0A88ngUhKxJgZbxLjyP29UVKrsUvBJl/hF5mLx/+/xcHZejD0PR/ay0 LtxJaH64mk3qpXOizx3/Ll66FWBtGbzXkXn9y0q+nkcLROlWN46a+GU+tQxuXGqs rXL8dnouW/mCqU3ooLK6DIChFRnIYUGDkBe5M1PCETN4VL75ABWodjN1/TLP5ea7 zC3fD01H3ng2ThwSQpAf1ZmMAtSXWKqnpmUPrNNZYuvjishtoUPQ5NcFEpu5Fv1B x/G053WnbNgYTvimcGCl/8kGoYqnVvYcnctSETekDdI+nT2CeByLGmtSLKz712ji G3R76dG5gealPO9PBwsIYZFqN5av0TNLu8Q7pG2Nk0GO2mAFcG4zVhdzqVWZGb7i Jb6EYy9s6cR9umD34xHSC/KDusNgdcWjLrvdRt3P3GsntPtfrgJFV4iOwWpKGbUK 9n/qvvkOZcH1d2+En94iR60lFbAlFeChHPGsbVozYpBqXOZtXbvDKTH6Yo8ZKh25 WvhuI4qxD1xc5Rwyxmd1knHsP7rCeo5lBnMim9ruk/7muJj5gUs= =s7dH -----END PGP SIGNATURE----- --Sig_/z9c+D=8yEXVEulNExJJ/WB5-- From nobody Sun May 5 13:30:27 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=web.de ARC-Seal: i=1; a=rsa-sha256; t=1616369604; cv=none; d=zohomail.com; s=zohoarc; b=NwYAO4eb92Z1UeILqGVU5SIwrJa+XKZ2fsi/ObT4dGjcXyonCP9icg8dJ8ScHcU//uYr6TfaQl0ulkOpvqxORe75SG1OndzazTmmKZX0JVmwXv+XcyzkMjgdtHsF1ARb4l19eihd6qx7pBX8iyiQlKVG+hYQua9KqPv3ly/Irdo= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1616369604; h=Content-Type: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=b8IpC29EeJJF4sgp8u67qbJqlj46nN+Mp1DXnFXzV6I=; b=NipBR8kdxKBGph279rpSUiccvnhqMijxGknzLzjzkNpGXSjH+vrTSxVb8lVVwhfORMyk3Hsh8SYDqYJZaYahcjFziQXbyx+nSg3fz2mx+0u4XMFHJtxhOIz6l1TjeX+DWCSgsM2RO0sooSJ9FFIW6Xltl4LY3wa6s7ENMMlc/sY= 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 1616369604884288.1929954813046; Sun, 21 Mar 2021 16:33:24 -0700 (PDT) Received: from localhost ([::1]:45542 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1lO7ZT-0000pK-Mp for importer@patchew.org; Sun, 21 Mar 2021 19:33:23 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:33220) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1lO7Y5-0007yO-7V for qemu-devel@nongnu.org; Sun, 21 Mar 2021 19:31:57 -0400 Received: from mout.web.de ([217.72.192.78]:50867) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1lO7Y3-0000Gh-9G for qemu-devel@nongnu.org; Sun, 21 Mar 2021 19:31:56 -0400 Received: from gecko.fritz.box ([94.134.180.131]) by smtp.web.de (mrweb101 [213.165.67.124]) with ESMTPSA (Nemesis) id 0LfAoO-1m3o5P0R9U-00omLQ; Mon, 22 Mar 2021 00:31:51 +0100 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=web.de; s=dbaedf251592; t=1616369511; bh=g96QoTGwedFUXA0fm9qRHfwgSpJ5sKc88uQ9mOYbs8g=; h=X-UI-Sender-Class:Date:From:To:Cc:Subject:In-Reply-To:References; b=HldLYBfnwi/K6EMLBLsNFThCJvvRXoNQ/MaQVybK1DBp8sswnUBpROay0kfcukKvq xXNPj8jBrprHmLhSg/qKqASmTbme1d5H4swnZdAOlrdV9dJV/3ok2gCDTNVSPL6ZAO TlZetMq4CR0D8hsTDVLfb6nBxo37bPRIYOZ/ukY4= X-UI-Sender-Class: c548c8c5-30a9-4db5-a2e7-cb6cb037b8f9 Date: Mon, 22 Mar 2021 00:31:49 +0100 From: Lukas Straub To: qemu-devel Subject: [PATCH 2/5] tests: Add tests for yank with the chardev-change Message-ID: In-Reply-To: References: MIME-Version: 1.0 Content-Type: multipart/signed; boundary="Sig_/0yWvBSAJSpD2xkySF0CPHfM"; protocol="application/pgp-signature"; micalg=pgp-sha256 X-Provags-ID: V03:K1:uBbVE/AGt6Cs/n1SLeOx7krvvnF1f2SiUvcmPVjMKH325m8ypIL 8HDXMcGIfdCtJbD424GTZKzaUb96B6XoMYH85a9cJjxx8WAgfCQO+vJWTK331WY26nr1DT+ nrTmWP9C0Z59swCDB97hcLrmbv8zF+ECb+TAgNjcgGr59W5B8ZZ63UNxs+ezHeyAPdqZ4T7 eiZGbUSvbUp2Es48mPekA== X-UI-Out-Filterresults: notjunk:1;V03:K0:5WIsvmLIz/4=:1IBQmfxnOCkHfc5bUSf6fy 05+f5jQ2y0tPwCKvgNnmlwyyorUPbBeADsUkCHt/Q/HCx0Jbe1y/k9aNCAoqISZstpIbfMZgL IAckN6EJ1YqRw/C17itkqN6oL56xLOaxihnyHCVLwgPo5UiqKrFOVnQY/qLQfQgIqeYD88HMg shzoY521bQduu/Ft2VGDEuCvtdOlr0iPUaoRCfvWfCPitYfx9MBo1QDJDgn1/yDjrMaPBXkY0 EuRzvFHCe6aYyf1WPwKBjQiw7mXEyJ+VfHQqes2muk6ji8jNytvYvuUGUICrqHPXK0bBDXzNb 1qAbxGk0TQ1Im1MS1RHvOjzUoBkGYW6OmD/AZd5e1/O+v/UuMyX1oe92U9rXCv17xWS7TNBpr CceVjNW+B24cWfMmoHIR2NVANH7mCBqCPkhBwbGrFINAUh1a61rdztu4uS1xLcfti2sHjWH7+ 5cztiVuWQg3FLYwBkgjqAk+HOVa0tr8oN7qwqkZn/O/df05inn1+zslM6sUQs9T8BYiGJmoMZ LfSncZm1Uxq5mm8r6jWXbkfvE7bVaDERRIGgc1JLVYCQsryNRJPLwGa5gplaoGVcrTcFykBT0 7ryI9/hkqUg9b41j1kfFUD6/M90+ebEg2K6e8kVsMxYG8NGOeY8TKcgC94Ld6qhCRdWOK2/mk CCEEffXJ4HFwANAnqCeteRdswRM2fSALw4yEeoxTIzVlinAHX2yCBm/ROxE2k53rtFAlsA5wn l3jP8GVNF2hUDk0jmx3ulZFubSTYYTA7HL6C/el5aXJmh5B2ZJWTl0v0ZDtPxXsM9TJ7TLxIp qWImah9BvgfS4Dzk8gW2qhtydcc9yh545NOdv6oaFo4dGqv9j9gIe8fQAw3p9L4CUJtQYuVlo Tv1KTCr/Eor/BrkGMR7cskgYj2SngW5DPfNE6e0yo= 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=217.72.192.78; envelope-from=lukasstraub2@web.de; helo=mout.web.de X-Spam_score_int: 8 X-Spam_score: 0.8 X-Spam_bar: / X-Spam_report: (0.8 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_ENVFROM_END_DIGIT=0.25, FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_LOW=-0.7, RCVD_IN_SBL_CSS=3.335, SPF_HELO_NONE=0.001, SPF_PASS=-0.001 autolearn=no 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: Laurent Vivier , Marc-Andre Lureau , Thomas Huth , Li Zhang , Paolo Bonzini Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-Zoho-Virus-Status: 1 X-ZohoMail-DKIM: fail (Header signature does not verify) --Sig_/0yWvBSAJSpD2xkySF0CPHfM Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Add tests for yank with the chardev-change case. Signed-off-by: Lukas Straub --- MAINTAINERS | 1 + tests/unit/meson.build | 3 +- tests/unit/test-yank.c | 240 +++++++++++++++++++++++++++++++++++++++++ 3 files changed, 243 insertions(+), 1 deletion(-) create mode 100644 tests/unit/test-yank.c diff --git a/MAINTAINERS b/MAINTAINERS index aa024eed17..a8a7f0d1c2 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -2817,6 +2817,7 @@ F: monitor/yank.c F: stubs/yank.c F: include/qemu/yank.h F: qapi/yank.json +F: tests/unit/test-yank.c COLO Framework M: zhanghailiang diff --git a/tests/unit/meson.build b/tests/unit/meson.build index 8ccf60af66..38e5dba920 100644 --- a/tests/unit/meson.build +++ b/tests/unit/meson.build @@ -123,7 +123,8 @@ if have_system 'test-util-sockets': ['socket-helpers.c'], 'test-base64': [], 'test-bufferiszero': [], - 'test-vmstate': [migration, io, '../../monitor/yank.c'] + 'test-vmstate': [migration, io, '../../monitor/yank.c'], + 'test-yank': ['socket-helpers.c', qom, io, chardev, '../../monitor/yan= k.c'] } if 'CONFIG_INOTIFY1' in config_host tests +=3D {'test-util-filemonitor': []} diff --git a/tests/unit/test-yank.c b/tests/unit/test-yank.c new file mode 100644 index 0000000000..44f24c45a8 --- /dev/null +++ b/tests/unit/test-yank.c @@ -0,0 +1,240 @@ +#include "qemu/osdep.h" +#include + +#include "qemu/config-file.h" +#include "qemu/module.h" +#include "qemu/option.h" +#include "qemu/sockets.h" +#include "chardev/char-fe.h" +#include "sysemu/sysemu.h" +#include "qapi/error.h" +#include "qapi/qapi-commands-char.h" +#include "qapi/qmp/qdict.h" +#include "qom/qom-qobject.h" +#include "io/channel-socket.h" +#include "qapi/qobject-input-visitor.h" +#include "qapi/qapi-visit-sockets.h" +#include "socket-helpers.h" +#include "qapi/qapi-commands-yank.h" +#include "qapi/qapi-types-yank.h" + +static int chardev_change(void *opaque) +{ + return 0; +} + +static bool is_yank_instance_registered(void) +{ + YankInstanceList *list; + bool ret; + + list =3D qmp_query_yank(&error_abort); + + ret =3D !!list; + + qapi_free_YankInstanceList(list); + + return ret; +} + +static void char_change_to_yank_test(gconstpointer opaque) +{ + SocketAddress *addr =3D (gpointer) opaque; + Chardev *chr; + CharBackend be; + ChardevReturn *ret; + QIOChannelSocket *ioc; + + /* + * Setup a listener socket and determine its address + * so we know the TCP port for the client later + */ + ioc =3D qio_channel_socket_new(); + g_assert_nonnull(ioc); + qio_channel_socket_listen_sync(ioc, addr, 1, &error_abort); + addr =3D qio_channel_socket_get_local_address(ioc, &error_abort); + g_assert_nonnull(addr); + + ChardevBackend old_backend =3D { .type =3D CHARDEV_BACKEND_KIND_NULL }; + ChardevBackend new_backend =3D { + .type =3D CHARDEV_BACKEND_KIND_SOCKET, + .u.socket.data =3D &(ChardevSocket) { + .addr =3D &(SocketAddressLegacy) { + .type =3D SOCKET_ADDRESS_LEGACY_KIND_INET, + .u.inet.data =3D &addr->u.inet + }, + .has_server =3D true, + .server =3D false + } }; + + g_assert(!is_yank_instance_registered()); + + ret =3D qmp_chardev_add("chardev", &old_backend, &error_abort); + qapi_free_ChardevReturn(ret); + chr =3D qemu_chr_find("chardev"); + g_assert_nonnull(chr); + + g_assert(!is_yank_instance_registered()); + + qemu_chr_wait_connected(chr, &error_abort); + qemu_chr_fe_init(&be, chr, &error_abort); + /* allow chardev-change */ + qemu_chr_fe_set_handlers(&be, NULL, NULL, + NULL, chardev_change, NULL, NULL, true); + + ret =3D qmp_chardev_change("chardev", &new_backend, &error_abort); + g_assert_nonnull(ret); + g_assert(be.chr !=3D chr); + g_assert(is_yank_instance_registered()); + + object_unparent(OBJECT(be.chr)); + object_unref(OBJECT(ioc)); + qapi_free_ChardevReturn(ret); +} + +static void char_change_yank_to_yank_test(gconstpointer opaque) +{ + SocketAddress *addr =3D (gpointer) opaque; + Chardev *chr; + CharBackend be; + ChardevReturn *ret; + QIOChannelSocket *ioc; + + /* + * Setup a listener socket and determine its address + * so we know the TCP port for the client later + */ + ioc =3D qio_channel_socket_new(); + g_assert_nonnull(ioc); + qio_channel_socket_listen_sync(ioc, addr, 1, &error_abort); + addr =3D qio_channel_socket_get_local_address(ioc, &error_abort); + g_assert_nonnull(addr); + + ChardevBackend backend =3D { + .type =3D CHARDEV_BACKEND_KIND_SOCKET, + .u.socket.data =3D &(ChardevSocket) { + .addr =3D &(SocketAddressLegacy) { + .type =3D SOCKET_ADDRESS_LEGACY_KIND_INET, + .u.inet.data =3D &addr->u.inet + }, + .has_server =3D true, + .server =3D false + } }; + + g_assert(!is_yank_instance_registered()); + + ret =3D qmp_chardev_add("chardev", &backend, &error_abort); + qapi_free_ChardevReturn(ret); + chr =3D qemu_chr_find("chardev"); + g_assert_nonnull(chr); + + g_assert(is_yank_instance_registered()); + + qemu_chr_wait_connected(chr, &error_abort); + qemu_chr_fe_init(&be, chr, &error_abort); + /* allow chardev-change */ + qemu_chr_fe_set_handlers(&be, NULL, NULL, + NULL, chardev_change, NULL, NULL, true); + + ret =3D qmp_chardev_change("chardev", &backend, &error_abort); + g_assert_nonnull(ret); + g_assert(be.chr !=3D chr); + g_assert(is_yank_instance_registered()); + + object_unparent(OBJECT(be.chr)); + object_unref(OBJECT(ioc)); + qapi_free_ChardevReturn(ret); +} + +static void char_change_from_yank_test(gconstpointer opaque) +{ + SocketAddress *addr =3D (gpointer) opaque; + Chardev *chr; + CharBackend be; + ChardevReturn *ret; + QIOChannelSocket *ioc; + + /* + * Setup a listener socket and determine its address + * so we know the TCP port for the client later + */ + ioc =3D qio_channel_socket_new(); + g_assert_nonnull(ioc); + qio_channel_socket_listen_sync(ioc, addr, 1, &error_abort); + addr =3D qio_channel_socket_get_local_address(ioc, &error_abort); + g_assert_nonnull(addr); + + ChardevBackend old_backend =3D { + .type =3D CHARDEV_BACKEND_KIND_SOCKET, + .u.socket.data =3D &(ChardevSocket) { + .addr =3D &(SocketAddressLegacy) { + .type =3D SOCKET_ADDRESS_LEGACY_KIND_INET, + .u.inet.data =3D &addr->u.inet + }, + .has_server =3D true, + .server =3D false + } }; + ChardevBackend new_backend =3D { .type =3D CHARDEV_BACKEND_KIND_NULL }; + + g_assert(!is_yank_instance_registered()); + + ret =3D qmp_chardev_add("chardev", &old_backend, &error_abort); + qapi_free_ChardevReturn(ret); + chr =3D qemu_chr_find("chardev"); + g_assert_nonnull(chr); + + g_assert(is_yank_instance_registered()); + + qemu_chr_wait_connected(chr, &error_abort); + qemu_chr_fe_init(&be, chr, &error_abort); + /* allow chardev-change */ + qemu_chr_fe_set_handlers(&be, NULL, NULL, + NULL, chardev_change, NULL, NULL, true); + + ret =3D qmp_chardev_change("chardev", &new_backend, &error_abort); + g_assert_nonnull(ret); + g_assert(be.chr !=3D chr); + g_assert(!is_yank_instance_registered()); + + object_unparent(OBJECT(be.chr)); + object_unref(OBJECT(ioc)); + qapi_free_ChardevReturn(ret); +} + +static SocketAddress tcpaddr =3D { + .type =3D SOCKET_ADDRESS_TYPE_INET, + .u.inet.host =3D (char *)"127.0.0.1", + .u.inet.port =3D (char *)"0", +}; + +int main(int argc, char **argv) +{ + bool has_ipv4, has_ipv6; + + qemu_init_main_loop(&error_abort); + socket_init(); + + g_test_init(&argc, &argv, NULL); + + if (socket_check_protocol_support(&has_ipv4, &has_ipv6) < 0) { + g_printerr("socket_check_protocol_support() failed\n"); + goto end; + } + + if (!has_ipv4) { + goto end; + } + + module_call_init(MODULE_INIT_QOM); + qemu_add_opts(&qemu_chardev_opts); + + g_test_add_data_func("/yank/char_change_to_yank", &tcpaddr, + char_change_to_yank_test); + g_test_add_data_func("/yank/char_change_yank_to_yank", &tcpaddr, + char_change_yank_to_yank_test); + g_test_add_data_func("/yank/char_change_from_yank", &tcpaddr, + char_change_from_yank_test); + +end: + return g_test_run(); +} -- 2.30.2 --Sig_/0yWvBSAJSpD2xkySF0CPHfM Content-Type: application/pgp-signature Content-Description: OpenPGP digital signature -----BEGIN PGP SIGNATURE----- iQIzBAEBCAAdFiEEg/qxWKDZuPtyYo+kNasLKJxdslgFAmBX12UACgkQNasLKJxd slhsTBAArDjIO2P2bCZ1s3lfcNPEgCioQMPOUdQnMWMmgRA+sYjw9ZeaNmHdEKom x0KPEC9fHrlvnxcyvCh9XQ05RE9bwRXGZ7AWDDLIqoFpugw99TcmRsjvTivpvIAg /vjz91g/K7QWuhOSExIujYoPt03YgHhHVzzsD6DBSAAokHDhxWMI7jPR9CY25SJa NnRHzTngdRyfi+CdE0VYmS41QTOGs8UxIz44qwuNFiaEk6KTsmM2QTIXi7RaufnL NDd1AEXDd2Rp37s4TEPlajrPbkXSr0huK/E7K4klY4z0ccywefQOjmZ63oSojjCm zykT0XDgp7GUdBUMiw/smtjOCWQWKLs8A2SqqKeMHwdo+LE/BGnUVD0zgirvBceU mYXQ3IV49Q7kubpdJ3DNdYIgj8BpoX/0pifvFZMNklB/ootn11RvFYt/yKavv/El 5GreGZ9k96Em8OLFNzVjBzcM9tf89JEE88nP3T7ctSZMP14TAhttffEreOGN1BQ+ 6VYmz/ASMVGOhtJ86NTbWkOasSG4Z3cEZnk60riTuvMSlGVfy6z7ydwbxiCZlVfO pdP1cV9l39K+RJ0ZJaBlTWg4sW4ye/7ikma36mmwrfeR9uUV8gzsaq1yZ8Ic1G/n FNRJEHLFGDK8lCPjNsU4E/vQjMPN44NroEyGngjgdrjV2HDJpNk= =KkWY -----END PGP SIGNATURE----- --Sig_/0yWvBSAJSpD2xkySF0CPHfM-- From nobody Sun May 5 13:30:27 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=web.de ARC-Seal: i=1; a=rsa-sha256; t=1616369834; cv=none; d=zohomail.com; s=zohoarc; b=RpEg3LuuzRmfh4gDXlo94TTHzkBCadNebDVsX8C3modJZQ42EhN+EDHr7SnN5vVD17RuJ6ZVLRKvNQslLpk5LSq+iBvGR+Zla3U+NY8k0btu6y3hvfCM4yv249LFPWg5bGccbDapTtt8iPV6KCqPmSyQ0xs92tOGnVord72lXRc= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1616369834; h=Content-Type: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=9SIG1d8JqGICbAlTuJg8fqj0n3NMfj+ZOhhXhEzDjUw=; b=cyH6B43xti3YMYO4UaeNIy6hSiDZKOyrJW6/eX4wrn30VZ/KSq6mxn1OU0HcWiIMFhCVrD/e0Wyq5qfKHLqDQilBs8VpcSgBvlW6Z2VFNgEmv1P+bDgvNpSBPMRoWyK/M91X9+/VTScc083uZEngrhxZ5NEAr/GTPzHgBrHm4T0= 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 1616369834874524.9479455555708; Sun, 21 Mar 2021 16:37:14 -0700 (PDT) Received: from localhost ([::1]:54744 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1lO7cs-0004ex-V6 for importer@patchew.org; Sun, 21 Mar 2021 19:36:54 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:33234) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1lO7Y9-00084w-3B for qemu-devel@nongnu.org; Sun, 21 Mar 2021 19:32:01 -0400 Received: from mout.web.de ([212.227.17.11]:37551) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1lO7Y7-0000Is-6N for qemu-devel@nongnu.org; Sun, 21 Mar 2021 19:32:00 -0400 Received: from gecko.fritz.box ([94.134.180.131]) by smtp.web.de (mrweb103 [213.165.67.124]) with ESMTPSA (Nemesis) id 0M09hY-1lgcMw3dgS-00uL6H; Mon, 22 Mar 2021 00:31:54 +0100 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=web.de; s=dbaedf251592; t=1616369515; bh=U/L3Q2NvRQad/NYnXasRUe7+xtntL2sQsIlsQlfr38A=; h=X-UI-Sender-Class:Date:From:To:Cc:Subject:In-Reply-To:References; b=kKpaFXVAci4MZu9ItDtR1CJwXE2JT+rzSfWL1XG76AS0rAXlz1Vndq4FE1Zo38k8R aSgMvAmJCoQFKcZ9/HtFAuJkjOtn3htCcehJ+cW//r9NKDcapjv0vykMqP8YD9lfK5 3PpEk/pdiTdFeJA4GifwxAfDGTxJuZPFFU2//nao= X-UI-Sender-Class: c548c8c5-30a9-4db5-a2e7-cb6cb037b8f9 Date: Mon, 22 Mar 2021 00:31:53 +0100 From: Lukas Straub To: qemu-devel Subject: [PATCH 3/5] chardev/char.c: Move object_property_try_add_child out of chardev_new Message-ID: <4d61f31f4c2ba1e768d91b2d8f946ea49b3a36fe.1616368879.git.lukasstraub2@web.de> In-Reply-To: References: MIME-Version: 1.0 Content-Type: multipart/signed; boundary="Sig_/_JVFYvpv4SSo7.qmy/1ZEKG"; protocol="application/pgp-signature"; micalg=pgp-sha256 X-Provags-ID: V03:K1:W77HgDixPhRrWxlghc1U5hPGmtWVGbMZ+2tQIL1UuyHrmG1A74A Aa/p5V9FzR+9MCCT+JXz6bxp2YH4gtiCYE+IecoWqJ2XMSsMYjPXjx3LeVPpScyzJzwy7Dw /6PPRffMxvaaR6dcluZM/usl0HJwDO9VbbL3m8N5cH0n1FxYixUr0NK0upzSXLJNd4g0edB gn4+pg8BF80iAbwNto0SA== X-UI-Out-Filterresults: notjunk:1;V03:K0:Vp3fwZzT1Ko=:Dy/cj/a/NC96eNZl7MwWLP cGoaesrDyCsJ1W/oFP9mLg4JMcIzorHvYwUTg7mu8UYun9aMmHYxvbAcEO213WYScc14M5Mel V6gcEQbt0vCZ26mV0lHmRgW8lcqSLg/goyw09Xvr1QgXefmTkW5P+I/yCTLoBHMGgpH3WjLHW 5eRs8itD2dht5X8gN+cChDqZGLH3OoFvRQ9aMM9WcyranMNn687G4R8CEboUIKNHc4EAP+OQ6 ajHekWtmHzBCO38Hiimn68pIRIcLGOOQfvrNwZeTU+0QPzBR/vJXif64XuyYxT2PftSVGvzRG 7iqcq3zbLzkimdhNhZpEaXFQsf1VQtflKBWdOXxC/BUFt6oXR0/RJCVjrqdb5b7RCr9Nl7mXE hFCUN5akUKuqHzdCfJmD0jvq8bZd5N3bX8cY9UG6Ha9zNHYkeSqig6mrZuaDbqxYTK16r+AgP T6QzkivSWxGEnILByMY007H+4ewo2hfzzSdqsqdKhUiTNrq6Ac0JtEFDcV+PTC79K944o2cVR lMqvnJqc0/PpyXxcTlJiZ7OxSz5u+sAWTFtb9KPQca2hABx0DGDL1I6fjOdiOYtFgJUZ0vLCJ eJtXaTlLNQD50tTY3+0XcrogCUXU2OWjP5Bxj+wWtNvwRhzRLNAzUBqFho8CcDHqIrc2TU75v Pic21/RGtJk0Q3WKLKmWcMSPUGHyiZFSc7hVY4ZGPHUDfHjuhU1ggorXGPh7BpxWxfQYZZhUB vBty2bGp2I4eXELJZ/AjyKg2S0lO+xwwoT6/SvkpArozx3fSGaG24XBRx6BdNbFC2hcz28sGk ZgmI5yDMoNhR1yD8sMCqW1r8F4n0VSvFJAXyq+0RMQpU2vQKs5bruYQq3uxM08BawtO2zn6Zf AdEMS5N9kz18VdyXMLghHSVBaH75WybrjUGjHbclazdai5kxbQieXNN9aJ9aSyCJ0QECUHVSI gRdW0Gz/3Kg+V7v/6fmfZSrHRR5zP/fjE9aiLy1hBF+gxbRWxMxVedL+iYryr92QKD2OIiZm7 bJ6z4xfPgEglLstF3pItoFUCfTpjDjBuUNUReMH1aZaJNbk/KckFegzQtH/07yml/u/TBlDLI sEyb2ZswR5vZEtrvsnfQzk2hu8O+uEFL6z1cTcoYhagoCBIlPAD1ilnMHvPDwklkYr1GilOtH WWtdOuexhhawmpq0wkk0yTuez2i0ldEpB9E6InWdi9KimiFe/kZmnT96Yldcamj6d3Zz6UTqE pSrLH7lA/j3fmzc7s5GPJiSRwCT/zF91i5td9Qw== 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=212.227.17.11; envelope-from=lukasstraub2@web.de; helo=mout.web.de X-Spam_score_int: 8 X-Spam_score: 0.8 X-Spam_bar: / X-Spam_report: (0.8 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_ENVFROM_END_DIGIT=0.25, FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_LOW=-0.7, RCVD_IN_MSPIKE_H3=-0.01, RCVD_IN_MSPIKE_WL=-0.01, RCVD_IN_SBL_CSS=3.335, SPF_HELO_NONE=0.001, SPF_PASS=-0.001 autolearn=no 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: Laurent Vivier , Marc-Andre Lureau , Thomas Huth , Li Zhang , Paolo Bonzini Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-Zoho-Virus-Status: 1 X-ZohoMail-DKIM: fail (Header signature does not verify) --Sig_/_JVFYvpv4SSo7.qmy/1ZEKG Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Move object_property_try_add_child out of chardev_new into it's callers. This is a preparation for the next patches to fix yank with the chardev-change case. Signed-off-by: Lukas Straub Reviewed-by: Marc-Andr=C3=A9 Lureau --- chardev/char.c | 42 ++++++++++++++++++++++++------------------ 1 file changed, 24 insertions(+), 18 deletions(-) diff --git a/chardev/char.c b/chardev/char.c index 97cafd6849..1584865027 100644 --- a/chardev/char.c +++ b/chardev/char.c @@ -972,7 +972,9 @@ static Chardev *chardev_new(const char *id, const char = *typename, qemu_char_open(chr, backend, &be_opened, &local_err); if (local_err) { - goto end; + error_propagate(errp, local_err); + object_unref(obj); + return NULL; } if (!chr->filename) { @@ -982,22 +984,6 @@ static Chardev *chardev_new(const char *id, const char= *typename, qemu_chr_be_event(chr, CHR_EVENT_OPENED); } - if (id) { - object_property_try_add_child(get_chardevs_root(), id, obj, - &local_err); - if (local_err) { - goto end; - } - object_unref(obj); - } - -end: - if (local_err) { - error_propagate(errp, local_err); - object_unref(obj); - return NULL; - } - return chr; } @@ -1006,6 +992,7 @@ Chardev *qemu_chardev_new(const char *id, const char *= typename, GMainContext *gcontext, Error **errp) { + Chardev *chr; g_autofree char *genid =3D NULL; if (!id) { @@ -1013,7 +1000,19 @@ Chardev *qemu_chardev_new(const char *id, const char= *typename, id =3D genid; } - return chardev_new(id, typename, backend, gcontext, errp); + chr =3D chardev_new(id, typename, backend, gcontext, errp); + if (!chr) { + return NULL; + } + + if (!object_property_try_add_child(get_chardevs_root(), id, OBJECT(chr= ), + errp)) { + object_unref(OBJECT(chr)); + return NULL; + } + object_unref(OBJECT(chr)); + + return chr; } ChardevReturn *qmp_chardev_add(const char *id, ChardevBackend *backend, @@ -1034,6 +1033,13 @@ ChardevReturn *qmp_chardev_add(const char *id, Chard= evBackend *backend, return NULL; } + if (!object_property_try_add_child(get_chardevs_root(), id, OBJECT(chr= ), + errp)) { + object_unref(OBJECT(chr)); + return NULL; + } + object_unref(OBJECT(chr)); + ret =3D g_new0(ChardevReturn, 1); if (CHARDEV_IS_PTY(chr)) { ret->pty =3D g_strdup(chr->filename + 4); -- 2.30.2 --Sig_/_JVFYvpv4SSo7.qmy/1ZEKG Content-Type: application/pgp-signature Content-Description: OpenPGP digital signature -----BEGIN PGP SIGNATURE----- iQIzBAEBCAAdFiEEg/qxWKDZuPtyYo+kNasLKJxdslgFAmBX12kACgkQNasLKJxd slh55xAAgwkKdyX3fCHQY6ZE4iGZPQRasaU4lvvMb3gqMhL+QQYpo9eTTR5qnxQM 4fJmEgEWndFGiP6zWpYY5ZPT71/3HfQqPPQtwy01dK7sXl0/KK9J/sgQJydB3InU TXV8V9MWpmnaow2B4Jgkcb8Ve0j7TgbbsF7bYXmwxZr6+R5+sSC2Hv5/f9qyJ04l Xrl571J65Bk0+q6ofReJo4G4/+nITHVwj8DV2HNCG1U5KWdCplZhViT0bC9mM1A4 vJXJlmXTDTR3lgaYn47QgTS05f/mFA0248FjAAH6AXa+pangU6SUy74gB6C3ymZm UGsp2b9+utGnDO8eX0nZ0A9611ClRXrvjNR4oSHuaqZQ1aSiwdkwzGKBcNvz+HOC 1+PbiU6HjXayr8kFguUOFRaw3N7nhSC+cn/CMyPpAgX+WVtPI9NPoS5CPUxuIobJ xi4O3IyUbiicnus0U/t4I6Le1O0MIZH6ujLD1yZZlj2jXKTJhUQ7gQ/9GdeGpNl/ 8h+qfHA3oc5X4iWoOpBHyWjEbIcqVnoTRbz254XnhYHXgXnUr0rqGphanJrd8tQg wY0127LDFjGVNtbMPotbeKSG7SUf26cRVX91wONbE3bzN22sD0morsUzNd+GpVXa uJedw+UkUxKB+9Rs+yEvldTDT0jMSq+PnExTB8i4yjElNYEZWU4= =NBd+ -----END PGP SIGNATURE----- --Sig_/_JVFYvpv4SSo7.qmy/1ZEKG-- From nobody Sun May 5 13:30:27 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=web.de ARC-Seal: i=1; a=rsa-sha256; t=1616369748; cv=none; d=zohomail.com; s=zohoarc; b=g+FprUOthfMJB0zpc9yZuhET4TxbwCy8Zyk+PDptMEj1eqow9c2+XEu4WwA3+O+Jo35pxRkWUWaV6Er4h/xsHK04NroDUUN1eE0oOFtv2cpYjTXgua3wJnpWcrm1Y4j4ay1mE9ctyoBOYq7CXRz2HrE/61NK0SvHzc+olynrQ0E= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1616369748; h=Content-Type: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=+hSJfPtYmupP2ik9do0LIp3rnro6TTuAcVGyHdSKB4g=; b=BxibedlHML1ZsVlOx0B3+ZX+PPWWowSFb3cow54r++xl/Sr1CReVviawOhh+43FgQeuRy7BwjQzIVCKmK2ayFx8eubTxDMTQSA52J3G0DLeQ6zGiXYGG8Jvk3xF2Jr5M2ZhciTRSQqOv7XZ4IlHtJNPRtx0ZTVnXAhxyTwMOVlY= 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 1616369748511537.7694797604705; Sun, 21 Mar 2021 16:35:48 -0700 (PDT) Received: from localhost ([::1]:51750 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1lO7bm-0003LH-F7 for importer@patchew.org; Sun, 21 Mar 2021 19:35:46 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:33246) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1lO7YC-0008BO-GK for qemu-devel@nongnu.org; Sun, 21 Mar 2021 19:32:04 -0400 Received: from mout.web.de ([212.227.17.11]:57431) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1lO7YA-0000KC-S2 for qemu-devel@nongnu.org; Sun, 21 Mar 2021 19:32:04 -0400 Received: from gecko.fritz.box ([94.134.180.131]) by smtp.web.de (mrweb102 [213.165.67.124]) with ESMTPSA (Nemesis) id 0Le4Po-1m2hcY1lng-00pvEv; Mon, 22 Mar 2021 00:31:58 +0100 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=web.de; s=dbaedf251592; t=1616369518; bh=/CQzRMKhJDxrhta8zXxxjDrA9hXcSEfYPXhsm1MQFlc=; h=X-UI-Sender-Class:Date:From:To:Cc:Subject:In-Reply-To:References; b=Mce/Ua3qGFBT0DShy0fvB8jH3M1B78hvnZBralvGKBvco/3CGqbd/h5osSqjnDhpX fFzLLXeHoi5CYQgSCudAVgu9mGcMsN9BhsCpXy9wXUvf/DcXeNE9LElKNcrE+vHqLq bJozCVi6pnLcgbTkdpMcBHBYjGe0//YVaw+OjEe8= X-UI-Sender-Class: c548c8c5-30a9-4db5-a2e7-cb6cb037b8f9 Date: Mon, 22 Mar 2021 00:31:57 +0100 From: Lukas Straub To: qemu-devel Subject: [PATCH 4/5] chardev/char.c: Always pass id to chardev_new Message-ID: In-Reply-To: References: MIME-Version: 1.0 Content-Type: multipart/signed; boundary="Sig_/nRBaFJWbg3e0zv2H3M=H/n_"; protocol="application/pgp-signature"; micalg=pgp-sha256 X-Provags-ID: V03:K1:N4LwLlL46OHDs6JuU8sO7Mw9Sn67CdlyRTUo3coUcSVo1lzyoLR J9IKeTnLVkqoeuNPlnrNDtIUuSFCFuhsLkAwCd603GnuXM/QNfvSTs4uYU4iEu/6oFj/0s3 ETzSDK6NAFmIkuYlMqMse0JBzTKHERXuUJ0cRMqmPeGm4qN++rQlk3N14TERKXA6f8OSVG0 FpFor3pH1ORYbyH/HasTw== X-UI-Out-Filterresults: notjunk:1;V03:K0:KaivfXGRMLU=:VSHuqhj4JGCKT//wfhssI2 YDU+7TLyauC0rPkgr82H8fAXy8SFJO0Ggbt7xljzjq3dM81n2GZfpxfjRWUK8MmkgwrjierQp tMTwXdQok3fA9ED1Iz6fvrd6Z51K0t1NVHmZjC83OQ1Z57aME+WPeMlyfLjg6CvwORpgwVGl1 TPvRDJjxmUFevmyEvDz/SH5mgVi+Jz4k9+tHlRENPzP+UyxjsqZy9lKzU9pxrkO1I/Uj+rpzh 0IebtwYLeT2H/bMwRZtC3TVjyJ1FAgknUUFt2hHHStHI0eWd0KrS7a/Bp1RYRCq89bnjNqOrZ Q1JGSnJm4Jb2rznjQ9VbMPCitoPNNK3fJTHs/c7dwV8HNawQTgcJ4/FGs5U/F1lpNzPp0o9pV BMB+jQdaPWHDolhWIcVn8krPdVYF6yA3MWz4/9Xd/ZjEB4yQ/jBynow4IQahsFafKiG3tsepl Z7a7cjf9wV0mZr0Dn193OjEdVKJ6Eiv5NDs/SLVQdtvO8c57H6vj4BsuNwCENcMrJZFqFVc6D i15U/h/ed3BdDNkTWo9cKVAesZG6F3rJ7sgN0Ui9iotkcVLQzK2x2KwKW7BMJUtTx43tZhFNs UXQLBJBCh1cBt9bo7MpXA2hhF3l/evKRTZm1kUMFMKKJxPl5cxl0EsCJjRc5/A5OuKqo4ileW KmLByFebXMAINzQEVp4cMEIsFckD2Qhit0VMcpPxNAQfoblKq2wnTBaRkQ/pFO89qiRuLewjc JmVUiPnK4fl3wVYdiOgly5Wv5dn05VipccCmgE76vB5hCheHVnL1+OZaTqZ9XmojZ0eVUfqbd 6wkq9yNGjJmr523drbTrupNDY1OQ5nnr1PdyiEov0PR2CB0bEnmTQwsAYl09g0Wd4W94MwxLp 92EFQNoF0CjE7dq+fdMDDseV3UMEMJA9yvFoMbFLuV0Eh01066vdEApSqI+Xxrjes8tFFjh9T sXauzyVwH6CkdC062Ac7BRGmXBmKmpK1YpdU4VT3QCVmc05BJ9SUDDfLkVNZ2Drqo33KoR/AB oSALj5ZK2BwsMUir2Al+jPj9p4C/9ZVHg+d8UF4XfpD7TPoYYWEonT4gzU1bgKkdG1cBY/LC1 vnKlqs65JGUnH44R67Bae0yTku+9eg7vxxGM320/ngP5IIx8DgBQUzjzWxmoCWt9WhcndDjY1 ZcClqU6IhNd6A1i+MbDNNBmWMBsGf1AoVO+UOX0iki2mttCqU4xOBsmBdDkVd2zE5BT2oouaZ 7fWkTciK6YfSbzmpr3g0KY97aswBCN/Tv2c4spA== 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=212.227.17.11; envelope-from=lukasstraub2@web.de; helo=mout.web.de X-Spam_score_int: 8 X-Spam_score: 0.8 X-Spam_bar: / X-Spam_report: (0.8 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_ENVFROM_END_DIGIT=0.25, FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_LOW=-0.7, RCVD_IN_MSPIKE_H3=-0.01, RCVD_IN_MSPIKE_WL=-0.01, RCVD_IN_SBL_CSS=3.335, SPF_HELO_NONE=0.001, SPF_PASS=-0.001 autolearn=no 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: Laurent Vivier , Marc-Andre Lureau , Thomas Huth , Li Zhang , Paolo Bonzini Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-Zoho-Virus-Status: 1 X-ZohoMail-DKIM: fail (Header signature does not verify) --Sig_/nRBaFJWbg3e0zv2H3M=H/n_ Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Always pass the id to chardev_new, since it is needed to register the yank instance for the chardev. Also, after checking that nothing calls chardev_new with id=3DNULL, assert() that id!=3DNULL. This fixes a crash when using chardev-change to change a chardev to chardev-socket, which attempts to register a yank instance. This in turn tries to dereference the NULL-pointer. Signed-off-by: Lukas Straub Reviewed-by: Marc-Andr=C3=A9 Lureau Tested-by: Li Zhang --- chardev/char.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/chardev/char.c b/chardev/char.c index 1584865027..ad416c0714 100644 --- a/chardev/char.c +++ b/chardev/char.c @@ -964,6 +964,7 @@ static Chardev *chardev_new(const char *id, const char = *typename, bool be_opened =3D true; assert(g_str_has_prefix(typename, "chardev-")); + assert(id); obj =3D object_new(typename); chr =3D CHARDEV(obj); @@ -1092,12 +1093,11 @@ ChardevReturn *qmp_chardev_change(const char *id, C= hardevBackend *backend, return NULL; } - chr_new =3D chardev_new(NULL, object_class_get_name(OBJECT_CLASS(cc)), + chr_new =3D chardev_new(id, object_class_get_name(OBJECT_CLASS(cc)), backend, chr->gcontext, errp); if (!chr_new) { return NULL; } - chr_new->label =3D g_strdup(id); if (chr->be_open && !chr_new->be_open) { qemu_chr_be_event(chr, CHR_EVENT_CLOSED); -- 2.30.2 --Sig_/nRBaFJWbg3e0zv2H3M=H/n_ Content-Type: application/pgp-signature Content-Description: OpenPGP digital signature -----BEGIN PGP SIGNATURE----- iQIzBAEBCAAdFiEEg/qxWKDZuPtyYo+kNasLKJxdslgFAmBX120ACgkQNasLKJxd sli+Mw/+P+01gQTG7dNYqTdEQ7qzL/cEcENavhDgsBDwm6o2GJivwViyZF9+v9TT BME9+gxAOTMYlfmW2+yrzLLwxyJr2FJ+PwQYA3sKyHFCJR1aDR/dn0dPraHSy5i6 gDsnEMxKaibZPZpJe1HcNvcHKrYTwF5Y1QGrvviH1Kt/2r63GJFOJzEOsO6hJA7E RJFfBHEg69CLPKplcGY06nZ0nnhFc3cT189FEPTZ3wtfsaYy8y7jviGbeT2U4Er5 zOLyeeMIz5RDLej9iX/pZJB11hOpaNI7UVi7VHVuAj/7t+r4rgUHOseoqfYo0aiE 8FpCcRpmADENn45HoIB9mJb+hW+D2Hj3p2u2mKMTM41SQiIjparHek8t8oryFKXA KC/4RgrkfNrG6HImq+ImYHPFdvZKjqd9Nr46Lj3uVnFUUIbBdH0DwmeUlxm4mtHN /Uro2K5S3uHh93uW62be8jFj/xx7YAuGsmhvg+lxpgmv6uzEWgYmpDMRsdcZdIW7 05az6+UcaY61c0UAYH9SVZBmDKbQpg0PyHzjXQfgN709ZadvXXMxIdgxS7H4GKAD NlQWtfuXjROQzboeiCGfi/xLvYXa2+AMs9Sm9sgsd9IcTz8eL8BAIvRNzjvLa8IL h7ERO5WvuevOGHoORS8le5DqaMd32AQWdpa3Osh+mE7p/HnCl9g= =DtBq -----END PGP SIGNATURE----- --Sig_/nRBaFJWbg3e0zv2H3M=H/n_-- From nobody Sun May 5 13:30:27 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=web.de ARC-Seal: i=1; a=rsa-sha256; t=1616369639; cv=none; d=zohomail.com; s=zohoarc; b=SL1PHqBxm0SBJr2vctDF6/mCnJwPnKf1VQ4DF6eycfajVSt+Qd+gWcSBO5UVyIDe4qVmWKV7LM6AeBK3b/i1/RfEv3L6j13OHbRi+y0/qM4gWxxdOkFzZIg8ZDUA5i/oHfD5cvzP3glDPiVFQ0OxrTU1PVr2VFvuM5Z2q1Fdh10= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1616369639; h=Content-Type: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=ZLxINZRxeNunmuBG1WlkDmmtGY+viGD6R16CsrCQAe8=; b=DAe18ixqi/KMZW97Hqb1vCIa8lJd3+bvIvcOahSgZdzJDNcZ3rN9oG2LEQ4E+3Ftl8by0kzxq0Q65xJjoFeaeOM3kcjfCRwj1+OSBGiizw+Ta0EHQDIJg52DZUsK/3LId4h4md3M/wzblYa9UZ9WREvnOyLAoACiO97bc9x1FsM= 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 1616369639073870.625091554312; Sun, 21 Mar 2021 16:33:59 -0700 (PDT) Received: from localhost ([::1]:47794 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1lO7a1-0001kp-JN for importer@patchew.org; Sun, 21 Mar 2021 19:33:57 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:33318) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1lO7YH-0008JY-1I for qemu-devel@nongnu.org; Sun, 21 Mar 2021 19:32:09 -0400 Received: from mout.web.de ([212.227.17.11]:54165) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1lO7YE-0000Lc-E3 for qemu-devel@nongnu.org; Sun, 21 Mar 2021 19:32:08 -0400 Received: from gecko.fritz.box ([94.134.180.131]) by smtp.web.de (mrweb102 [213.165.67.124]) with ESMTPSA (Nemesis) id 0MfHfy-1l3gz20B4s-00Onto; Mon, 22 Mar 2021 00:32:02 +0100 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=web.de; s=dbaedf251592; t=1616369522; bh=wqQ4An/TMM6FIxQ0nDBdd6hH4ru3wVj1eu+s7vxDmBQ=; h=X-UI-Sender-Class:Date:From:To:Cc:Subject:In-Reply-To:References; b=odZvEgvLEUT6i7run8acvREwUhUzLS3njqc8gnGLzR//bGvjgep/OvJDt4SoYjvfO Kl0l73LL+nZhp5UIURRSDgzoQ7gvE2CYodlwyXyiRFNd3PDjIatAYd5npI5n5xWIRT bI40d3+VsDsrWtTAKn3jDg9IhfIH6Ft1Mr779Rxk= X-UI-Sender-Class: c548c8c5-30a9-4db5-a2e7-cb6cb037b8f9 Date: Mon, 22 Mar 2021 00:32:00 +0100 From: Lukas Straub To: qemu-devel Subject: [PATCH 5/5] chardev: Fix yank with the chardev-change case Message-ID: In-Reply-To: References: MIME-Version: 1.0 Content-Type: multipart/signed; boundary="Sig_/Z=ntiXKo1oaCXqVW37womtB"; protocol="application/pgp-signature"; micalg=pgp-sha256 X-Provags-ID: V03:K1:+IRZnfbiFq3UcLckaA+3TzYJPUKBusUva1ERotabTiz1xiBiFdK BIACuDqLgT5dCQxWne0z52JwqLOt0dMnwH716q68dopj6O3yA1rFR8GtLl2/W4vcDjim1Cp UckzjlOnKecdHdH9YvRXMo0/tXp2mHeqA8bt6fqu6pOa7JsMjPF/0ODD4fSrUrfmQ9Ybx9u HUiyJdCmhpltCqEaTBs0w== X-UI-Out-Filterresults: notjunk:1;V03:K0:5+YUTg2wbP4=:wEQVyDQggLzrSmy/L+o2Jw Sv5017iV8dQ0qSxDVrhNIfM8P30dUMt74ikqIfkB8w/tLZbevBuPWcXtjtiM/4qHrDJoqILfa GsLtgqVGkjXSbgH9DZhYxNK1VTzijMOPSnufBrKFCHqCjLyuqqCZAXMu+2hceEUg/+JFX9CmO 6pu0FVSBy1yENFL23vZUnEKwC7W42S251hbpJIH9nwRHywhXa7aLk5HIjJ32TPwJ6NgyDKRIb drs6OFd3mhEQDoW01du6sSHZ3oW8D6y/E7knJUBNm/zOyrF3ILs+Kx+rGHhsZ4SmXXmnXUrV+ Y+TGlTZSPuTLIUspnXbgFSCHBWLuccFdQdcFB2K4j59bC22Ovd471VRSmhSVJMGO087ja0x19 R6Px3WSGyoF2NerIWkPi7SV92Dv14Vwsbj+x61UnBKhYwz9a0GaCaFfZG1aehEJaF6DCbuGWo 9CSw1BgXn3MmhJhkZFkpQ2sPlCfYt9nTAWjLj/Uos0S/nLvDbviBkYiGzKjxb77nbnPf2pcVR eeyyUDVSTCDgzZ/SakE6DD/GrKJrCYA19b9qCFMv4qHyq9Pv0xMz1KAFl5MQVGLi0USDeVUko rWrVejJ5G72hnczIuLsfXaROnDAWsfK40jGY8cI0/mdm8J4aElvlZJdtgkdBePEOeG0Jo4VFj Vndeg5xB+D1kVbr6nPsSK3QqA0wjl249mbPuqlkkHvTTWUHho6zflp9nTqfE3aSyBri+fvIfd fiAGefLhq+5RUg8TbDp2qw8FsymY0BMAjP30nU3pPi5FGpM9wEm8Pu9jXRrO51EV/iuz24lfu cSXVThHqfKsmp4nBCNPB0YO0zY1t3L4HiHD3Q2ieySw6k3af4HKr0nSjTAYSTubI92KB2whtH zwyUBtPFVQqXW7ntvqz/z4csR58a4ePTH8OuEkSIfwx4jTp+Qirsf41TiSw3TzuOjC1iEvGVy k9vnSv1rHakU63xLEYfe1ql2CrY7H1Imvjx64zT8a0NwjoyZ8LfIhHQ4cblTkYtSWUE4UScPy 3G3occ4APinXFbxeCRLpig+iA1CVQPVuRWdiEvNt5h7HGNnXm35wYeintz5TmqFVu4zUUbbrs qTdUxPhvuqmb3xs39w/P70VLjBFvKL8Sv2UOgPsk65aV7n7wA0UB3oAiCx+oDIinchP8qbJ8X l7qj9eO7x5xWDPcFCX8ett6DyYQCl2EjkGqLbbQuHC0pm7e9q4Aiq3JOgfFg5Vn8vzsK6nIYK rgGdFVPmXH2NOnanHZhrqfVmkXtVK1TiOkNohnA== 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=212.227.17.11; envelope-from=lukasstraub2@web.de; helo=mout.web.de X-Spam_score_int: 8 X-Spam_score: 0.8 X-Spam_bar: / X-Spam_report: (0.8 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_ENVFROM_END_DIGIT=0.25, FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_LOW=-0.7, RCVD_IN_MSPIKE_H3=-0.01, RCVD_IN_MSPIKE_WL=-0.01, RCVD_IN_SBL_CSS=3.335, SPF_HELO_NONE=0.001, SPF_PASS=-0.001 autolearn=no 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: Laurent Vivier , Marc-Andre Lureau , Thomas Huth , Li Zhang , Paolo Bonzini Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-Zoho-Virus-Status: 1 X-ZohoMail-DKIM: fail (Header signature does not verify) --Sig_/Z=ntiXKo1oaCXqVW37womtB Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" When changing from chardev-socket (which supports yank) to chardev-socket again, it fails, because the new chardev attempts to register a new yank instance. This in turn fails, as there still is the yank instance from the current chardev. Also, the old chardev shouldn't unregister the yank instance when it is freed. To fix this, now the new chardev only registers a yank instance if the current chardev doesn't support yank and thus hasn't registered one already. Also, when the old chardev is freed, it now only unregisters the yank instance if the new chardev doesn't need it. s->registered_yank is always true here, as chardev-change only works on user-visible chardevs and those are guraranteed to register a yank instance as they are initialized via chardev_new() qemu_char_open() cc->open() (qmp_chardev_open_socket()). Signed-off-by: Lukas Straub Reviewed-by: Marc-Andr=C3=A9 Lureau --- chardev/char-socket.c | 20 +++++++++++++++++--- chardev/char.c | 32 +++++++++++++++++++++++++------- include/chardev/char.h | 4 ++++ 3 files changed, 46 insertions(+), 10 deletions(-) diff --git a/chardev/char-socket.c b/chardev/char-socket.c index c8bced76b7..8186b6a205 100644 --- a/chardev/char-socket.c +++ b/chardev/char-socket.c @@ -1119,7 +1119,13 @@ static void char_socket_finalize(Object *obj) } g_free(s->tls_authz); if (s->registered_yank) { - yank_unregister_instance(CHARDEV_YANK_INSTANCE(chr->label)); + /* + * In the chardev-change special-case, we shouldn't unregister the= yank + * instance, as it still may be needed. + */ + if (chr->yank_unregister_instance) { + yank_unregister_instance(CHARDEV_YANK_INSTANCE(chr->label)); + } } qemu_chr_be_event(chr, CHR_EVENT_CLOSED); @@ -1421,8 +1427,14 @@ static void qmp_chardev_open_socket(Chardev *chr, qemu_chr_set_feature(chr, QEMU_CHAR_FEATURE_FD_PASS); } - if (!yank_register_instance(CHARDEV_YANK_INSTANCE(chr->label), errp)) { - return; + /* + * In the chardev-change special-case, we shouldn't register a new yank + * instance, as there already may be one. + */ + if (chr->yank_register_instance) { + if (!yank_register_instance(CHARDEV_YANK_INSTANCE(chr->label), err= p)) { + return; + } } s->registered_yank =3D true; @@ -1564,6 +1576,8 @@ static void char_socket_class_init(ObjectClass *oc, v= oid *data) { ChardevClass *cc =3D CHARDEV_CLASS(oc); + cc->supports_yank =3D true; + cc->parse =3D qemu_chr_parse_socket; cc->open =3D qmp_chardev_open_socket; cc->chr_wait_connected =3D tcp_chr_wait_connected; diff --git a/chardev/char.c b/chardev/char.c index ad416c0714..245f8be201 100644 --- a/chardev/char.c +++ b/chardev/char.c @@ -39,6 +39,7 @@ #include "qemu/option.h" #include "qemu/id.h" #include "qemu/coroutine.h" +#include "qemu/yank.h" #include "chardev-internal.h" @@ -266,6 +267,8 @@ static void char_init(Object *obj) { Chardev *chr =3D CHARDEV(obj); + chr->yank_register_instance =3D true; + chr->yank_unregister_instance =3D true; chr->logfd =3D -1; qemu_mutex_init(&chr->chr_write_lock); @@ -956,6 +959,7 @@ void qemu_chr_set_feature(Chardev *chr, static Chardev *chardev_new(const char *id, const char *typename, ChardevBackend *backend, GMainContext *gcontext, + bool yank_register_instance, Error **errp) { Object *obj; @@ -968,6 +972,7 @@ static Chardev *chardev_new(const char *id, const char = *typename, obj =3D object_new(typename); chr =3D CHARDEV(obj); + chr->yank_register_instance =3D yank_register_instance; chr->label =3D g_strdup(id); chr->gcontext =3D gcontext; @@ -1001,7 +1006,7 @@ Chardev *qemu_chardev_new(const char *id, const char = *typename, id =3D genid; } - chr =3D chardev_new(id, typename, backend, gcontext, errp); + chr =3D chardev_new(id, typename, backend, gcontext, true, errp); if (!chr) { return NULL; } @@ -1029,7 +1034,7 @@ ChardevReturn *qmp_chardev_add(const char *id, Charde= vBackend *backend, } chr =3D chardev_new(id, object_class_get_name(OBJECT_CLASS(cc)), - backend, NULL, errp); + backend, NULL, true, errp); if (!chr) { return NULL; } @@ -1054,7 +1059,7 @@ ChardevReturn *qmp_chardev_change(const char *id, Cha= rdevBackend *backend, Error **errp) { CharBackend *be; - const ChardevClass *cc; + const ChardevClass *cc, *cc_new; Chardev *chr, *chr_new; bool closed_sent =3D false; ChardevReturn *ret; @@ -1088,13 +1093,20 @@ ChardevReturn *qmp_chardev_change(const char *id, C= hardevBackend *backend, return NULL; } - cc =3D char_get_class(ChardevBackendKind_str(backend->type), errp); - if (!cc) { + cc =3D CHARDEV_GET_CLASS(chr); + cc_new =3D char_get_class(ChardevBackendKind_str(backend->type), errp); + if (!cc_new) { return NULL; } - chr_new =3D chardev_new(id, object_class_get_name(OBJECT_CLASS(cc)), - backend, chr->gcontext, errp); + /* + * Only register a yank instance if the current chardev hasn't registe= red + * one already. + */ + chr_new =3D chardev_new(id, object_class_get_name(OBJECT_CLASS(cc_new)= ), + backend, chr->gcontext, + /* yank_register_instance =3D */ !cc->supports_y= ank, + errp); if (!chr_new) { return NULL; } @@ -1118,6 +1130,12 @@ ChardevReturn *qmp_chardev_change(const char *id, Ch= ardevBackend *backend, return NULL; } + /* + * When the old chardev is freed, it should only unregister the yank + * instance if the new chardev doesn't need it. + */ + chr->yank_unregister_instance =3D !cc_new->supports_yank; + object_unparent(OBJECT(chr)); object_property_add_child(get_chardevs_root(), chr_new->label, OBJECT(chr_new)); diff --git a/include/chardev/char.h b/include/chardev/char.h index 4181a2784a..ff38bb3af0 100644 --- a/include/chardev/char.h +++ b/include/chardev/char.h @@ -65,6 +65,9 @@ struct Chardev { char *filename; int logfd; int be_open; + /* used to coordinate the chardev-change special-case: */ + bool yank_register_instance; + bool yank_unregister_instance; GSource *gsource; GMainContext *gcontext; DECLARE_BITMAP(features, QEMU_CHAR_FEATURE_LAST); @@ -251,6 +254,7 @@ struct ChardevClass { ObjectClass parent_class; bool internal; /* TODO: eventually use TYPE_USER_CREATABLE */ + bool supports_yank; void (*parse)(QemuOpts *opts, ChardevBackend *backend, Error **errp); void (*open)(Chardev *chr, ChardevBackend *backend, -- 2.30.2 --Sig_/Z=ntiXKo1oaCXqVW37womtB Content-Type: application/pgp-signature Content-Description: OpenPGP digital signature -----BEGIN PGP SIGNATURE----- iQIzBAEBCAAdFiEEg/qxWKDZuPtyYo+kNasLKJxdslgFAmBX13AACgkQNasLKJxd sljd8hAAkZkNjeu9BKi+uViMkn+RwMwnjYPsuM+Wxc4g29t2dirpvI3d+JsikwgQ bGCEC2y5nAht/iVlSyhx1DNc63wwWaiPz/yIGAVJjXjB20sVZnxGQfMzvfb1M0gc 3StnoLzzwIB94ylDTdLS6tIpG5ZZLuKV+xD827LNBOPLBux7M0uNiPrxeoJBYKHV owvB72chFX0Ra0DfO5g6/VXvOCfCd8DNyS9oKw66qTPLbYffwvlr0QVeklcc6Xlq N4QKWKSNqIKF8AsAudtM7pCYiJ2Vqn1RBzgGep7ZVT5vYrpn2JwQYuaXNk3fvWfJ VrIJdy6/zsUn1pV+DP+HP7E0lrRxyoJUfV89w7Qmf4ia5A3RbnLZkuHi1ATKtS0B KkTZcxZ9HNNCjtw3LQnlafLZlsvdEVdeYtjrRZY7bUSJheE6YZXWTaVokZXrCLWD rDN6Cbof85gY2eg0m7Hy3jKIW8MToh9pMrjHU1SRkMSMEPdtXF6vuntrfahN/eUU nGB3bp+QvtcadI8x7pel982RPVgUFJDC6YDYRr6wyTNwPMLKYgli+QvsN90hlvLT kOhOhofJY71wg8Vgj7ZqLlASKKBcCqTpNYv/1+md9xwaWEMdrnKJCyJjFZJ9ga0S jf0mC72wlmeWILLDJXPKNqW2c8cC3y07ioZjhFco8uKtlLlAiFA= =YDqN -----END PGP SIGNATURE----- --Sig_/Z=ntiXKo1oaCXqVW37womtB--