Re: [PATCH] Adding new ioctl for updating Vdagent state

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

 



On Mon, Aug 08, 2016 at 06:19:41PM +0300, Sameeh Jubran wrote:
> On Thu, Aug 4, 2016 at 3:17 PM, Christophe Fergeau <cfergeau@xxxxxxxxxx>
> wrote:
> > On Thu, Aug 04, 2016 at 02:02:08PM +0300, Sameeh Jubran wrote:
> > > New ioctl for updating the driver with vdagent running state.
> >
> > Please add some rationale to the commit log for having this (ie what is
> > going to be achieved with that escape).
> >
> > For what it's worth, I don't think the QXL driver should need to know
> > whether we are in server mode or not (which does not necessarily
> > correspond to vdagent running/not running). I'm talking about server
> > mode here as I think you added that in order to fix issues with the
> > pointer not showing up in server mode.
> >
> > I believe this issue is not limited to Win8+, and I've seen that too
> > on linux. Showing/not showing the cursor in server mode is handled
> > mostly client side. The client gets 'move' events from the server,
> > and in response to that, blends the server-side cursor. Currently,
> > I'm not getting these 'move' events regardless of the OS. Could this
> > patch be meant to address such a bug too, or is this fixing something
> > else?
> >
> This patch enables the driver to send move commands to the QXL device when
> vdagent is off.
> This maybe the reason you are not getting any move commands.

For the record, my issues were caused by vmmouse being used.
<features><vmport state=off/></features> in my libvirt XML solved it

Christophe

Attachment: signature.asc
Description: PGP signature

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel

[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]