Re: [RFC] support -b <baudrate> when starting gdb

Also, I found 'set remotebaud' and 'set debug remote 1' to do this.

I'd like to add the code just to support the same -b flag as gdb (so
-r can also be used with a non-standard serial port.)

Thanks,



Adrian

On 15 January 2013 21:15, Adrian Chadd <adrian_at_freebsd.org> wrote:
> Hi,
>
> There doesn't seem to be a blessed way to set the baudrate from inside
> gdb/kgdb. It seems to be set from '-b' on the command line.
>
> However kgdb doesn't have this support.
>
> This patch adds -b support so kgdb so I can override the default speed
> (9600 it seems) to speak kgdb over serial to a 115200 console MIPS
> device.
>
> The MIPS stuff has other issues; I'll talk about those later.
>
> Thanks,
>
>
>
> Adrian
>
>
> Index: gnu/usr.bin/gdb/kgdb/main.c
> ===================================================================
> --- gnu/usr.bin/gdb/kgdb/main.c (revision 245281)
> +++ gnu/usr.bin/gdb/kgdb/main.c (working copy)
> _at__at_ -333,11 +333,24 _at__at_
>         args.argv = malloc(sizeof(char *));
>         args.argv[0] = argv[0];
>
> -       while ((ch = getopt(argc, argv, "ac:d:fn:qr:vw")) != -1) {
> +       while ((ch = getopt(argc, argv, "ab:c:d:fn:qr:vw")) != -1) {
>                 switch (ch) {
>                 case 'a':
>                         annotation_level++;
>                         break;
> +               case 'b':
> +                       {
> +                               int i;
> +                               char *p;
> +
> +                               i = strtol (optarg, &p, 0);
> +                               if (i == 0 && p == optarg)
> +                               warnx("warning: could not set baud
> rate to `%s'.\n",
> +                                   optarg);
> +                               else
> +                                       baud_rate = i;
> +                       }
> +                       break;
>                 case 'c':       /* use given core file. */
>                         if (vmcore != NULL) {
>                                 warnx("option %c: can only be specified once",