[PATCH] Add a return value to set_cursor_position() to allow it to report failure

Keith Packard keithp at keithp.com
Wed Apr 30 10:23:37 PDT 2014

Michael Thayer <michael.thayer at oracle.com> writes:

> On 22/04/14 07:00, Keith Packard wrote:
>> Michael Thayer <michael.thayer at oracle.com> writes:
>>> set_cursor_position() may need to be able to fail and have the server fall
>>> back to a software cursor in at least the situation in which we are running
>>> on virtual hardware and using the host cursor as a hardware cursor for the
>>> guest but cannot change its position.
>> Frankly, the usual "solution" for nested or virtual X servers is to just
>> ignore the cursor position assignment.
> I would still very much like to have the change if it considered 
> acceptable, and can submit an fixed patch like the one for 
> set_cursor_argb() and friends to force a driver build break.  Of course 
> if you tell me it is something that you would rather not have in I can 
> leave it at that.

I don't see any value in allowing pointer warping to fail and setting a
software cursor in response. If the next input event from the user isn't
going to happen at the warped position, then you should be setting the
cursor position before sending the next input event anyways.

The alternative is for you to simply always use a software cursor, which
would be easy to manage.

keith.packard at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 810 bytes
Desc: not available
URL: <http://lists.x.org/archives/xorg-devel/attachments/20140430/796efceb/attachment.sig>

More information about the xorg-devel mailing list