[igt-dev] [PATCH i-g-t] lib/igt_aux: Fix igt_mem_sleep doc

[igt-dev] [PATCH i-g-t] lib/igt_aux: Fix igt_mem_sleep doc

[Rodrigo Vivi]

I recently had updated the names to use the official names, but
forgot to update the doc.

CC: Ashutosh Dixit <ashutosh.dixit@intel.com>
Fixes: 2983efbdb1a ("lib/igt_aux: Let's use the official mem_sleep names")
Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
---
 lib/igt_aux.h | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/lib/igt_aux.h b/lib/igt_aux.h
index 89437a90..c8d487b6 100644
--- a/lib/igt_aux.h
+++ b/lib/igt_aux.h
@@ -197,9 +197,9 @@ enum igt_suspend_test {
 /**
  * igt_mem_sleep:
  * @MEM_SLEEP_NONE: no support
- * @MEM_SLEEP_FREEZE: suspend-to-idle target state, aka S0ix or freeze,
- * @MEM_SLEEP_STANDBY: standby target state, aka S1
- * @MEM_SLEEP_MEM: suspend-to-mem target state aka S3
+ * @MEM_SLEEP_S2IDLE: suspend-to-idle target state, aka S0ix or freeze,
+ * @MEM_SLEEP_SHALLOW: standby target state, aka S1
+ * @MEM_SLEEP_DEEP: suspend-to-mem target state aka S3
  */
 enum igt_mem_sleep {
 	MEM_SLEEP_NONE,
-- 
2.37.1

[igt-dev] ✗ GitLab.Pipeline: warning for lib/igt_aux: Fix igt_mem_sleep doc

[Patchwork]

== Series Details ==

Series: lib/igt_aux: Fix igt_mem_sleep doc
URL   : https://patchwork.freedesktop.org/series/107046/
State : warning

== Summary ==

Pipeline status: FAILED.

see https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/-/pipelines/657211 for the overview.

test:ninja-test-mips has failed (https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/-/jobs/26437133):
  Ok:                   22
  Expected Fail:         3
  Fail:                290
  Unexpected Pass:       0
  Skipped:               0
  Timeout:               0
  
  Full log written to /builds/gfx-ci/igt-ci-tags/build/meson-logs/testlog.txt
  section_end:1659739333:step_script
  section_start:1659739333:upload_artifacts_on_failure
  Uploading artifacts for failed job
  Uploading artifacts...
  build: found 1755 matching files and directories   
  Uploading artifacts as "archive" to coordinator... 201 Created  id=26437133 responseStatus=201 Created token=xnSRhpg6
  section_end:1659739340:upload_artifacts_on_failure
  section_start:1659739340:cleanup_file_variables
  Cleaning up project directory and file based variables
  section_end:1659739341:cleanup_file_variables
  ERROR: Job failed: exit code 1

== Logs ==

For more details see: https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/-/pipelines/657211

[igt-dev] ✗ Fi.CI.BAT: failure for lib/igt_aux: Fix igt_mem_sleep doc

[Patchwork]

[-- Attachment #1: Type: text/plain, Size: 7942 bytes --]

== Series Details ==

Series: lib/igt_aux: Fix igt_mem_sleep doc
URL   : https://patchwork.freedesktop.org/series/107046/
State : failure

== Summary ==

CI Bug Log - changes from IGT_6615 -> IGTPW_7617
====================================================

Summary
-------

  **FAILURE**

  Serious unknown changes coming with IGTPW_7617 absolutely need to be
  verified manually.
  
  If you think the reported changes have nothing to do with the changes
  introduced in IGTPW_7617, please notify your bug team to allow them
  to document this new failure mode, which will reduce false positives in CI.

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

Participating hosts (41 -> 41)
------------------------------

  Additional (1): fi-bsw-kefka 
  Missing    (1): bat-dg2-8 

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

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

### IGT changes ###

#### Possible regressions ####

  * igt@i915_selftest@live@gt_pm:
    - fi-apl-guc:         [PASS][1] -> [DMESG-FAIL][2]
   [1]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_6615/fi-apl-guc/igt@i915_selftest@live@gt_pm.html
   [2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7617/fi-apl-guc/igt@i915_selftest@live@gt_pm.html

  
#### Suppressed ####

  The following results come from untrusted machines, tests, or statuses.
  They do not affect the overall result.

  * igt@i915_pm_rpm@module-reload:
    - {bat-dg2-10}:       NOTRUN -> [DMESG-WARN][3]
   [3]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7617/bat-dg2-10/igt@i915_pm_rpm@module-reload.html

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

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

### IGT changes ###

#### Issues hit ####

  * igt@gem_lmem_swapping@parallel-random-engines:
    - fi-glk-j4005:       NOTRUN -> [SKIP][4] ([fdo#109271] / [i915#4613]) +3 similar issues
   [4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7617/fi-glk-j4005/igt@gem_lmem_swapping@parallel-random-engines.html

  * igt@i915_pm_rps@basic-api:
    - bat-dg1-6:          [PASS][5] -> [FAIL][6] ([i915#4032])
   [5]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_6615/bat-dg1-6/igt@i915_pm_rps@basic-api.html
   [6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7617/bat-dg1-6/igt@i915_pm_rps@basic-api.html

  * igt@i915_selftest@live@hangcheck:
    - fi-hsw-4770:        [PASS][7] -> [INCOMPLETE][8] ([i915#4785])
   [7]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_6615/fi-hsw-4770/igt@i915_selftest@live@hangcheck.html
   [8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7617/fi-hsw-4770/igt@i915_selftest@live@hangcheck.html

  * igt@kms_chamelium@common-hpd-after-suspend:
    - fi-hsw-g3258:       NOTRUN -> [SKIP][9] ([fdo#109271] / [fdo#111827])
   [9]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7617/fi-hsw-g3258/igt@kms_chamelium@common-hpd-after-suspend.html
    - fi-pnv-d510:        NOTRUN -> [SKIP][10] ([fdo#109271])
   [10]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7617/fi-pnv-d510/igt@kms_chamelium@common-hpd-after-suspend.html
    - fi-glk-j4005:       NOTRUN -> [SKIP][11] ([fdo#109271] / [fdo#111827])
   [11]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7617/fi-glk-j4005/igt@kms_chamelium@common-hpd-after-suspend.html

  * igt@kms_chamelium@hdmi-hpd-fast:
    - fi-bsw-kefka:       NOTRUN -> [SKIP][12] ([fdo#109271] / [fdo#111827]) +8 similar issues
   [12]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7617/fi-bsw-kefka/igt@kms_chamelium@hdmi-hpd-fast.html

  * igt@kms_cursor_legacy@basic-busy-flip-before-cursor@atomic-transitions-varying-size:
    - fi-bsw-kefka:       NOTRUN -> [FAIL][13] ([i915#6298])
   [13]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7617/fi-bsw-kefka/igt@kms_cursor_legacy@basic-busy-flip-before-cursor@atomic-transitions-varying-size.html

  * igt@prime_vgem@basic-fence-flip:
    - fi-bsw-kefka:       NOTRUN -> [SKIP][14] ([fdo#109271]) +16 similar issues
   [14]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7617/fi-bsw-kefka/igt@prime_vgem@basic-fence-flip.html

  * igt@runner@aborted:
    - fi-hsw-4770:        NOTRUN -> [FAIL][15] ([fdo#109271] / [i915#4312] / [i915#5594] / [i915#6246])
   [15]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7617/fi-hsw-4770/igt@runner@aborted.html

  
#### Possible fixes ####

  * igt@gem_exec_suspend@basic-s3@smem:
    - {bat-rplp-1}:       [DMESG-WARN][16] ([i915#2867]) -> [PASS][17]
   [16]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_6615/bat-rplp-1/igt@gem_exec_suspend@basic-s3@smem.html
   [17]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7617/bat-rplp-1/igt@gem_exec_suspend@basic-s3@smem.html

  * igt@i915_module_load@reload:
    - {bat-dg2-10}:       [DMESG-WARN][18] -> [PASS][19]
   [18]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_6615/bat-dg2-10/igt@i915_module_load@reload.html
   [19]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7617/bat-dg2-10/igt@i915_module_load@reload.html

  * igt@i915_selftest@live@hangcheck:
    - fi-hsw-g3258:       [INCOMPLETE][20] ([i915#3303] / [i915#4785]) -> [PASS][21]
   [20]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_6615/fi-hsw-g3258/igt@i915_selftest@live@hangcheck.html
   [21]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7617/fi-hsw-g3258/igt@i915_selftest@live@hangcheck.html
    - bat-dg1-5:          [DMESG-FAIL][22] ([i915#4494] / [i915#4957]) -> [PASS][23]
   [22]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_6615/bat-dg1-5/igt@i915_selftest@live@hangcheck.html
   [23]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7617/bat-dg1-5/igt@i915_selftest@live@hangcheck.html

  * igt@i915_selftest@live@requests:
    - fi-pnv-d510:        [DMESG-FAIL][24] ([i915#4528]) -> [PASS][25]
   [24]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_6615/fi-pnv-d510/igt@i915_selftest@live@requests.html
   [25]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7617/fi-pnv-d510/igt@i915_selftest@live@requests.html

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

  [fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
  [fdo#111827]: https://bugs.freedesktop.org/show_bug.cgi?id=111827
  [i915#2582]: https://gitlab.freedesktop.org/drm/intel/issues/2582
  [i915#2867]: https://gitlab.freedesktop.org/drm/intel/issues/2867
  [i915#3303]: https://gitlab.freedesktop.org/drm/intel/issues/3303
  [i915#4032]: https://gitlab.freedesktop.org/drm/intel/issues/4032
  [i915#4312]: https://gitlab.freedesktop.org/drm/intel/issues/4312
  [i915#4494]: https://gitlab.freedesktop.org/drm/intel/issues/4494
  [i915#4528]: https://gitlab.freedesktop.org/drm/intel/issues/4528
  [i915#4613]: https://gitlab.freedesktop.org/drm/intel/issues/4613
  [i915#4785]: https://gitlab.freedesktop.org/drm/intel/issues/4785
  [i915#4957]: https://gitlab.freedesktop.org/drm/intel/issues/4957
  [i915#4983]: https://gitlab.freedesktop.org/drm/intel/issues/4983
  [i915#5087]: https://gitlab.freedesktop.org/drm/intel/issues/5087
  [i915#5537]: https://gitlab.freedesktop.org/drm/intel/issues/5537
  [i915#5594]: https://gitlab.freedesktop.org/drm/intel/issues/5594
  [i915#5950]: https://gitlab.freedesktop.org/drm/intel/issues/5950
  [i915#6246]: https://gitlab.freedesktop.org/drm/intel/issues/6246
  [i915#6257]: https://gitlab.freedesktop.org/drm/intel/issues/6257
  [i915#6298]: https://gitlab.freedesktop.org/drm/intel/issues/6298


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

  * CI: CI-20190529 -> None
  * IGT: IGT_6615 -> IGTPW_7617

  CI-20190529: 20190529
  CI_DRM_11971: 2bdae66c9988dd0f07633629c0a85383cfc05940 @ git://anongit.freedesktop.org/gfx-ci/linux
  IGTPW_7617: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7617/index.html
  IGT_6615: a98a66399db9939b16e1cb3b47055f400834affb @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git

== Logs ==

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

[-- Attachment #2: Type: text/html, Size: 9147 bytes --]

Re: [igt-dev] [PATCH i-g-t] lib/igt_aux: Fix igt_mem_sleep doc

[Dixit, Ashutosh]

On Fri, 05 Aug 2022 15:21:55 -0700, Rodrigo Vivi wrote:
>
> I recently had updated the names to use the official names, but
> forgot to update the doc.

Thanks!

Reviewed-by: Ashutosh Dixit <ashutosh.dixit@intel.com>

> CC: Ashutosh Dixit <ashutosh.dixit@intel.com>
> Fixes: 2983efbdb1a ("lib/igt_aux: Let's use the official mem_sleep names")
> Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
> ---
>  lib/igt_aux.h | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/lib/igt_aux.h b/lib/igt_aux.h
> index 89437a90..c8d487b6 100644
> --- a/lib/igt_aux.h
> +++ b/lib/igt_aux.h
> @@ -197,9 +197,9 @@ enum igt_suspend_test {
>  /**
>   * igt_mem_sleep:
>   * @MEM_SLEEP_NONE: no support
> - * @MEM_SLEEP_FREEZE: suspend-to-idle target state, aka S0ix or freeze,
> - * @MEM_SLEEP_STANDBY: standby target state, aka S1
> - * @MEM_SLEEP_MEM: suspend-to-mem target state aka S3
> + * @MEM_SLEEP_S2IDLE: suspend-to-idle target state, aka S0ix or freeze,
> + * @MEM_SLEEP_SHALLOW: standby target state, aka S1
> + * @MEM_SLEEP_DEEP: suspend-to-mem target state aka S3
>   */
>  enum igt_mem_sleep {
>	MEM_SLEEP_NONE,
> --
> 2.37.1
>