Re: [PATCH 1/2] drm/radeon: Restrict offset for legacy hardware cursor.

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

 



On Mit, 2012-03-14 at 11:08 -0400, Alex Deucher wrote: 
> 2012/3/14 Michel Dänzer <michel@xxxxxxxxxxx>:
> > From: Michel Dänzer <michel.daenzer@xxxxxxx>
> >
> > The hardware only takes 27 bits for the offset, so larger offsets are
> > truncated, and the hardware cursor shows random bits other than the intended
> > ones.
> >
> > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=46796
> >
> > Cc: stable@xxxxxxxxxxxxxxx
> > Signed-off-by: Michel Dänzer <michel.daenzer@xxxxxxx>
> 
> Reviewed-by: Alex Deucher <alexander.deucher@xxxxxxx>

Thanks. In the meantime, it occurred to me there might be a problem when
there's less than 128M of VRAM in the first place (so bo->placement.lpfn
ends up larger than rdev->mman.bdev.man[TTM_PL_VRAM].size). To be safe,
I'll send a v2 patch preventing that.


> FWIW, I think we may need a similar fix for the crtc base addresses.
> They are also limited to 27 bit offsets from the mc address specified
> in DISPLAY[2]_BASE_ADDR.

Hmm, good point.


-- 
Earthling Michel Dänzer           |                   http://www.amd.com
Libre software enthusiast         |          Debian, X and DRI developer
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/dri-devel



[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux