Re: ✗ Fi.CI.BAT: failure for series starting with [1/2] drm/i915/guc: Don't enable GuC when vGPU is active

[Tomi Sarvela <tomi.p.sarvela@intel.com>]

On 15/01/18 12:28, Zhenyu Wang wrote:
> On 2018.01.15 12:07:28 +0200, Joonas Lahtinen wrote:
>> On Fri, 2018-01-12 at 14:08 +0800, Du, Changbin wrote:
>>> On Fri, Jan 12, 2018 at 11:32:30AM +0530, Sagar Arun Kamble wrote:
>>>> Is skl-gvtdvm not having vGPU active?
>>>>
>>>> It has flag X86_FEATURE_HYPERVISOR set however it might be set on host too
>>>> so relying intel_vgpu_active().
>>>>
>>>
>>> Do you mean flag X86_FEATURE_HYPERVISOR is set on host, too? This is weird since this
>>> flag indicates the OS is running on a hypervisor.
>>
>> + CI folks and Zhenyu
>>
>> Somehow, magically, the virtual machine seems to starts skipping all
>> tests when GuC is disabled?
>>
>> Has somebody actually validated that the tests results are valid for
>> the virtual machine? Or is this a one-off CI quirk?
> 
> Are these tests really run in VM with GVT-g enabled on host?

These tests are ran on VM running on GVT-d (as name implies), not GVT-g.

Tomi

>> PS. I'd do the check as early return condition at the top, like vGPU
>> feature disabling elsewhere. It's not much point calculating the
>> details (calling into functions like intel_uc_fw_is_selected) and then
>> throwing results out afterwards :)
>>
> 
> yeah, better, and we do plan to support vGuC from ICL for vGPU.
> 
>>>
>>>>
>>>> On 1/11/2018 10:27 PM, Patchwork wrote:
>>>>> == Series Details ==
>>>>>
>>>>> Series: series starting with [1/2] drm/i915/guc: Don't enable GuC when vGPU is active
>>>>> URL   : https://patchwork.freedesktop.org/series/36343/
>>>>> State : failure
>>>>>
>>>>> == Summary ==
>>>>>
>>>>> Series 36343v1 series starting with [1/2] drm/i915/guc: Don't enable GuC when vGPU is active
>>>>> https://patchwork.freedesktop.org/api/1.0/series/36343/revisions/1/mbox/
>>>>>
>>>>> Test core_auth:
>>>>>           Subgroup basic-auth:
>>>>>                   pass       -> SKIP       (fi-skl-gvtdvm)
>>>>> Test core_prop_blob:
>>>>>           Subgroup basic:
>>>>>                   pass       -> SKIP       (fi-skl-gvtdvm)
>>>>> Test debugfs_test:
>>>>>           Subgroup read_all_entries:
>>>>>                   dmesg-warn -> DMESG-FAIL (fi-elk-e7500) fdo#103989
>>>>>                   pass       -> SKIP       (fi-skl-gvtdvm)
>>>>> Test drv_getparams_basic:
>>>>>           Subgroup basic-eu-total:
>>>>>                   pass       -> SKIP       (fi-skl-gvtdvm)
>>>>>           Subgroup basic-subslice-total:
>>>>>                   pass       -> SKIP       (fi-skl-gvtdvm)
>>>>> Test drv_hangman:
>>>>>           Subgroup error-state-basic:
>>>>>                   pass       -> SKIP       (fi-skl-gvtdvm)
>>>>> Test gem_basic:
>>>>>           Subgroup bad-close:
>>>>>                   pass       -> SKIP       (fi-skl-gvtdvm)
>>>>>           Subgroup create-close:
>>>>>                   pass       -> SKIP       (fi-skl-gvtdvm)
>>>>>           Subgroup create-fd-close:
>>>>>                   pass       -> SKIP       (fi-skl-gvtdvm)
>>>>> Test gem_busy:
>>>>>           Subgroup basic-busy-default:
>>>>>                   pass       -> SKIP       (fi-skl-gvtdvm)
>>>>>           Subgroup basic-hang-default:
>>>>>                   pass       -> SKIP       (fi-skl-gvtdvm) fdo#104108 +2
>>>>> Test gem_close_race:
>>>>>           Subgroup basic-process:
>>>>>                   pass       -> SKIP       (fi-skl-gvtdvm)
>>>>>           Subgroup basic-threads:
>>>>>                   pass       -> SKIP       (fi-skl-gvtdvm)
>>>>> Test gem_cpu_reloc:
>>>>>           Subgroup basic:
>>>>>                   pass       -> SKIP       (fi-skl-gvtdvm)
>>>>> Test gem_cs_tlb:
>>>>>           Subgroup basic-default:
>>>>>                   pass       -> SKIP       (fi-skl-gvtdvm)
>>>>> Test gem_ctx_create:
>>>>>           Subgroup basic:
>>>>>                   pass       -> SKIP       (fi-skl-gvtdvm)
>>>>>           Subgroup basic-files:
>>>>>                   pass       -> SKIP       (fi-skl-gvtdvm)
>>>>> Test gem_ctx_exec:
>>>>>           Subgroup basic:
>>>>>                   pass       -> SKIP       (fi-skl-gvtdvm)
>>>>> Test gem_ctx_param:
>>>>>           Subgroup basic:
>>>>>                   pass       -> SKIP       (fi-skl-gvtdvm)
>>>>>           Subgroup basic-default:
>>>>>                   pass       -> SKIP       (fi-skl-gvtdvm)
>>>>> Test gem_ctx_switch:
>>>>>           Subgroup basic-default:
>>>>>                   pass       -> SKIP       (fi-skl-gvtdvm)
>>>>>           Subgroup basic-default-heavy:
>>>>>                   pass       -> SKIP       (fi-skl-gvtdvm)
>>>>> Test gem_exec_basic:
>>>>>           Subgroup basic-blt:
>>>>>                   pass       -> SKIP       (fi-skl-gvtdvm)
>>>>>           Subgroup basic-bsd:
>>>>>                   pass       -> SKIP       (fi-skl-gvtdvm)
>>>>>           Subgroup basic-bsd1:
>>>>>                   pass       -> SKIP       (fi-skl-gvtdvm)
>>>>>           Subgroup basic-bsd2:
>>>>>                   pass       -> SKIP       (fi-skl-gvtdvm)
>>>>>           Subgroup basic-default:
>>>>>                   pass       -> SKIP       (fi-skl-gvtdvm)
>>>>>           Subgroup basic-render:
>>>>>                   pass       -> SKIP       (fi-skl-gvtdvm)
>>>>>           Subgroup basic-vebox:
>>>>>                   pass       -> SKIP       (fi-skl-gvtdvm)
>>>>>           Subgroup gtt-blt:
>>>>>                   pass       -> SKIP       (fi-skl-gvtdvm)
>>>>>           Subgroup gtt-bsd:
>>>>>                   pass       -> SKIP       (fi-skl-gvtdvm)
>>>>>           Subgroup gtt-bsd1:
>>>>>                   pass       -> SKIP       (fi-skl-gvtdvm)
>>>>>           Subgroup gtt-bsd2:
>>>>>                   pass       -> SKIP       (fi-skl-gvtdvm)
>>>>>           Subgroup gtt-default:
>>>>>                   pass       -> SKIP       (fi-skl-gvtdvm)
>>>>>           Subgroup gtt-render:
>>>>>                   pass       -> SKIP       (fi-skl-gvtdvm)
>>>>>           Subgroup gtt-vebox:
>>>>>                   pass       -> SKIP       (fi-skl-gvtdvm)
>>>>>           Subgroup readonly-blt:
>>>>>                   pass       -> SKIP       (fi-skl-gvtdvm)
>>>>>           Subgroup readonly-bsd:
>>>>>                   pass       -> SKIP       (fi-skl-gvtdvm)
>>>>>           Subgroup readonly-bsd1:
>>>>>                   pass       -> SKIP       (fi-skl-gvtdvm)
>>>>>           Subgroup readonly-bsd2:
>>>>>                   pass       -> SKIP       (fi-skl-gvtdvm)
>>>>>           Subgroup readonly-default:
>>>>> WARNING: Long output truncated
>>>>>
>>>>> da4321c7c088d01ad62b18d37dcfe5ef8d0b855d drm-tip: 2018y-01m-11d-14h-59m-37s UTC integration manifest
>>>>> a0e8e84432b0 HAX: drm/i915/uc: Enable GuC/HuC for CI
>>>>> 973357da236a drm/i915/guc: Don't enable GuC when vGPU is active
>>>>>
>>>>> == Logs ==
>>>>>
>>>>> For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_7651/issues.html
>>>
>>>
>> -- 
>> Joonas Lahtinen
>> Open Source Technology Center
>> Intel Corporation
> 


Tomi
-- 
Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx