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 <[email protected]> Mikolaj
