On 01/16/2018 11:42 AM, Felix Kuehling wrote: > On 2018-01-16 10:18 AM, Andrey Grodzovsky wrote: >> Avoid OOM on syatem pages allocations. >> >> v2: >> Remove modeprobe parameter, making this behaviour the only option. >> >> Signed-off-by: Andrey Grodzovsky <andrey.grodzovsky at amd.com> >> --- >> drivers/gpu/drm/amd/amdgpu/amdgpu_object.c | 4 ++++ >> 1 file changed, 4 insertions(+) >> >> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_object.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_object.c >> index 5c4c3e0..b4dc3bd 100644 >> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_object.c >> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_object.c >> @@ -420,6 +420,10 @@ static int amdgpu_bo_do_create(struct amdgpu_device *adev, >> #endif >> >> bo->tbo.bdev = &adev->mman.bdev; >> + >> + /* We opt to avoid OOM on system pages allocations */ >> + bo->tbo.bdev->no_retry = true; >> + > Couldn't this be done once in amdgpu_ttm_init just after calling > ttm_bo_device_init? No need to do this for every BO creation. > > Regards, > Â Felix Good catch, will fix. Thanks, Andrey > >> amdgpu_ttm_placement_from_domain(bo, domain); >> >> r = ttm_bo_init_reserved(&adev->mman.bdev, &bo->tbo, size, type, > _______________________________________________ > amd-gfx mailing list > amd-gfx at lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/amd-gfx
