From patchwork Thu Jun 27 13:37:43 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Akihiko Odaki X-Patchwork-Id: 1953272 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@legolas.ozlabs.org Authentication-Results: legolas.ozlabs.org; dkim=pass (2048-bit key; unprotected) header.d=daynix-com.20230601.gappssmtp.com header.i=@daynix-com.20230601.gappssmtp.com header.a=rsa-sha256 header.s=20230601 header.b=RLU4ZjHT; dkim-atps=neutral Authentication-Results: legolas.ozlabs.org; spf=pass (sender SPF authorized) smtp.mailfrom=nongnu.org (client-ip=209.51.188.17; helo=lists.gnu.org; envelope-from=qemu-ppc-bounces+incoming=patchwork.ozlabs.org@nongnu.org; receiver=patchwork.ozlabs.org) Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) (using TLSv1.2 with cipher ECDHE-ECDSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by legolas.ozlabs.org (Postfix) with ESMTPS id 4W905Z718xz20Xg for ; Thu, 27 Jun 2024 23:38:42 +1000 (AEST) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1sMpKJ-0000wJ-MX; Thu, 27 Jun 2024 09:38:15 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1sMpKF-0000u1-Vj for qemu-ppc@nongnu.org; Thu, 27 Jun 2024 09:38:13 -0400 Received: from mail-pg1-x52e.google.com ([2607:f8b0:4864:20::52e]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1sMpKD-0006K3-Vk for qemu-ppc@nongnu.org; Thu, 27 Jun 2024 09:38:11 -0400 Received: by mail-pg1-x52e.google.com with SMTP id 41be03b00d2f7-6eab07ae82bso5671103a12.3 for ; Thu, 27 Jun 2024 06:38:08 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=daynix-com.20230601.gappssmtp.com; s=20230601; t=1719495487; x=1720100287; darn=nongnu.org; h=cc:to:content-transfer-encoding:mime-version:message-id:date :subject:from:from:to:cc:subject:date:message-id:reply-to; bh=R7xdJXPQ/0h5i2oefmhppGMlFOw41XX+zUeXH5357TE=; b=RLU4ZjHTJ/lDAqzkVnqCB7ldV98yxbN5XPfBoCnJoyv+v2H3E4AP65QT2RCrPZdXVo DTw+Zf6XYAFJ0bCHCPWi1BSFhlLEqpnszg50lyHY7rNrRkY8qtIbe/ZwJw4/xqMFMvr/ P66pdTPMb3Byr+khfw7wq9bsq4Puo3GkW/GET/B+osr5AEBZuSbdebY76dP2bgtqrVSz G3jHvwNtto7bMK67tYK3VbzkrfWXcfdL/YcIJcpfyFR1+kMwrsFDl5MHNo0nceraopTy JtInjf1YmCFjUfPcy8p87tDOEi6SROGRcQfA/6DeJxdsOnO/yGt1a0VfRIRrIPBet8iQ szGg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1719495487; x=1720100287; h=cc:to:content-transfer-encoding:mime-version:message-id:date :subject:from:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=R7xdJXPQ/0h5i2oefmhppGMlFOw41XX+zUeXH5357TE=; b=l2uIWHYnwITV3u5MCNY89gF/P3u1tE2C8UgrKdSyA0k2h00FBOrllaowsta3P3rJNd lMa6syCXSMeDkN+e1h4NJ0CSb+cvqAZE+LqCtGFcz9h+lTdPvYnDuwweFVf3g6KBGkdf 75k0Cc+vOajskk9j5Jjif1mcGgfkc6JZmQ3FNg3o9vbQcdrwAUZq2sqc6tXE9kWSQZ2+ SVuHkOgAARiFkwFTBxKUi1vgNRlt08X26LAmlct1Aa/NefjiViQc8UeGnn3vC6PnVODt beGG0dFiFhs6JrJP14CD242mbcTgRV6QDOyZSbOsPcv6j10it0YDELkMms2S9o8RE6EV mC2Q== X-Forwarded-Encrypted: i=1; AJvYcCX8gmD9t6ACB+IqwXqIpM/49wyG5kOWyT5iDjc3xAsOjnWmCiU5dAlRp0yuoFm86T1kqI8InsBXoyAiqWs61rCpEwIH X-Gm-Message-State: AOJu0YxcBNQXZpFTAOLbuTbVpV4YUldeyshBj13ZTotTWdiSw9Fcyjav N/7Yk9GSNk4J2sCl+9QUNtY1P2TBDFC46yMseck+YulnzWntYYv/m4bbHWWxBl8= X-Google-Smtp-Source: AGHT+IGwTrPdvWef3J4iAXhB+zx8QCinKcVnw3a3AwZKpx32cS5pjuRoAHRckaAb07ZY39+jtK2d/A== X-Received: by 2002:a05:6a20:4e18:b0:1bd:2520:c595 with SMTP id adf61e73a8af0-1bd2520cc54mr6173708637.31.1719495487456; Thu, 27 Jun 2024 06:38:07 -0700 (PDT) Received: from localhost ([2400:4050:a840:1e00:9ac7:6d57:2b16:6932]) by smtp.gmail.com with UTF8SMTPSA id d2e1a72fcca58-706b491bf0fsm1337964b3a.76.2024.06.27.06.38.02 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Thu, 27 Jun 2024 06:38:07 -0700 (PDT) From: Akihiko Odaki Subject: [PATCH v2 00/15] Fix check-qtest-ppc64 sanitizer errors Date: Thu, 27 Jun 2024 22:37:43 +0900 Message-Id: <20240627-san-v2-0-750bb0946dbd@daynix.com> MIME-Version: 1.0 X-B4-Tracking: v=1; b=H4sIACdrfWYC/zWMQQ7CIBBFr9LMWgxMsdquvIfpglCwsxAMY0ibh ruLJC7fz3/vAHaJHMPUHZBcJqYYKuCpA7ua8HSClsqAErUc8CLYBCHHq/HGa68sQn2+k/O0tcp jrrwSf2LaWzSr3/r3h+ZnJaTwvbUae4nqNt4Xswfazja+YC6lfAGYbqKRlwAAAA== To: Eduardo Habkost , Marcel Apfelbaum , =?utf-8?q?Philippe_Mathieu-?= =?utf-8?q?Daud=C3=A9?= , Yanan Wang , John Snow , BALATON Zoltan , Jiaxun Yang , Nicholas Piggin , Daniel Henrique Barboza , David Gibson , Harsh Prateek Bora , Alexey Kardashevskiy , "Michael S. Tsirkin" , =?utf-8?q?Alex_Benn=C3=A9e?= , Peter Xu , Fabiano Rosas , Paolo Bonzini , David Hildenbrand , Thomas Huth , Laurent Vivier Cc: qemu-devel@nongnu.org, qemu-block@nongnu.org, qemu-ppc@nongnu.org, Akihiko Odaki X-Mailer: b4 0.14-dev-fd6e3 Received-SPF: none client-ip=2607:f8b0:4864:20::52e; envelope-from=akihiko.odaki@daynix.com; helo=mail-pg1-x52e.google.com X-Spam_score_int: -18 X-Spam_score: -1.9 X-Spam_bar: - X-Spam_report: (-1.9 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001, SPF_NONE=0.001 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-ppc@nongnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-ppc-bounces+incoming=patchwork.ozlabs.org@nongnu.org Sender: qemu-ppc-bounces+incoming=patchwork.ozlabs.org@nongnu.org Based-on: <3ad18bc590ef28e1526e8053568086b453e7ffde.1718211878.git.quic_mathbern@quicinc.com> ("[PATCH] cpu: fix memleak of 'halt_cond' and 'thread'") I saw various sanitizer errors when running check-qtest-ppc64. While I could just turn off sanitizers, I decided to tackle them this time. Unfortunately, GLib does not free test data in some cases so some sanitizer errors remain. All sanitizer errors will be gone with this patch series combined with the following change for GLib: https://gitlab.gnome.org/GNOME/glib/-/merge_requests/4120 Signed-off-by: Akihiko Odaki Reviewed-by: Michael S. Tsirkin --- Changes in v2: - Rebased to "[PATCH] cpu: fix memleak of 'halt_cond' and 'thread'". (Philippe Mathieu-Daudé) - Converted IRQs into GPIO lines and removed one qemu_irq usage. (Peter Maydell) - s/suppresses/fixes/ (Michael S. Tsirkin) - Corrected title of patch "hw/virtio: Free vqs after vhost_dev_cleanup()" (was "hw/virtio: Free vqs before vhost_dev_cleanup()") - Link to v1: https://lore.kernel.org/r/20240626-san-v1-0-f3cc42302189@daynix.com --- Akihiko Odaki (15): cpu: Free cpu_ases hw/ide: Convert macio ide_irq into GPIO line hw/ide: Remove internal DMA qemu_irq hw/isa/vt82c686: Define a GPIO line between vt82c686 and i8259 spapr: Free stdout path ppc/vof: Fix unaligned FDT property access hw/virtio: Free vqs after vhost_dev_cleanup() migration: Free removed SaveStateEntry memory: Do not create circular reference with subregion tests/qtest: Use qtest_add_data_func_full() tests/qtest: Free unused QMP response tests/qtest: Free old machine variable name tests/qtest: Delete previous boot file tests/qtest: Free paths tests/qtest: Free GThread include/hw/ppc/mac_dbdma.h | 5 +++-- hw/core/cpu-common.c | 1 + hw/ide/macio.c | 18 +++++++++++++----- hw/isa/vt82c686.c | 7 ++++--- hw/misc/macio/mac_dbdma.c | 10 +++++----- hw/ppc/spapr_vof.c | 2 +- hw/ppc/vof.c | 2 +- hw/virtio/vhost-user-base.c | 2 ++ migration/savevm.c | 2 ++ system/memory.c | 11 +++++++++-- tests/qtest/device-introspect-test.c | 7 +++---- tests/qtest/libqtest.c | 3 +++ tests/qtest/migration-test.c | 18 +++++++++++------- tests/qtest/qos-test.c | 16 ++++++++++++---- tests/qtest/vhost-user-test.c | 6 +++--- 15 files changed, 73 insertions(+), 37 deletions(-) --- base-commit: af799a2337c3e39994411f90631905d809a41da4 change-id: 20240625-san-097afaf4f1c2 Best regards,