Am 13.11.2013 21:29, schrieb Alex Deucher: > If we fail to allocate an indirect buffer (ib) when updating > the ptes, return an error instead of trying to use the ib. > Avoids a null pointer dereference. > > Bug: > https://bugzilla.kernel.org/show_bug.cgi?id=58621 > > Signed-off-by: Alex Deucher <alexander.deucher at amd.com> > Cc: stable at vger.kernel.org
I'm getting old. I've seen that as well while working on the VM page table init, but forgotten to fix it... Reviewed-by: Christian K?nig <christian.koenig at amd.com> > --- > drivers/gpu/drm/radeon/radeon_gart.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/gpu/drm/radeon/radeon_gart.c > b/drivers/gpu/drm/radeon/radeon_gart.c > index cd7489b..3044e50 100644 > --- a/drivers/gpu/drm/radeon/radeon_gart.c > +++ b/drivers/gpu/drm/radeon/radeon_gart.c > @@ -1209,6 +1209,8 @@ int radeon_vm_bo_update_pte(struct radeon_device *rdev, > return -ENOMEM; > > r = radeon_ib_get(rdev, R600_RING_TYPE_DMA_INDEX, &ib, NULL, ndw * 4); > + if (r) > + return r; > ib.length_dw = 0; > > r = radeon_vm_update_pdes(rdev, vm, &ib, bo_va->soffset, > bo_va->eoffset);
