From nobody Sat Sep 21 07:35:01 2024 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=pass; 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=pass(p=none dis=none) header.from=kernel.org ARC-Seal: i=1; a=rsa-sha256; t=1705055458; cv=none; d=zohomail.com; s=zohoarc; b=X8k0G2Vs1Rx4ACfDLtc1sE5vmLRUo+EyLS5wwJZ5RS7+2uXZ+fLRhJPR+HuWmmczMCANoLGAq3CDQWNuPMHwXbrVQEXTpsgBz3slmIg9w8O0AOoAd+MgYiqLoak5pLXilVgBbuC08cS6BNQKI7y9Jo51eC7BLwN8yWlt8pOtpys= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1705055458; h=Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:Subject:To:To:Message-Id:Reply-To; bh=Ngjf47Vce3JhVb8qWGutCY/a6ackwxxYMnAO7DilxHs=; b=MrrX/Db/QyHuA3kc1KHfXap8CuqJjNlGqWDarGdUiZRcFSyEwIc0iAJMXZmj3NJGDj0IDY79vLK6ppwxSyUtCKFhG2GherMXP/nV0fiZCCg33z/HL+6svewlA9h5KZ7X/08CRQlc0GmJ/J37zJb+m+wj39MpagQQliS91vHTebQ= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; 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=pass header.from= (p=none dis=none) Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1705055458521685.2846596059456; Fri, 12 Jan 2024 02:30:58 -0800 (PST) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1rOEnH-0007xK-JL; Fri, 12 Jan 2024 05:29:43 -0500 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 1rOEnF-0007wD-0L for qemu-devel@nongnu.org; Fri, 12 Jan 2024 05:29:41 -0500 Received: from dfw.source.kernel.org ([2604:1380:4641:c500::1]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1rOEnD-0006Hs-Dd for qemu-devel@nongnu.org; Fri, 12 Jan 2024 05:29:40 -0500 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by dfw.source.kernel.org (Postfix) with ESMTP id 328CF61CB5; Fri, 12 Jan 2024 10:29:38 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id CE0D1C43390; Fri, 12 Jan 2024 10:29:36 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1705055377; bh=RlkAaN0yX2H3dqDoNpVJQaqZRzFOKfP/8dh+xq5mKls=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=a/xLzBjObEd46yAAGpqfz0SRSzwPEH8iyNUNnt74P4KyRrRGuYEZZFTgSmEfwlVXa AWTDBmCpU+zVAQkbBIZxqDEMWa/UVJ819CFRI9Ap/YxcBHXnou/UsASZS+dcUfVsk/ U2L0XSEeRKaDHjtWdJFhI7zDigw5KYeBsMZNWBNtKSt9GEbbyHK566Bbx8xF465iGY xcGXrKZJi5tgDMz+mXd3k9V/zqVJLJEznGwQFIp9r1uvyZvtBiswQ3eN9+yNsL+vx2 AJm3LfafXjYDVsNJqB1ire2/BOugYbndfll0EvbRkQU5KqgRxuKBlAp7con/bpx20A PINtZuSBUmXiQ== From: deller@kernel.org To: qemu-devel@nongnu.org Cc: Richard Henderson , Helge Deller , Bruno Haible Subject: [PATCH v3 5/9] hw/hppa: Move software power button address back into PDC Date: Fri, 12 Jan 2024 11:29:23 +0100 Message-ID: <20240112102927.35406-6-deller@kernel.org> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20240112102927.35406-1-deller@kernel.org> References: <20240112102927.35406-1-deller@kernel.org> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable 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=2604:1380:4641:c500::1; envelope-from=deller@kernel.org; helo=dfw.source.kernel.org X-Spam_score_int: -68 X-Spam_score: -6.9 X-Spam_bar: ------ X-Spam_report: (-6.9 / 5.0 requ) BAYES_00=-1.9, DKIMWL_WL_HIGH=-2.467, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_MED=-2.3, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, T_SCC_BODY_TEXT_LINE=-0.01 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+importer=patchew.org@nongnu.org Sender: qemu-devel-bounces+importer=patchew.org@nongnu.org X-ZohoMail-DKIM: pass (identity @kernel.org) X-ZM-MESSAGEID: 1705055460483100003 Content-Type: text/plain; charset="utf-8" From: Helge Deller The various operating systems (e.g. Linux, NetBSD) have issues mapping the power button when it's stored in page zero. NetBSD even crashes, because it fails to map that page and then accesses unmapped memory. Since we now have a consistent memory mapping of PDC in 32-bit and 64-bit address space (the lower 32-bits of the address are in sync) the power button can be moved back to PDC space. This patch fixes the power button on Linux, NetBSD and HP-UX. Signed-off-by: Helge Deller Tested-by: Bruno Haible Reviewed-by: Richard Henderson --- hw/hppa/machine.c | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/hw/hppa/machine.c b/hw/hppa/machine.c index 54ca2fd91a..9e611620cc 100644 --- a/hw/hppa/machine.c +++ b/hw/hppa/machine.c @@ -36,8 +36,8 @@ =20 #define MIN_SEABIOS_HPPA_VERSION 12 /* require at least this fw version */ =20 -/* Power button address at &PAGE0->pad[4] */ -#define HPA_POWER_BUTTON (0x40 + 4 * sizeof(uint32_t)) +#define HPA_POWER_BUTTON (FIRMWARE_END - 0x10) +static hwaddr soft_power_reg; =20 #define enable_lasi_lan() 0 =20 @@ -45,7 +45,6 @@ static DeviceState *lasi_dev; =20 static void hppa_powerdown_req(Notifier *n, void *opaque) { - hwaddr soft_power_reg =3D HPA_POWER_BUTTON; uint32_t val; =20 val =3D ldl_be_phys(&address_space_memory, soft_power_reg); @@ -221,7 +220,7 @@ static FWCfgState *create_fw_cfg(MachineState *ms, PCIB= us *pci_bus, fw_cfg_add_file(fw_cfg, "/etc/hppa/machine", g_memdup(mc->name, len), len); =20 - val =3D cpu_to_le64(HPA_POWER_BUTTON); + val =3D cpu_to_le64(soft_power_reg); fw_cfg_add_file(fw_cfg, "/etc/hppa/power-button-addr", g_memdup(&val, sizeof(val)), sizeof(val)); =20 @@ -295,6 +294,8 @@ static TranslateFn *machine_HP_common_init_cpus(Machine= State *machine) ram_max =3D 0xf0000000; /* 3.75 GB (32-bit CPU) */ } =20 + soft_power_reg =3D translate(NULL, HPA_POWER_BUTTON); + for (unsigned int i =3D 0; i < smp_cpus; i++) { g_autofree char *name =3D g_strdup_printf("cpu%u-io-eir", i); =20 --=20 2.43.0