NetBSD Problem Report #59419

From www@netbsd.org  Tue May 13 19:53:01 2025
Return-Path: <www@netbsd.org>
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 82D1E1A923E
	for <gnats-bugs@gnats.NetBSD.org>; Tue, 13 May 2025 19:53:01 +0000 (UTC)
Message-Id: <20250513195300.3977C1A9241@mollari.NetBSD.org>
Date: Tue, 13 May 2025 19:53:00 +0000 (UTC)
From: jamesbrowning137@gmail.com
Reply-To: jamesbrowning137@gmail.com
To: gnats-bugs@NetBSD.org
Subject: KGDB build fails for evbarm
X-Send-Pr-Version: www-1.0

>Number:         59419
>Category:       port-evbarm
>Synopsis:       KGDB build fails for evbarm
>Confidential:   no
>Severity:       non-critical
>Priority:       medium
>Responsible:    port-evbarm-maintainer
>State:          closed
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Tue May 13 19:55:00 +0000 2025
>Closed-Date:    Wed Sep 02 18:53:50 +0000 2026
>Last-Modified:  Wed Sep 02 18:53:50 +0000 2026
>Originator:     James Browning
>Release:        hg src-draft trunk tip (as of April 7th 2025)
>Organization:
>Environment:
Linux phobos 6.12.21-gentoo #1 SMP PREEMPT_DYNAMIC Sun Apr 27 20:43:52
PDT 2025 x86_64 AMD Ryzen 5 PRO 4650U with Radeon Graphics
AuthenticAMD GNU/Linux
>Description:
When attempting to enable kgdb support for evbarm, I run into a couple of build failures:

1. conflicting definitions of "db_active"
2. unrecognized symbol "db_error"

Here is the output for error 1:
  --- db_interface.o ---
  /home/james/NetBSD/src-draft/sys/arch/arm/arm32/db_interface.c:77:9:
  error: conflicting type qualifiers for 'db_active'
     77 | int     db_active = 0;
        |         ^~~~~~~~~
  In file included from
  /home/james/NetBSD/src-draft/sys/arch/arm/arm32/db_interface.c:59:
  /home/james/NetBSD/src-draft/sys/ddb/db_active.h:41:18: note: previous
  definition of 'db_active' with type 'int'
     41 | static const int db_active __unused = 0;
        |     

And after adding a work around for error 1, here is error 2:
  /home/james/NetBSD/tools/bin/armv7--netbsdelf-eabihf-ld: db_access.o:
  in function `db_get_qvalue':
  /home/james/NetBSD/src-draft/sys/ddb/db_access.c:99:(.text+0xd8):
  undefined reference to `db_error'
  + exit 1

Not sure if it matters, but I'm cross compiling from gentoo. I've never encountered any other issues with my build environment besides this.

>How-To-Repeat:
In your GENERIC.local, add the following to enable KGDB in the build:

  no options		DDB
  no options		DDB_ONPANIC
  no options      DDB_HISTORY_SIZE

  options    KGDB
  options KGDB_DEVNAME="\"com\"",KGDB_DEVADDR=0x48022000,KGDB_DEVRATE=115200,KGDB_DEV=123 #DEVADDR is UART1 on AM335x  
  makeoptions DEBUG="-g"

Compile the GENERIC kernel with "evbarm" as the machine and "earmv7hf" for the architecture.
>Fix:
Temporary work around:

diff -r d708c4dcdd7d sys/arch/arm/arm32/db_interface.c
--- a/sys/arch/arm/arm32/db_interface.c	Sun Apr 27 23:30:39 2025 +0000
+++ b/sys/arch/arm/arm32/db_interface.c	Tue May 13 12:08:07 2025 -0700
@@ -74,7 +74,7 @@

 int db_trapper(u_int, u_int, trapframe_t *, int);

-int	db_active = 0;
+//int	db_active = 0;
 db_regs_t ddb_regs;	/* register state */
 db_regs_t *ddb_regp;
diff -r d708c4dcdd7d sys/ddb/db_access.c
--- a/sys/ddb/db_access.c	Sun Apr 27 23:30:39 2025 +0000
+++ b/sys/ddb/db_access.c	Tue May 13 12:08:07 2025 -0700
@@ -93,10 +93,12 @@
 		return (uint32_t)db_get_value(addr, size, false);
 	}

-	if (size != sizeof(data)) {
-		db_error("unsupported size\n");
-		/*NOTREACHED*/
-	}
+
+//	if (size != sizeof(data)) {
+//
+//		db_error("unsupported size\n");
+//		/*NOTREACHED*/
+//	}

 	db_read_bytes(addr, sizeof(data), (char *)&data);
 	return data;


>Release-Note:

>Audit-Trail:
From: "Nick Hudson" <skrll@netbsd.org>
To: gnats-bugs@gnats.NetBSD.org
Cc: 
Subject: PR/59419 CVS commit: src/sys
Date: Wed, 25 Feb 2026 05:34:42 +0000

 Module Name:	src
 Committed By:	skrll
 Date:		Wed Feb 25 05:34:42 UTC 2026

 Modified Files:
 	src/sys/arch/arm/arm32: db_interface.c
 	src/sys/arch/mips/mips: db_interface.c
 	src/sys/arch/powerpc/powerpc: db_interface.c
 	src/sys/arch/sparc64/sparc64: db_interface.c
 	src/sys/ddb: db_access.c

 Log Message:
 PR/59419: KGDB build fails for evbarm

 Fix compilation of db_interface.c when KGDB is defined, but DDB is not.


 To generate a diff of this commit:
 cvs rdiff -u -r1.66 -r1.67 src/sys/arch/arm/arm32/db_interface.c
 cvs rdiff -u -r1.100 -r1.101 src/sys/arch/mips/mips/db_interface.c
 cvs rdiff -u -r1.62 -r1.63 src/sys/arch/powerpc/powerpc/db_interface.c
 cvs rdiff -u -r1.139 -r1.140 src/sys/arch/sparc64/sparc64/db_interface.c
 cvs rdiff -u -r1.27 -r1.28 src/sys/ddb/db_access.c

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

State-Changed-From-To: open->pending-pullups
State-Changed-By: skrll@NetBSD.org
State-Changed-When: Thu, 26 Feb 2026 10:23:45 +0000
State-Changed-Why:
[pullup-11 #202]
[pullup-10 #1244]


From: "Martin Husemann" <martin@netbsd.org>
To: gnats-bugs@gnats.NetBSD.org
Cc: 
Subject: PR/59419 CVS commit: [netbsd-11] src/sys
Date: Wed, 4 Mar 2026 20:26:01 +0000

 Module Name:	src
 Committed By:	martin
 Date:		Wed Mar  4 20:26:01 UTC 2026

 Modified Files:
 	src/sys/arch/arm/arm32 [netbsd-11]: db_interface.c
 	src/sys/arch/mips/mips [netbsd-11]: db_interface.c
 	src/sys/arch/powerpc/powerpc [netbsd-11]: db_interface.c
 	src/sys/arch/sparc64/sparc64 [netbsd-11]: db_interface.c
 	src/sys/ddb [netbsd-11]: db_access.c

 Log Message:
 Pull up following revision(s) (requested by skrll in ticket #202):

 	sys/arch/powerpc/powerpc/db_interface.c: revision 1.63
 	sys/ddb/db_access.c: revision 1.28
 	sys/arch/arm/arm32/db_interface.c: revision 1.67
 	sys/arch/mips/mips/db_interface.c: revision 1.101
 	sys/arch/sparc64/sparc64/db_interface.c: revision 1.140

 PR/59419: KGDB build fails for evbarm

 Fix compilation of db_interface.c when KGDB is defined, but DDB is not.


 To generate a diff of this commit:
 cvs rdiff -u -r1.65 -r1.65.8.1 src/sys/arch/arm/arm32/db_interface.c
 cvs rdiff -u -r1.99 -r1.99.8.1 src/sys/arch/mips/mips/db_interface.c
 cvs rdiff -u -r1.61 -r1.61.2.1 src/sys/arch/powerpc/powerpc/db_interface.c
 cvs rdiff -u -r1.138 -r1.138.10.1 src/sys/arch/sparc64/sparc64/db_interface.c
 cvs rdiff -u -r1.27 -r1.27.8.1 src/sys/ddb/db_access.c

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

From: "Martin Husemann" <martin@netbsd.org>
To: gnats-bugs@gnats.NetBSD.org
Cc: 
Subject: PR/59419 CVS commit: [netbsd-10] src/sys
Date: Wed, 4 Mar 2026 20:27:42 +0000

 Module Name:	src
 Committed By:	martin
 Date:		Wed Mar  4 20:27:42 UTC 2026

 Modified Files:
 	src/sys/arch/arm/arm32 [netbsd-10]: db_interface.c
 	src/sys/arch/mips/mips [netbsd-10]: db_interface.c
 	src/sys/arch/powerpc/powerpc [netbsd-10]: db_interface.c
 	src/sys/arch/sparc64/sparc64 [netbsd-10]: db_interface.c
 	src/sys/ddb [netbsd-10]: db_access.c

 Log Message:
 Pull up following revision(s) (requested by skrll in ticket #1244):

 	sys/arch/powerpc/powerpc/db_interface.c: revision 1.63
 	sys/ddb/db_access.c: revision 1.28
 	sys/arch/arm/arm32/db_interface.c: revision 1.67
 	sys/arch/mips/mips/db_interface.c: revision 1.101
 	sys/arch/sparc64/sparc64/db_interface.c: revision 1.140

 PR/59419: KGDB build fails for evbarm

 Fix compilation of db_interface.c when KGDB is defined, but DDB is not.


 To generate a diff of this commit:
 cvs rdiff -u -r1.64 -r1.64.2.1 src/sys/arch/arm/arm32/db_interface.c
 cvs rdiff -u -r1.97 -r1.97.2.1 src/sys/arch/mips/mips/db_interface.c
 cvs rdiff -u -r1.60 -r1.60.2.1 src/sys/arch/powerpc/powerpc/db_interface.c
 cvs rdiff -u -r1.138 -r1.138.2.1 src/sys/arch/sparc64/sparc64/db_interface.c
 cvs rdiff -u -r1.26 -r1.26.26.1 src/sys/ddb/db_access.c

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

State-Changed-From-To: pending-pullups->closed
State-Changed-By: maya@NetBSD.org
State-Changed-When: Wed, 02 Sep 2026 18:53:50 +0000
State-Changed-Why:
Pullups done


>Unformatted:

Home	PR Database Search