qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH] debugcon: support for debugging consoles (e.g.


From: Gerd Hoffmann
Subject: Re: [Qemu-devel] [PATCH] debugcon: support for debugging consoles (e.g. Bochs port 0xe9)
Date: Wed, 25 Nov 2009 23:54:06 +0100
User-agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.4pre) Gecko/20091014 Fedora/3.0-2.8.b4.fc11 Lightning/1.0pre Thunderbird/3.0b4

On 11/25/09 23:31, H. Peter Anvin wrote:
On 11/25/2009 02:26 PM, H. Peter Anvin wrote:

As such, I think there is still a place for the simple command line
options like "-hda", "-serial" and my proposed "-debugcon", in addition
to the full-featured connect-anything options and configuration files.
That's all.


I guess one question is if the simplified options could be better
implemented via some kind of builtin macros or translation submodule
rather than being treated as first-class options which they basically
are now.  Still, that's a future development task.

Wouldn't be that hard I think.  Something along the lines of

    case QEMU_OPTION_debugcon:
         opts = qemu_chr_parse_compat("debugcon", optarg);
         if (NULL == opts) {
              fprintf(stderr, "already have a debugcon chardev\");
              exit(1);
         }
         opts = qemu_opts_create(&qemu_device_opts, "debugcon", 1);
         if (NULL == opts) {
              fprintf(stderr, "already have a debugcon device\");
              exit(1);
         }
         qemu_opt_set(opts, "driver", isa-debugcon);
         qemu_opt_set(opts, "chardev", "debugcon");
         break;

should do the trick. Depends on the chardev fix mentioned in this thread too. Allows to define a single device with the default port via shortcut.

I tend to not add new command line switches for every little new device. For this one it might be reasonable nevertheless given the use cases.

cheers,
  Gerd





reply via email to

[Prev in Thread] Current Thread [Next in Thread]