drm/i915/gvt: Replace duplicated code with exist function
authorPing Gao <ping.a.gao@intel.com>
Tue, 4 Jul 2017 08:11:16 +0000 (16:11 +0800)
committerZhenyu Wang <zhenyuw@linux.intel.com>
Thu, 10 Aug 2017 02:26:08 +0000 (10:26 +0800)
Use the exist function intel_gvt_ggtt_validate_range to replace
these duplicated code that do the same thing.

Signed-off-by: Ping Gao <ping.a.gao@intel.com>
Signed-off-by: Zhenyu Wang <zhenyuw@linux.intel.com>
drivers/gpu/drm/i915/gvt/cmd_parser.c

index 7ed8c551a5a1ae4c291195d410aba4968d1442b6..72b97ce525e8c713b067630b1d0978f3ccdc0632 100644 (file)
@@ -1382,13 +1382,13 @@ static inline int cmd_address_audit(struct parser_exec_state *s,
                        ret = -EINVAL;
                        goto err;
                }
-       } else if ((!vgpu_gmadr_is_valid(s->vgpu, guest_gma)) ||
-                       (!vgpu_gmadr_is_valid(s->vgpu,
-                                             guest_gma + op_size - 1))) {
+       } else if (!intel_gvt_ggtt_validate_range(vgpu, guest_gma, op_size)) {
                ret = -EINVAL;
                goto err;
        }
+
        return 0;
+
 err:
        gvt_vgpu_err("cmd_parser: Malicious %s detected, addr=0x%lx, len=%d!\n",
                        s->info->name, guest_gma, op_size);