With per-process address spaces in place, a rogue process submitting bogus command streams can only hurt itself. There is no need to validate the command stream before execution anymore.
Unified · Split
| @@ -517,7 +517,8 @@ int etnaviv_ioctl_gem_submit(struct drm_device *dev, void *data, · 516 unmodified lines | |||
| 517 | if (ret) | 517 | if (ret) |
| 518 | goto err_submit_objects; | 518 | goto err_submit_objects; |
| 519 | 519 | ||
| 520 | if (!etnaviv_cmd_validate_one(gpu, stream, args->stream_size / 4, | 520 | if ((priv->mmu_global->version != ETNAVIV_IOMMU_V2) && |
| 521 | !etnaviv_cmd_validate_one(gpu, stream, args->stream_size / 4, | ||
| 521 | relocs, args->nr_relocs)) { | 522 | relocs, args->nr_relocs)) { |
| 522 | ret = -EINVAL; | 523 | ret = -EINVAL; |
| 523 | goto err_submit_objects; | 524 | goto err_submit_objects; |