diff options
| author | Rob Clark <robdclark@gmail.com> | 2016-05-24 18:29:38 -0400 |
|---|---|---|
| committer | Rob Clark <robdclark@gmail.com> | 2016-06-04 14:45:48 -0400 |
| commit | 69a834c28fb514403eb91a4f0120da214a52e056 (patch) | |
| tree | 718733cb1bdaea6feb4a831e14838d03d19ef85c /drivers/gpu/drm/msm/adreno | |
| parent | ab3ab68493aaac6fea4ad1bb597def9f48f41c71 (diff) | |
| download | lwn-69a834c28fb514403eb91a4f0120da214a52e056.tar.gz lwn-69a834c28fb514403eb91a4f0120da214a52e056.zip | |
drm/msm: deal with exhausted vmap space better
Some, but not all, callers of obj->vmap() would check if return
IS_ERR(). So let's actually return an error if vmap() fails. And fixup
the call-sites that were not handling this properly.
Signed-off-by: Rob Clark <robdclark@gmail.com>
Diffstat (limited to 'drivers/gpu/drm/msm/adreno')
| -rw-r--r-- | drivers/gpu/drm/msm/adreno/adreno_gpu.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/gpu/drm/msm/adreno/adreno_gpu.c b/drivers/gpu/drm/msm/adreno/adreno_gpu.c index fbe304ee6c80..2aec27dbb5bb 100644 --- a/drivers/gpu/drm/msm/adreno/adreno_gpu.c +++ b/drivers/gpu/drm/msm/adreno/adreno_gpu.c @@ -408,7 +408,7 @@ int adreno_gpu_init(struct drm_device *drm, struct platform_device *pdev, } adreno_gpu->memptrs = msm_gem_vaddr(adreno_gpu->memptrs_bo); - if (!adreno_gpu->memptrs) { + if (IS_ERR(adreno_gpu->memptrs)) { dev_err(drm->dev, "could not vmap memptrs\n"); return -ENOMEM; } |