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
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
