On Tue, Jul 17, 2012 at 1:02 PM, Michel Dänzer <michel@xxxxxxxxxxx> wrote: > From: Michel Dänzer <michel.daenzer@xxxxxxx> > > This could previously fail if either of the enabled displays was using a > horizontal resolution that is a multiple of 128, and only the leftmost column > of the cursor was (supposed to be) visible at the right edge of that display. > > The solution is to move the cursor one pixel to the left in that case. > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=33183 > > Cc: stable@xxxxxxxxxxxxxxx > Signed-off-by: Michel Dänzer <michel.daenzer@xxxxxxx> Reviewed-by: Alex Deucher <alexander.deucher@xxxxxxx> > --- > drivers/gpu/drm/radeon/radeon_cursor.c | 8 +++++++- > 1 file changed, 7 insertions(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/radeon/radeon_cursor.c b/drivers/gpu/drm/radeon/radeon_cursor.c > index 42acc64..711e95a 100644 > --- a/drivers/gpu/drm/radeon/radeon_cursor.c > +++ b/drivers/gpu/drm/radeon/radeon_cursor.c > @@ -262,8 +262,14 @@ int radeon_crtc_cursor_move(struct drm_crtc *crtc, > if (!(cursor_end & 0x7f)) > w--; > } > - if (w <= 0) > + if (w <= 0) { > w = 1; > + cursor_end = x - xorigin + w; > + if (!(cursor_end & 0x7f)) { > + x--; > + WARN_ON_ONCE(x < 0); > + } > + } > } > } > > -- > 1.7.10 > > _______________________________________________ > dri-devel mailing list > dri-devel@xxxxxxxxxxxxxxxxxxxxx > http://lists.freedesktop.org/mailman/listinfo/dri-devel _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel
