All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH i-g-t] tests/kms_flip: Add test to check suspend/resume
@ 2019-01-18 11:29 ` Shayenne Moura
  0 siblings, 0 replies; 8+ messages in thread
From: Shayenne Moura @ 2019-01-18 11:29 UTC (permalink / raw)
  To: Petri Latvala, Arkadiusz Hiler
  Cc: igt-dev, Daniel Vetter, intel-gfx, Rodrigo Siqueira

This patch adds one test to evaluate suspend/resume operations using kms_flip.

Signed-off-by: Shayenne Moura <shayenneluzmoura@gmail.com>
---
 tests/kms_flip.c | 1 +
 1 file changed, 1 insertion(+)
 mode change 100644 => 100755 tests/kms_flip.c

diff --git a/tests/kms_flip.c b/tests/kms_flip.c
old mode 100644
new mode 100755
index f28272dd..3ca2fdfc
--- a/tests/kms_flip.c
+++ b/tests/kms_flip.c
@@ -1567,6 +1567,7 @@ int main(int argc, char **argv)
 		{ 10, TEST_DPMS_OFF | TEST_DPMS | TEST_VBLANK_RACE, "dpms-vs-vblank-race" },
 		{ 10, TEST_MODESET | TEST_VBLANK_RACE, "modeset-vs-vblank-race" },
 		{ 0, TEST_BO_TOOBIG | TEST_NO_2X_OUTPUT, "bo-too-big" },
+		{ 30, TEST_FLIP | TEST_SUSPEND, "flip-vs-suspend" },
 	};
 	int i;
 
-- 
2.17.1

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply related	[flat|nested] 8+ messages in thread

* [igt-dev] [PATCH i-g-t] tests/kms_flip: Add test to check suspend/resume
@ 2019-01-18 11:29 ` Shayenne Moura
  0 siblings, 0 replies; 8+ messages in thread
From: Shayenne Moura @ 2019-01-18 11:29 UTC (permalink / raw)
  To: Petri Latvala, Arkadiusz Hiler; +Cc: igt-dev, intel-gfx

This patch adds one test to evaluate suspend/resume operations using kms_flip.

Signed-off-by: Shayenne Moura <shayenneluzmoura@gmail.com>
---
 tests/kms_flip.c | 1 +
 1 file changed, 1 insertion(+)
 mode change 100644 => 100755 tests/kms_flip.c

diff --git a/tests/kms_flip.c b/tests/kms_flip.c
old mode 100644
new mode 100755
index f28272dd..3ca2fdfc
--- a/tests/kms_flip.c
+++ b/tests/kms_flip.c
@@ -1567,6 +1567,7 @@ int main(int argc, char **argv)
 		{ 10, TEST_DPMS_OFF | TEST_DPMS | TEST_VBLANK_RACE, "dpms-vs-vblank-race" },
 		{ 10, TEST_MODESET | TEST_VBLANK_RACE, "modeset-vs-vblank-race" },
 		{ 0, TEST_BO_TOOBIG | TEST_NO_2X_OUTPUT, "bo-too-big" },
+		{ 30, TEST_FLIP | TEST_SUSPEND, "flip-vs-suspend" },
 	};
 	int i;
 
-- 
2.17.1

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

^ permalink raw reply related	[flat|nested] 8+ messages in thread

* [igt-dev] ✓ Fi.CI.BAT: success for tests/kms_flip: Add test to check suspend/resume
  2019-01-18 11:29 ` [igt-dev] " Shayenne Moura
  (?)
@ 2019-01-18 12:06 ` Patchwork
  -1 siblings, 0 replies; 8+ messages in thread
From: Patchwork @ 2019-01-18 12:06 UTC (permalink / raw)
  To: Shayenne Moura; +Cc: igt-dev

== Series Details ==

Series: tests/kms_flip: Add test to check suspend/resume
URL   : https://patchwork.freedesktop.org/series/55403/
State : success

== Summary ==

CI Bug Log - changes from CI_DRM_5447 -> IGTPW_2260
====================================================

Summary
-------

  **SUCCESS**

  No regressions found.

  External URL: https://patchwork.freedesktop.org/api/1.0/series/55403/revisions/1/mbox/

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

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

### IGT changes ###

#### Issues hit ####

  * igt@amdgpu/amd_basic@cs-compute:
    - fi-kbl-8809g:       NOTRUN -> FAIL [fdo#108094]

  * igt@amdgpu/amd_prime@amd-to-i915:
    - fi-kbl-8809g:       NOTRUN -> FAIL [fdo#107341]

  * igt@i915_selftest@live_hangcheck:
    - fi-bwr-2160:        PASS -> DMESG-FAIL [fdo#108735]

  * igt@kms_frontbuffer_tracking@basic:
    - fi-hsw-peppy:       PASS -> DMESG-WARN [fdo#102614]

  * igt@kms_pipe_crc_basic@read-crc-pipe-b-frame-sequence:
    - fi-byt-clapper:     PASS -> FAIL [fdo#103191] / [fdo#107362] +1

  * igt@prime_vgem@basic-fence-flip:
    - fi-gdg-551:         PASS -> FAIL [fdo#103182]

  
#### Possible fixes ####

  * igt@amdgpu/amd_basic@userptr:
    - fi-kbl-8809g:       DMESG-WARN [fdo#108965] -> PASS

  * igt@gem_exec_suspend@basic-s4-devices:
    - fi-blb-e6850:       INCOMPLETE [fdo#107718] -> PASS

  * igt@kms_pipe_crc_basic@nonblocking-crc-pipe-a:
    - fi-byt-clapper:     FAIL [fdo#107362] -> PASS

  * igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a:
    - fi-byt-clapper:     FAIL [fdo#103191] / [fdo#107362] -> PASS +2

  * igt@prime_vgem@basic-fence-flip:
    - fi-ilk-650:         FAIL [fdo#104008] -> PASS

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

  [fdo#102614]: https://bugs.freedesktop.org/show_bug.cgi?id=102614
  [fdo#103182]: https://bugs.freedesktop.org/show_bug.cgi?id=103182
  [fdo#103191]: https://bugs.freedesktop.org/show_bug.cgi?id=103191
  [fdo#104008]: https://bugs.freedesktop.org/show_bug.cgi?id=104008
  [fdo#107341]: https://bugs.freedesktop.org/show_bug.cgi?id=107341
  [fdo#107362]: https://bugs.freedesktop.org/show_bug.cgi?id=107362
  [fdo#107718]: https://bugs.freedesktop.org/show_bug.cgi?id=107718
  [fdo#108094]: https://bugs.freedesktop.org/show_bug.cgi?id=108094
  [fdo#108735]: https://bugs.freedesktop.org/show_bug.cgi?id=108735
  [fdo#108965]: https://bugs.freedesktop.org/show_bug.cgi?id=108965
  [fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
  [fdo#109278]: https://bugs.freedesktop.org/show_bug.cgi?id=109278


Participating hosts (47 -> 44)
------------------------------

  Additional (1): fi-skl-6260u 
  Missing    (4): fi-kbl-soraka fi-ilk-m540 fi-byt-squawks fi-bsw-cyan 


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

    * IGT: IGT_4778 -> IGTPW_2260

  CI_DRM_5447: 1fb7b31e074e222be369480ecef1c5c3a48991c9 @ git://anongit.freedesktop.org/gfx-ci/linux
  IGTPW_2260: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2260/
  IGT_4778: 0c80bd7f2a964605f307e0220c6bb756685582b9 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools



== Testlist changes ==

+igt@kms_flip@2x-flip-vs-suspend
+igt@kms_flip@2x-flip-vs-suspend-interruptible
+igt@kms_flip@flip-vs-suspend
+igt@kms_flip@flip-vs-suspend-interruptible

== Logs ==

For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2260/
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [igt-dev] ✓ Fi.CI.IGT: success for tests/kms_flip: Add test to check suspend/resume
  2019-01-18 11:29 ` [igt-dev] " Shayenne Moura
  (?)
  (?)
@ 2019-01-18 17:43 ` Patchwork
  -1 siblings, 0 replies; 8+ messages in thread
From: Patchwork @ 2019-01-18 17:43 UTC (permalink / raw)
  To: Shayenne Moura; +Cc: igt-dev

== Series Details ==

Series: tests/kms_flip: Add test to check suspend/resume
URL   : https://patchwork.freedesktop.org/series/55403/
State : success

== Summary ==

CI Bug Log - changes from CI_DRM_5447_full -> IGTPW_2260_full
====================================================

Summary
-------

  **SUCCESS**

  No regressions found.

  External URL: https://patchwork.freedesktop.org/api/1.0/series/55403/revisions/1/mbox/

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

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

### IGT changes ###

#### Issues hit ####

  * igt@gem_exec_await@wide-all:
    - shard-apl:          NOTRUN -> INCOMPLETE [fdo#103927]

  * igt@gem_exec_whisper@normal:
    - shard-glk:          PASS -> INCOMPLETE [fdo#103359] / [k.org#198133]

  * igt@kms_busy@extended-pageflip-modeset-hang-oldfb-render-b:
    - shard-snb:          NOTRUN -> DMESG-WARN [fdo#107956]

  * igt@kms_ccs@pipe-a-crc-sprite-planes-basic:
    - shard-glk:          PASS -> FAIL [fdo#108145]
    - shard-kbl:          PASS -> FAIL [fdo#107725] / [fdo#108145]

  * igt@kms_color@pipe-b-legacy-gamma:
    - shard-apl:          PASS -> FAIL [fdo#104782]

  * igt@kms_cursor_crc@cursor-128x128-suspend:
    - shard-apl:          PASS -> FAIL [fdo#103191] / [fdo#103232]
    - shard-kbl:          PASS -> FAIL [fdo#103191] / [fdo#103232]

  * igt@kms_cursor_crc@cursor-64x21-sliding:
    - shard-apl:          PASS -> FAIL [fdo#103232] +5

  * igt@kms_cursor_crc@cursor-64x64-random:
    - shard-kbl:          PASS -> FAIL [fdo#103232] +2

  * igt@kms_cursor_crc@cursor-64x64-sliding:
    - shard-glk:          PASS -> FAIL [fdo#103232] +2

  * igt@kms_cursor_legacy@2x-long-nonblocking-modeset-vs-cursor-atomic:
    - shard-glk:          PASS -> FAIL [fdo#105454] / [fdo#106509]

  * igt@kms_draw_crc@draw-method-xrgb8888-mmap-wc-xtiled:
    - shard-glk:          PASS -> FAIL [fdo#107791]

  * igt@kms_frontbuffer_tracking@fbc-1p-primscrn-spr-indfb-draw-mmap-wc:
    - shard-apl:          PASS -> FAIL [fdo#103167]

  * igt@kms_frontbuffer_tracking@fbc-2p-primscrn-spr-indfb-draw-pwrite:
    - shard-glk:          PASS -> FAIL [fdo#103167] +2

  * igt@kms_plane@plane-position-covered-pipe-a-planes:
    - shard-apl:          PASS -> FAIL [fdo#103166] +4

  * igt@kms_plane_multiple@atomic-pipe-b-tiling-none:
    - shard-glk:          PASS -> FAIL [fdo#103166] +2

  * igt@kms_plane_multiple@atomic-pipe-c-tiling-x:
    - shard-kbl:          PASS -> FAIL [fdo#103166]

  
#### Possible fixes ####

  * igt@kms_available_modes_crc@available_mode_test_crc:
    - shard-apl:          FAIL [fdo#106641] -> PASS

  * igt@kms_color@pipe-a-ctm-max:
    - shard-apl:          FAIL [fdo#108147] -> PASS

  * igt@kms_color@pipe-b-degamma:
    - shard-apl:          FAIL [fdo#104782] -> PASS

  * igt@kms_cursor_crc@cursor-128x128-random:
    - shard-apl:          FAIL [fdo#103232] -> PASS +4

  * igt@kms_cursor_crc@cursor-256x256-suspend:
    - shard-apl:          FAIL [fdo#103191] / [fdo#103232] -> PASS

  * igt@kms_cursor_crc@cursor-64x21-random:
    - shard-kbl:          DMESG-WARN [fdo#103313] / [fdo#105345] -> PASS +1

  * igt@kms_cursor_crc@cursor-64x64-offscreen:
    - shard-hsw:          DMESG-FAIL [fdo#102614] / [fdo#103232] -> PASS +1

  * igt@kms_cursor_crc@cursor-64x64-onscreen:
    - shard-glk:          FAIL [fdo#103232] -> PASS +1

  * igt@kms_cursor_legacy@2x-long-flip-vs-cursor-legacy:
    - shard-glk:          FAIL [fdo#104873] -> PASS

  * igt@kms_cursor_legacy@cursora-vs-flipb-atomic-transitions:
    - shard-hsw:          DMESG-WARN [fdo#102614] -> PASS +2

  * igt@kms_fbcon_fbt@fbc-suspend:
    - shard-kbl:          INCOMPLETE [fdo#103665] -> PASS +1

  * igt@kms_frontbuffer_tracking@fbc-1p-primscrn-cur-indfb-draw-pwrite:
    - shard-apl:          FAIL [fdo#103167] -> PASS +1
    - shard-kbl:          FAIL [fdo#103167] -> PASS

  * igt@kms_frontbuffer_tracking@fbc-2p-primscrn-shrfb-plflip-blt:
    - shard-hsw:          DMESG-FAIL [fdo#102614] / [fdo#103167] -> PASS

  * igt@kms_frontbuffer_tracking@fbc-2p-primscrn-spr-indfb-onoff:
    - shard-glk:          FAIL [fdo#103167] -> PASS +4

  * igt@kms_plane@pixel-format-pipe-a-planes:
    - shard-kbl:          FAIL [fdo#103166] -> PASS

  * igt@kms_plane_alpha_blend@pipe-a-alpha-opaque-fb:
    - shard-apl:          FAIL [fdo#108145] -> PASS

  * igt@kms_plane_multiple@atomic-pipe-c-tiling-yf:
    - shard-apl:          FAIL [fdo#103166] -> PASS +2

  * igt@kms_rotation_crc@sprite-rotation-180:
    - shard-apl:          INCOMPLETE [fdo#103927] -> PASS

  * igt@kms_universal_plane@universal-plane-pipe-c-functional:
    - shard-glk:          FAIL [fdo#103166] -> PASS +3

  * igt@perf@oa-exponents:
    - shard-glk:          FAIL [fdo#105483] -> PASS

  * igt@testdisplay:
    - shard-kbl:          DMESG-WARN [fdo#105345] -> PASS

  
#### Warnings ####

  * igt@i915_suspend@shrink:
    - shard-kbl:          INCOMPLETE [fdo#103665] / [fdo#106886] -> DMESG-WARN [fdo#109244]

  * igt@kms_frontbuffer_tracking@fbc-2p-indfb-fliptrack:
    - shard-snb:          {SKIP} [fdo#109271] -> INCOMPLETE [fdo#105411]

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

  [fdo#102614]: https://bugs.freedesktop.org/show_bug.cgi?id=102614
  [fdo#103166]: https://bugs.freedesktop.org/show_bug.cgi?id=103166
  [fdo#103167]: https://bugs.freedesktop.org/show_bug.cgi?id=103167
  [fdo#103191]: https://bugs.freedesktop.org/show_bug.cgi?id=103191
  [fdo#103232]: https://bugs.freedesktop.org/show_bug.cgi?id=103232
  [fdo#103313]: https://bugs.freedesktop.org/show_bug.cgi?id=103313
  [fdo#103359]: https://bugs.freedesktop.org/show_bug.cgi?id=103359
  [fdo#103540]: https://bugs.freedesktop.org/show_bug.cgi?id=103540
  [fdo#103665]: https://bugs.freedesktop.org/show_bug.cgi?id=103665
  [fdo#103927]: https://bugs.freedesktop.org/show_bug.cgi?id=103927
  [fdo#104782]: https://bugs.freedesktop.org/show_bug.cgi?id=104782
  [fdo#104873]: https://bugs.freedesktop.org/show_bug.cgi?id=104873
  [fdo#105345]: https://bugs.freedesktop.org/show_bug.cgi?id=105345
  [fdo#105411]: https://bugs.freedesktop.org/show_bug.cgi?id=105411
  [fdo#105454]: https://bugs.freedesktop.org/show_bug.cgi?id=105454
  [fdo#105483]: https://bugs.freedesktop.org/show_bug.cgi?id=105483
  [fdo#106509]: https://bugs.freedesktop.org/show_bug.cgi?id=106509
  [fdo#106641]: https://bugs.freedesktop.org/show_bug.cgi?id=106641
  [fdo#106886]: https://bugs.freedesktop.org/show_bug.cgi?id=106886
  [fdo#107725]: https://bugs.freedesktop.org/show_bug.cgi?id=107725
  [fdo#107791]: https://bugs.freedesktop.org/show_bug.cgi?id=107791
  [fdo#107956]: https://bugs.freedesktop.org/show_bug.cgi?id=107956
  [fdo#108145]: https://bugs.freedesktop.org/show_bug.cgi?id=108145
  [fdo#108147]: https://bugs.freedesktop.org/show_bug.cgi?id=108147
  [fdo#109244]: https://bugs.freedesktop.org/show_bug.cgi?id=109244
  [fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
  [fdo#109278]: https://bugs.freedesktop.org/show_bug.cgi?id=109278
  [k.org#198133]: https://bugzilla.kernel.org/show_bug.cgi?id=198133


Participating hosts (7 -> 5)
------------------------------

  Missing    (2): shard-skl shard-iclb 


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

    * IGT: IGT_4778 -> IGTPW_2260
    * Piglit: piglit_4509 -> None

  CI_DRM_5447: 1fb7b31e074e222be369480ecef1c5c3a48991c9 @ git://anongit.freedesktop.org/gfx-ci/linux
  IGTPW_2260: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2260/
  IGT_4778: 0c80bd7f2a964605f307e0220c6bb756685582b9 @ 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_2260/
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH i-g-t] tests/kms_flip: Add test to check suspend/resume
  2019-01-18 11:29 ` [igt-dev] " Shayenne Moura
@ 2019-01-21  9:34   ` Rodrigo Siqueira
  -1 siblings, 0 replies; 8+ messages in thread
From: Rodrigo Siqueira @ 2019-01-21  9:34 UTC (permalink / raw)
  To: Shayenne Moura; +Cc: Daniel Vetter, intel-gfx, igt-dev


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

Hi,

On 01/18, Shayenne Moura wrote:
> This patch adds one test to evaluate suspend/resume operations using kms_flip.
> 
> Signed-off-by: Shayenne Moura <shayenneluzmoura@gmail.com>
> ---
>  tests/kms_flip.c | 1 +
>  1 file changed, 1 insertion(+)
>  mode change 100644 => 100755 tests/kms_flip.c
> 
> diff --git a/tests/kms_flip.c b/tests/kms_flip.c
> old mode 100644
> new mode 100755
> index f28272dd..3ca2fdfc
> --- a/tests/kms_flip.c
> +++ b/tests/kms_flip.c
> @@ -1567,6 +1567,7 @@ int main(int argc, char **argv)
>  		{ 10, TEST_DPMS_OFF | TEST_DPMS | TEST_VBLANK_RACE, "dpms-vs-vblank-race" },
>  		{ 10, TEST_MODESET | TEST_VBLANK_RACE, "modeset-vs-vblank-race" },
>  		{ 0, TEST_BO_TOOBIG | TEST_NO_2X_OUTPUT, "bo-too-big" },
> +		{ 30, TEST_FLIP | TEST_SUSPEND, "flip-vs-suspend" },

I remember to follow a conversation in the IRC that you said that VKMS
pass this test. I tried it here on my VM, and after running the test my
system got freeze. My VM uses Arch Linux with Kernel 5.0.0-rc1. Did I
miss something?

Also, I tested it on my host machine with i915 driver and I noticed that
the test took much more than 30 seconds to finish. Is it right?

Thanks
Best Regards

>  	};
>  	int i;
>  
> -- 
> 2.17.1
> 

-- 
Rodrigo Siqueira
https://siqueira.tech
Graduate Student
Department of Computer Science
University of São Paulo

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

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

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [igt-dev] [PATCH i-g-t] tests/kms_flip: Add test to check suspend/resume
@ 2019-01-21  9:34   ` Rodrigo Siqueira
  0 siblings, 0 replies; 8+ messages in thread
From: Rodrigo Siqueira @ 2019-01-21  9:34 UTC (permalink / raw)
  To: Shayenne Moura; +Cc: Petri Latvala, intel-gfx, igt-dev


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

Hi,

On 01/18, Shayenne Moura wrote:
> This patch adds one test to evaluate suspend/resume operations using kms_flip.
> 
> Signed-off-by: Shayenne Moura <shayenneluzmoura@gmail.com>
> ---
>  tests/kms_flip.c | 1 +
>  1 file changed, 1 insertion(+)
>  mode change 100644 => 100755 tests/kms_flip.c
> 
> diff --git a/tests/kms_flip.c b/tests/kms_flip.c
> old mode 100644
> new mode 100755
> index f28272dd..3ca2fdfc
> --- a/tests/kms_flip.c
> +++ b/tests/kms_flip.c
> @@ -1567,6 +1567,7 @@ int main(int argc, char **argv)
>  		{ 10, TEST_DPMS_OFF | TEST_DPMS | TEST_VBLANK_RACE, "dpms-vs-vblank-race" },
>  		{ 10, TEST_MODESET | TEST_VBLANK_RACE, "modeset-vs-vblank-race" },
>  		{ 0, TEST_BO_TOOBIG | TEST_NO_2X_OUTPUT, "bo-too-big" },
> +		{ 30, TEST_FLIP | TEST_SUSPEND, "flip-vs-suspend" },

I remember to follow a conversation in the IRC that you said that VKMS
pass this test. I tried it here on my VM, and after running the test my
system got freeze. My VM uses Arch Linux with Kernel 5.0.0-rc1. Did I
miss something?

Also, I tested it on my host machine with i915 driver and I noticed that
the test took much more than 30 seconds to finish. Is it right?

Thanks
Best Regards

>  	};
>  	int i;
>  
> -- 
> 2.17.1
> 

-- 
Rodrigo Siqueira
https://siqueira.tech
Graduate Student
Department of Computer Science
University of São Paulo

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 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

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH i-g-t] tests/kms_flip: Add test to check suspend/resume
  2019-01-21  9:34   ` [igt-dev] " Rodrigo Siqueira
@ 2019-01-21 10:10     ` Daniel Vetter
  -1 siblings, 0 replies; 8+ messages in thread
From: Daniel Vetter @ 2019-01-21 10:10 UTC (permalink / raw)
  To: Rodrigo Siqueira; +Cc: Daniel Vetter, intel-gfx, igt-dev, Shayenne Moura

On Mon, Jan 21, 2019 at 07:34:32AM -0200, Rodrigo Siqueira wrote:
> Hi,
> 
> On 01/18, Shayenne Moura wrote:
> > This patch adds one test to evaluate suspend/resume operations using kms_flip.
> > 
> > Signed-off-by: Shayenne Moura <shayenneluzmoura@gmail.com>
> > ---
> >  tests/kms_flip.c | 1 +
> >  1 file changed, 1 insertion(+)
> >  mode change 100644 => 100755 tests/kms_flip.c
> > 
> > diff --git a/tests/kms_flip.c b/tests/kms_flip.c
> > old mode 100644
> > new mode 100755
> > index f28272dd..3ca2fdfc
> > --- a/tests/kms_flip.c
> > +++ b/tests/kms_flip.c
> > @@ -1567,6 +1567,7 @@ int main(int argc, char **argv)
> >  		{ 10, TEST_DPMS_OFF | TEST_DPMS | TEST_VBLANK_RACE, "dpms-vs-vblank-race" },
> >  		{ 10, TEST_MODESET | TEST_VBLANK_RACE, "modeset-vs-vblank-race" },
> >  		{ 0, TEST_BO_TOOBIG | TEST_NO_2X_OUTPUT, "bo-too-big" },
> > +		{ 30, TEST_FLIP | TEST_SUSPEND, "flip-vs-suspend" },
> 
> I remember to follow a conversation in the IRC that you said that VKMS
> pass this test. I tried it here on my VM, and after running the test my
> system got freeze. My VM uses Arch Linux with Kernel 5.0.0-rc1. Did I
> miss something?
> 
> Also, I tested it on my host machine with i915 driver and I noticed that
> the test took much more than 30 seconds to finish. Is it right?

Yeah should probably reduce to 10 or something. The susend-resume will
take 10+ seconds at least, and we'lld do one per CRTC, so this is going to
take a while anyway. Note that the test timeout is per-CRTC iirc.
-Daniel

> 
> Thanks
> Best Regards
> 
> >  	};
> >  	int i;
> >  
> > -- 
> > 2.17.1
> > 
> 
> -- 
> Rodrigo Siqueira
> https://siqueira.tech
> Graduate Student
> Department of Computer Science
> University of São Paulo



-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [igt-dev] [PATCH i-g-t] tests/kms_flip: Add test to check suspend/resume
@ 2019-01-21 10:10     ` Daniel Vetter
  0 siblings, 0 replies; 8+ messages in thread
From: Daniel Vetter @ 2019-01-21 10:10 UTC (permalink / raw)
  To: Rodrigo Siqueira; +Cc: Petri Latvala, intel-gfx, igt-dev

On Mon, Jan 21, 2019 at 07:34:32AM -0200, Rodrigo Siqueira wrote:
> Hi,
> 
> On 01/18, Shayenne Moura wrote:
> > This patch adds one test to evaluate suspend/resume operations using kms_flip.
> > 
> > Signed-off-by: Shayenne Moura <shayenneluzmoura@gmail.com>
> > ---
> >  tests/kms_flip.c | 1 +
> >  1 file changed, 1 insertion(+)
> >  mode change 100644 => 100755 tests/kms_flip.c
> > 
> > diff --git a/tests/kms_flip.c b/tests/kms_flip.c
> > old mode 100644
> > new mode 100755
> > index f28272dd..3ca2fdfc
> > --- a/tests/kms_flip.c
> > +++ b/tests/kms_flip.c
> > @@ -1567,6 +1567,7 @@ int main(int argc, char **argv)
> >  		{ 10, TEST_DPMS_OFF | TEST_DPMS | TEST_VBLANK_RACE, "dpms-vs-vblank-race" },
> >  		{ 10, TEST_MODESET | TEST_VBLANK_RACE, "modeset-vs-vblank-race" },
> >  		{ 0, TEST_BO_TOOBIG | TEST_NO_2X_OUTPUT, "bo-too-big" },
> > +		{ 30, TEST_FLIP | TEST_SUSPEND, "flip-vs-suspend" },
> 
> I remember to follow a conversation in the IRC that you said that VKMS
> pass this test. I tried it here on my VM, and after running the test my
> system got freeze. My VM uses Arch Linux with Kernel 5.0.0-rc1. Did I
> miss something?
> 
> Also, I tested it on my host machine with i915 driver and I noticed that
> the test took much more than 30 seconds to finish. Is it right?

Yeah should probably reduce to 10 or something. The susend-resume will
take 10+ seconds at least, and we'lld do one per CRTC, so this is going to
take a while anyway. Note that the test timeout is per-CRTC iirc.
-Daniel

> 
> Thanks
> Best Regards
> 
> >  	};
> >  	int i;
> >  
> > -- 
> > 2.17.1
> > 
> 
> -- 
> Rodrigo Siqueira
> https://siqueira.tech
> Graduate Student
> Department of Computer Science
> University of São Paulo



-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2019-01-21 10:10 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-01-18 11:29 [PATCH i-g-t] tests/kms_flip: Add test to check suspend/resume Shayenne Moura
2019-01-18 11:29 ` [igt-dev] " Shayenne Moura
2019-01-18 12:06 ` [igt-dev] ✓ Fi.CI.BAT: success for " Patchwork
2019-01-18 17:43 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
2019-01-21  9:34 ` [PATCH i-g-t] " Rodrigo Siqueira
2019-01-21  9:34   ` [igt-dev] " Rodrigo Siqueira
2019-01-21 10:10   ` Daniel Vetter
2019-01-21 10:10     ` [igt-dev] " Daniel Vetter

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.