NetBSD Problem Report #58817

From martin@duskware.de  Fri Nov  8 12:34:05 2024
Return-Path: <martin@duskware.de>
Received: from mail.netbsd.org (mail.netbsd.org [199.233.217.200])
	(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)
	 key-exchange X25519 server-signature RSA-PSS (2048 bits)
	 client-signature RSA-PSS (2048 bits))
	(Client CN "mail.NetBSD.org", Issuer "mail.NetBSD.org CA" (not verified))
	by mollari.NetBSD.org (Postfix) with ESMTPS id 0E2041A9238
	for <gnats-bugs@gnats.NetBSD.org>; Fri,  8 Nov 2024 12:34:05 +0000 (UTC)
Message-Id: <20241108123356.6DB865CC79B@emmas.aprisoft.de>
Date: Fri,  8 Nov 2024 13:33:56 +0100 (CET)
From: martin@NetBSD.org
Reply-To: martin@NetBSD.org
To: gnats-bugs@NetBSD.org
Subject: acpidump -dt requires ACPIDUMP_USE_DEVMEM to work
X-Send-Pr-Version: 3.95

>Number:         58817
>Category:       bin
>Synopsis:       acpidump -dt requires ACPIDUMP_USE_DEVMEM to work
>Confidential:   no
>Severity:       serious
>Priority:       medium
>Responsible:    bin-bug-people
>State:          open
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Fri Nov 08 12:35:00 +0000 2024
>Last-Modified:  Tue Nov 12 21:50:01 +0000 2024
>Originator:     Martin Husemann
>Release:        NetBSD 10.99.12
>Organization:
The NetBSD Foundation, Inc.
>Environment:
System: NetBSD master-of-confusion.duskware.de 10.99.12 NetBSD 10.99.12 (GENERIC) #747: Thu Nov 7 16:08:27 CET 2024 martin@seven-days-to-the-wolves.aprisoft.de:/work/src/sys/arch/amd64/compile/GENERIC amd64
Architecture: x86_64
Machine: amd64
>Description:

Trying to dump tables with acpdidump fails unless you set ACPIDUMP_USE_DEVMEM=1

>How-To-Repeat:

# acpidump -dt > foo.asl
acpidump: can't read table
# ACPIDUMP_USE_DEVMEM=1 acpidump -dt > foo.asl
#

>Fix:
n/a

>Audit-Trail:
From: mlelstv@serpens.de (Michael van Elst)
To: gnats-bugs@netbsd.org
Cc: 
Subject: Re: bin/58817: acpidump -dt requires ACPIDUMP_USE_DEVMEM to work
Date: Fri, 8 Nov 2024 16:25:40 -0000 (UTC)

 martin@NetBSD.org writes:

 ># acpidump -dt > foo.asl
 >acpidump: can't read table
 ># ACPIDUMP_USE_DEVMEM=1 acpidump -dt > foo.asl


 Without INSECURE kernel, that also fails...

From: Christos Zoulas <christos@zoulas.com>
To: gnats-bugs@netbsd.org
Cc: gnats-admin@netbsd.org,
 netbsd-bugs@netbsd.org,
 "martin@netbsd.org" <martin@NetBSD.org>
Subject: Re: bin/58817: acpidump -dt requires ACPIDUMP_USE_DEVMEM to work
Date: Fri, 8 Nov 2024 11:44:14 -0500

 --Apple-Mail=_F8501FBD-BB7B-46B0-BD19-6E965B733BD0
 Content-Transfer-Encoding: 7bit
 Content-Type: text/plain;
 	charset=us-ascii

 Worked for me. Perhaps it is host-specific?

 christos


 --Apple-Mail=_F8501FBD-BB7B-46B0-BD19-6E965B733BD0
 Content-Transfer-Encoding: 7bit
 Content-Disposition: attachment;
 	filename=signature.asc
 Content-Type: application/pgp-signature;
 	name=signature.asc
 Content-Description: Message signed with OpenPGP

 -----BEGIN PGP SIGNATURE-----
 Comment: GPGTools - http://gpgtools.org

 iF0EARECAB0WIQS+BJlbqPkO0MDBdsRxESqxbLM7OgUCZy4/3gAKCRBxESqxbLM7
 OvfWAJ4+LtaA4eGT29k6Ekx1YqDcJRDNywCg8yQV4IsgWnt22lwDG2T9krKt6yQ=
 =ImOF
 -----END PGP SIGNATURE-----

 --Apple-Mail=_F8501FBD-BB7B-46B0-BD19-6E965B733BD0--

From: Jared McNeill <jmcneill@invisible.ca>
To: Christos Zoulas <christos@zoulas.com>
Cc: gnats-bugs@netbsd.org, gnats-admin@netbsd.org, netbsd-bugs@netbsd.org, 
    "martin@netbsd.org" <martin@NetBSD.org>
Subject: Re: bin/58817: acpidump -dt requires ACPIDUMP_USE_DEVMEM to work
Date: Fri, 8 Nov 2024 13:26:53 -0400 (AST)

 On Fri, 8 Nov 2024, Christos Zoulas wrote:

 > Worked for me. Perhaps it is host-specific?

 Yes certainly host specific. sys/dev/acpi_dev.c only allows known tables 
 to be read, so we need to understand why it is failing on this 
 system. Let's start with dmesg + acpidump output from /dev/mem + ktrace of 
 the acpidump process to understand how it's failing.

From: Martin Husemann <martin@duskware.de>
To: Jared McNeill <jmcneill@invisible.ca>
Cc: Christos Zoulas <christos@zoulas.com>, gnats-bugs@netbsd.org
Subject: Re: bin/58817: acpidump -dt requires ACPIDUMP_USE_DEVMEM to work
Date: Sat, 9 Nov 2024 16:32:46 +0100

 Here are the requested logs:

 dmesg:
 	https://www.netbsd.org/~martin/acer_dmesg_nbsd_verbose2.txt

 acpidump output with ACPIDUMP_USE_DEVMEM=1:

 	https://www.netbsd.org/~martin/acer.2.asl.xz

 ktrace|kdump output for acpidump -dt w/o ACPIDUMP_USE_DEVMEM:

 	https://www.netbsd.org/~martin/acer.dump.gz

 Martin

From: "Martin Husemann" <martin@netbsd.org>
To: gnats-bugs@gnats.NetBSD.org
Cc: 
Subject: PR/58817 CVS commit: src/sys/dev/acpi
Date: Tue, 12 Nov 2024 13:24:01 +0000

 Module Name:	src
 Committed By:	martin
 Date:		Tue Nov 12 13:24:01 UTC 2024

 Modified Files:
 	src/sys/dev/acpi: acpi_dev.c

 Log Message:
 PR 58817: allow userland access to the TCPA. Patch from Jared.


 To generate a diff of this commit:
 cvs rdiff -u -r1.2 -r1.3 src/sys/dev/acpi/acpi_dev.c

 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.

From: "Taylor R Campbell" <riastradh@netbsd.org>
To: gnats-bugs@gnats.NetBSD.org
Cc: 
Subject: PR/58817 CVS commit: src/sys/dev/acpi
Date: Tue, 12 Nov 2024 21:49:11 +0000

 Module Name:	src
 Committed By:	riastradh
 Date:		Tue Nov 12 21:49:11 UTC 2024

 Modified Files:
 	src/sys/dev/acpi: acpi_dev.c

 Log Message:
 acpi(4): KNF

 - Nix trailing whitespace.
 - Fix operator spacing.
 - No need to cast AcpiOsMapMemory result.

 No functional change intended.

 Followup commit for PR 58817.


 To generate a diff of this commit:
 cvs rdiff -u -r1.3 -r1.4 src/sys/dev/acpi/acpi_dev.c

 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.

Home	PR Database Search