{"id":2226560,"url":"http://patchwork.ozlabs.org/api/1.1/covers/2226560/?format=json","web_url":"http://patchwork.ozlabs.org/project/qemu-devel/cover/20260422161202.34150-1-viking4@gmail.com/","project":{"id":14,"url":"http://patchwork.ozlabs.org/api/1.1/projects/14/?format=json","name":"QEMU Development","link_name":"qemu-devel","list_id":"qemu-devel.nongnu.org","list_email":"qemu-devel@nongnu.org","web_url":"","scm_url":"","webscm_url":""},"msgid":"<20260422161202.34150-1-viking4@gmail.com>","date":"2026-04-22T16:12:01","name":"[0/1] migration/multifd: fix channel count TOCTOU race on cancel and retry","submitter":{"id":92831,"url":"http://patchwork.ozlabs.org/api/1.1/people/92831/?format=json","name":"Trieu Huynh","email":"vikingtc4@gmail.com"},"mbox":"http://patchwork.ozlabs.org/project/qemu-devel/cover/20260422161202.34150-1-viking4@gmail.com/mbox/","series":[{"id":501039,"url":"http://patchwork.ozlabs.org/api/1.1/series/501039/?format=json","web_url":"http://patchwork.ozlabs.org/project/qemu-devel/list/?series=501039","date":"2026-04-22T16:12:01","name":"migration/multifd: fix channel count TOCTOU race on cancel and retry","version":1,"mbox":"http://patchwork.ozlabs.org/series/501039/mbox/"}],"comments":"http://patchwork.ozlabs.org/api/covers/2226560/comments/","headers":{"Return-Path":"<qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org>","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@legolas.ozlabs.org","Authentication-Results":["legolas.ozlabs.org;\n\tdkim=pass (2048-bit key;\n unprotected) header.d=gmail.com header.i=@gmail.com header.a=rsa-sha256\n header.s=20251104 header.b=cPkWmns+;\n\tdkim-atps=neutral","legolas.ozlabs.org;\n spf=pass (sender SPF authorized) smtp.mailfrom=nongnu.org\n (client-ip=209.51.188.17; helo=lists1p.gnu.org;\n envelope-from=qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org;\n receiver=patchwork.ozlabs.org)"],"Received":["from lists1p.gnu.org (lists1p.gnu.org [209.51.188.17])\n\t(using TLSv1.2 with cipher ECDHE-ECDSA-AES256-GCM-SHA384 (256/256 bits))\n\t(No client certificate requested)\n\tby legolas.ozlabs.org (Postfix) with ESMTPS id 4g145H6T43z1yCv\n\tfor <incoming@patchwork.ozlabs.org>; Thu, 23 Apr 2026 02:13:06 +1000 (AEST)","from localhost ([::1] helo=lists1p.gnu.org)\n\tby lists1p.gnu.org with esmtp (Exim 4.90_1)\n\t(envelope-from <qemu-devel-bounces@nongnu.org>)\n\tid 1wFaBd-0000Pz-LU; Wed, 22 Apr 2026 12:12:25 -0400","from eggs.gnu.org ([2001:470:142:3::10])\n by lists1p.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256)\n (Exim 4.90_1) (envelope-from <vikingtc4@gmail.com>)\n id 1wFaBZ-0000PG-RP\n for qemu-devel@nongnu.org; Wed, 22 Apr 2026 12:12:22 -0400","from mail-pf1-x434.google.com ([2607:f8b0:4864:20::434])\n by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128)\n (Exim 4.90_1) (envelope-from <vikingtc4@gmail.com>)\n id 1wFaBX-0005P1-3b\n for qemu-devel@nongnu.org; Wed, 22 Apr 2026 12:12:20 -0400","by mail-pf1-x434.google.com with SMTP id\n d2e1a72fcca58-82f8b60e54dso3743296b3a.2\n for <qemu-devel@nongnu.org>; Wed, 22 Apr 2026 09:12:18 -0700 (PDT)","from localhost.localdomain ([42.114.219.141])\n by smtp.gmail.com with ESMTPSA id\n d2e1a72fcca58-82f8e9cbb28sm16558138b3a.13.2026.04.22.09.12.12\n (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256);\n Wed, 22 Apr 2026 09:12:13 -0700 (PDT)"],"DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=gmail.com; s=20251104; t=1776874335; x=1777479135; darn=nongnu.org;\n h=content-transfer-encoding:mime-version:message-id:date:subject:cc\n :to:from:from:to:cc:subject:date:message-id:reply-to;\n bh=XkwnlAaIhmzRWKnpfKUMWyjcWDJ6wWPYevSxB2grds0=;\n b=cPkWmns+ER0PU3gWQU4st5iCvQouQtBk8Pb1K7E6hXOcYNhlB482Cy8cKuBL5q9NQt\n pwmARcGBy5WVk8bbqNTKpywOEc5OIoxm/6pw/P51SjogkmOndV87pY8cb0zQtbsWptFF\n g8gN2cE9rLdz+6td5fnS2lcHt8EndrfjiGWdgX4CPjGWFvgb85aqqzAyzVlclEC+dnWX\n fiZCc8gUXbvjZ5gAhFDVQmjjV3pMreUA3ha3Sy/t8Y0ec5QoD0qZmsC2ZxXBTLiDttXc\n S6keIpYqj5Mt0G0vu/sd7KkqT7ZjC+m04Rx7YUwfT41J2e/Z+AMaYgNFDCeXE4gGLgYk\n raOA==","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=1e100.net; s=20251104; t=1776874335; x=1777479135;\n h=content-transfer-encoding:mime-version:message-id:date:subject:cc\n :to:from:x-gm-gg:x-gm-message-state:from:to:cc:subject:date\n :message-id:reply-to;\n bh=XkwnlAaIhmzRWKnpfKUMWyjcWDJ6wWPYevSxB2grds0=;\n b=L2oWLIdQhc6qHu8C4N+ZkOQ6bqZeXk+n/U2WEtOubuBwUQGokr7iHjXE6UjhJhTc5F\n 93Om+fLIsWf7BZTKjKbyEyXMza0OWrB9jpiYw+nl3D76kmc88BqkXpVJ6YxViVaS2Qj0\n eP+rq7dGfy2hfdMLbdj+6ULhSe/jJmgZfiH8TU0+QeSJPGs62aw5WpyD28jGhmOW6tEJ\n rXdFxUj8Cgq9zXgXMuRAINj5mlEHB1HAKEVKqYefxEs0YKRd5O1IYDOjgNdMWzr7S5qu\n oAGYQi60M/PnA5VU1HmfpdW5DouQ67P3CqIG+dRbBUSv4QFtg7d15lGDAo9+Q9MNXrtb\n 1Q4w==","X-Gm-Message-State":"AOJu0YztzxbyEdqgqLBibzNQuiiCgIC/km3TKe2dHoUQEn7NZE37h1DB\n ylpLJHf5en/YSe85YASauD88TN8dCLhMLOi9UWV/QV5rwwOakfZzXz14c5DLxw==","X-Gm-Gg":"AeBDiesqWeMwsBu4cyY85BmTHEL30KGKECMCQvbL47Ix8Ucbu/57asVP0SznDZn6Fol\n C7potUqadeS3g62IftxN01txibtK1qMmCJQ9KSmxe0FI30JKVXoR8hAvp/TOnjL/V9ik73c8Jlx\n KoKrJl/mJJ615mhHQWOKAv64qROfpmut4iK7MwAXuqRkiTM2dYAyVsh4k8zG0TwKYCdqK/kKsLE\n J5K4ahvipbGb1DNlCyW+WIzhcGWiee0mYcsI73TcLDJj/oA1QECPGIdIl1y77tQpKFkdebI6WXs\n 8djQNXfGfsf87ARCfPrBZHmbQmhojaWCDezfwYnq2B76+tV8V7UpSi7CbfmohNJREcUt5kb5bMq\n WsUtqG2X1UOeB4QmuKXFAh4OecZlT0U//zSQSyu3yNCBQTX4elS3ZDtL3WoKrcK68tILiaWnXAz\n COQj78gDahuPyXH7HmJTTBsdsnQfwWaibuexPr06+CDiQk4FdtExSO3vgwiA==","X-Received":"by 2002:a05:6a00:2da2:b0:829:8942:2c85 with SMTP id\n d2e1a72fcca58-82f8c86b594mr23653843b3a.17.1776874334657;\n Wed, 22 Apr 2026 09:12:14 -0700 (PDT)","From":"Trieu Huynh <vikingtc4@gmail.com>","X-Google-Original-From":"Trieu Huynh <viking4@gmail.com>","To":"qemu-devel@nongnu.org","Cc":"Trieu Huynh <vikingtc4@gmail.com>","Subject":"[PATCH 0/1] migration/multifd: fix channel count TOCTOU race on\n cancel and retry","Date":"Wed, 22 Apr 2026 23:12:01 +0700","Message-ID":"<20260422161202.34150-1-viking4@gmail.com>","X-Mailer":"git-send-email 2.43.0","MIME-Version":"1.0","Content-Transfer-Encoding":"8bit","Received-SPF":"pass client-ip=2607:f8b0:4864:20::434;\n envelope-from=vikingtc4@gmail.com; helo=mail-pf1-x434.google.com","X-Spam_score_int":"-17","X-Spam_score":"-1.8","X-Spam_bar":"-","X-Spam_report":"(-1.8 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1,\n DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1,\n FREEMAIL_ENVFROM_END_DIGIT=0.25, FREEMAIL_FROM=0.001,\n RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001,\n 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.29","Precedence":"list","List-Id":"qemu development <qemu-devel.nongnu.org>","List-Unsubscribe":"<https://lists.nongnu.org/mailman/options/qemu-devel>,\n <mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>","List-Archive":"<https://lists.nongnu.org/archive/html/qemu-devel>","List-Post":"<mailto:qemu-devel@nongnu.org>","List-Help":"<mailto:qemu-devel-request@nongnu.org?subject=help>","List-Subscribe":"<https://lists.nongnu.org/mailman/listinfo/qemu-devel>,\n <mailto:qemu-devel-request@nongnu.org?subject=subscribe>","Errors-To":"qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org","Sender":"qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org"},"content":"From: Trieu Huynh <vikingtc4@gmail.com>\n\nQEMU aborts on src during multifd migration when multifd-channels is\nchanged after migrate_cancel:\n\n~ # qemu-system-x86_64: ../util/yank.c:107: yank_unregister_instance:\nAssertion `QLIST_EMPTY(&entry->yankfns)' failed.\nAborted (core dumped)\n\nBelow is the backtrace:\n#7  yank_unregister_instance (instance=0x7fffffffd870) at ../util/yank.c:107\n#8  migration_cleanup (s=0x555557649d10) at ../migration/migration.c:1338\n#9  migration_cleanup_bh (opaque=0x555557649d10) at ../migration/migration.c:1345\n#10 migration_bh_dispatch_bh (opaque=0x7ffe500051d0) at ../migration/migration.c:350\n#11 aio_bh_call (bh=0x7ffe5006d670) at ../util/async.c:173\n#12 aio_bh_poll (ctx=0x555557649210) at ../util/async.c:220\n#13 aio_dispatch (ctx=0x555557649210) at ../util/aio-posix.c:390\n#14 aio_ctx_dispatch (source=0x555557649210) at ../util/async.c:365\n...\n\nSteps to reproduce:\n\n* 1. Start VM on src\n./qemu-system-x86_64 \\\n  -<other_options>\n\n* 2. Start dest with -incoming defer\n./qemu-system-x86_64 \\\n  -incoming defer \\\n  -<other_options>\n\n* 3. Enable multifd with 4 channels on both src and dest\n{ \"execute\": \"qmp_capabilities\" }\n{ \"execute\": \"migrate-set-capabilities\",\n  \"arguments\": {\"capabilities\": [{\"capability\": \"multifd\",\n                                  \"state\": true}]} }\n{ \"execute\": \"migrate-set-parameters\",\n  \"arguments\": {\"multifd-channels\": 4} }\n{ \"execute\": \"migrate-incoming\",\n  \"arguments\": {\"uri\": \"tcp:0:4444\"} }\n\n* 4. Trigger migration\n** dest:\n{ \"execute\": \"migrate-incoming\", \"arguments\": {\"uri\": \"tcp:0:4444\"} }\n** src:\n{ \"execute\": \"migrate\", \"arguments\": {\"uri\": \"tcp:127.0.0.1:4444\"} }\n\n* 5. Cancel migration on src while status is active\n{ \"execute\": \"migrate_cancel\" }\n\n* 6. Immediately change multifd-channels to 2 on src\n{ \"execute\": \"migrate-set-parameters\",\n  \"arguments\": {\"multifd-channels\": 2} }\n\nQEMU aborts as shown above\n\nTrieu Huynh (1):\n  migration/multifd: fix channel count TOCTOU race on cancel and retry\n\n migration/multifd.c | 13 ++++++++-----\n 1 file changed, 8 insertions(+), 5 deletions(-)"}