[PATCH] drm: don't override possible_crtcs for primary/cursor planes

2016-11-08 Thread Daniel Vetter
On Sat, Nov 05, 2016 at 10:52:01AM -0400, Rob Clark wrote: > It is kind of a pointless restriction. If userspace does silly things > like using crtcA's cursor plane on crtcB, and then setcursor on crtcA, > it will end up with the overlay disabled on crtcB. But userspace is > allowed to shoot itse

[PATCH] drm: don't override possible_crtcs for primary/cursor planes

2016-11-05 Thread Rob Clark
It is kind of a pointless restriction. If userspace does silly things like using crtcA's cursor plane on crtcB, and then setcursor on crtcA, it will end up with the overlay disabled on crtcB. But userspace is allowed to shoot itself like this. v2: don't WARN_ON() if caller did not set ->possible

[PATCH] drm: don't override possible_crtcs for primary/cursor planes

2016-10-20 Thread Ville Syrjälä
On Thu, Oct 20, 2016 at 03:06:06PM -0400, Rob Clark wrote: > It is kind of a pointless restriction. If userspace does silly things > like using crtcA's cursor plane on crtcB, and then setcursor on crtcA, > it will end up with the overlay disabled on crtcB. But userspace is > allowed to shoot itse

[PATCH] drm: don't override possible_crtcs for primary/cursor planes

2016-10-20 Thread Rob Clark
On Thu, Oct 20, 2016 at 3:19 PM, Ville Syrjälä wrote: > On Thu, Oct 20, 2016 at 03:06:06PM -0400, Rob Clark wrote: >> It is kind of a pointless restriction. If userspace does silly things >> like using crtcA's cursor plane on crtcB, and then setcursor on crtcA, >> it will end up with the overla

[PATCH] drm: don't override possible_crtcs for primary/cursor planes

2016-10-20 Thread Sean Paul
On Thu, Oct 20, 2016 at 3:06 PM, Rob Clark wrote: > It is kind of a pointless restriction. If userspace does silly things > like using crtcA's cursor plane on crtcB, and then setcursor on crtcA, > it will end up with the overlay disabled on crtcB. But userspace is > allowed to shoot itself like

[PATCH] drm: don't override possible_crtcs for primary/cursor planes

2016-10-20 Thread Rob Clark
It is kind of a pointless restriction. If userspace does silly things like using crtcA's cursor plane on crtcB, and then setcursor on crtcA, it will end up with the overlay disabled on crtcB. But userspace is allowed to shoot itself like this. Signed-off-by: Rob Clark --- Note that cursor and p