Module Name:    src
Committed By:   joerg
Date:           Thu May 26 15:34:14 UTC 2011

Modified Files:
        src/sys/arch/alpha/include: db_machdep.h
        src/sys/arch/amd64/include: db_machdep.h
        src/sys/arch/arm/include: db_machdep.h
        src/sys/arch/hppa/include: db_machdep.h
        src/sys/arch/i386/include: db_machdep.h
        src/sys/arch/ia64/include: db_machdep.h
        src/sys/arch/m68k/include: db_machdep.h
        src/sys/arch/mips/include: db_machdep.h
        src/sys/arch/powerpc/include: db_machdep.h
        src/sys/arch/sh3/include: db_machdep.h
        src/sys/arch/sparc/include: db_machdep.h
        src/sys/arch/sparc64/include: db_machdep.h
        src/sys/arch/vax/include: db_machdep.h
        src/sys/ddb: db_examine.c db_interface.h db_lex.c

Log Message:
Introduce DDB_EXPR_FMT and replace the logic around DB_EXPR_T_IS_QUAD.


To generate a diff of this commit:
cvs rdiff -u -r1.17 -r1.18 src/sys/arch/alpha/include/db_machdep.h
cvs rdiff -u -r1.10 -r1.11 src/sys/arch/amd64/include/db_machdep.h
cvs rdiff -u -r1.16 -r1.17 src/sys/arch/arm/include/db_machdep.h
cvs rdiff -u -r1.10 -r1.11 src/sys/arch/hppa/include/db_machdep.h
cvs rdiff -u -r1.29 -r1.30 src/sys/arch/i386/include/db_machdep.h
cvs rdiff -u -r1.2 -r1.3 src/sys/arch/ia64/include/db_machdep.h
cvs rdiff -u -r1.29 -r1.30 src/sys/arch/m68k/include/db_machdep.h
cvs rdiff -u -r1.26 -r1.27 src/sys/arch/mips/include/db_machdep.h
cvs rdiff -u -r1.22 -r1.23 src/sys/arch/powerpc/include/db_machdep.h
cvs rdiff -u -r1.15 -r1.16 src/sys/arch/sh3/include/db_machdep.h
cvs rdiff -u -r1.25 -r1.26 src/sys/arch/sparc/include/db_machdep.h
cvs rdiff -u -r1.29 -r1.30 src/sys/arch/sparc64/include/db_machdep.h
cvs rdiff -u -r1.17 -r1.18 src/sys/arch/vax/include/db_machdep.h
cvs rdiff -u -r1.34 -r1.35 src/sys/ddb/db_examine.c
cvs rdiff -u -r1.26 -r1.27 src/sys/ddb/db_interface.h
cvs rdiff -u -r1.21 -r1.22 src/sys/ddb/db_lex.c

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

Modified files:

Index: src/sys/arch/alpha/include/db_machdep.h
diff -u src/sys/arch/alpha/include/db_machdep.h:1.17 src/sys/arch/alpha/include/db_machdep.h:1.18
--- src/sys/arch/alpha/include/db_machdep.h:1.17	Wed Feb 21 22:59:37 2007
+++ src/sys/arch/alpha/include/db_machdep.h	Thu May 26 15:34:12 2011
@@ -1,4 +1,4 @@
-/* $NetBSD: db_machdep.h,v 1.17 2007/02/21 22:59:37 thorpej Exp $ */
+/* $NetBSD: db_machdep.h,v 1.18 2011/05/26 15:34:12 joerg Exp $ */
 
 /*
  * Copyright (c) 1995 Carnegie-Mellon University.
@@ -39,6 +39,7 @@
 #include <machine/frame.h>
 
 typedef	vaddr_t		db_addr_t;	/* address - unsigned */
+#define	DDB_EXPR_FMT	"l"		/* expression is long */
 typedef	long		db_expr_t;	/* expression - signed */
 
 typedef struct trapframe db_regs_t;

Index: src/sys/arch/amd64/include/db_machdep.h
diff -u src/sys/arch/amd64/include/db_machdep.h:1.10 src/sys/arch/amd64/include/db_machdep.h:1.11
--- src/sys/arch/amd64/include/db_machdep.h:1.10	Sun Apr 10 20:36:49 2011
+++ src/sys/arch/amd64/include/db_machdep.h	Thu May 26 15:34:12 2011
@@ -1,4 +1,4 @@
-/*	$NetBSD: db_machdep.h,v 1.10 2011/04/10 20:36:49 christos Exp $	*/
+/*	$NetBSD: db_machdep.h,v 1.11 2011/05/26 15:34:12 joerg Exp $	*/
 
 /* 
  * Mach Operating System
@@ -41,6 +41,7 @@
 #include <machine/trap.h>
 
 typedef	vaddr_t		db_addr_t;	/* address - unsigned */
+#define	DDB_EXPR_FMT	"l"		/* expression is long */
 typedef	long		db_expr_t;	/* expression - signed */
 
 typedef struct trapframe db_regs_t;

Index: src/sys/arch/arm/include/db_machdep.h
diff -u src/sys/arch/arm/include/db_machdep.h:1.16 src/sys/arch/arm/include/db_machdep.h:1.17
--- src/sys/arch/arm/include/db_machdep.h:1.16	Sat Mar 14 14:45:55 2009
+++ src/sys/arch/arm/include/db_machdep.h	Thu May 26 15:34:12 2011
@@ -1,4 +1,4 @@
-/*	$NetBSD: db_machdep.h,v 1.16 2009/03/14 14:45:55 dsl Exp $	*/
+/*	$NetBSD: db_machdep.h,v 1.17 2011/05/26 15:34:12 joerg Exp $	*/
 
 /*
  * Copyright (c) 1996 Scott K Stevens
@@ -43,6 +43,7 @@
 /* end of mangling */
 
 typedef	vaddr_t		db_addr_t;	/* address - unsigned */
+#define	DDB_EXPR_FMT	"l"		/* expression is long */
 typedef	long		db_expr_t;	/* expression - signed */
 
 typedef trapframe_t db_regs_t;

Index: src/sys/arch/hppa/include/db_machdep.h
diff -u src/sys/arch/hppa/include/db_machdep.h:1.10 src/sys/arch/hppa/include/db_machdep.h:1.11
--- src/sys/arch/hppa/include/db_machdep.h:1.10	Fri Jan 14 02:06:26 2011
+++ src/sys/arch/hppa/include/db_machdep.h	Thu May 26 15:34:12 2011
@@ -1,4 +1,4 @@
-/*	$NetBSD: db_machdep.h,v 1.10 2011/01/14 02:06:26 rmind Exp $	*/
+/*	$NetBSD: db_machdep.h,v 1.11 2011/05/26 15:34:12 joerg Exp $	*/
 
 /*	$OpenBSD: db_machdep.h,v 1.5 2001/02/16 19:20:13 mickey Exp $	*/
 
@@ -39,6 +39,7 @@
 
 /* types the generic ddb module needs */
 typedef	vaddr_t db_addr_t;
+#define	DDB_EXPR_FMT	"l"		/* expression is long */
 typedef	long db_expr_t;
 
 typedef struct trapframe db_regs_t;

Index: src/sys/arch/i386/include/db_machdep.h
diff -u src/sys/arch/i386/include/db_machdep.h:1.29 src/sys/arch/i386/include/db_machdep.h:1.30
--- src/sys/arch/i386/include/db_machdep.h:1.29	Mon Apr 11 04:22:30 2011
+++ src/sys/arch/i386/include/db_machdep.h	Thu May 26 15:34:13 2011
@@ -1,4 +1,4 @@
-/*	$NetBSD: db_machdep.h,v 1.29 2011/04/11 04:22:30 mrg Exp $	*/
+/*	$NetBSD: db_machdep.h,v 1.30 2011/05/26 15:34:13 joerg Exp $	*/
 
 /* 
  * Mach Operating System
@@ -41,6 +41,7 @@
 #include <machine/trap.h>
 
 typedef	vaddr_t		db_addr_t;	/* address - unsigned */
+#define	DDB_EXPR_FMT	"l"		/* expression is long */
 typedef	long		db_expr_t;	/* expression - signed */
 
 typedef struct trapframe db_regs_t;

Index: src/sys/arch/ia64/include/db_machdep.h
diff -u src/sys/arch/ia64/include/db_machdep.h:1.2 src/sys/arch/ia64/include/db_machdep.h:1.3
--- src/sys/arch/ia64/include/db_machdep.h:1.2	Wed Aug 30 11:12:04 2006
+++ src/sys/arch/ia64/include/db_machdep.h	Thu May 26 15:34:13 2011
@@ -1,4 +1,4 @@
-/* $NetBSD: db_machdep.h,v 1.2 2006/08/30 11:12:04 cherry Exp $ */
+/* $NetBSD: db_machdep.h,v 1.3 2011/05/26 15:34:13 joerg Exp $ */
 
 /*
  * Copyright (c) 1995 Carnegie-Mellon University.
@@ -41,6 +41,7 @@
 #include <machine/ia64_cpu.h>
 
 typedef	vaddr_t		db_addr_t;	/* address - unsigned */
+#define	DDB_EXPR_FMT	"l"		/* expression is long */
 typedef	long		db_expr_t;	/* expression - signed */
 
 typedef struct trapframe db_regs_t;

Index: src/sys/arch/m68k/include/db_machdep.h
diff -u src/sys/arch/m68k/include/db_machdep.h:1.29 src/sys/arch/m68k/include/db_machdep.h:1.30
--- src/sys/arch/m68k/include/db_machdep.h:1.29	Mon Apr 11 04:22:30 2011
+++ src/sys/arch/m68k/include/db_machdep.h	Thu May 26 15:34:13 2011
@@ -1,4 +1,4 @@
-/*	$NetBSD: db_machdep.h,v 1.29 2011/04/11 04:22:30 mrg Exp $	*/
+/*	$NetBSD: db_machdep.h,v 1.30 2011/05/26 15:34:13 joerg Exp $	*/
 
 /* 
  * Mach Operating System
@@ -46,6 +46,7 @@
 #include <machine/trap.h>
 
 typedef	vaddr_t		db_addr_t;	/* address - unsigned */
+#define	DDB_EXPR_FMT	"l"		/* expression is long */
 typedef	long		db_expr_t;	/* expression - signed */
 typedef struct trapframe db_regs_t;
 

Index: src/sys/arch/mips/include/db_machdep.h
diff -u src/sys/arch/mips/include/db_machdep.h:1.26 src/sys/arch/mips/include/db_machdep.h:1.27
--- src/sys/arch/mips/include/db_machdep.h:1.26	Thu Apr 14 05:07:56 2011
+++ src/sys/arch/mips/include/db_machdep.h	Thu May 26 15:34:13 2011
@@ -1,4 +1,4 @@
-/* $NetBSD: db_machdep.h,v 1.26 2011/04/14 05:07:56 cliff Exp $ */
+/* $NetBSD: db_machdep.h,v 1.27 2011/05/26 15:34:13 joerg Exp $ */
 
 /*
  * Copyright (c) 1997 Jonathan Stone (hereinafter referred to as the author)
@@ -42,6 +42,7 @@
 
 
 typedef	vaddr_t		db_addr_t;	/* address - unsigned */
+#define	DDB_EXPR_FMT	"l"		/* expression is long */
 typedef	long		db_expr_t;	/* expression - signed */
 
 typedef struct reg db_regs_t;

Index: src/sys/arch/powerpc/include/db_machdep.h
diff -u src/sys/arch/powerpc/include/db_machdep.h:1.22 src/sys/arch/powerpc/include/db_machdep.h:1.23
--- src/sys/arch/powerpc/include/db_machdep.h:1.22	Tue Jan 18 01:02:54 2011
+++ src/sys/arch/powerpc/include/db_machdep.h	Thu May 26 15:34:13 2011
@@ -1,5 +1,5 @@
 /*	$OpenBSD: db_machdep.h,v 1.2 1997/03/21 00:48:48 niklas Exp $	*/
-/*	$NetBSD: db_machdep.h,v 1.22 2011/01/18 01:02:54 matt Exp $	*/
+/*	$NetBSD: db_machdep.h,v 1.23 2011/05/26 15:34:13 joerg Exp $	*/
 
 /* 
  * Mach Operating System
@@ -45,6 +45,7 @@
 #define	DB_ELFSIZE	32
 
 typedef	vaddr_t		db_addr_t;	/* address - unsigned */
+#define	DDB_EXPR_FMT	"l"		/* expression is long */
 typedef	long		db_expr_t;	/* expression - signed */
 struct powerpc_saved_state {
 	u_int32_t	r[32];		/* data registers */

Index: src/sys/arch/sh3/include/db_machdep.h
diff -u src/sys/arch/sh3/include/db_machdep.h:1.15 src/sys/arch/sh3/include/db_machdep.h:1.16
--- src/sys/arch/sh3/include/db_machdep.h:1.15	Fri Jan 14 02:06:30 2011
+++ src/sys/arch/sh3/include/db_machdep.h	Thu May 26 15:34:13 2011
@@ -1,4 +1,4 @@
-/*	$NetBSD: db_machdep.h,v 1.15 2011/01/14 02:06:30 rmind Exp $	*/
+/*	$NetBSD: db_machdep.h,v 1.16 2011/05/26 15:34:13 joerg Exp $	*/
 
 /*
  * Mach Operating System
@@ -39,6 +39,7 @@
 #include <sh3/pcb.h>
 
 typedef	vaddr_t		db_addr_t;	/* address - unsigned */
+#define	DDB_EXPR_FMT	"l"		/* expression is long */
 typedef	long		db_expr_t;	/* expression - signed */
 
 typedef struct trapframe db_regs_t;

Index: src/sys/arch/sparc/include/db_machdep.h
diff -u src/sys/arch/sparc/include/db_machdep.h:1.25 src/sys/arch/sparc/include/db_machdep.h:1.26
--- src/sys/arch/sparc/include/db_machdep.h:1.25	Mon Apr 11 04:22:31 2011
+++ src/sys/arch/sparc/include/db_machdep.h	Thu May 26 15:34:13 2011
@@ -1,4 +1,4 @@
-/*	$NetBSD: db_machdep.h,v 1.25 2011/04/11 04:22:31 mrg Exp $ */
+/*	$NetBSD: db_machdep.h,v 1.26 2011/05/26 15:34:13 joerg Exp $ */
 
 /*
  * Mach Operating System
@@ -40,6 +40,7 @@
 #include <machine/reg.h>
 
 typedef	vaddr_t		db_addr_t;	/* address - unsigned */
+#define	DDB_EXPR_FMT	"l"		/* expression is long */
 typedef	long		db_expr_t;	/* expression - signed */
 
 typedef struct {

Index: src/sys/arch/sparc64/include/db_machdep.h
diff -u src/sys/arch/sparc64/include/db_machdep.h:1.29 src/sys/arch/sparc64/include/db_machdep.h:1.30
--- src/sys/arch/sparc64/include/db_machdep.h:1.29	Wed Apr 13 03:22:49 2011
+++ src/sys/arch/sparc64/include/db_machdep.h	Thu May 26 15:34:13 2011
@@ -1,4 +1,4 @@
-/*	$NetBSD: db_machdep.h,v 1.29 2011/04/13 03:22:49 mrg Exp $ */
+/*	$NetBSD: db_machdep.h,v 1.30 2011/05/26 15:34:13 joerg Exp $ */
 
 /*
  * Mach Operating System
@@ -46,6 +46,7 @@
 
 /* use 64-bit types explicitly for 32-bit kernels */
 typedef	vaddr_t		db_addr_t;	/* address - unsigned */
+#define	DDB_EXPR_FMT	"l"		/* expression is int64_t (long) */
 typedef	int64_t		db_expr_t;	/* expression - signed */
 
 struct trapstate {

Index: src/sys/arch/vax/include/db_machdep.h
diff -u src/sys/arch/vax/include/db_machdep.h:1.17 src/sys/arch/vax/include/db_machdep.h:1.18
--- src/sys/arch/vax/include/db_machdep.h:1.17	Sat Nov  6 11:46:03 2010
+++ src/sys/arch/vax/include/db_machdep.h	Thu May 26 15:34:14 2011
@@ -1,4 +1,4 @@
-/*	$NetBSD: db_machdep.h,v 1.17 2010/11/06 11:46:03 uebayasi Exp $	*/
+/*	$NetBSD: db_machdep.h,v 1.18 2011/05/26 15:34:14 joerg Exp $	*/
 
 /* 
  * Mach Operating System
@@ -40,6 +40,7 @@
 #include <machine/psl.h>
 
 typedef	vaddr_t		db_addr_t;	/* address - unsigned */
+#define	DDB_EXPR_FMT	"l"		/* expression is long */
 typedef	long		db_expr_t;	/* expression - signed */
 
 typedef struct trapframe db_regs_t;

Index: src/sys/ddb/db_examine.c
diff -u src/sys/ddb/db_examine.c:1.34 src/sys/ddb/db_examine.c:1.35
--- src/sys/ddb/db_examine.c:1.34	Sat Mar  7 22:02:17 2009
+++ src/sys/ddb/db_examine.c	Thu May 26 15:34:14 2011
@@ -1,4 +1,4 @@
-/*	$NetBSD: db_examine.c,v 1.34 2009/03/07 22:02:17 ad Exp $	*/
+/*	$NetBSD: db_examine.c,v 1.35 2011/05/26 15:34:14 joerg Exp $	*/
 
 /*
  * Mach Operating System
@@ -30,7 +30,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: db_examine.c,v 1.34 2009/03/07 22:02:17 ad Exp $");
+__KERNEL_RCSID(0, "$NetBSD: db_examine.c,v 1.35 2011/05/26 15:34:14 joerg Exp $");
 
 #include <sys/param.h>
 #include <sys/systm.h>
@@ -117,8 +117,7 @@
 			case 'x':	/* unsigned hex */
 				value = db_get_value(addr, size, false);
 				addr += size;
-				db_printf(DB_EXPR_T_IS_QUAD ? "%-*qx" : "%-*lx",
-				    width, value);
+				db_printf("%-*" DDB_EXPR_FMT "x", width, value);
 				break;
 			case 'm':	/* hex dump */
 				/*
@@ -135,8 +134,8 @@
  						    db_get_value(addr+bytes, 1,
 							false);
 						db_printf(
-						    DB_EXPR_T_IS_QUAD ? "%02qx":
-						    "%02lx", value);
+						    "%02" DDB_EXPR_FMT "x",
+						    value);
 						bytes++;
 						if (!(bytes % 4))
 							db_printf(" ");
@@ -165,20 +164,17 @@
 			case 'd':	/* signed decimal */
 				value = db_get_value(addr, size, true);
 				addr += size;
-				db_printf(DB_EXPR_T_IS_QUAD ? "%-*qd" : "%-*ld",
-				    width, value);
+				db_printf("%-*" DDB_EXPR_FMT "d", width, value);
 				break;
 			case 'u':	/* unsigned decimal */
 				value = db_get_value(addr, size, false);
 				addr += size;
-				db_printf(DB_EXPR_T_IS_QUAD ? "%-*qu" : "%-*lu",
-				    width, value);
+				db_printf("%-*" DDB_EXPR_FMT "u", width, value);
 				break;
 			case 'o':	/* unsigned octal */
 				value = db_get_value(addr, size, false);
 				addr += size;
-				db_printf(DB_EXPR_T_IS_QUAD ? "%-*qo" : "%-*lo",
-				    width, value);
+				db_printf("%-*" DDB_EXPR_FMT "o", width, value);
 				break;
 			case 'c':	/* character */
 				value = db_get_value(addr, 1, false);
@@ -244,7 +240,7 @@
 			break;
 		}
 	case 'x':
-		db_printf(DB_EXPR_T_IS_QUAD ? "%16qx" : "%8lx", addr);
+		db_printf("%16" DDB_EXPR_FMT "x", addr);
 		break;
 	case 'z':
 		{
@@ -255,13 +251,13 @@
 			break;
 		}
 	case 'd':
-		db_printf(DB_EXPR_T_IS_QUAD ? "%11qd" : "%11ld", addr);
+		db_printf("%11" DDB_EXPR_FMT "d", addr);
 		break;
 	case 'u':
-		db_printf(DB_EXPR_T_IS_QUAD ? "%11qu" : "%11lu", addr);
+		db_printf("%11" DDB_EXPR_FMT "u", addr);
 		break;
 	case 'o':
-		db_printf(DB_EXPR_T_IS_QUAD ? "%15qo" : "%16lo", addr);
+		db_printf("%15" DDB_EXPR_FMT "o", addr);
 		break;
 	case 'c':
 		value = addr & 0xFF;

Index: src/sys/ddb/db_interface.h
diff -u src/sys/ddb/db_interface.h:1.26 src/sys/ddb/db_interface.h:1.27
--- src/sys/ddb/db_interface.h:1.26	Sun Apr 10 20:59:22 2011
+++ src/sys/ddb/db_interface.h	Thu May 26 15:34:14 2011
@@ -1,4 +1,4 @@
-/*	$NetBSD: db_interface.h,v 1.26 2011/04/10 20:59:22 christos Exp $	*/
+/*	$NetBSD: db_interface.h,v 1.27 2011/05/26 15:34:14 joerg Exp $	*/
 
 /*-
  * Copyright (c) 1995 The NetBSD Foundation, Inc.
@@ -72,10 +72,4 @@
 /* kern/subr_vmem.c */
 void		db_show_all_vmems(db_expr_t, bool, db_expr_t, const char *);
 
-/*
- * This is used in several places to determine which printf format
- * string is appropriate for displaying a variable of type db_expr_t.
- */
-#define	DB_EXPR_T_IS_QUAD (/* CONSTCOND */ sizeof(db_expr_t) > sizeof(long))
-
 #endif /* _DDB_DB_INTERFACE_H_ */

Index: src/sys/ddb/db_lex.c
diff -u src/sys/ddb/db_lex.c:1.21 src/sys/ddb/db_lex.c:1.22
--- src/sys/ddb/db_lex.c:1.21	Sat Mar  7 22:02:17 2009
+++ src/sys/ddb/db_lex.c	Thu May 26 15:34:14 2011
@@ -1,4 +1,4 @@
-/*	$NetBSD: db_lex.c,v 1.21 2009/03/07 22:02:17 ad Exp $	*/
+/*	$NetBSD: db_lex.c,v 1.22 2011/05/26 15:34:14 joerg Exp $	*/
 
 /*
  * Mach Operating System
@@ -34,7 +34,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: db_lex.c,v 1.21 2009/03/07 22:02:17 ad Exp $");
+__KERNEL_RCSID(0, "$NetBSD: db_lex.c,v 1.22 2011/05/26 15:34:14 joerg Exp $");
 
 #include <sys/param.h>
 #include <sys/systm.h>
@@ -150,14 +150,11 @@
 	static char buf[25];
 
 	if (db_radix == 16)
-		snprintf(buf, sizeof(buf), DB_EXPR_T_IS_QUAD ? "%#qx" : "%#lx",
-		    val);
+		snprintf(buf, sizeof(buf), "%" DDB_EXPR_FMT "x", val);
 	else if (db_radix == 8)
-		snprintf(buf, sizeof(buf), DB_EXPR_T_IS_QUAD ? "%#qo" : "%#lo",
-		    val);
+		snprintf(buf, sizeof(buf), "%" DDB_EXPR_FMT "o", val);
 	else
-		snprintf(buf, sizeof(buf), DB_EXPR_T_IS_QUAD ? "%qu" : "%lu",
-		    val);
+		snprintf(buf, sizeof(buf), "%" DDB_EXPR_FMT "u", val);
 
 	return (buf);
 }

Reply via email to