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

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

 



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?

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]