[igt-dev] [PATCH i-g-t v11 0/2] Add test for asynchronous flips

[igt-dev] [PATCH i-g-t v11 0/2] Add test for asynchronous flips

[Karthik B S]

Add test for validation of asynchronous flips.

Karthik B S (2):
  lib/igt_ioctl_wrappers: Add igt_has_drm_cap ioctl wrapper
  tests/kms_async_flips: Add test to validate asynchronous flips

 lib/ioctl_wrappers.c    |  17 ++
 lib/ioctl_wrappers.h    |   1 +
 tests/Makefile.sources  |   1 +
 tests/kms_async_flips.c | 425 ++++++++++++++++++++++++++++++++++++++++
 tests/meson.build       |   1 +
 5 files changed, 445 insertions(+)
 create mode 100644 tests/kms_async_flips.c

-- 
2.22.0

_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

[igt-dev] [PATCH i-g-t v11 1/2] lib/igt_ioctl_wrappers: Add igt_has_drm_cap ioctl wrapper

[Karthik B S]

Add a generic helper for DRM_IOCTL_GET_CAP ioctl.

v10: -No changes.

v11: -Pass cap ID instead of the whole structure. (Ville)
     -Fix the patch ordering. (Ville)

Signed-off-by: Karthik B S <karthik.b.s@intel.com>
---
 lib/ioctl_wrappers.c | 17 +++++++++++++++++
 lib/ioctl_wrappers.h |  1 +
 2 files changed, 18 insertions(+)

diff --git a/lib/ioctl_wrappers.c b/lib/ioctl_wrappers.c
index 3781286d..3f35609a 100644
--- a/lib/ioctl_wrappers.c
+++ b/lib/ioctl_wrappers.c
@@ -1318,3 +1318,20 @@ int __kms_addfb(int fd, uint32_t handle,
 
 	return ret < 0 ? -errno : ret;
 }
+
+/**
+ * igt_has_drm_cap:
+ * @fd: Open DRM file descriptor.
+ * @cap: drm_get_cap structure.
+ *
+ * This helper verifies if the passed cap is supported by the kernel
+ *
+ * Returns: Whether the cap is supported or not.
+ */
+bool igt_has_drm_cap(int fd, uint64_t flag)
+{
+	struct drm_get_cap cap = { .capability = flag };
+
+	igt_assert(drmIoctl(fd, DRM_IOCTL_GET_CAP, &cap) == 0);
+	return cap.value;
+}
diff --git a/lib/ioctl_wrappers.h b/lib/ioctl_wrappers.h
index 870ac8b7..e0ec203d 100644
--- a/lib/ioctl_wrappers.h
+++ b/lib/ioctl_wrappers.h
@@ -196,6 +196,7 @@ struct local_drm_mode_fb_cmd2 {
 
 bool igt_has_fb_modifiers(int fd);
 void igt_require_fb_modifiers(int fd);
+bool igt_has_drm_cap(int fd, uint64_t flag);
 
 /**
  * __kms_addfb:
-- 
2.22.0

_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

Re: [igt-dev] [PATCH i-g-t v11 1/2] lib/igt_ioctl_wrappers: Add igt_has_drm_cap ioctl wrapper

[Ville Syrjälä]

On Fri, Sep 25, 2020 at 09:56:12AM +0530, Karthik B S wrote:
> Add a generic helper for DRM_IOCTL_GET_CAP ioctl.
> 
> v10: -No changes.
> 
> v11: -Pass cap ID instead of the whole structure. (Ville)
>      -Fix the patch ordering. (Ville)
> 
> Signed-off-by: Karthik B S <karthik.b.s@intel.com>
> ---
>  lib/ioctl_wrappers.c | 17 +++++++++++++++++
>  lib/ioctl_wrappers.h |  1 +
>  2 files changed, 18 insertions(+)
> 
> diff --git a/lib/ioctl_wrappers.c b/lib/ioctl_wrappers.c
> index 3781286d..3f35609a 100644
> --- a/lib/ioctl_wrappers.c
> +++ b/lib/ioctl_wrappers.c
> @@ -1318,3 +1318,20 @@ int __kms_addfb(int fd, uint32_t handle,
>  
>  	return ret < 0 ? -errno : ret;
>  }
> +
> +/**
> + * igt_has_drm_cap:
> + * @fd: Open DRM file descriptor.
> + * @cap: drm_get_cap structure.
> + *
> + * This helper verifies if the passed cap is supported by the kernel
> + *
> + * Returns: Whether the cap is supported or not.
> + */
> +bool igt_has_drm_cap(int fd, uint64_t flag)

The docs and code here don't match. I fixed that up and
s/flag/capability/ for clarity.

There was also an int vs. pointer compiler warning in the
othr patch  which I fixed.

Series pushed to master. Thanks.

> +{
> +	struct drm_get_cap cap = { .capability = flag };
> +
> +	igt_assert(drmIoctl(fd, DRM_IOCTL_GET_CAP, &cap) == 0);
> +	return cap.value;
> +}
> diff --git a/lib/ioctl_wrappers.h b/lib/ioctl_wrappers.h
> index 870ac8b7..e0ec203d 100644
> --- a/lib/ioctl_wrappers.h
> +++ b/lib/ioctl_wrappers.h
> @@ -196,6 +196,7 @@ struct local_drm_mode_fb_cmd2 {
>  
>  bool igt_has_fb_modifiers(int fd);
>  void igt_require_fb_modifiers(int fd);
> +bool igt_has_drm_cap(int fd, uint64_t flag);
>  
>  /**
>   * __kms_addfb:
> -- 
> 2.22.0

-- 
Ville Syrjälä
Intel
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

Re: [igt-dev] [PATCH i-g-t v11 1/2] lib/igt_ioctl_wrappers: Add igt_has_drm_cap ioctl wrapper

[Karthik B S]

On 9/25/2020 8:38 PM, Ville Syrjälä wrote:
> On Fri, Sep 25, 2020 at 09:56:12AM +0530, Karthik B S wrote:
>> Add a generic helper for DRM_IOCTL_GET_CAP ioctl.
>>
>> v10: -No changes.
>>
>> v11: -Pass cap ID instead of the whole structure. (Ville)
>>       -Fix the patch ordering. (Ville)
>>
>> Signed-off-by: Karthik B S <karthik.b.s@intel.com>
>> ---
>>   lib/ioctl_wrappers.c | 17 +++++++++++++++++
>>   lib/ioctl_wrappers.h |  1 +
>>   2 files changed, 18 insertions(+)
>>
>> diff --git a/lib/ioctl_wrappers.c b/lib/ioctl_wrappers.c
>> index 3781286d..3f35609a 100644
>> --- a/lib/ioctl_wrappers.c
>> +++ b/lib/ioctl_wrappers.c
>> @@ -1318,3 +1318,20 @@ int __kms_addfb(int fd, uint32_t handle,
>>   
>>   	return ret < 0 ? -errno : ret;
>>   }
>> +
>> +/**
>> + * igt_has_drm_cap:
>> + * @fd: Open DRM file descriptor.
>> + * @cap: drm_get_cap structure.
>> + *
>> + * This helper verifies if the passed cap is supported by the kernel
>> + *
>> + * Returns: Whether the cap is supported or not.
>> + */
>> +bool igt_has_drm_cap(int fd, uint64_t flag)
> 
> The docs and code here don't match. I fixed that up and
> s/flag/capability/ for clarity.
> 
> There was also an int vs. pointer compiler warning in the
> othr patch  which I fixed.
> 
> Series pushed to master. Thanks.
> 

Thanks for the merge.

Thanks,
Karthik.B.S
>> +{
>> +	struct drm_get_cap cap = { .capability = flag };
>> +
>> +	igt_assert(drmIoctl(fd, DRM_IOCTL_GET_CAP, &cap) == 0);
>> +	return cap.value;
>> +}
>> diff --git a/lib/ioctl_wrappers.h b/lib/ioctl_wrappers.h
>> index 870ac8b7..e0ec203d 100644
>> --- a/lib/ioctl_wrappers.h
>> +++ b/lib/ioctl_wrappers.h
>> @@ -196,6 +196,7 @@ struct local_drm_mode_fb_cmd2 {
>>   
>>   bool igt_has_fb_modifiers(int fd);
>>   void igt_require_fb_modifiers(int fd);
>> +bool igt_has_drm_cap(int fd, uint64_t flag);
>>   
>>   /**
>>    * __kms_addfb:
>> -- 
>> 2.22.0
> 
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

Re: [igt-dev] [PATCH i-g-t v11 1/2] lib/igt_ioctl_wrappers: Add igt_has_drm_cap ioctl wrapper

[Ville Syrjälä]

On Tue, Sep 29, 2020 at 03:16:56PM +0530, Karthik B S wrote:
> 
> 
> On 9/25/2020 8:38 PM, Ville Syrjälä wrote:
> > On Fri, Sep 25, 2020 at 09:56:12AM +0530, Karthik B S wrote:
> >> Add a generic helper for DRM_IOCTL_GET_CAP ioctl.
> >>
> >> v10: -No changes.
> >>
> >> v11: -Pass cap ID instead of the whole structure. (Ville)
> >>       -Fix the patch ordering. (Ville)
> >>
> >> Signed-off-by: Karthik B S <karthik.b.s@intel.com>
> >> ---
> >>   lib/ioctl_wrappers.c | 17 +++++++++++++++++
> >>   lib/ioctl_wrappers.h |  1 +
> >>   2 files changed, 18 insertions(+)
> >>
> >> diff --git a/lib/ioctl_wrappers.c b/lib/ioctl_wrappers.c
> >> index 3781286d..3f35609a 100644
> >> --- a/lib/ioctl_wrappers.c
> >> +++ b/lib/ioctl_wrappers.c
> >> @@ -1318,3 +1318,20 @@ int __kms_addfb(int fd, uint32_t handle,
> >>   
> >>   	return ret < 0 ? -errno : ret;
> >>   }
> >> +
> >> +/**
> >> + * igt_has_drm_cap:
> >> + * @fd: Open DRM file descriptor.
> >> + * @cap: drm_get_cap structure.
> >> + *
> >> + * This helper verifies if the passed cap is supported by the kernel
> >> + *
> >> + * Returns: Whether the cap is supported or not.
> >> + */
> >> +bool igt_has_drm_cap(int fd, uint64_t flag)
> > 
> > The docs and code here don't match. I fixed that up and
> > s/flag/capability/ for clarity.
> > 
> > There was also an int vs. pointer compiler warning in the
> > othr patch  which I fixed.
> > 
> > Series pushed to master. Thanks.
> > 
> 
> Thanks for the merge.

CI is not very happy with these. Pls investigate.

> 
> Thanks,
> Karthik.B.S
> >> +{
> >> +	struct drm_get_cap cap = { .capability = flag };
> >> +
> >> +	igt_assert(drmIoctl(fd, DRM_IOCTL_GET_CAP, &cap) == 0);
> >> +	return cap.value;
> >> +}
> >> diff --git a/lib/ioctl_wrappers.h b/lib/ioctl_wrappers.h
> >> index 870ac8b7..e0ec203d 100644
> >> --- a/lib/ioctl_wrappers.h
> >> +++ b/lib/ioctl_wrappers.h
> >> @@ -196,6 +196,7 @@ struct local_drm_mode_fb_cmd2 {
> >>   
> >>   bool igt_has_fb_modifiers(int fd);
> >>   void igt_require_fb_modifiers(int fd);
> >> +bool igt_has_drm_cap(int fd, uint64_t flag);
> >>   
> >>   /**
> >>    * __kms_addfb:
> >> -- 
> >> 2.22.0
> > 

-- 
Ville Syrjälä
Intel
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

Re: [igt-dev] [PATCH i-g-t v11 1/2] lib/igt_ioctl_wrappers: Add igt_has_drm_cap ioctl wrapper

[Karthik B S]

On 9/29/2020 10:07 PM, Ville Syrjälä wrote:
> On Tue, Sep 29, 2020 at 03:16:56PM +0530, Karthik B S wrote:
>>
>>
>> On 9/25/2020 8:38 PM, Ville Syrjälä wrote:
>>> On Fri, Sep 25, 2020 at 09:56:12AM +0530, Karthik B S wrote:
>>>> Add a generic helper for DRM_IOCTL_GET_CAP ioctl.
>>>>
>>>> v10: -No changes.
>>>>
>>>> v11: -Pass cap ID instead of the whole structure. (Ville)
>>>>        -Fix the patch ordering. (Ville)
>>>>
>>>> Signed-off-by: Karthik B S <karthik.b.s@intel.com>
>>>> ---
>>>>    lib/ioctl_wrappers.c | 17 +++++++++++++++++
>>>>    lib/ioctl_wrappers.h |  1 +
>>>>    2 files changed, 18 insertions(+)
>>>>
>>>> diff --git a/lib/ioctl_wrappers.c b/lib/ioctl_wrappers.c
>>>> index 3781286d..3f35609a 100644
>>>> --- a/lib/ioctl_wrappers.c
>>>> +++ b/lib/ioctl_wrappers.c
>>>> @@ -1318,3 +1318,20 @@ int __kms_addfb(int fd, uint32_t handle,
>>>>    
>>>>    	return ret < 0 ? -errno : ret;
>>>>    }
>>>> +
>>>> +/**
>>>> + * igt_has_drm_cap:
>>>> + * @fd: Open DRM file descriptor.
>>>> + * @cap: drm_get_cap structure.
>>>> + *
>>>> + * This helper verifies if the passed cap is supported by the kernel
>>>> + *
>>>> + * Returns: Whether the cap is supported or not.
>>>> + */
>>>> +bool igt_has_drm_cap(int fd, uint64_t flag)
>>>
>>> The docs and code here don't match. I fixed that up and
>>> s/flag/capability/ for clarity.
>>>
>>> There was also an int vs. pointer compiler warning in the
>>> othr patch  which I fixed.
>>>
>>> Series pushed to master. Thanks.
>>>
>>
>> Thanks for the merge.
> 
> CI is not very happy with these. Pls investigate.
> 

I went through the results for kms_async_flips on CI.
Other than alternate-sync-async-flip subtest on SKL which is giving warn 
in dmesg "Atomic update on pipe (A) took 325 us, max time under evasion 
is 250 us", all the other failures are sporadic and failing due to one 
of the async flips in between taking longer time.

Two things are coming to my mind to handle this.
1. Reduce the MIN_FLIPS_PER_FRAME to 5 may be and check if this can 
handle the error across all platforms.

(Or)

2. Remove the assert where we check the flip interval of each async flip.

For subtest async-flip-with-page-flip-events, anyway we're having a FPS 
check at the end which measures the overall flip rate.

For subtest alternate-sync-async-flip, may be I add a buffer to allow a 
couple of flips to take longer than the threshold before actually 
failing the test.

Which approach would you suggest? or any other way you would suggest to 
handle this better?

Also looks like some of the HW are faster than others.
For Ex, we're not seeing any failures on ICL, whereas many failures are 
seen on KBL. Would you suggest we have MIN_FLIPS_PER_FRAME at a platform 
level?

I will also try to find out what is causing the dmesg warn only for SKL 
in case of alternate async and sync flips.
Any thing that you could point to, that should be changed for SKL in 
particular to take care of the dmesg warning?

Thanks,
Karthik.B.S
>>
>> Thanks,
>> Karthik.B.S
>>>> +{
>>>> +	struct drm_get_cap cap = { .capability = flag };
>>>> +
>>>> +	igt_assert(drmIoctl(fd, DRM_IOCTL_GET_CAP, &cap) == 0);
>>>> +	return cap.value;
>>>> +}
>>>> diff --git a/lib/ioctl_wrappers.h b/lib/ioctl_wrappers.h
>>>> index 870ac8b7..e0ec203d 100644
>>>> --- a/lib/ioctl_wrappers.h
>>>> +++ b/lib/ioctl_wrappers.h
>>>> @@ -196,6 +196,7 @@ struct local_drm_mode_fb_cmd2 {
>>>>    
>>>>    bool igt_has_fb_modifiers(int fd);
>>>>    void igt_require_fb_modifiers(int fd);
>>>> +bool igt_has_drm_cap(int fd, uint64_t flag);
>>>>    
>>>>    /**
>>>>     * __kms_addfb:
>>>> -- 
>>>> 2.22.0
>>>
> 
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

Re: [igt-dev] [PATCH i-g-t v11 1/2] lib/igt_ioctl_wrappers: Add igt_has_drm_cap ioctl wrapper

[Karthik B S]

On 9/30/2020 11:48 AM, Karthik B S wrote:
> 
> 
> On 9/29/2020 10:07 PM, Ville Syrjälä wrote:
>> On Tue, Sep 29, 2020 at 03:16:56PM +0530, Karthik B S wrote:
>>>
>>>
>>> On 9/25/2020 8:38 PM, Ville Syrjälä wrote:
>>>> On Fri, Sep 25, 2020 at 09:56:12AM +0530, Karthik B S wrote:
>>>>> Add a generic helper for DRM_IOCTL_GET_CAP ioctl.
>>>>>
>>>>> v10: -No changes.
>>>>>
>>>>> v11: -Pass cap ID instead of the whole structure. (Ville)
>>>>>        -Fix the patch ordering. (Ville)
>>>>>
>>>>> Signed-off-by: Karthik B S <karthik.b.s@intel.com>
>>>>> ---
>>>>>    lib/ioctl_wrappers.c | 17 +++++++++++++++++
>>>>>    lib/ioctl_wrappers.h |  1 +
>>>>>    2 files changed, 18 insertions(+)
>>>>>
>>>>> diff --git a/lib/ioctl_wrappers.c b/lib/ioctl_wrappers.c
>>>>> index 3781286d..3f35609a 100644
>>>>> --- a/lib/ioctl_wrappers.c
>>>>> +++ b/lib/ioctl_wrappers.c
>>>>> @@ -1318,3 +1318,20 @@ int __kms_addfb(int fd, uint32_t handle,
>>>>>        return ret < 0 ? -errno : ret;
>>>>>    }
>>>>> +
>>>>> +/**
>>>>> + * igt_has_drm_cap:
>>>>> + * @fd: Open DRM file descriptor.
>>>>> + * @cap: drm_get_cap structure.
>>>>> + *
>>>>> + * This helper verifies if the passed cap is supported by the kernel
>>>>> + *
>>>>> + * Returns: Whether the cap is supported or not.
>>>>> + */
>>>>> +bool igt_has_drm_cap(int fd, uint64_t flag)
>>>>
>>>> The docs and code here don't match. I fixed that up and
>>>> s/flag/capability/ for clarity.
>>>>
>>>> There was also an int vs. pointer compiler warning in the
>>>> othr patch  which I fixed.
>>>>
>>>> Series pushed to master. Thanks.
>>>>
>>>
>>> Thanks for the merge.
>>
>> CI is not very happy with these. Pls investigate.
>>
> 

I've sent a patch with minor tweaks to the existing logic,
to fix the failures seen on CI.
https://patchwork.freedesktop.org/series/82567/

Could you please take a look.

Thanks,
Karthik.B.S
> I went through the results for kms_async_flips on CI.
> Other than alternate-sync-async-flip subtest on SKL which is giving warn 
> in dmesg "Atomic update on pipe (A) took 325 us, max time under evasion 
> is 250 us", all the other failures are sporadic and failing due to one 
> of the async flips in between taking longer time.
> 
> Two things are coming to my mind to handle this.
> 1. Reduce the MIN_FLIPS_PER_FRAME to 5 may be and check if this can 
> handle the error across all platforms.
> 
> (Or)
> 
> 2. Remove the assert where we check the flip interval of each async flip.
> 
> For subtest async-flip-with-page-flip-events, anyway we're having a FPS 
> check at the end which measures the overall flip rate.
> 
> For subtest alternate-sync-async-flip, may be I add a buffer to allow a 
> couple of flips to take longer than the threshold before actually 
> failing the test.
> 
> Which approach would you suggest? or any other way you would suggest to 
> handle this better?
> 
> Also looks like some of the HW are faster than others.
> For Ex, we're not seeing any failures on ICL, whereas many failures are 
> seen on KBL. Would you suggest we have MIN_FLIPS_PER_FRAME at a platform 
> level?
> 
> I will also try to find out what is causing the dmesg warn only for SKL 
> in case of alternate async and sync flips.
> Any thing that you could point to, that should be changed for SKL in 
> particular to take care of the dmesg warning?
> 
> Thanks,
> Karthik.B.S
>>>
>>> Thanks,
>>> Karthik.B.S
>>>>> +{
>>>>> +    struct drm_get_cap cap = { .capability = flag };
>>>>> +
>>>>> +    igt_assert(drmIoctl(fd, DRM_IOCTL_GET_CAP, &cap) == 0);
>>>>> +    return cap.value;
>>>>> +}
>>>>> diff --git a/lib/ioctl_wrappers.h b/lib/ioctl_wrappers.h
>>>>> index 870ac8b7..e0ec203d 100644
>>>>> --- a/lib/ioctl_wrappers.h
>>>>> +++ b/lib/ioctl_wrappers.h
>>>>> @@ -196,6 +196,7 @@ struct local_drm_mode_fb_cmd2 {
>>>>>    bool igt_has_fb_modifiers(int fd);
>>>>>    void igt_require_fb_modifiers(int fd);
>>>>> +bool igt_has_drm_cap(int fd, uint64_t flag);
>>>>>    /**
>>>>>     * __kms_addfb:
>>>>> -- 
>>>>> 2.22.0
>>>>
>>
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

[igt-dev] [PATCH i-g-t v11 2/2] tests/kms_async_flips: Add test to validate asynchronous flips

[Karthik B S]

Asynchronous flips are issued using the page flip IOCTL.
The test consists of two subtests. The first subtest waits for
the page flip event to be received before giving the next flip,
and the second subtest doesn't wait for page flip events.

The test passes if the IOCTL is successful.

v2: -Add authors in the test file. (Paulo)
    -Reduce the run time and timeouts to suit IGT needs. (Paulo)
    -Replace igt_debug's with igt_assert's to catch slow flips. (Paulo)
    -Follow IGT coding style regarding spaces. (Paulo)
    -Make set up code part of igt_fixture. (Paulo)
    -Skip the test if async flips are not supported. (Paulo)
    -Replace suggested-by. (Paulo)
    -Added description for test and subtests.

v3: -Rename the test to kms_async_flips. (Paulo)
    -Modify the TODO comment. (Paulo)
    -Remove igt_debug in flip_handler. (Paulo)
    -Use drmIoctl() in has_async function. (Paulo)
    -Add more details in igt_assert in flip_handler. (Paulo)
    -Remove flag variable in flip_handler. (Paulo)
    -Call igt_assert in flip_handler after the warm up time.

v4: -Calculate the time stamp in flip_handler from userspace, as the
     kernel will return vbl timestamps and this cannot be used
     for async flips.
    -Add a new subtest to verify that the async flip time stamp
     lies in between the previous and next vblank time stamp. (Daniel)

v5: -Add test that alternates between sync and async flips. (Ville)
    -Add test to verify invalid async flips. (Ville)
    -Remove the subtest async-flip-without-page-flip-events. (Michel)
    -Remove the intel gpu restriction and make the test generic. (Michel)

v6: -Change the THRESHOLD from 10 to 8 as failures are seen on CI
     on platforms <= gen10.
    -In older platforms(<= gen10), async address update bit in plane ctl
     is double buffered. Made changes in subtests to accomodate this.
    -Moved the igt_assert from flip_handler to individual subtest as we
     now have four subtests and adding conditions for the assert in
     flip handler is messy.

v7: -Change flip_interval from int to float for more precision.
    -Remove the fb height change case in 'invalid' subtest as per the
     feedback received on the kernel patches.
    -Add subtest to verify legacy cursor IOCTL. (Ville)

v8: -Add a cursor flip before async flip in cursor test. (Ville)
    -Make flip_interval double for more precision as failures are seen
     on older platforms on CI.

v9: -Replace BUFS macro. (Ville)
    -Query cursor resolution from the kernel. (Ville)
    -Rename THRESHOLD to MIN_FLIPS_PER_FRAME. (Ville)
    -Use XRGB8888 instead of ARGB8888. (Ville)
    -Rename has_monotonic_timestamp to require_monotonic_timestamp. (Ville)
    -Make logic in alternate_sync_async subtest more readable. (Ville)
    -Rename get_vbl_timestamp_us() function. (Ville)
    -Rename has_async() to has_async_flip. (Ville)
    -Make data_t static. (Ville)
    -Fix leaking fb and res pointers. (Ville)
    -Remove async_capable variable. (Ville)
    -Move initialization to a function. (Ville)
    -Have a generic has_drm_cap() helper. (Ville)
    -Warm up logic removed.
    -Added TODO's for follow up work.

v10: -Move flip timestamps inside data_t. (Ville)
     -Remove the redundant first flip in invalid subtest. (Ville)

v11: -Pass cap ID to the igt_has_drm_cap helper instead
      of the whole structure. (Ville)
     -Fix the patch ordering. (Ville)

Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
Signed-off-by: Karthik B S <karthik.b.s@intel.com>
---
 tests/Makefile.sources  |   1 +
 tests/kms_async_flips.c | 425 ++++++++++++++++++++++++++++++++++++++++
 tests/meson.build       |   1 +
 3 files changed, 427 insertions(+)
 create mode 100644 tests/kms_async_flips.c

diff --git a/tests/Makefile.sources b/tests/Makefile.sources
index 6ae95155..f32ea9cf 100644
--- a/tests/Makefile.sources
+++ b/tests/Makefile.sources
@@ -32,6 +32,7 @@ TESTS_progs = \
 	feature_discovery \
 	kms_3d \
 	kms_addfb_basic \
+	kms_async_flips \
 	kms_atomic \
 	kms_atomic_interruptible \
 	kms_atomic_transition \
diff --git a/tests/kms_async_flips.c b/tests/kms_async_flips.c
new file mode 100644
index 00000000..56663096
--- /dev/null
+++ b/tests/kms_async_flips.c
@@ -0,0 +1,425 @@
+/*
+ * Copyright © 2020 Intel Corporation
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining a
+ * copy of this software and associated documentation files (the "Software"),
+ * to deal in the Software without restriction, including without limitation
+ * the rights to use, copy, modify, merge, publish, distribute, sublicense,
+ * and/or sell copies of the Software, and to permit persons to whom the
+ * Software is furnished to do so, subject to the following conditions:
+ *
+ * The above copyright notice and this permission notice (including the next
+ * paragraph) shall be included in all copies or substantial portions of the
+ * Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
+ * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
+ * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
+ * IN THE SOFTWARE.
+ *
+ * Authors:
+ *  Paulo Zanoni <paulo.r.zanoni@intel.com>
+ *  Karthik B S <karthik.b.s@intel.com>
+ */
+
+#include "igt.h"
+#include "igt_aux.h"
+#include <sys/ioctl.h>
+#include <sys/time.h>
+#include <poll.h>
+
+#define CURSOR_POS 128
+
+/*
+ * These constants can be tuned in case we start getting unexpected
+ * results in CI.
+ */
+
+#define RUN_TIME 2
+#define MIN_FLIPS_PER_FRAME 8
+
+IGT_TEST_DESCRIPTION("Test asynchronous page flips.");
+
+typedef struct {
+	int drm_fd;
+	uint32_t crtc_id;
+	uint32_t refresh_rate;
+	struct igt_fb bufs[4];
+	igt_display_t display;
+	drmModeConnectorPtr connector;
+	unsigned long flip_timestamp_us;
+	double flip_interval;
+} data_t;
+
+static drmModeConnectorPtr find_connector_for_modeset(data_t *data)
+{
+	igt_output_t *output;
+	drmModeConnectorPtr ret = NULL;
+
+	for_each_connected_output(&data->display, output) {
+		if (output->config.connector->count_modes > 0) {
+			ret = output->config.connector;
+			break;
+		}
+	}
+
+	igt_assert_f(ret, "Connector NOT found\n");
+	return ret;
+}
+
+static void flip_handler(int fd_, unsigned int sequence, unsigned int tv_sec,
+			 unsigned int tv_usec, void *_data)
+{
+	data_t *data = _data;
+	static double last_ms;
+	double cur_ms;
+	struct timespec ts;
+
+	clock_gettime(CLOCK_MONOTONIC, &ts);
+
+	cur_ms =  ts.tv_sec * 1000.0 + ts.tv_nsec / 1000000.0;
+
+	if (last_ms)
+		data->flip_interval = cur_ms - last_ms;
+	else
+		data->flip_interval = 0;
+
+	last_ms = cur_ms;
+
+	data->flip_timestamp_us = ts.tv_sec * 1000000 + ts.tv_nsec / 1000;
+}
+
+static void wait_flip_event(data_t *data)
+{
+	int ret;
+	drmEventContext evctx;
+	struct pollfd pfd;
+
+	evctx.version = 2;
+	evctx.vblank_handler = NULL;
+	evctx.page_flip_handler = flip_handler;
+
+	pfd.fd = data->drm_fd;
+	pfd.events = POLLIN;
+	pfd.revents = 0;
+
+	ret = poll(&pfd, 1, 2000);
+
+	switch (ret) {
+	case 0:
+		igt_assert_f(0, "Flip Timeout\n");
+		break;
+	case 1:
+		ret = drmHandleEvent(data->drm_fd, &evctx);
+		igt_assert(ret == 0);
+		break;
+	default:
+		/* unexpected */
+		igt_assert(0);
+	}
+}
+
+static void make_fb(data_t *data, struct igt_fb *fb,
+		    drmModeConnectorPtr connector, int index)
+{
+	uint32_t width, height;
+	int rec_width;
+
+	width = connector->modes[0].hdisplay;
+	height = connector->modes[0].vdisplay;
+
+	rec_width = width / (ARRAY_SIZE(data->bufs) * 2);
+
+	igt_create_fb(data->drm_fd, width, height, DRM_FORMAT_XRGB8888,
+		      LOCAL_I915_FORMAT_MOD_X_TILED, fb);
+	igt_draw_fill_fb(data->drm_fd, fb, 0x88);
+	igt_draw_rect_fb(data->drm_fd, NULL, NULL, fb, IGT_DRAW_MMAP_CPU,
+			 rec_width * 2 + rec_width * index,
+			 height / 4, rec_width,
+			 height / 2, rand());
+}
+
+static void require_monotonic_timestamp(int fd)
+{
+	igt_require_f(igt_has_drm_cap(fd, DRM_CAP_TIMESTAMP_MONOTONIC),
+		      "Monotonic timestamps not supported\n");
+}
+
+static void test_async_flip(data_t *data, bool alternate_sync_async)
+{
+	int ret, frame;
+	long long int fps;
+	struct timeval start, end, diff;
+
+	require_monotonic_timestamp(data->drm_fd);
+
+	gettimeofday(&start, NULL);
+	frame = 1;
+	do {
+		int flags = DRM_MODE_PAGE_FLIP_ASYNC | DRM_MODE_PAGE_FLIP_EVENT;
+
+		if (alternate_sync_async) {
+			flags &= ~DRM_MODE_PAGE_FLIP_ASYNC;
+
+			ret = drmModePageFlip(data->drm_fd, data->crtc_id,
+					      data->bufs[frame % 4].fb_id,
+					      flags, data);
+
+			igt_assert(ret == 0);
+
+			wait_flip_event(data);
+
+			flags |= DRM_MODE_PAGE_FLIP_ASYNC;
+
+			/*
+			 * In older platforms (<= Gen10), async address update bit is double buffered.
+			 * So flip timestamp can be verified only from the second flip.
+			 * The first async flip just enables the async address update.
+			 */
+			if (is_i915_device(data->drm_fd)) {
+				uint32_t devid = intel_get_drm_devid(data->drm_fd);
+
+				if (IS_GEN9(devid) || IS_GEN10(devid)) {
+					ret = drmModePageFlip(data->drm_fd, data->crtc_id,
+							      data->bufs[frame % 4].fb_id,
+							      flags, data);
+
+					igt_assert(ret == 0);
+
+					wait_flip_event(data);
+				}
+			}
+		}
+
+		ret = drmModePageFlip(data->drm_fd, data->crtc_id,
+				      data->bufs[frame % 4].fb_id,
+				      flags, data);
+
+		igt_assert(ret == 0);
+
+		wait_flip_event(data);
+
+		gettimeofday(&end, NULL);
+		timersub(&end, &start, &diff);
+
+		igt_assert_f(data->flip_interval < 1000.0 / (data->refresh_rate * MIN_FLIPS_PER_FRAME),
+			     "Flip interval not significantly smaller than vblank interval\n"
+			     "Flip interval: %lfms, Refresh Rate = %dHz, Threshold = %d\n",
+			     data->flip_interval, data->refresh_rate, MIN_FLIPS_PER_FRAME);
+
+		frame++;
+	} while (diff.tv_sec < RUN_TIME);
+
+	if (!alternate_sync_async) {
+		fps = frame * 1000 / RUN_TIME;
+		igt_assert_f((fps / 1000) > (data->refresh_rate * MIN_FLIPS_PER_FRAME),
+			     "FPS should be significantly higher than the refresh rate\n");
+	}
+}
+
+static void wait_for_vblank(data_t *data, unsigned long *vbl_time, unsigned int *seq)
+{
+	drmVBlank wait_vbl;
+	uint32_t pipe_id_flag;
+	int pipe;
+
+	memset(&wait_vbl, 0, sizeof(wait_vbl));
+	pipe = kmstest_get_pipe_from_crtc_id(data->drm_fd, data->crtc_id);
+	pipe_id_flag = kmstest_get_vbl_flag(pipe);
+
+	wait_vbl.request.type = DRM_VBLANK_RELATIVE | pipe_id_flag;
+	wait_vbl.request.sequence = 1;
+
+	igt_assert(drmIoctl(data->drm_fd, DRM_IOCTL_WAIT_VBLANK, &wait_vbl) == 0);
+	*vbl_time = wait_vbl.reply.tval_sec * 1000000 + wait_vbl.reply.tval_usec;
+	*seq = wait_vbl.reply.sequence;
+}
+
+static void test_timestamp(data_t *data)
+{
+	int flags = DRM_MODE_PAGE_FLIP_ASYNC | DRM_MODE_PAGE_FLIP_EVENT;
+	unsigned long vbl_time, vbl_time1;
+	unsigned int seq, seq1;
+	int ret;
+
+	require_monotonic_timestamp(data->drm_fd);
+
+	/*
+	 * In older platforms(<= gen10), async address update bit is double buffered.
+	 * So flip timestamp can be verified only from the second flip.
+	 * The first async flip just enables the async address update.
+	 */
+	ret = drmModePageFlip(data->drm_fd, data->crtc_id,
+			      data->bufs[0].fb_id,
+			      flags, data);
+
+	igt_assert(ret == 0);
+
+	wait_flip_event(data);
+
+	wait_for_vblank(data, &vbl_time, &seq);
+
+	ret = drmModePageFlip(data->drm_fd, data->crtc_id,
+			      data->bufs[0].fb_id,
+			      flags, data);
+
+	igt_assert(ret == 0);
+
+	wait_flip_event(data);
+
+	wait_for_vblank(data, &vbl_time1, &seq1);
+
+	/* TODO: Make changes to do as many flips as possbile between two vblanks */
+
+	igt_assert_f(seq1 == seq + 1,
+		     "Vblank sequence is expected to be incremented by one(%d != (%d + 1)\n", seq1, seq);
+
+	igt_info("vbl1_timestamp = %ldus\nflip_timestamp = %ldus\nvbl2_timestamp = %ldus\n",
+		 vbl_time, data->flip_timestamp_us, vbl_time1);
+
+	igt_assert_f(vbl_time < data->flip_timestamp_us && vbl_time1 > data->flip_timestamp_us,
+		     "Async flip time stamp is expected to be in between 2 vblank time stamps\n");
+}
+
+static void test_cursor(data_t *data)
+{
+	int flags = DRM_MODE_PAGE_FLIP_ASYNC | DRM_MODE_PAGE_FLIP_EVENT;
+	int ret;
+	uint64_t width, height;
+	struct igt_fb cursor_fb;
+	struct drm_mode_cursor cur;
+
+	do_or_die(drmGetCap(data->drm_fd, DRM_CAP_CURSOR_WIDTH, &width));
+	do_or_die(drmGetCap(data->drm_fd, DRM_CAP_CURSOR_WIDTH, &height));
+
+	igt_create_color_fb(data->drm_fd, width, height, DRM_FORMAT_ARGB8888,
+			    LOCAL_DRM_FORMAT_MOD_NONE, 1., 1., 1., &cursor_fb);
+
+	cur.flags = DRM_MODE_CURSOR_BO;
+	cur.crtc_id = data->crtc_id;
+	cur.width = width;
+	cur.height = height;
+	cur.handle = cursor_fb.gem_handle;
+
+	do_ioctl(data->drm_fd, DRM_IOCTL_MODE_CURSOR, &cur);
+
+	ret = drmModePageFlip(data->drm_fd, data->crtc_id,
+			      data->bufs[0].fb_id,
+			      flags, data);
+
+	igt_assert(ret == 0);
+
+	wait_flip_event(data);
+
+	cur.flags = DRM_MODE_CURSOR_MOVE;
+	cur.x = CURSOR_POS;
+	cur.y = CURSOR_POS;
+
+	do_ioctl(data->drm_fd, DRM_IOCTL_MODE_CURSOR, &cur);
+
+	igt_remove_fb(data->drm_fd, &cursor_fb);
+}
+
+static void test_invalid(data_t *data)
+{
+	int flags = DRM_MODE_PAGE_FLIP_ASYNC | DRM_MODE_PAGE_FLIP_EVENT;
+	int ret;
+	uint32_t width, height;
+	struct igt_fb fb;
+
+	width = data->connector->modes[0].hdisplay;
+	height = data->connector->modes[0].vdisplay;
+
+	igt_create_fb(data->drm_fd, width, height, DRM_FORMAT_XRGB8888,
+		      LOCAL_I915_FORMAT_MOD_Y_TILED, &fb);
+
+	/* Flip with a different fb modifier which is expected to be rejected */
+	ret = drmModePageFlip(data->drm_fd, data->crtc_id,
+			      fb.fb_id, flags, data);
+
+	igt_assert(ret == -EINVAL);
+
+	/* TODO: Add verification for changes in stride, pixel format */
+
+	igt_remove_fb(data->drm_fd, &fb);
+}
+
+static void test_init(data_t *data)
+{
+	drmModeResPtr res;
+	int i, ret;
+
+	res = drmModeGetResources(data->drm_fd);
+	igt_assert(res);
+
+	kmstest_unset_all_crtcs(data->drm_fd, res);
+
+	data->connector = find_connector_for_modeset(data);
+	data->crtc_id = kmstest_find_crtc_for_connector(data->drm_fd,
+							res, data->connector, 0);
+
+	data->refresh_rate = data->connector->modes[0].vrefresh;
+
+	for (i = 0; i < ARRAY_SIZE(data->bufs); i++)
+		make_fb(data, &data->bufs[i], data->connector, i);
+
+	ret = drmModeSetCrtc(data->drm_fd, data->crtc_id, data->bufs[0].fb_id, 0, 0,
+			     &data->connector->connector_id, 1, &data->connector->modes[0]);
+
+	igt_assert(ret == 0);
+
+	drmModeFreeResources(res);
+}
+
+igt_main
+{
+	static data_t data;
+	int i;
+
+	igt_fixture {
+		data.drm_fd = drm_open_driver_master(DRIVER_ANY);
+		kmstest_set_vt_graphics_mode();
+		igt_display_require(&data.display, data.drm_fd);
+		igt_display_require_output(&data.display);
+
+		igt_require_f(igt_has_drm_cap(data.drm_fd, DRM_CAP_ASYNC_PAGE_FLIP),
+			      "Async Flip is not supported\n");
+	}
+
+	igt_describe("Verify the async flip functionality and the fps during async flips");
+	igt_subtest_group {
+		igt_fixture
+			test_init(&data);
+
+		igt_describe("Wait for page flip events in between successive asynchronous flips");
+		igt_subtest("async-flip-with-page-flip-events")
+			test_async_flip(&data, false);
+
+		igt_describe("Alternate between sync and async flips");
+		igt_subtest("alternate-sync-async-flip")
+			test_async_flip(&data, true);
+
+		igt_describe("Verify that the async flip timestamp does not coincide with either previous or next vblank");
+		igt_subtest("test-time-stamp")
+			test_timestamp(&data);
+
+		igt_describe("Verify that the DRM_IOCTL_MODE_CURSOR passes after async flip");
+		igt_subtest("test-cursor")
+			test_cursor(&data);
+
+		igt_describe("Negative case to verify if changes in fb are rejected from kernel as expected");
+		igt_subtest("invalid-async-flip")
+			test_invalid(&data);
+
+		igt_fixture {
+			for (i = 0; i < ARRAY_SIZE(data.bufs); i++)
+				igt_remove_fb(data.drm_fd, &data.bufs[i]);
+		}
+	}
+
+	igt_fixture
+		igt_display_fini(&data.display);
+}
diff --git a/tests/meson.build b/tests/meson.build
index 5eb2d2fc..515f7528 100644
--- a/tests/meson.build
+++ b/tests/meson.build
@@ -16,6 +16,7 @@ test_progs = [
 	'feature_discovery',
 	'kms_3d',
 	'kms_addfb_basic',
+	'kms_async_flips',
 	'kms_atomic',
 	'kms_atomic_interruptible',
 	'kms_atomic_transition',
-- 
2.22.0

_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

[igt-dev] ✓ Fi.CI.BAT: success for Add test for asynchronous flips

[Patchwork]

[-- Attachment #1.1: Type: text/plain, Size: 5003 bytes --]

== Series Details ==

Series: Add test for asynchronous flips
URL   : https://patchwork.freedesktop.org/series/82075/
State : success

== Summary ==

CI Bug Log - changes from CI_DRM_9054 -> IGTPW_5011
====================================================

Summary
-------

  **SUCCESS**

  No regressions found.

  External URL: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5011/index.html

Known issues
------------

  Here are the changes found in IGTPW_5011 that come from known issues:

### IGT changes ###

#### Issues hit ####

  * igt@i915_pm_rpm@basic-pci-d3-state:
    - fi-byt-j1900:       [PASS][1] -> [DMESG-WARN][2] ([i915#1982])
   [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9054/fi-byt-j1900/igt@i915_pm_rpm@basic-pci-d3-state.html
   [2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5011/fi-byt-j1900/igt@i915_pm_rpm@basic-pci-d3-state.html

  * igt@kms_busy@basic@flip:
    - fi-kbl-x1275:       [PASS][3] -> [DMESG-WARN][4] ([i915#62] / [i915#92] / [i915#95])
   [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9054/fi-kbl-x1275/igt@kms_busy@basic@flip.html
   [4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5011/fi-kbl-x1275/igt@kms_busy@basic@flip.html

  * igt@kms_cursor_legacy@basic-flip-before-cursor-legacy:
    - fi-icl-u2:          [PASS][5] -> [DMESG-WARN][6] ([i915#1982])
   [5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9054/fi-icl-u2/igt@kms_cursor_legacy@basic-flip-before-cursor-legacy.html
   [6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5011/fi-icl-u2/igt@kms_cursor_legacy@basic-flip-before-cursor-legacy.html

  * igt@vgem_basic@unload:
    - fi-skl-guc:         [PASS][7] -> [DMESG-WARN][8] ([i915#2203])
   [7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9054/fi-skl-guc/igt@vgem_basic@unload.html
   [8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5011/fi-skl-guc/igt@vgem_basic@unload.html

  
#### Warnings ####

  * igt@gem_exec_suspend@basic-s0:
    - fi-kbl-x1275:       [DMESG-WARN][9] ([i915#62] / [i915#92] / [i915#95]) -> [DMESG-WARN][10] ([i915#1982] / [i915#62] / [i915#92] / [i915#95])
   [9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9054/fi-kbl-x1275/igt@gem_exec_suspend@basic-s0.html
   [10]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5011/fi-kbl-x1275/igt@gem_exec_suspend@basic-s0.html

  * igt@gem_exec_suspend@basic-s3:
    - fi-kbl-x1275:       [DMESG-WARN][11] ([i915#1982] / [i915#62] / [i915#92] / [i915#95]) -> [DMESG-WARN][12] ([i915#62] / [i915#92] / [i915#95])
   [11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9054/fi-kbl-x1275/igt@gem_exec_suspend@basic-s3.html
   [12]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5011/fi-kbl-x1275/igt@gem_exec_suspend@basic-s3.html

  * igt@i915_pm_rpm@basic-rte:
    - fi-kbl-x1275:       [DMESG-WARN][13] ([i915#62] / [i915#92] / [i915#95]) -> [DMESG-WARN][14] ([i915#62] / [i915#92])
   [13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9054/fi-kbl-x1275/igt@i915_pm_rpm@basic-rte.html
   [14]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5011/fi-kbl-x1275/igt@i915_pm_rpm@basic-rte.html

  * igt@kms_cursor_legacy@basic-flip-after-cursor-legacy:
    - fi-kbl-x1275:       [DMESG-WARN][15] ([i915#62] / [i915#92]) -> [DMESG-WARN][16] ([i915#62] / [i915#92] / [i915#95]) +3 similar issues
   [15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9054/fi-kbl-x1275/igt@kms_cursor_legacy@basic-flip-after-cursor-legacy.html
   [16]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5011/fi-kbl-x1275/igt@kms_cursor_legacy@basic-flip-after-cursor-legacy.html

  
  {name}: This element is suppressed. This means it is ignored when computing
          the status of the difference (SUCCESS, WARNING, or FAILURE).

  [i915#1635]: https://gitlab.freedesktop.org/drm/intel/issues/1635
  [i915#1982]: https://gitlab.freedesktop.org/drm/intel/issues/1982
  [i915#2203]: https://gitlab.freedesktop.org/drm/intel/issues/2203
  [i915#62]: https://gitlab.freedesktop.org/drm/intel/issues/62
  [i915#92]: https://gitlab.freedesktop.org/drm/intel/issues/92
  [i915#95]: https://gitlab.freedesktop.org/drm/intel/issues/95


Participating hosts (46 -> 39)
------------------------------

  Missing    (7): fi-ilk-m540 fi-hsw-4200u fi-byt-squawks fi-bsw-cyan fi-tgl-y fi-byt-clapper fi-bdw-samus 


Build changes
-------------

  * CI: CI-20190529 -> None
  * IGT: IGT_5787 -> IGTPW_5011

  CI-20190529: 20190529
  CI_DRM_9054: 04b9e672bb8daa5e9ac3a41abf5fc26d75ef9b51 @ git://anongit.freedesktop.org/gfx-ci/linux
  IGTPW_5011: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5011/index.html
  IGT_5787: 0ec962017c8131de14e0cb038f7f76b1f17ed637 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools



== Testlist changes ==

+igt@kms_async_flips@alternate-sync-async-flip
+igt@kms_async_flips@async-flip-with-page-flip-events
+igt@kms_async_flips@invalid-async-flip
+igt@kms_async_flips@test-cursor
+igt@kms_async_flips@test-time-stamp

== Logs ==

For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5011/index.html

[-- Attachment #1.2: Type: text/html, Size: 7363 bytes --]

[-- Attachment #2: Type: text/plain, Size: 154 bytes --]

_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

[igt-dev] ✓ Fi.CI.IGT: success for Add test for asynchronous flips

[Patchwork]

[-- Attachment #1.1: Type: text/plain, Size: 13502 bytes --]

== Series Details ==

Series: Add test for asynchronous flips
URL   : https://patchwork.freedesktop.org/series/82075/
State : success

== Summary ==

CI Bug Log - changes from CI_DRM_9054_full -> IGTPW_5011_full
====================================================

Summary
-------

  **SUCCESS**

  No regressions found.

  External URL: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5011/index.html

Possible new issues
-------------------

  Here are the unknown changes that may have been introduced in IGTPW_5011_full:

### IGT changes ###

#### Possible regressions ####

  * {igt@kms_async_flips@invalid-async-flip} (NEW):
    - shard-tglb:         NOTRUN -> [SKIP][1] +4 similar issues
   [1]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5011/shard-tglb5/igt@kms_async_flips@invalid-async-flip.html

  * {igt@kms_async_flips@test-cursor} (NEW):
    - shard-iclb:         NOTRUN -> [SKIP][2] +4 similar issues
   [2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5011/shard-iclb8/igt@kms_async_flips@test-cursor.html

  
New tests
---------

  New tests have been introduced between CI_DRM_9054_full and IGTPW_5011_full:

### New IGT tests (5) ###

  * igt@kms_async_flips@alternate-sync-async-flip:
    - Statuses : 7 skip(s)
    - Exec time: [0.0] s

  * igt@kms_async_flips@async-flip-with-page-flip-events:
    - Statuses : 7 skip(s)
    - Exec time: [0.0] s

  * igt@kms_async_flips@invalid-async-flip:
    - Statuses : 7 skip(s)
    - Exec time: [0.0] s

  * igt@kms_async_flips@test-cursor:
    - Statuses : 7 skip(s)
    - Exec time: [0.0] s

  * igt@kms_async_flips@test-time-stamp:
    - Statuses : 7 skip(s)
    - Exec time: [0.0] s

  

Known issues
------------

  Here are the changes found in IGTPW_5011_full that come from known issues:

### IGT changes ###

#### Issues hit ####

  * igt@gem_exec_reloc@basic-many-active@rcs0:
    - shard-apl:          [PASS][3] -> [FAIL][4] ([i915#1635] / [i915#2389])
   [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9054/shard-apl2/igt@gem_exec_reloc@basic-many-active@rcs0.html
   [4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5011/shard-apl2/igt@gem_exec_reloc@basic-many-active@rcs0.html

  * igt@gem_exec_whisper@basic-contexts-forked:
    - shard-glk:          [PASS][5] -> [DMESG-WARN][6] ([i915#118] / [i915#95])
   [5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9054/shard-glk6/igt@gem_exec_whisper@basic-contexts-forked.html
   [6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5011/shard-glk7/igt@gem_exec_whisper@basic-contexts-forked.html

  * igt@kms_big_fb@y-tiled-64bpp-rotate-0:
    - shard-apl:          [PASS][7] -> [DMESG-WARN][8] ([i915#1635] / [i915#1982])
   [7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9054/shard-apl8/igt@kms_big_fb@y-tiled-64bpp-rotate-0.html
   [8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5011/shard-apl6/igt@kms_big_fb@y-tiled-64bpp-rotate-0.html

  * igt@kms_flip@2x-plain-flip-ts-check@ab-vga1-hdmi-a1:
    - shard-hsw:          [PASS][9] -> [DMESG-WARN][10] ([i915#1982])
   [9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9054/shard-hsw7/igt@kms_flip@2x-plain-flip-ts-check@ab-vga1-hdmi-a1.html
   [10]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5011/shard-hsw6/igt@kms_flip@2x-plain-flip-ts-check@ab-vga1-hdmi-a1.html

  * igt@kms_flip@flip-vs-expired-vblank-interruptible@c-dp1:
    - shard-apl:          [PASS][11] -> [FAIL][12] ([i915#1635] / [i915#79])
   [11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9054/shard-apl1/igt@kms_flip@flip-vs-expired-vblank-interruptible@c-dp1.html
   [12]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5011/shard-apl2/igt@kms_flip@flip-vs-expired-vblank-interruptible@c-dp1.html

  * igt@kms_flip@flip-vs-suspend@c-hdmi-a1:
    - shard-hsw:          [PASS][13] -> [INCOMPLETE][14] ([i915#2055])
   [13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9054/shard-hsw4/igt@kms_flip@flip-vs-suspend@c-hdmi-a1.html
   [14]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5011/shard-hsw1/igt@kms_flip@flip-vs-suspend@c-hdmi-a1.html

  * igt@kms_flip@plain-flip-fb-recreate-interruptible@a-hdmi-a1:
    - shard-glk:          [PASS][15] -> [FAIL][16] ([i915#2122])
   [15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9054/shard-glk1/igt@kms_flip@plain-flip-fb-recreate-interruptible@a-hdmi-a1.html
   [16]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5011/shard-glk4/igt@kms_flip@plain-flip-fb-recreate-interruptible@a-hdmi-a1.html

  * igt@kms_hdr@bpc-switch-suspend:
    - shard-kbl:          [PASS][17] -> [DMESG-WARN][18] ([i915#180])
   [17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9054/shard-kbl1/igt@kms_hdr@bpc-switch-suspend.html
   [18]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5011/shard-kbl6/igt@kms_hdr@bpc-switch-suspend.html

  * igt@kms_pipe_crc_basic@read-crc-pipe-a-frame-sequence:
    - shard-tglb:         [PASS][19] -> [DMESG-WARN][20] ([i915#1982]) +6 similar issues
   [19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9054/shard-tglb5/igt@kms_pipe_crc_basic@read-crc-pipe-a-frame-sequence.html
   [20]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5011/shard-tglb2/igt@kms_pipe_crc_basic@read-crc-pipe-a-frame-sequence.html

  * igt@kms_psr@psr2_cursor_mmap_cpu:
    - shard-iclb:         [PASS][21] -> [SKIP][22] ([fdo#109441]) +2 similar issues
   [21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9054/shard-iclb2/igt@kms_psr@psr2_cursor_mmap_cpu.html
   [22]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5011/shard-iclb8/igt@kms_psr@psr2_cursor_mmap_cpu.html

  
#### Possible fixes ####

  * igt@gem_exec_endless@dispatch@vcs0:
    - shard-iclb:         [INCOMPLETE][23] ([i915#1958]) -> [PASS][24]
   [23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9054/shard-iclb6/igt@gem_exec_endless@dispatch@vcs0.html
   [24]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5011/shard-iclb5/igt@gem_exec_endless@dispatch@vcs0.html

  * igt@gem_exec_reloc@basic-many-active@vecs0:
    - shard-glk:          [FAIL][25] ([i915#2389]) -> [PASS][26] +2 similar issues
   [25]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9054/shard-glk2/igt@gem_exec_reloc@basic-many-active@vecs0.html
   [26]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5011/shard-glk3/igt@gem_exec_reloc@basic-many-active@vecs0.html

  * igt@gen9_exec_parse@allowed-all:
    - shard-kbl:          [DMESG-WARN][27] ([i915#1436] / [i915#716]) -> [PASS][28]
   [27]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9054/shard-kbl2/igt@gen9_exec_parse@allowed-all.html
   [28]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5011/shard-kbl2/igt@gen9_exec_parse@allowed-all.html

  * igt@i915_pm_dc@dc6-psr:
    - shard-iclb:         [FAIL][29] ([i915#1899]) -> [PASS][30]
   [29]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9054/shard-iclb3/igt@i915_pm_dc@dc6-psr.html
   [30]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5011/shard-iclb8/igt@i915_pm_dc@dc6-psr.html

  * igt@kms_cursor_legacy@2x-long-flip-vs-cursor-atomic:
    - shard-glk:          [FAIL][31] ([i915#72]) -> [PASS][32]
   [31]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9054/shard-glk9/igt@kms_cursor_legacy@2x-long-flip-vs-cursor-atomic.html
   [32]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5011/shard-glk4/igt@kms_cursor_legacy@2x-long-flip-vs-cursor-atomic.html

  * igt@kms_flip@flip-vs-expired-vblank-interruptible@c-hdmi-a2:
    - shard-glk:          [FAIL][33] ([i915#79]) -> [PASS][34]
   [33]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9054/shard-glk5/igt@kms_flip@flip-vs-expired-vblank-interruptible@c-hdmi-a2.html
   [34]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5011/shard-glk3/igt@kms_flip@flip-vs-expired-vblank-interruptible@c-hdmi-a2.html

  * igt@kms_flip@flip-vs-suspend-interruptible@c-hdmi-a1:
    - shard-hsw:          [INCOMPLETE][35] ([i915#2055]) -> [PASS][36]
   [35]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9054/shard-hsw6/igt@kms_flip@flip-vs-suspend-interruptible@c-hdmi-a1.html
   [36]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5011/shard-hsw4/igt@kms_flip@flip-vs-suspend-interruptible@c-hdmi-a1.html

  * igt@kms_flip@flip-vs-suspend@b-vga1:
    - shard-snb:          [INCOMPLETE][37] ([i915#82]) -> [PASS][38]
   [37]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9054/shard-snb1/igt@kms_flip@flip-vs-suspend@b-vga1.html
   [38]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5011/shard-snb7/igt@kms_flip@flip-vs-suspend@b-vga1.html

  * igt@kms_flip@plain-flip-fb-recreate-interruptible@b-dp1:
    - shard-kbl:          [FAIL][39] ([i915#2122]) -> [PASS][40]
   [39]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9054/shard-kbl7/igt@kms_flip@plain-flip-fb-recreate-interruptible@b-dp1.html
   [40]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5011/shard-kbl7/igt@kms_flip@plain-flip-fb-recreate-interruptible@b-dp1.html

  * igt@kms_frontbuffer_tracking@fbc-1p-primscrn-spr-indfb-fullscreen:
    - shard-tglb:         [DMESG-WARN][41] ([i915#1982]) -> [PASS][42] +3 similar issues
   [41]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9054/shard-tglb3/igt@kms_frontbuffer_tracking@fbc-1p-primscrn-spr-indfb-fullscreen.html
   [42]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5011/shard-tglb5/igt@kms_frontbuffer_tracking@fbc-1p-primscrn-spr-indfb-fullscreen.html

  * igt@kms_psr@psr2_sprite_plane_move:
    - shard-iclb:         [SKIP][43] ([fdo#109441]) -> [PASS][44] +2 similar issues
   [43]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9054/shard-iclb5/igt@kms_psr@psr2_sprite_plane_move.html
   [44]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5011/shard-iclb2/igt@kms_psr@psr2_sprite_plane_move.html

  * igt@kms_setmode@basic:
    - shard-apl:          [FAIL][45] ([i915#1635] / [i915#31]) -> [PASS][46]
   [45]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9054/shard-apl4/igt@kms_setmode@basic.html
   [46]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5011/shard-apl7/igt@kms_setmode@basic.html

  * igt@kms_vblank@pipe-a-ts-continuation-suspend:
    - shard-kbl:          [DMESG-WARN][47] ([i915#180]) -> [PASS][48] +2 similar issues
   [47]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9054/shard-kbl4/igt@kms_vblank@pipe-a-ts-continuation-suspend.html
   [48]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5011/shard-kbl2/igt@kms_vblank@pipe-a-ts-continuation-suspend.html

  * igt@kms_vblank@pipe-b-query-busy:
    - shard-apl:          [DMESG-WARN][49] ([i915#1635] / [i915#1982]) -> [PASS][50]
   [49]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9054/shard-apl8/igt@kms_vblank@pipe-b-query-busy.html
   [50]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5011/shard-apl8/igt@kms_vblank@pipe-b-query-busy.html

  * igt@perf@polling-small-buf:
    - shard-iclb:         [FAIL][51] ([i915#1722]) -> [PASS][52]
   [51]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9054/shard-iclb3/igt@perf@polling-small-buf.html
   [52]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5011/shard-iclb2/igt@perf@polling-small-buf.html

  * igt@perf_pmu@module-unload:
    - shard-iclb:         [DMESG-WARN][53] ([i915#1982]) -> [PASS][54]
   [53]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9054/shard-iclb3/igt@perf_pmu@module-unload.html
   [54]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5011/shard-iclb1/igt@perf_pmu@module-unload.html

  
  {name}: This element is suppressed. This means it is ignored when computing
          the status of the difference (SUCCESS, WARNING, or FAILURE).

  [fdo#109441]: https://bugs.freedesktop.org/show_bug.cgi?id=109441
  [i915#118]: https://gitlab.freedesktop.org/drm/intel/issues/118
  [i915#1436]: https://gitlab.freedesktop.org/drm/intel/issues/1436
  [i915#1635]: https://gitlab.freedesktop.org/drm/intel/issues/1635
  [i915#1722]: https://gitlab.freedesktop.org/drm/intel/issues/1722
  [i915#180]: https://gitlab.freedesktop.org/drm/intel/issues/180
  [i915#1899]: https://gitlab.freedesktop.org/drm/intel/issues/1899
  [i915#1958]: https://gitlab.freedesktop.org/drm/intel/issues/1958
  [i915#1982]: https://gitlab.freedesktop.org/drm/intel/issues/1982
  [i915#2055]: https://gitlab.freedesktop.org/drm/intel/issues/2055
  [i915#2122]: https://gitlab.freedesktop.org/drm/intel/issues/2122
  [i915#2389]: https://gitlab.freedesktop.org/drm/intel/issues/2389
  [i915#2469]: https://gitlab.freedesktop.org/drm/intel/issues/2469
  [i915#2476]: https://gitlab.freedesktop.org/drm/intel/issues/2476
  [i915#31]: https://gitlab.freedesktop.org/drm/intel/issues/31
  [i915#716]: https://gitlab.freedesktop.org/drm/intel/issues/716
  [i915#72]: https://gitlab.freedesktop.org/drm/intel/issues/72
  [i915#79]: https://gitlab.freedesktop.org/drm/intel/issues/79
  [i915#82]: https://gitlab.freedesktop.org/drm/intel/issues/82
  [i915#95]: https://gitlab.freedesktop.org/drm/intel/issues/95


Participating hosts (10 -> 8)
------------------------------

  Missing    (2): pig-skl-6260u pig-glk-j5005 


Build changes
-------------

  * CI: CI-20190529 -> None
  * IGT: IGT_5787 -> IGTPW_5011
  * Piglit: piglit_4509 -> None

  CI-20190529: 20190529
  CI_DRM_9054: 04b9e672bb8daa5e9ac3a41abf5fc26d75ef9b51 @ git://anongit.freedesktop.org/gfx-ci/linux
  IGTPW_5011: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5011/index.html
  IGT_5787: 0ec962017c8131de14e0cb038f7f76b1f17ed637 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
  piglit_4509: fdc5a4ca11124ab8413c7988896eec4c97336694 @ git://anongit.freedesktop.org/piglit

== Logs ==

For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5011/index.html

[-- Attachment #1.2: Type: text/html, Size: 15850 bytes --]

[-- Attachment #2: Type: text/plain, Size: 154 bytes --]

_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev