From nobody Fri Nov 14 13:41:18 2025 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 ARC-Seal: i=1; a=rsa-sha256; t=1587929831; cv=none; d=zohomail.com; s=zohoarc; b=W1NkUNjkT/amK1wZObwwckPa2XSjjqFnUF27m4vn1vwA/MPu5HE+VNXqauwO5iUII90HIWNKmPcaKAM/gRBOw1HtYkllHhkXjpYEoiUeLFpvqXT9+SRzHMaUGViGAfs5YC9jc9vKcYa63h2K3SB17rijTmH+HGREt1yDVrvhVlw= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1587929831; 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=qRWSqwM2fCu1pU9beHihTmDheFs1RDYc89JAMHB88mw=; b=bzhckSjWm1ibwxsnuXLx3sKTplvL6+daKUJsVoUtiKc9Hh12B6R6VGrQXx4dtDWRr7tSinRSK+TsyjzESwHFn64AN6ChsCaFG+jPW/7bKfonzDbzn3oXxkzyOwTIHZlPz5NKF4s6kfwJSk0jBq7vAktf2cwaiq8U/TNVjPpem5g= 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 Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1587929830878786.6676176160983; Sun, 26 Apr 2020 12:37:10 -0700 (PDT) Received: from localhost ([::1]:43702 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1jSn5O-0005K8-3H for importer@patchew.org; Sun, 26 Apr 2020 15:37:07 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:47706) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1jSmud-0001Cu-Rz for qemu-devel@nongnu.org; Sun, 26 Apr 2020 15:26:00 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.90_1) (envelope-from ) id 1jSmud-0005nJ-Cz for qemu-devel@nongnu.org; Sun, 26 Apr 2020 15:25:59 -0400 Received: from mout.web.de ([217.72.192.78]:42417) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1jSmuc-0005iC-P7 for qemu-devel@nongnu.org; Sun, 26 Apr 2020 15:25:58 -0400 Received: from luklap ([89.247.255.95]) by smtp.web.de (mrweb102 [213.165.67.124]) with ESMTPSA (Nemesis) id 0M09dQ-1jC62K0IlY-00uKv9; Sun, 26 Apr 2020 21:25:40 +0200 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=web.de; s=dbaedf251592; t=1587929140; bh=6pc9YXNKhuW8jIo+3vRiEjlLgHIMsoHErd6rjWBDDRk=; h=X-UI-Sender-Class:Date:From:To:Cc:Subject:In-Reply-To:References; b=YwWoBmLJDlw2wpSHh/fJOU8LgyEgfCctJzFNUz2fLQgmQDjWXa4p2VbSWVTfp79sn Zn/beSM+eQDSPpP5nlMHUJ43SMiIykmao5frkaFpmGjgtokJK8/4n0zdSUL3b7i4uY xnWDGDHZpn+z/cnkp3tKiC5Uf6dMRPYyTEJLen3E= X-UI-Sender-Class: c548c8c5-30a9-4db5-a2e7-cb6cb037b8f9 Date: Sun, 26 Apr 2020 21:25:38 +0200 From: Lukas Straub To: qemu-devel Subject: [PATCH v2 1/6] net/colo-compare.c: Create event_bh with the right AioContext Message-ID: In-Reply-To: References: MIME-Version: 1.0 Content-Type: multipart/signed; boundary="Sig_/x=njM09_KyZCR5NOU5o5MQ_"; protocol="application/pgp-signature"; micalg=pgp-sha512 X-Provags-ID: V03:K1:JlkzBR+7HxwSYRWS4OUWJUhDmB/2kMywrHnAnQ7UBqYh9h9TwMY OSN0sM3cSjIbv5pVtUHzVzfsUryRTwkgSOAANyr3ZvMtH3ljmxQpqqLecn5B/C3Mvk141SK ifaZBVe+AtJMC1568mz5q32eBGocLXzs2bTCnfPUur13xebqsi+Jp0KZXXdtIn/c7HmpoeH 0F/C6zP3GV9imHhGfVdZw== X-UI-Out-Filterresults: notjunk:1;V03:K0:6xeLxWDG2qU=:5pgQqo9w8wXgC8tnJlxFYC /dWZ6e+qeO2jBe/nxW4tG2AQ/uebqafJwMlowJUgSnh5cDtIOGBEahxSSjd/6bb7rbs+xfPVo 2aSC2pGBrlrXZRiXYSqSa07XIi8XLWJ3WUVd5i8wzth3lyPRYzYvX1uWrfDf3AloC5K6upfsr jDYE1OAlqhGm9x5qSMUfkZwBEm7Y4pcuGYkit5WTqpdr8VdQsero4L7YF2IbLVFlpVEXLgooH 6pTR6shmTyNuyt/t41GoJdZLKXdQ7bdDUYHxuBmh8O37w4bCs7A/zAy121OzI1KLLaRFrf5VX 0ylqlCSmYVrgoWbYaiAfdYxjiY3+PR9hSwu21gObkX9yUdrY7+8lWCxN6zsMJ+rXPXCco+iuf PhfluGigjxxo9Yx/FLuffR2YzcAITs5S4yearA5GRy9GNaVd3AjaE4sDLmbqWGiZUVeuF8Ico 13LlgvHfZ800BzHYMy/vpYXkC3F3VUSdDoDTFb9vy50/CFh3NW/ZuddyHFu/tcwUIszTnyj6m s4fiQvHSN12EVW8bH47eNCxyLw9LL/mg4hrFQzd4QsOUKpWsVRzN7ga4NIQgdtqQbe2jJF1nm 96Rq5ONiVqDZjaoJ5GSBTU19jWJ5sJmP0Jv9kej/8TvA4znxz2LnCodHIr7ZKjl4zBSZBs9rj foXikZLuNDr68GBLUMdKHVt74Ptj1GKVi2ACmVh+DNtZbhcTnYrE8KL5W1U/b7T8SkCey98AA Dz6Q2RiDp/z6hoToPjkA3RdVvBtQO35+XH8vjvhPqifjhnaHp/CpfZH0osvcFlPgvSPHTYthx e1YRN3zv4tV7e+m+ws0Z9hXrgiJHguflMMSs65V0HHVCoKG0s2YeJnrbA0OPOjUkeXxJxhj/B 9MZbqhAbLmIS2ZecFQZfftwrQdE42PjooyOtwIPLHv8Gj+Z06J55sMgctmYGRBPO4k5L6xZ8U x3m8+kSeFY8nkCxj5I8HAsOV1ER/n2MkncXiAa0biW9Yby/wuE01UrZ/KGAPTegP4riMkzY0v k/S6b7W7vJjvnWdLx+RxT8PcNY83Eu3DhkOW2/ja9SuenoFP+OPHZINlkaQZgflVquqvjOGXm ZY9GHH33bhLyW0xERNjVN3XICCbmh7MH1Z9C6/3wPjy37XQoFftpv1zCSp3BUZS48SBIfm2K9 iXPTQFpeB/YNZ/q3Ot8tkbL28saenzB4uJR2oajld0iq6lmQQXnnV1fT6iMjD018a+MwAbpiT 7hrqexU+pyMGsAB6s 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-detected-operating-system: by eggs.gnu.org: First seen = 2020/04/26 15:25:57 X-ACL-Warn: Detected OS = Linux 2.2.x-3.x [generic] X-Received-From: 217.72.192.78 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: Zhang Chen , Jason Wang , Paolo Bonzini , Li Zhijian , =?UTF-8?B?TWFyYy1BbmRyw6k=?= Lureau 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_/x=njM09_KyZCR5NOU5o5MQ_ Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" qemu_bh_new will set the bh to be executed in the main loop. This causes crashes as colo_compare_handle_event assumes that it has exclusive access the queues, which are also concurrently accessed in the iothread. Create the bh with the AioContext of the iothread to fulfill these assumptions and fix the crashes. This is safe, because the bh already takes the appropriate locks. Signed-off-by: Lukas Straub Reviewed-by: Zhang Chen Reviewed-by: Derek Su Tested-by: Derek Su --- net/colo-compare.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/net/colo-compare.c b/net/colo-compare.c index 10c0239f9d..1de4220fe2 100644 --- a/net/colo-compare.c +++ b/net/colo-compare.c @@ -890,6 +890,7 @@ static void colo_compare_handle_event(void *opaque) =20 static void colo_compare_iothread(CompareState *s) { + AioContext *ctx =3D iothread_get_aio_context(s->iothread); object_ref(OBJECT(s->iothread)); s->worker_context =3D iothread_get_g_main_context(s->iothread); =20 @@ -906,7 +907,7 @@ static void colo_compare_iothread(CompareState *s) } =20 colo_compare_timer_init(s); - s->event_bh =3D qemu_bh_new(colo_compare_handle_event, s); + s->event_bh =3D aio_bh_new(ctx, colo_compare_handle_event, s); } =20 static char *compare_get_pri_indev(Object *obj, Error **errp) --=20 2.20.1 --Sig_/x=njM09_KyZCR5NOU5o5MQ_ Content-Type: application/pgp-signature Content-Description: OpenPGP digital signature -----BEGIN PGP SIGNATURE----- iQIzBAEBCgAdFiEEg/qxWKDZuPtyYo+kNasLKJxdslgFAl6l4DIACgkQNasLKJxd sli/Qw//ck9gGY1Er61CekXwYNaF89ZkAUNCiSrCM0bX2rKoV5exbreMgV1Vx3wa MpCJqDcOsIk8izknKGxnmRj5XAFVy9XoLmegx1Agy/9leBO9kGhiDFjdyDTcDLuG v4F+ttgO46w3F1yLUm1o01GkH6kna5BjBcH8Q2qH8Ue5Z5eZiaz/oZNVfSgd+V1o s488UobY8bCgKX95hhfhauF/wcmOdxNuAuAbSvK9EJNC7u3ClX+65L9urfUcAzjk Usb2bjelVfx0g+xyVMs6hhWG4AjKFFhttyz3tl9MM9QVyl45cgY24zDiPALFY7hN wUIR2W2KzyWa/KE8nZh5KaLENv4KKuxjnXNifIlfXPkDxWqOGQgq1FmUXX/UHN0P 3LHGoSD88CoeCH8QIfoLUK8jAMlCh6ne8qe9thxyw6TlQvKAhuCn1DBQwQgAE1JR bP2AiX0+9nitcS7MGbTcBfCslQYRxL5CWhh+X99O9vBr97kzoFLVVC50smOi8HNm Th+JQ5X5uQHIZU0aPPH1FGdA/dS6xuZVi8EJSiP37Ma/O6intGADXs3azbeceiKg QrpjdFIqAevF+TQVsgcgS9vG0X2E1yYgavsjf1qu+YG2yxyK6dySAxH2NF6/oQTL IjHverPi3dIX8823boOO/tKPlvqvVu2DbGaGJlK/Har5KdcqrM4= =UUtZ -----END PGP SIGNATURE----- --Sig_/x=njM09_KyZCR5NOU5o5MQ_--