[Intel-gfx] [PATCH] drm/i915/gvt: Fix workload status after wait

Joonas Lahtinen joonas.lahtinen at linux.intel.com
Wed Nov 2 06:47:52 UTC 2016


On ke, 2016-11-02 at 13:41 +0800, Zhenyu Wang wrote:
> From commit e95433c73a11759203af1cae5958f998c9673370, workload status setting
> was changed to only capture on error path, but we need to set it properly in
> normal path too, otherwise we'll fail to complete workload which could lead
> guest VM vGPU reset.
> 

Should have Fixes tag with the above commit.

> @@ -455,7 +455,8 @@ static int workload_thread(void *priv)
>  		if (lret < 0) {
>  			workload->status = lret;
>  			gvt_err("fail to wait workload, skip\n");
> -		}
> +		} else
> +			workload->status = 0;

All branches of if-else continuum must use braces if one does, so
"} else {" here

Regards, Joonas
-- 
Joonas Lahtinen
Open Source Technology Center
Intel Corporation


More information about the Intel-gfx mailing list