From patchwork Tue Apr 8 18:55:32 2025 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?q?Philippe_Mathieu-Daud=C3=A9?= X-Patchwork-Id: 879044 Delivered-To: patch@linaro.org Received: by 2002:a5d:6dae:0:b0:38f:210b:807b with SMTP id u14csp5636109wrs; Tue, 8 Apr 2025 11:57:02 -0700 (PDT) X-Forwarded-Encrypted: i=2; AJvYcCV2x2Sb+EVX99k6amFVyK2LtwzQQE9xliBvz3Ms95uNwv1qli6gTV6Q+l5RNZ2nOBUtz6nlhg==@linaro.org X-Google-Smtp-Source: AGHT+IF+D/e6IyOFca9J7yrarkQ5OozMdsJCdZ2jJxsAcD20kneZA63ZbYoESTnN4ZhroLaUpF/P X-Received: by 2002:ac8:7dcd:0:b0:476:639e:edf1 with SMTP id d75a77b69052e-4795f2dda2bmr979981cf.23.1744138622441; Tue, 08 Apr 2025 11:57:02 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1744138622; cv=none; d=google.com; s=arc-20240605; b=hbksZgjBSAfZG5jr4haDR4kDj7Dup8nFJwQpziHKsM+V2L+cCEOf4mYGm2LgkohGsP fMjCuyLnFZWZPG7M/P/mCqD8qobtm0uJn5+TCevFXxBkhMZb8ou3riJpZM//tXDdMRnI E4UFjN6npDK6jq71b160baAoBxg2Y+TkulPPv8KUUQHQ5qy1drLF8nLnxWE1MU6sgTM8 Sox5hJ6JZ77Ax728nOQiiIVz6EL4MHT4U2AhWyZuPyVMlxJh0vPOqpV25fG4FpdjzGZb ngyY+sqK/4GKb+X48HTt9/qr4YGaCR6XD4PBPJG2Tj90NNjDZpL9CWAE7HWHSr58oaIf VNxw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20240605; h=sender:errors-to:list-subscribe:list-help:list-post:list-archive :list-unsubscribe:list-id:precedence:content-transfer-encoding :mime-version:references:in-reply-to:message-id:date:subject:cc:to :from:dkim-signature; bh=gHTmarhdzx/+kGuibyF5scVpYUQxoz4xhKrDhSMfcxQ=; fh=pRKhRQ5C8okbjnnEhRSvEkj5whlgEuxk3RJlYTtO6/w=; b=kQ5ZGYR+dv7So7sgTwMgT+j05xmVRVK7gIPXro7ybTypK4kXSYsj7egD03Ndy7IpQG v1ocdH+WPZeeCDCNT7d9s144hLC1Wd9XZFpax4n23aexiJReP23oV4tXrcTV3NH5REWW tMY+BYIJovBd6PBFZJoDOyTivsKL8Jh4n55dbndpr+TrrQYsqs2WiHggJeSPXwxS9wyE auX3fnGZybxmG7BO+9iqXGXf07d6lhpKGFhfx0N5iuCODw+Lmg0PCwho/cadGqFNE4rd C1v4sxAMOW7u0V4yeeVGjcNbmBBLt1nClXispwKDC1PgGlnbwUY1KF/zrtJbzCAT47wg 9g5w==; dara=google.com ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=tdE9NBVU; spf=pass (google.com: domain of qemu-devel-bounces+patch=linaro.org@nongnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom="qemu-devel-bounces+patch=linaro.org@nongnu.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org; dara=neutral header.i=@linaro.org Return-Path: Received: from lists.gnu.org (lists.gnu.org. [209.51.188.17]) by mx.google.com with ESMTPS id d75a77b69052e-4791b1c3a51si107151741cf.523.2025.04.08.11.57.02 for (version=TLS1_2 cipher=ECDHE-ECDSA-CHACHA20-POLY1305 bits=256/256); Tue, 08 Apr 2025 11:57:02 -0700 (PDT) Received-SPF: pass (google.com: domain of qemu-devel-bounces+patch=linaro.org@nongnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=tdE9NBVU; spf=pass (google.com: domain of qemu-devel-bounces+patch=linaro.org@nongnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom="qemu-devel-bounces+patch=linaro.org@nongnu.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org; dara=neutral header.i=@linaro.org Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1u2E78-0004Yp-6L; Tue, 08 Apr 2025 14:56:02 -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 1u2E71-0004Xx-V8 for qemu-devel@nongnu.org; Tue, 08 Apr 2025 14:55:57 -0400 Received: from mail-wm1-x329.google.com ([2a00:1450:4864:20::329]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1u2E6x-0004Cx-Od for qemu-devel@nongnu.org; Tue, 08 Apr 2025 14:55:55 -0400 Received: by mail-wm1-x329.google.com with SMTP id 5b1f17b1804b1-43cebe06e9eso40398255e9.3 for ; Tue, 08 Apr 2025 11:55:51 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1744138549; x=1744743349; darn=nongnu.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=gHTmarhdzx/+kGuibyF5scVpYUQxoz4xhKrDhSMfcxQ=; b=tdE9NBVU7MaQwfuVml2H3+LQ9l/leoMiTqL7eegQLjFKHVwHlha5f0qesDdJ6/fJxn FtqqKNFpphYJILbX5YhMIVDTI+n37rqmZBmz/yxU+xmlFjBkylDMJNYv3fcTKk+HQfAg btKHgeQPhIb/g/f8lmMGvUiYkjZ/CfmbsP10Xee/U3ZRKqghspAvnMTHqeL2vsJR1qHk 4AG0J/5u8yyeCpy2EOIDVhbheU0himqsagt2F0CUN0UTjduJqafDqiM4c4nlRVdMJnG4 Bl7qxRwdcZxVBpxUZ4pdYpFztCdeNoWg82/5cW5+SSKNVsw3P2DnN0s0fHJOQh0jt7cK FETQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1744138549; x=1744743349; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=gHTmarhdzx/+kGuibyF5scVpYUQxoz4xhKrDhSMfcxQ=; b=SVWmGGlPV/efxXpR2uOe6+vpE/LtCpETYhemyc3dj8O4LXRJZZRwSuTxt5Qc5g/SYb 3UNpOs9k/vu1FidV6nLnxhoLQwxgTEsF9ZQqTAH8Tj6iadGBnOalh9w4hLEVuuQ8b5Js MesKIp1CiR+XOuH/OSny0hR41Zzj/22o+tKzSJBXg5OkUJB6GV9uIPMPgIWPENDbnpzP lKqfaIZDBw0yzeTmu+Xfbdh8WTB1ct5XHtsOQJlyfkRncLGGxGSzjG/2HIHDOrbIZ4Uo QRS5ytA4fmqv5PbiC6Ujd1MF3GpNxRJdSGRUO+BPbcK8Ew/cuAVlgRu/m8SgXRTT6opV VbHA== X-Gm-Message-State: AOJu0YzTCv3ejxHzUi7iD2pEllV+jfC1oeCytalU4kZCFjlp/yWw7QJs 7xOVw4jVYP5aGHc2JyjfIVH2ftc5C+z5+xAqkSicU4ar9PaQxLLaOl0i98gcAJw4J3Q34Vdp/28 0vno= X-Gm-Gg: ASbGnctVw58Hr3WrXneHtcSWdv6rPertrVQXSX08Ok7cmbvaNWT2Z5C5vEGlMxfhlQM iaDMxR1DNE/jGsfAwJp28J+Dxzy9E+0QFoWYlDCsI4yW+QNPRgHmraavJ6kLp1TDcElPhCvyLyu 2AslDKSL10WkW4L+1UU2vUJdSHbOVH2IzHo/LyfHEz09JoTcE1mZXPRy0imBiAxyC37+ZUfDi1I SPVSlg0jUhNkwOWA/lXp3SONIgTN7ahkD29BRUOjExyV+sEea3y572lcXS1ZwbkaUebHTsjS6YN pUySlAM4chN0ESre1YYhVNww56ci+0By03QvUGIzo/ZVS2E+iuYSQ0xfnqIU/1f3wu5uHWCRfad dUaZrTW7+sR7Vivma3i4= X-Received: by 2002:a05:600c:1c02:b0:43b:ca39:6c7d with SMTP id 5b1f17b1804b1-43f1ec7cd2dmr2584905e9.3.1744138549410; Tue, 08 Apr 2025 11:55:49 -0700 (PDT) Received: from localhost.localdomain (88-187-86-199.subs.proxad.net. [88.187.86.199]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-43ec1663046sm175470495e9.13.2025.04.08.11.55.48 (version=TLS1_3 cipher=TLS_CHACHA20_POLY1305_SHA256 bits=256/256); Tue, 08 Apr 2025 11:55:48 -0700 (PDT) From: =?utf-8?q?Philippe_Mathieu-Daud=C3=A9?= To: qemu-devel@nongnu.org Cc: Joel Stanley , Alistair Francis , =?utf-8?q?Philippe_Mathieu-Dau?= =?utf-8?q?d=C3=A9?= Subject: [PULL 2/8] hw/core/machine: Fix -machine dumpdtb=file.dtb Date: Tue, 8 Apr 2025 20:55:32 +0200 Message-ID: <20250408185538.85538-3-philmd@linaro.org> X-Mailer: git-send-email 2.47.1 In-Reply-To: <20250408185538.85538-1-philmd@linaro.org> References: <20250408185538.85538-1-philmd@linaro.org> MIME-Version: 1.0 Received-SPF: pass client-ip=2a00:1450:4864:20::329; envelope-from=philmd@linaro.org; helo=mail-wm1-x329.google.com 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, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, 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.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+patch=linaro.org@nongnu.org Sender: qemu-devel-bounces+patch=linaro.org@nongnu.org From: Joel Stanley In commit 8fd2518ef2f8 ("hw: Centralize handling of -machine dumpdtb option") the call to dump was moved with respect to the init of the machine. This resulted in the device tree missing parts of the machine description, depending on how they construct their device tree. The arm virt machine is missing some PSCI nodes, while the riscv one is missing most of its content. Move the dump to after the notifiers have been run, allowing virt_machine_done to be called and the device tree to be fully populated. Fixes: 8fd2518ef2f8 ("hw: Centralize handling of -machine dumpdtb option") Signed-off-by: Joel Stanley Reviewed-by: Alistair Francis Reviewed-by: Philippe Mathieu-Daudé Message-ID: <20250401041509.719153-1-joel@jms.id.au> Signed-off-by: Philippe Mathieu-Daudé --- hw/core/machine.c | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/hw/core/machine.c b/hw/core/machine.c index f52a4f2273b..63c6ef93d29 100644 --- a/hw/core/machine.c +++ b/hw/core/machine.c @@ -1731,12 +1731,6 @@ void qdev_machine_creation_done(void) phase_advance(PHASE_MACHINE_READY); qdev_assert_realized_properly(); - /* - * If the user used -machine dumpdtb=file.dtb to request that we - * dump the DTB to a file, do it now, and exit. - */ - handle_machine_dumpdtb(current_machine); - /* TODO: once all bus devices are qdevified, this should be done * when bus is created by qdev.c */ /* @@ -1750,6 +1744,12 @@ void qdev_machine_creation_done(void) notifier_list_notify(&machine_init_done_notifiers, NULL); + /* + * If the user used -machine dumpdtb=file.dtb to request that we + * dump the DTB to a file, do it now, and exit. + */ + handle_machine_dumpdtb(current_machine); + if (rom_check_and_register_reset() != 0) { exit(1); }