On (Mon) Aug 02 2010 [04:33:49], Alon Levy wrote:
> Hi,
>
> This patch adds three CHR_IOCTLs and uses them in virtserial devices, to be
> used
> by a chardev backend, such as a spice vm channel (spice is a vdi solution).
>
> Basically virtio-serial provides three driver initiated events for guest
> open of
> a device, guest close, and guest ready (driver port init complete) that
> before this
> patch are not exposed to the chardev backend.
>
> With the spicevmc backend this is used like this:
> qemu -chardev spicevmc,id=vdiport,name=vdiport -device
> virtserialport,chardev=vdiport,name=com.redhat.spice.0
>
> I'd appreciate any feedback if this seems the right way to accomplish this,
> and
> for the numbers I grabbed.
Looks fine to me, maybe you don't need the notifications for
virtio-console ports, only the virtio-serial ports could be of interest,
but I'm fine with both notifying as well.
Also, this description could be made part of the patch description
itself.
Amit
