diff options
author | Marek Olšák <marek.olsak@amd.com> | 2014-05-27 02:56:36 +0200 |
---|---|---|
committer | Jiri Slaby <jslaby@suse.cz> | 2014-07-17 15:04:36 +0200 |
commit | 20badb285d5c13d6c4c8ab2a47203d73378da50b (patch) | |
tree | 25b9af08daad5b32776ba25d579bda2c3e705638 /drivers | |
parent | 4a88652c5b6dae30f51c877df7beb9eed82675f8 (diff) |
drm/radeon: don't allow RADEON_GEM_DOMAIN_CPU for command submission
commit ec65da385d46f63740c1c9230b891a6dcbd64c71 upstream.
It hangs the hardware.
Signed-off-by: Marek Olšák <marek.olsak@amd.com>
Reviewed-by: Christian König <christian.koenig@amd.com>
Signed-off-by: Jiri Slaby <jslaby@suse.cz>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/gpu/drm/radeon/radeon_cs.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/drivers/gpu/drm/radeon/radeon_cs.c b/drivers/gpu/drm/radeon/radeon_cs.c index b2b8b38f0319..ed9a997c99a3 100644 --- a/drivers/gpu/drm/radeon/radeon_cs.c +++ b/drivers/gpu/drm/radeon/radeon_cs.c @@ -97,6 +97,12 @@ static int radeon_cs_parser_relocs(struct radeon_cs_parser *p) uint32_t domain = r->write_domain ? r->write_domain : r->read_domains; + if (domain & RADEON_GEM_DOMAIN_CPU) { + DRM_ERROR("RADEON_GEM_DOMAIN_CPU is not valid " + "for command submission\n"); + return -EINVAL; + } + p->relocs[i].lobj.domain = domain; if (domain == RADEON_GEM_DOMAIN_VRAM) domain |= RADEON_GEM_DOMAIN_GTT; |