On Mon, 2012-07-09 at 17:22 +0200, Christian König wrote:
> On 09.07.2012 17:06, Michel Dänzer wrote:
> > On Mon, 2012-07-09 at 12:42 +0200, Christian König wrote:
> >> Making it easier to controlwhen it is executed.
> >>
> >> Signed-off-by: Christian König <deathsimple@xxxxxxxxxxx>
> > [...]
> >> diff --git a/drivers/gpu/drm/radeon/radeon_device.c b/drivers/gpu/drm/radeon/radeon_device.c
> >> index 254fdb4..bbd0971 100644
> >> --- a/drivers/gpu/drm/radeon/radeon_device.c
> >> +++ b/drivers/gpu/drm/radeon/radeon_device.c
> >> @@ -822,6 +822,10 @@ int radeon_device_init(struct radeon_device *rdev,
> >> if (r)
> >> return r;
> >>
> >> + r = radeon_ib_ring_tests(rdev);
> >> + if (r)
> >> + DRM_ERROR("ib ring test failed (%d).\n", r);
> >> +
> >> if (rdev->flags & RADEON_IS_AGP && !rdev->accel_working) {
> >> /* Acceleration not working on AGP card try again
> >> * with fallback to PCI or PCIE GART
> > I think this needs to set rdev->accel_working = false on failure, so the
> > AGP -> PCI(e) fallback can kick in.
>
> See the implementation of radeon_ib_ring_tests, it is already handling
> that internally.
Oops, I actually did check this when writing the above, but somehow I
failed to see what I was looking for. :}
--
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
