Re: [Qemu-devel] [PATCH 1/9] chardev: add greeting

[Gerd Hoffmann]

On 11/20/09 18:41, Paul Brook wrote:
> On Tuesday 17 November 2009, Gerd Hoffmann wrote:
> > Add a greeting string to CharDriverState which is printed after
> > initialization.  Used to have the qemu vc consoles labeled.  This
> > way we can avoid walking all the chardevs a second time after
> > initialization just to print the greeting.
>
> I think "greeting" is propagating a bad idea into new code. Much better would
> be some form of ID and/or human readable description that can also be used
> elsewhere.

The naming is only one part of the problem.  The second part is that the 
greeting is printed only for the 'vc' backend (where you really need it 
because there is no other way to figure what chardev you are looking at 
when switching screens via Ctrl-Alt-<nr>).

There already is a 'label' field.  So we could add a flag instead of a 
string, then do:

   if (chr->want_greeting)
       qemu_chr_printf(chr, "%s console\n", chr->label);

How about this?

cheers,
  Gerd