Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 19 Feb 2013 02:09:19 +0000 (UTC)
From:      Adrian Chadd <adrian@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r246958 - head/gnu/usr.bin/gdb/kgdb
Message-ID:  <201302190209.r1J29Ja8009856@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: adrian
Date: Tue Feb 19 02:09:18 2013
New Revision: 246958
URL: http://svnweb.freebsd.org/changeset/base/246958

Log:
  kgdb enhancements!
  
  * document the kgdb -b flag
  * better verify what's valid with -b
  * add more comprehensive command line help
  
  PR:		kern/175743
  Submitted by:	Christoph Mallon <christoph.mallon@gmx.de>

Modified:
  head/gnu/usr.bin/gdb/kgdb/kgdb.1
  head/gnu/usr.bin/gdb/kgdb/main.c

Modified: head/gnu/usr.bin/gdb/kgdb/kgdb.1
==============================================================================
--- head/gnu/usr.bin/gdb/kgdb/kgdb.1	Tue Feb 19 02:07:36 2013	(r246957)
+++ head/gnu/usr.bin/gdb/kgdb/kgdb.1	Tue Feb 19 02:09:18 2013	(r246958)
@@ -33,6 +33,7 @@
 .Sh SYNOPSIS
 .Nm
 .Op Fl a | Fl f | Fl fullname
+.Op Fl b Ar rate
 .Op Fl q | Fl quiet
 .Op Fl v
 .Op Fl w
@@ -62,6 +63,9 @@ The
 or
 .Fl fullname
 options are supported for backward compatibility as well.
+.It Fl b Ar rate
+Set the baudrate to
+.Ar rate .
 .It Fl q
 Suppress printing of the banner when the debugger starts.
 The

Modified: head/gnu/usr.bin/gdb/kgdb/main.c
==============================================================================
--- head/gnu/usr.bin/gdb/kgdb/main.c	Tue Feb 19 02:07:36 2013	(r246957)
+++ head/gnu/usr.bin/gdb/kgdb/main.c	Tue Feb 19 02:09:18 2013	(r246958)
@@ -83,7 +83,7 @@ usage(void)
 {
 
 	fprintf(stderr,
-	    "usage: %s [-afqvw] [-d crashdir] [-c core | -n dumpnr | -r device]\n"
+	    "usage: %s [-afqvw] [-b rate] [-d crashdir] [-c core | -n dumpnr | -r device]\n"
 	    "\t[kernel [core]]\n", getprogname());
 	exit(1);
 }
@@ -338,19 +338,18 @@ main(int argc, char *argv[])
 		case 'a':
 			annotation_level++;
 			break;
-		case 'b':
-			{
-				int i;
-				char *p;
+		case 'b': {
+			int i;
+			char *p;
 
-				i = strtol (optarg, &p, 0);
-				if (i == 0 && p == optarg)
+			i = strtol(optarg, &p, 0);
+			if (*p != '\0' || p == optarg)
 				warnx("warning: could not set baud rate to `%s'.\n",
 				    optarg);
-				else
-					baud_rate = i;
-			}
+			else
+				baud_rate = i;
 			break;
+		}
 		case 'c':	/* use given core file. */
 			if (vmcore != NULL) {
 				warnx("option %c: can only be specified once",



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201302190209.r1J29Ja8009856>