diff options
author | Christian König <christian.koenig@amd.com> | 2014-05-21 11:43:59 -0400 |
---|---|---|
committer | Christian König <christian.koenig@amd.com> | 2014-05-30 08:21:44 -0400 |
commit | 60a445405a1731bac2279eab354173c9c3156747 (patch) | |
tree | 2a3c1a801b553e66c8f2026852ef33c2eb0c27bc | |
parent | 4b21ce1b4b5d262e7d4656b8ececc891fc3cb806 (diff) |
drm/radeon: avoid crash if VM command submission isn't available
Signed-off-by: Christian König <christian.koenig@amd.com>
CC: stable@vger.kernel.org
-rw-r--r-- | drivers/gpu/drm/radeon/radeon_cs.c | 15 |
1 files changed, 11 insertions, 4 deletions
diff --git a/drivers/gpu/drm/radeon/radeon_cs.c b/drivers/gpu/drm/radeon/radeon_cs.c index 2b6e0ebcc13a..4b6f5b824153 100644 --- a/drivers/gpu/drm/radeon/radeon_cs.c +++ b/drivers/gpu/drm/radeon/radeon_cs.c | |||
@@ -342,10 +342,17 @@ int radeon_cs_parser_init(struct radeon_cs_parser *p, void *data) | |||
342 | return -EINVAL; | 342 | return -EINVAL; |
343 | 343 | ||
344 | /* we only support VM on some SI+ rings */ | 344 | /* we only support VM on some SI+ rings */ |
345 | if ((p->rdev->asic->ring[p->ring]->cs_parse == NULL) && | 345 | if ((p->cs_flags & RADEON_CS_USE_VM) == 0) { |
346 | ((p->cs_flags & RADEON_CS_USE_VM) == 0)) { | 346 | if (p->rdev->asic->ring[p->ring]->cs_parse == NULL) { |
347 | DRM_ERROR("Ring %d requires VM!\n", p->ring); | 347 | DRM_ERROR("Ring %d requires VM!\n", p->ring); |
348 | return -EINVAL; | 348 | return -EINVAL; |
349 | } | ||
350 | } else { | ||
351 | if (p->rdev->asic->ring[p->ring]->ib_parse == NULL) { | ||
352 | DRM_ERROR("VM not supported on ring %d!\n", | ||
353 | p->ring); | ||
354 | return -EINVAL; | ||
355 | } | ||
349 | } | 356 | } |
350 | } | 357 | } |
351 | 358 | ||