Re: [libvirt] [PATCH v2 1/2] conf: add debugcon chardev guest interface

On Mon, 2019-02-11 at 08:14 +0000, Nikolay Shirokovskiy wrote: > On 08.02.2019 17:34, Andrea Bolognani wrote: > > Even with that fixed, while your code prevents a <console> element > > associated to the isa-debugcon to be automatically created, it > > doesn't prevent something like > > > > <console type='pty'/> > > <serial type='file'> > > <source path='...'/> > > <target type='isa-serial'> > > <model name='isa-debugcon'/> > > </target> > > </serial> > > > > to result in the same problematic configuration, while the user > > clearly wanted to have both a regular serial console *and* the > > isa-debugcon. > > Yeah I noticed that too but I though this is like case of usb-serial > for example. We do not add missing console in that case but allow > existing console to be alias of usb-serial. Hm, that looks like a bug. I think we should be consistent about this: either the aliasing between <serial/> and <console/> works both ways, or it should not happen at all IMHO.

> Can usb-serial actually be console? No idea :)