Hi Krzysztof

> +     if (!ppgtt->vm.allocate_va_range) {
> +             i915_vm_put(&ppgtt->vm);
> +             return 0;
> +     }

I don't know if it feels more in line with kernel style, but consider
changing it to a label before second `i915_vm_put` at end of function
plus goto instead of creating new cleanup section.

Otherwise looks alright.
Reviewed-by: Mikolaj Wasiak <mikolaj.was...@intel.com>

Mikolaj

Reply via email to