[PATCH] xf86/cursor: Check hw cursor after xf86SeCursor() call, too
Takashi Iwai
tiwai at suse.de
Thu Jul 17 12:04:19 PDT 2014
At Thu, 17 Jul 2014 11:09:19 -0700,
Keith Packard wrote:
>
> Takashi Iwai <tiwai at suse.de> writes:
>
> > When a video driver changed the availability of hw cursor inside the
> > xf86SetCursor() call, f86CursorSetCursor() still assumes the hw
> > cursor and clears SWCursor flag of the screen unconditionally. This
> > results in the inconsistent state, typically the invisible cursor
> > pointer. For example, the problem is seen with modesetting driver
> > with Cirrus or MGA KMS where the hw cursor isn't always available.
> >
> > This patch adds the check of availability of hw cursor again after the
> > call of xf86SetCursor(), and proceeds only when it passes the test
> > after the call.
>
> This has been fixed with Michael Thayer's patch which has xf86SetCursor
> return whether the hw cursor was enabled successfully.
OK, thanks! I myself forgot about this patch completely :)
Takashi
More information about the xorg-devel
mailing list