[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 2/4] virtio-serial: Clean up virtser_bus_dev_pri
From: |
Amit Shah |
Subject: |
Re: [Qemu-devel] [PATCH 2/4] virtio-serial: Clean up virtser_bus_dev_print() output |
Date: |
Thu, 19 May 2011 19:54:19 +0530 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On (Thu) 19 May 2011 [16:18:29], Markus Armbruster wrote:
> Amit Shah <address@hidden> writes:
>
> > On (Thu) 19 May 2011 [13:37:15], Markus Armbruster wrote:
> >> Old version looks like this in info qtree (last four lines):
> >>
> >> dev: virtconsole, id ""
> >> dev-prop: is_console = 1
> >> dev-prop: nr = 0
> >> dev-prop: chardev = <null>
> >> dev-prop: name = <null>
> >> dev-prop-int: id: 0
> >> dev-prop-int: guest_connected: 1
> >> dev-prop-int: host_connected: 0
> >> dev-prop-int: throttled: 0
> >>
> >> Indentation is off, and "dev-prop-int" suggests these are properties
> >> you can configure with -device, which isn't the case. The other
> >> buses' print_dev() callbacks don't do that. For instance, PCI's
> >> output looks like this:
> >>
> >> class Ethernet controller, addr 00:03.0, pci id 1af4:1000 (sub
> >> 1af4:0001)
> >> bar 0: i/o at 0xffffffffffffffff [0x1e]
> >> bar 1: mem at 0xffffffffffffffff [0xffe]
> >> bar 6: mem at 0xffffffffffffffff [0xfffe]
> >>
> >> Change virtser_bus_dev_print() to that style. Result:
> >>
> >> dev: virtconsole, id ""
> >> dev-prop: is_console = 1
> >> dev-prop: nr = 0
> >> dev-prop: chardev = <null>
> >> dev-prop: name = <null>
> >> port 0, guest on, host off, throttle off
> >
> > Here the original guest_connected and host_connected meant whether the
> > endpoints were open. guest on/off, host on/off don't convey that
> > meaning. Can't think of a short version, can you?
>
> I chose on/off to stay consistent with how qdev shows bool properties
> (print_bit() in qdev-properties.c). May be misguided. Like you, I'm
> having difficulties coming up with a better version that is still
> consise.
>
> But: should "info qtree" show such device state? It's about
> configuration of the device tree, isn't it?
Ah; right. guest on/off, throttle are for debug (like debugfs
counterpart in the kernel). I guess we'd need a new monitor command
to print these out?
Amit
[Qemu-devel] [PATCH 1/4] usb-ccid: Drop unused CCIDCardInfo callback print(), Markus Armbruster, 2011/05/19
Re: [Qemu-devel] [PATCH 0/4] A few cleanups of qdev users, Gerd Hoffmann, 2011/05/19