All of lore.kernel.org
 help / color / mirror / Atom feed
* [igt-dev] [PATCH i-g-t] lib/igt_device_scan: Add slot selector
@ 2021-01-04  9:34 Łukasz Łaguna
  2021-01-04 10:21 ` [igt-dev] ✓ Fi.CI.BAT: success for " Patchwork
                   ` (2 more replies)
  0 siblings, 3 replies; 5+ messages in thread
From: Łukasz Łaguna @ 2021-01-04  9:34 UTC (permalink / raw)
  To: igt-dev

Add selector for direct device selection, based on hardware (PCIe) path.
It allows to choose device in specified domain, bus, slot and function.

Example:
gem_exec_basic --device "pci:slot=0000:01:00.0"

Signed-off-by: Łukasz Łaguna <lukasz.laguna@intel.com>
---
 lib/igt_device_scan.c | 18 +++++++++++++++---
 1 file changed, 15 insertions(+), 3 deletions(-)

diff --git a/lib/igt_device_scan.c b/lib/igt_device_scan.c
index b3b2bf4d..4704ba6f 100644
--- a/lib/igt_device_scan.c
+++ b/lib/igt_device_scan.c
@@ -83,9 +83,9 @@
  *   device selection, e.g. in automated execution setting. In such scenarios
  *   please consider using sys, pci or platform filters instead.
  *
- * - pci: select device using PCI vendor and device properties
+ * - pci: select device using PCI slot or vendor and device properties
  *   |[<!-- language="plain" -->
- *   pci:[vendor=%04x/name][,device=%04x][,card=%d]
+ *   pci:[vendor=%04x/name][,device=%04x][,card=%d] | [slot=%04x:%02x:%02x.%x]
  *   ]|
  *
  *   Filter allows device selection using vendor (hex or name), device id
@@ -117,6 +117,12 @@
  *
  *   It selects the second one.
  *
+ *   Another possibility is to select device using a PCI slot:
+ *
+ *   |[<!-- language="plain" -->
+ *   pci:slot=0000:01:00.0
+ *   ]|
+ *
  *   As order the on PCI bus doesn't change (unless you'll add new device or
  *   reorder existing one) device selection using this filter will always
  *   return you same device regardless the order of enumeration.
@@ -1138,6 +1144,7 @@ struct filter {
 		char *vendor;
 		char *device;
 		char *card;
+		char *slot;
 		char *drm;
 		char *driver;
 	} data;
@@ -1154,6 +1161,7 @@ static void fill_filter_data(struct filter *filter, const char *key, const char
 	__fill_key(vendor);
 	__fill_key(device);
 	__fill_key(card);
+	__fill_key(slot);
 	__fill_key(drm);
 	__fill_key(driver);
 #undef __fill_key
@@ -1271,6 +1279,10 @@ static struct igt_list_head *filter_pci(const struct filter_class *fcls,
 		if (!is_pci_subsystem(dev))
 			continue;
 
+		/* Skip if 'slot' doesn't match */
+		if (filter->data.slot && !strequal(filter->data.slot, dev->pci_slot_name))
+			continue;
+
 		/* Skip if 'vendor' doesn't match (hex or name) */
 		if (filter->data.vendor && !is_vendor_matched(dev, filter->data.vendor))
 			continue;
@@ -1325,7 +1337,7 @@ static struct filter_class filter_definition_list[] = {
 	{
 		.name = "pci",
 		.filter_function = filter_pci,
-		.help = "pci:[vendor=%04x/name][,device=%04x][,card=%d]",
+		.help = "pci:[vendor=%04x/name][,device=%04x][,card=%d] | [slot=%04x:%02x:%02x.%x]",
 		.detail = "vendor is hex number or vendor name\n",
 	},
 	{
-- 
2.28.0

---------------------------------------------------------------------
Intel Technology Poland sp. z o.o.
ul. Sowackiego 173 | 80-298 Gdask | Sd Rejonowy Gdask Pnoc | VII Wydzia Gospodarczy Krajowego Rejestru Sdowego - KRS 101882 | NIP 957-07-52-316 | Kapita zakadowy 200.000 PLN.
Ta wiadomo wraz z zacznikami jest przeznaczona dla okrelonego adresata i moe zawiera informacje poufne. W razie przypadkowego otrzymania tej wiadomoci, prosimy o powiadomienie nadawcy oraz trwae jej usunicie; jakiekolwiek przegldanie lub rozpowszechnianie jest zabronione.
This e-mail and any attachments may contain confidential material for the sole use of the intended recipient(s). If you are not the intended recipient, please contact the sender and delete all copies; any review or distribution by others is strictly prohibited.
 
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

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

* [igt-dev] ✓ Fi.CI.BAT: success for lib/igt_device_scan: Add slot selector
  2021-01-04  9:34 [igt-dev] [PATCH i-g-t] lib/igt_device_scan: Add slot selector Łukasz Łaguna
@ 2021-01-04 10:21 ` Patchwork
  2021-01-04 11:55 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
  2021-01-04 13:33 ` [igt-dev] [PATCH i-g-t] " Zbigniew Kempczyński
  2 siblings, 0 replies; 5+ messages in thread
From: Patchwork @ 2021-01-04 10:21 UTC (permalink / raw)
  To: Łukasz Łaguna; +Cc: igt-dev


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

== Series Details ==

Series: lib/igt_device_scan: Add slot selector
URL   : https://patchwork.freedesktop.org/series/85435/
State : success

== Summary ==

CI Bug Log - changes from IGT_5941 -> IGTPW_5346
====================================================

Summary
-------

  **SUCCESS**

  No regressions found.

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

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

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

### IGT changes ###

#### Issues hit ####

  * igt@amdgpu/amd_basic@cs-gfx:
    - fi-hsw-4770:        NOTRUN -> [SKIP][1] ([fdo#109271]) +25 similar issues
   [1]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/fi-hsw-4770/igt@amdgpu/amd_basic@cs-gfx.html

  * igt@amdgpu/amd_basic@query-info:
    - fi-tgl-y:           NOTRUN -> [SKIP][2] ([fdo#109315] / [i915#2575])
   [2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/fi-tgl-y/igt@amdgpu/amd_basic@query-info.html

  * igt@core_hotunplug@unbind-rebind:
    - fi-hsw-4770:        NOTRUN -> [WARN][3] ([i915#2283])
   [3]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/fi-hsw-4770/igt@core_hotunplug@unbind-rebind.html

  * igt@debugfs_test@read_all_entries:
    - fi-tgl-y:           [PASS][4] -> [DMESG-WARN][5] ([i915#402]) +2 similar issues
   [4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5941/fi-tgl-y/igt@debugfs_test@read_all_entries.html
   [5]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/fi-tgl-y/igt@debugfs_test@read_all_entries.html

  * igt@gem_huc_copy@huc-copy:
    - fi-bxt-dsi:         NOTRUN -> [SKIP][6] ([fdo#109271] / [i915#2190])
   [6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/fi-bxt-dsi/igt@gem_huc_copy@huc-copy.html

  * igt@i915_module_load@reload:
    - fi-bxt-dsi:         NOTRUN -> [DMESG-WARN][7] ([i915#1982])
   [7]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/fi-bxt-dsi/igt@i915_module_load@reload.html

  * igt@kms_chamelium@common-hpd-after-suspend:
    - fi-bxt-dsi:         NOTRUN -> [SKIP][8] ([fdo#109271] / [fdo#111827]) +8 similar issues
   [8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/fi-bxt-dsi/igt@kms_chamelium@common-hpd-after-suspend.html
    - fi-hsw-4770:        NOTRUN -> [SKIP][9] ([fdo#109271] / [fdo#111827]) +8 similar issues
   [9]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/fi-hsw-4770/igt@kms_chamelium@common-hpd-after-suspend.html

  * igt@kms_force_connector_basic@force-load-detect:
    - fi-bxt-dsi:         NOTRUN -> [SKIP][10] ([fdo#109271]) +27 similar issues
   [10]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/fi-bxt-dsi/igt@kms_force_connector_basic@force-load-detect.html

  * igt@kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-d:
    - fi-bxt-dsi:         NOTRUN -> [SKIP][11] ([fdo#109271] / [i915#533])
   [11]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/fi-bxt-dsi/igt@kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-d.html

  * igt@runner@aborted:
    - fi-bdw-5557u:       NOTRUN -> [FAIL][12] ([i915#2029])
   [12]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/fi-bdw-5557u/igt@runner@aborted.html

  
#### Possible fixes ####

  * igt@kms_chamelium@dp-crc-fast:
    - fi-kbl-7500u:       [FAIL][13] ([i915#1161] / [i915#262]) -> [PASS][14]
   [13]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5941/fi-kbl-7500u/igt@kms_chamelium@dp-crc-fast.html
   [14]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/fi-kbl-7500u/igt@kms_chamelium@dp-crc-fast.html

  * igt@prime_vgem@basic-fence-flip:
    - fi-tgl-y:           [DMESG-WARN][15] ([i915#402]) -> [PASS][16] +2 similar issues
   [15]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5941/fi-tgl-y/igt@prime_vgem@basic-fence-flip.html
   [16]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/fi-tgl-y/igt@prime_vgem@basic-fence-flip.html

  
  [fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
  [fdo#109315]: https://bugs.freedesktop.org/show_bug.cgi?id=109315
  [fdo#111827]: https://bugs.freedesktop.org/show_bug.cgi?id=111827
  [i915#1161]: https://gitlab.freedesktop.org/drm/intel/issues/1161
  [i915#1982]: https://gitlab.freedesktop.org/drm/intel/issues/1982
  [i915#2029]: https://gitlab.freedesktop.org/drm/intel/issues/2029
  [i915#2190]: https://gitlab.freedesktop.org/drm/intel/issues/2190
  [i915#2283]: https://gitlab.freedesktop.org/drm/intel/issues/2283
  [i915#2575]: https://gitlab.freedesktop.org/drm/intel/issues/2575
  [i915#262]: https://gitlab.freedesktop.org/drm/intel/issues/262
  [i915#402]: https://gitlab.freedesktop.org/drm/intel/issues/402
  [i915#533]: https://gitlab.freedesktop.org/drm/intel/issues/533


Participating hosts (38 -> 36)
------------------------------

  Additional (2): fi-hsw-4770 fi-bxt-dsi 
  Missing    (4): fi-ilk-m540 fi-dg1-1 fi-bsw-cyan fi-bdw-samus 


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

  * CI: CI-20190529 -> None
  * IGT: IGT_5941 -> IGTPW_5346

  CI-20190529: 20190529
  CI_DRM_9540: 031143817afd9bb0877e723f741fb711d4d0c55b @ git://anongit.freedesktop.org/gfx-ci/linux
  IGTPW_5346: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/index.html
  IGT_5941: 58b135e66be4fa4db8f668fa5d125b31537cb9a6 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools

== Logs ==

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

[-- Attachment #1.2: Type: text/html, Size: 6684 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] 5+ messages in thread

* [igt-dev] ✓ Fi.CI.IGT: success for lib/igt_device_scan: Add slot selector
  2021-01-04  9:34 [igt-dev] [PATCH i-g-t] lib/igt_device_scan: Add slot selector Łukasz Łaguna
  2021-01-04 10:21 ` [igt-dev] ✓ Fi.CI.BAT: success for " Patchwork
@ 2021-01-04 11:55 ` Patchwork
  2021-01-04 13:33 ` [igt-dev] [PATCH i-g-t] " Zbigniew Kempczyński
  2 siblings, 0 replies; 5+ messages in thread
From: Patchwork @ 2021-01-04 11:55 UTC (permalink / raw)
  To: Łukasz Łaguna; +Cc: igt-dev


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

== Series Details ==

Series: lib/igt_device_scan: Add slot selector
URL   : https://patchwork.freedesktop.org/series/85435/
State : success

== Summary ==

CI Bug Log - changes from IGT_5941_full -> IGTPW_5346_full
====================================================

Summary
-------

  **SUCCESS**

  No regressions found.

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

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

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

### IGT changes ###

#### Issues hit ####

  * igt@device_reset@unbind-reset-rebind:
    - shard-glk:          [PASS][1] -> [DMESG-WARN][2] ([i915#2283])
   [1]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5941/shard-glk4/igt@device_reset@unbind-reset-rebind.html
   [2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-glk2/igt@device_reset@unbind-reset-rebind.html
    - shard-apl:          [PASS][3] -> [DMESG-WARN][4] ([i915#2283])
   [3]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5941/shard-apl3/igt@device_reset@unbind-reset-rebind.html
   [4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-apl7/igt@device_reset@unbind-reset-rebind.html
    - shard-kbl:          [PASS][5] -> [DMESG-WARN][6] ([i915#2283])
   [5]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5941/shard-kbl2/igt@device_reset@unbind-reset-rebind.html
   [6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-kbl7/igt@device_reset@unbind-reset-rebind.html
    - shard-tglb:         [PASS][7] -> [DMESG-WARN][8] ([i915#1602])
   [7]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5941/shard-tglb1/igt@device_reset@unbind-reset-rebind.html
   [8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-tglb8/igt@device_reset@unbind-reset-rebind.html
    - shard-iclb:         [PASS][9] -> [DMESG-WARN][10] ([i915#2283])
   [9]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5941/shard-iclb4/igt@device_reset@unbind-reset-rebind.html
   [10]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-iclb3/igt@device_reset@unbind-reset-rebind.html

  * igt@gem_ctx_persistence@engines-hang:
    - shard-hsw:          NOTRUN -> [SKIP][11] ([fdo#109271] / [i915#1099]) +4 similar issues
   [11]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-hsw4/igt@gem_ctx_persistence@engines-hang.html

  * igt@gem_ctx_sseu@mmap-args:
    - shard-kbl:          NOTRUN -> [SKIP][12] ([fdo#109271]) +11 similar issues
   [12]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-kbl7/igt@gem_ctx_sseu@mmap-args.html
    - shard-tglb:         NOTRUN -> [SKIP][13] ([i915#280])
   [13]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-tglb5/igt@gem_ctx_sseu@mmap-args.html

  * igt@gem_exec_reloc@basic-many-active@vcs1:
    - shard-iclb:         NOTRUN -> [FAIL][14] ([i915#2389])
   [14]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-iclb4/igt@gem_exec_reloc@basic-many-active@vcs1.html

  * igt@i915_pm_rpm@modeset-non-lpsp:
    - shard-iclb:         NOTRUN -> [SKIP][15] ([fdo#110892])
   [15]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-iclb8/igt@i915_pm_rpm@modeset-non-lpsp.html
    - shard-tglb:         NOTRUN -> [SKIP][16] ([fdo#111644] / [i915#1397] / [i915#2411])
   [16]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-tglb3/igt@i915_pm_rpm@modeset-non-lpsp.html

  * igt@kms_async_flips@test-time-stamp:
    - shard-tglb:         [PASS][17] -> [FAIL][18] ([i915#2597])
   [17]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5941/shard-tglb3/igt@kms_async_flips@test-time-stamp.html
   [18]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-tglb5/igt@kms_async_flips@test-time-stamp.html

  * igt@kms_atomic_transition@plane-toggle-modeset-transition:
    - shard-glk:          [PASS][19] -> [DMESG-WARN][20] ([i915#118] / [i915#95])
   [19]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5941/shard-glk8/igt@kms_atomic_transition@plane-toggle-modeset-transition.html
   [20]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-glk9/igt@kms_atomic_transition@plane-toggle-modeset-transition.html

  * igt@kms_big_fb@yf-tiled-addfb-size-overflow:
    - shard-tglb:         NOTRUN -> [SKIP][21] ([fdo#111615]) +1 similar issue
   [21]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-tglb7/igt@kms_big_fb@yf-tiled-addfb-size-overflow.html

  * igt@kms_chamelium@hdmi-hpd-with-enabled-mode:
    - shard-hsw:          NOTRUN -> [SKIP][22] ([fdo#109271] / [fdo#111827]) +18 similar issues
   [22]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-hsw4/igt@kms_chamelium@hdmi-hpd-with-enabled-mode.html

  * igt@kms_chamelium@hdmi-mode-timings:
    - shard-kbl:          NOTRUN -> [SKIP][23] ([fdo#109271] / [fdo#111827]) +1 similar issue
   [23]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-kbl2/igt@kms_chamelium@hdmi-mode-timings.html
    - shard-iclb:         NOTRUN -> [SKIP][24] ([fdo#109284] / [fdo#111827]) +1 similar issue
   [24]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-iclb5/igt@kms_chamelium@hdmi-mode-timings.html
    - shard-glk:          NOTRUN -> [SKIP][25] ([fdo#109271] / [fdo#111827]) +1 similar issue
   [25]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-glk7/igt@kms_chamelium@hdmi-mode-timings.html
    - shard-apl:          NOTRUN -> [SKIP][26] ([fdo#109271] / [fdo#111827]) +1 similar issue
   [26]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-apl4/igt@kms_chamelium@hdmi-mode-timings.html

  * igt@kms_color@pipe-c-legacy-gamma:
    - shard-kbl:          NOTRUN -> [FAIL][27] ([i915#71])
   [27]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-kbl1/igt@kms_color@pipe-c-legacy-gamma.html
    - shard-apl:          NOTRUN -> [FAIL][28] ([i915#71])
   [28]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-apl6/igt@kms_color@pipe-c-legacy-gamma.html

  * igt@kms_color_chamelium@pipe-a-ctm-red-to-blue:
    - shard-tglb:         NOTRUN -> [SKIP][29] ([fdo#109284] / [fdo#111827]) +1 similar issue
   [29]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-tglb7/igt@kms_color_chamelium@pipe-a-ctm-red-to-blue.html

  * igt@kms_cursor_crc@pipe-d-cursor-128x42-offscreen:
    - shard-iclb:         NOTRUN -> [SKIP][30] ([fdo#109278]) +1 similar issue
   [30]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-iclb2/igt@kms_cursor_crc@pipe-d-cursor-128x42-offscreen.html

  * igt@kms_flip@plain-flip-fb-recreate@c-edp1:
    - shard-tglb:         [PASS][31] -> [FAIL][32] ([i915#2122])
   [31]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5941/shard-tglb8/igt@kms_flip@plain-flip-fb-recreate@c-edp1.html
   [32]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-tglb2/igt@kms_flip@plain-flip-fb-recreate@c-edp1.html

  * igt@kms_frontbuffer_tracking@fbc-2p-primscrn-pri-indfb-draw-render:
    - shard-iclb:         NOTRUN -> [SKIP][33] ([fdo#109280]) +3 similar issues
   [33]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-iclb5/igt@kms_frontbuffer_tracking@fbc-2p-primscrn-pri-indfb-draw-render.html

  * igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-cur-indfb-onoff:
    - shard-tglb:         NOTRUN -> [SKIP][34] ([fdo#111825]) +3 similar issues
   [34]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-tglb6/igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-cur-indfb-onoff.html

  * igt@kms_frontbuffer_tracking@fbcpsr-badstride:
    - shard-iclb:         [PASS][35] -> [FAIL][36] ([i915#49])
   [35]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5941/shard-iclb8/igt@kms_frontbuffer_tracking@fbcpsr-badstride.html
   [36]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-iclb5/igt@kms_frontbuffer_tracking@fbcpsr-badstride.html

  * igt@kms_frontbuffer_tracking@psr-1p-primscrn-shrfb-pgflip-blt:
    - shard-apl:          NOTRUN -> [SKIP][37] ([fdo#109271]) +11 similar issues
   [37]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-apl4/igt@kms_frontbuffer_tracking@psr-1p-primscrn-shrfb-pgflip-blt.html

  * igt@kms_frontbuffer_tracking@psr-rgb565-draw-mmap-cpu:
    - shard-glk:          NOTRUN -> [SKIP][38] ([fdo#109271]) +8 similar issues
   [38]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-glk1/igt@kms_frontbuffer_tracking@psr-rgb565-draw-mmap-cpu.html

  * igt@kms_plane_lowres@pipe-b-tiling-yf:
    - shard-hsw:          NOTRUN -> [SKIP][39] ([fdo#109271]) +277 similar issues
   [39]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-hsw4/igt@kms_plane_lowres@pipe-b-tiling-yf.html

  * igt@kms_psr@psr2_cursor_plane_onoff:
    - shard-iclb:         [PASS][40] -> [SKIP][41] ([fdo#109441]) +1 similar issue
   [40]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5941/shard-iclb2/igt@kms_psr@psr2_cursor_plane_onoff.html
   [41]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-iclb4/igt@kms_psr@psr2_cursor_plane_onoff.html

  * igt@kms_psr@psr2_suspend:
    - shard-iclb:         NOTRUN -> [SKIP][42] ([fdo#109441])
   [42]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-iclb7/igt@kms_psr@psr2_suspend.html

  * igt@perf@gen8-unprivileged-single-ctx-counters:
    - shard-tglb:         NOTRUN -> [SKIP][43] ([fdo#109289])
   [43]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-tglb2/igt@perf@gen8-unprivileged-single-ctx-counters.html

  * igt@perf@polling-parameterized:
    - shard-hsw:          [PASS][44] -> [FAIL][45] ([i915#1542])
   [44]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5941/shard-hsw4/igt@perf@polling-parameterized.html
   [45]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-hsw6/igt@perf@polling-parameterized.html

  * igt@prime_nv_api@i915_self_import_to_different_fd:
    - shard-tglb:         NOTRUN -> [SKIP][46] ([fdo#109291])
   [46]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-tglb8/igt@prime_nv_api@i915_self_import_to_different_fd.html
    - shard-iclb:         NOTRUN -> [SKIP][47] ([fdo#109291])
   [47]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-iclb8/igt@prime_nv_api@i915_self_import_to_different_fd.html

  
#### Possible fixes ####

  * {igt@gem_exec_fair@basic-deadline}:
    - shard-glk:          [FAIL][48] ([i915#2846]) -> [PASS][49]
   [48]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5941/shard-glk6/igt@gem_exec_fair@basic-deadline.html
   [49]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-glk5/igt@gem_exec_fair@basic-deadline.html

  * {igt@gem_exec_fair@basic-none-share@rcs0}:
    - shard-iclb:         [FAIL][50] ([i915#2842]) -> [PASS][51]
   [50]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5941/shard-iclb4/igt@gem_exec_fair@basic-none-share@rcs0.html
   [51]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-iclb5/igt@gem_exec_fair@basic-none-share@rcs0.html

  * {igt@gem_exec_fair@basic-none@rcs0}:
    - shard-glk:          [FAIL][52] ([i915#2842]) -> [PASS][53]
   [52]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5941/shard-glk1/igt@gem_exec_fair@basic-none@rcs0.html
   [53]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-glk5/igt@gem_exec_fair@basic-none@rcs0.html

  * {igt@gem_exec_fair@basic-none@vecs0}:
    - shard-kbl:          [FAIL][54] ([i915#2842]) -> [PASS][55] +1 similar issue
   [54]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5941/shard-kbl7/igt@gem_exec_fair@basic-none@vecs0.html
   [55]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-kbl2/igt@gem_exec_fair@basic-none@vecs0.html
    - shard-apl:          [FAIL][56] ([i915#2842]) -> [PASS][57]
   [56]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5941/shard-apl8/igt@gem_exec_fair@basic-none@vecs0.html
   [57]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-apl4/igt@gem_exec_fair@basic-none@vecs0.html

  * {igt@gem_exec_fair@basic-pace@vcs1}:
    - shard-kbl:          [SKIP][58] ([fdo#109271]) -> [PASS][59]
   [58]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5941/shard-kbl3/igt@gem_exec_fair@basic-pace@vcs1.html
   [59]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-kbl4/igt@gem_exec_fair@basic-pace@vcs1.html

  * {igt@gem_exec_fair@basic-pace@vecs0}:
    - shard-tglb:         [FAIL][60] ([i915#2842]) -> [PASS][61] +1 similar issue
   [60]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5941/shard-tglb2/igt@gem_exec_fair@basic-pace@vecs0.html
   [61]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-tglb3/igt@gem_exec_fair@basic-pace@vecs0.html

  * igt@gem_exec_whisper@basic-contexts-forked:
    - shard-glk:          [DMESG-WARN][62] ([i915#118] / [i915#95]) -> [PASS][63]
   [62]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5941/shard-glk6/igt@gem_exec_whisper@basic-contexts-forked.html
   [63]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-glk2/igt@gem_exec_whisper@basic-contexts-forked.html

  * {igt@gem_spin_batch@legacy-resubmit-bsd2}:
    - shard-iclb:         [SKIP][64] ([fdo#109276]) -> [PASS][65]
   [64]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5941/shard-iclb7/igt@gem_spin_batch@legacy-resubmit-bsd2.html
   [65]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-iclb1/igt@gem_spin_batch@legacy-resubmit-bsd2.html

  * {igt@gem_spin_batch@resubmit-new-all-vcs1}:
    - shard-iclb:         [SKIP][66] -> [PASS][67]
   [66]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5941/shard-iclb8/igt@gem_spin_batch@resubmit-new-all-vcs1.html
   [67]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-iclb4/igt@gem_spin_batch@resubmit-new-all-vcs1.html

  * igt@i915_pm_rpm@modeset-non-lpsp:
    - shard-hsw:          [SKIP][68] ([fdo#109271]) -> [PASS][69]
   [68]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5941/shard-hsw6/igt@i915_pm_rpm@modeset-non-lpsp.html
   [69]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-hsw4/igt@i915_pm_rpm@modeset-non-lpsp.html

  * igt@i915_suspend@fence-restore-untiled:
    - shard-hsw:          [DMESG-WARN][70] ([i915#2637]) -> [PASS][71]
   [70]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5941/shard-hsw6/igt@i915_suspend@fence-restore-untiled.html
   [71]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-hsw4/igt@i915_suspend@fence-restore-untiled.html

  * igt@kms_cursor_legacy@cursor-vs-flip-toggle:
    - shard-hsw:          [FAIL][72] ([i915#2370]) -> [PASS][73]
   [72]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5941/shard-hsw6/igt@kms_cursor_legacy@cursor-vs-flip-toggle.html
   [73]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-hsw6/igt@kms_cursor_legacy@cursor-vs-flip-toggle.html

  * igt@kms_hdmi_inject@inject-audio:
    - shard-tglb:         [SKIP][74] ([i915#433]) -> [PASS][75]
   [74]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5941/shard-tglb3/igt@kms_hdmi_inject@inject-audio.html
   [75]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-tglb2/igt@kms_hdmi_inject@inject-audio.html

  * igt@kms_psr@psr2_primary_page_flip:
    - shard-iclb:         [SKIP][76] ([fdo#109441]) -> [PASS][77] +2 similar issues
   [76]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5941/shard-iclb6/igt@kms_psr@psr2_primary_page_flip.html
   [77]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-iclb2/igt@kms_psr@psr2_primary_page_flip.html

  
#### Warnings ####

  * igt@i915_pm_rc6_residency@rc6-idle:
    - shard-iclb:         [WARN][78] ([i915#1804] / [i915#2684]) -> [WARN][79] ([i915#2684])
   [78]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5941/shard-iclb7/igt@i915_pm_rc6_residency@rc6-idle.html
   [79]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-iclb5/igt@i915_pm_rc6_residency@rc6-idle.html

  * igt@runner@aborted:
    - shard-iclb:         [FAIL][80] ([i915#2295] / [i915#2724]) -> ([FAIL][81], [FAIL][82]) ([i915#2283] / [i915#2295] / [i915#2724] / [i915#483])
   [80]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5941/shard-iclb4/igt@runner@aborted.html
   [81]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-iclb1/igt@runner@aborted.html
   [82]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-iclb3/igt@runner@aborted.html
    - shard-apl:          [FAIL][83] ([i915#2295]) -> ([FAIL][84], [FAIL][85], [FAIL][86]) ([i915#1610] / [i915#2295] / [i915#2426])
   [83]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5941/shard-apl2/igt@runner@aborted.html
   [84]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-apl7/igt@runner@aborted.html
   [85]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-apl7/igt@runner@aborted.html
   [86]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/shard-apl8/igt@runner@aborted.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#109276]: https://bugs.freedesktop.org/show_bug.cgi?id=109276
  [fdo#109278]: https://bugs.freedesktop.org/show_bug.cgi?id=109278
  [fdo#109280]: https://bugs.freedesktop.org/show_bug.cgi?id=109280
  [fdo#109284]: https://bugs.freedesktop.org/show_bug.cgi?id=109284
  [fdo#109289]: https://bugs.freedesktop.org/show_bug.cgi?id=109289
  [fdo#109291]: https://bugs.freedesktop.org/show_bug.cgi?id=109291
  [fdo#109441]: https://bugs.freedesktop.org/show_bug.cgi?id=109441
  [fdo#110892]: https://bugs.freedesktop.org/show_bug.cgi?id=110892
  [fdo#111615]: https://bugs.freedesktop.org/show_bug.cgi?id=111615
  [fdo#111644]: https://bugs.freedesktop.org/show_bug.cgi?id=111644
  [fdo#111825]: https://bugs.freedesktop.org/show_bug.cgi?id=111825
  [fdo#111827]: https://bugs.freedesktop.org/show_bug.cgi?id=111827
  [i915#1099]: https://gitlab.freedesktop.org/drm/intel/issues/1099
  [i915#118]: https://gitlab.freedesktop.org/drm/intel/issues/118
  [i915#1397]: https://gitlab.freedesktop.org/drm/intel/issues/1397
  [i915#1542]: https://gitlab.freedesktop.org/drm/intel/issues/1542
  [i915#1602]: https://gitlab.freedesktop.org/drm/intel/issues/1602
  [i915#1610]: https://gitlab.freedesktop.org/drm/intel/issues/1610
  [i915#1804]: https://gitlab.freedesktop.org/drm/intel/issues/1804
  [i915#2122]: https://gitlab.freedesktop.org/drm/intel/issues/2122
  [i915#2283]: https://gitlab.freedesktop.org/drm/intel/issues/2283
  [i915#2295]: https://gitlab.freedesktop.org/drm/intel/issues/2295
  [i915#2370]: https://gitlab.freedesktop.org/drm/intel/issues/2370
  [i915#2389]: https://gitlab.freedesktop.org/drm/intel/issues/2389
  [i915#2411]: https://gitlab.freedesktop.org/drm/intel/issues/2411
  [i915#2426]: https://gitlab.freedesktop.org/drm/intel/issues/2426
  [i915#2597]: https://gitlab.freedesktop.org/drm/intel/issues/2597
  [i915#2637]: https://gitlab.freedesktop.org/drm/intel/issues/2637
  [i915#2684]: https://gitlab.freedesktop.org/drm/intel/issues/2684
  [i915#2724]: https://gitlab.freedesktop.org/drm/intel/issues/2724
  [i915#280]: https://gitlab.freedesktop.org/drm/intel/issues/280
  [i915#2802]: https://gitlab.freedesktop.org/drm/intel/issues/2802
  [i915#2804]: https://gitlab.freedesktop.org/drm/intel/issues/2804
  [i915#2842]: https://gitlab.freedesktop.org/drm/intel/issues/2842
  [i915#2846]: https://gitlab.freedesktop.org/drm/intel/issues/2846
  [i915#433]: https://gitlab.freedesktop.org/drm/intel/issues/433
  [i915#483]: https://gitlab.freedesktop.org/drm/intel/issues/483
  [i915#49]: https://gitlab.freedesktop.org/drm/intel/issues/49
  [i915#71]: https://gitlab.freedesktop.org/drm/intel/issues/71
  [i915#95]: https://gitlab.freedesktop.org/drm/intel/issues/95


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

  No changes in participating hosts


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

  * CI: CI-20190529 -> None
  * IGT: IGT_5941 -> IGTPW_5346

  CI-20190529: 20190529
  CI_DRM_9540: 031143817afd9bb0877e723f741fb711d4d0c55b @ git://anongit.freedesktop.org/gfx-ci/linux
  IGTPW_5346: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5346/index.html
  IGT_5941: 58b135e66be4fa4db8f668fa5d125b31537cb9a6 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools

== Logs ==

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

[-- Attachment #1.2: Type: text/html, Size: 24013 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] 5+ messages in thread

* Re: [igt-dev] [PATCH i-g-t] lib/igt_device_scan: Add slot selector
  2021-01-04  9:34 [igt-dev] [PATCH i-g-t] lib/igt_device_scan: Add slot selector Łukasz Łaguna
  2021-01-04 10:21 ` [igt-dev] ✓ Fi.CI.BAT: success for " Patchwork
  2021-01-04 11:55 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
@ 2021-01-04 13:33 ` Zbigniew Kempczyński
  2021-01-05 13:43   ` Laguna, Lukasz
  2 siblings, 1 reply; 5+ messages in thread
From: Zbigniew Kempczyński @ 2021-01-04 13:33 UTC (permalink / raw)
  To: Łukasz Łaguna; +Cc: igt-dev

On Mon, Jan 04, 2021 at 10:34:41AM +0100, Łukasz Łaguna wrote:
> Add selector for direct device selection, based on hardware (PCIe) path.
> It allows to choose device in specified domain, bus, slot and function.
> 
> Example:
> gem_exec_basic --device "pci:slot=0000:01:00.0"
> 
> Signed-off-by: Łukasz Łaguna <lukasz.laguna@intel.com>
> ---
>  lib/igt_device_scan.c | 18 +++++++++++++++---
>  1 file changed, 15 insertions(+), 3 deletions(-)
> 
> diff --git a/lib/igt_device_scan.c b/lib/igt_device_scan.c
> index b3b2bf4d..4704ba6f 100644
> --- a/lib/igt_device_scan.c
> +++ b/lib/igt_device_scan.c
> @@ -83,9 +83,9 @@
>   *   device selection, e.g. in automated execution setting. In such scenarios
>   *   please consider using sys, pci or platform filters instead.
>   *
> - * - pci: select device using PCI vendor and device properties
> + * - pci: select device using PCI slot or vendor and device properties
>   *   |[<!-- language="plain" -->
> - *   pci:[vendor=%04x/name][,device=%04x][,card=%d]
> + *   pci:[vendor=%04x/name][,device=%04x][,card=%d] | [slot=%04x:%02x:%02x.%x]
>   *   ]|
>   *
>   *   Filter allows device selection using vendor (hex or name), device id
> @@ -117,6 +117,12 @@
>   *
>   *   It selects the second one.
>   *
> + *   Another possibility is to select device using a PCI slot:
> + *
> + *   |[<!-- language="plain" -->
> + *   pci:slot=0000:01:00.0
> + *   ]|
> + *
>   *   As order the on PCI bus doesn't change (unless you'll add new device or
>   *   reorder existing one) device selection using this filter will always
>   *   return you same device regardless the order of enumeration.
> @@ -1138,6 +1144,7 @@ struct filter {
>  		char *vendor;
>  		char *device;
>  		char *card;
> +		char *slot;
>  		char *drm;
>  		char *driver;
>  	} data;
> @@ -1154,6 +1161,7 @@ static void fill_filter_data(struct filter *filter, const char *key, const char
>  	__fill_key(vendor);
>  	__fill_key(device);
>  	__fill_key(card);
> +	__fill_key(slot);
>  	__fill_key(drm);
>  	__fill_key(driver);
>  #undef __fill_key
> @@ -1271,6 +1279,10 @@ static struct igt_list_head *filter_pci(const struct filter_class *fcls,
>  		if (!is_pci_subsystem(dev))
>  			continue;
>  
> +		/* Skip if 'slot' doesn't match */
> +		if (filter->data.slot && !strequal(filter->data.slot, dev->pci_slot_name))
> +			continue;
> +

selector - pci:[vendor=%04x/name][,device=%04x][,card=%d] | [slot=%04x:%02x:%02x.%x]

According to pci selector I assume you want to have:

1. pci:vendor=...,device=...
  
or 

2. pci:slot=xxxx:xx:xx.x 

So slot should be exclusive and no vendor/device/card parameters should be allowed
and vice versa. 

Now I'm able to pass:

pci:slot=0000:00:02.0,vendor=8081

but I expect vendor check shouldn't be processed here (passing slot gives you explicit
device and according to selector description these two are exclusive).

So - disable vendor/device/card if slot is used and slot when vendor/device/card
are used. I think you even should exit with an error in such case.

--
Zbigniew

>  		/* Skip if 'vendor' doesn't match (hex or name) */
>  		if (filter->data.vendor && !is_vendor_matched(dev, filter->data.vendor))
>  			continue;
> @@ -1325,7 +1337,7 @@ static struct filter_class filter_definition_list[] = {
>  	{
>  		.name = "pci",
>  		.filter_function = filter_pci,
> -		.help = "pci:[vendor=%04x/name][,device=%04x][,card=%d]",
> +		.help = "pci:[vendor=%04x/name][,device=%04x][,card=%d] | [slot=%04x:%02x:%02x.%x]",
>  		.detail = "vendor is hex number or vendor name\n",
>  	},
>  	{
> -- 
> 2.28.0
> 
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

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

* Re: [igt-dev] [PATCH i-g-t] lib/igt_device_scan: Add slot selector
  2021-01-04 13:33 ` [igt-dev] [PATCH i-g-t] " Zbigniew Kempczyński
@ 2021-01-05 13:43   ` Laguna, Lukasz
  0 siblings, 0 replies; 5+ messages in thread
From: Laguna, Lukasz @ 2021-01-05 13:43 UTC (permalink / raw)
  To: Kempczynski, Zbigniew; +Cc: igt-dev

> -----Original Message-----
> From: Zbigniew Kempczyński <zbigniew.kempczynski@intel.com>
> Sent: poniedziałek, 4 stycznia 2021 14:33
> To: Laguna, Lukasz <lukasz.laguna@intel.com>
> Cc: igt-dev@lists.freedesktop.org
> Subject: Re: [PATCH i-g-t] lib/igt_device_scan: Add slot selector
> 
> On Mon, Jan 04, 2021 at 10:34:41AM +0100, Łukasz Łaguna wrote:
> > Add selector for direct device selection, based on hardware (PCIe) path.
> > It allows to choose device in specified domain, bus, slot and function.
> >
> > Example:
> > gem_exec_basic --device "pci:slot=0000:01:00.0"
> >
> > Signed-off-by: Łukasz Łaguna <lukasz.laguna@intel.com>
> > ---
> >  lib/igt_device_scan.c | 18 +++++++++++++++---
> >  1 file changed, 15 insertions(+), 3 deletions(-)
> >
> > diff --git a/lib/igt_device_scan.c b/lib/igt_device_scan.c index
> > b3b2bf4d..4704ba6f 100644
> > --- a/lib/igt_device_scan.c
> > +++ b/lib/igt_device_scan.c
> > @@ -83,9 +83,9 @@
> >   *   device selection, e.g. in automated execution setting. In such scenarios
> >   *   please consider using sys, pci or platform filters instead.
> >   *
> > - * - pci: select device using PCI vendor and device properties
> > + * - pci: select device using PCI slot or vendor and device
> > + properties
> >   *   |[<!-- language="plain" -->
> > - *   pci:[vendor=%04x/name][,device=%04x][,card=%d]
> > + *   pci:[vendor=%04x/name][,device=%04x][,card=%d] |
> [slot=%04x:%02x:%02x.%x]
> >   *   ]|
> >   *
> >   *   Filter allows device selection using vendor (hex or name), device id
> > @@ -117,6 +117,12 @@
> >   *
> >   *   It selects the second one.
> >   *
> > + *   Another possibility is to select device using a PCI slot:
> > + *
> > + *   |[<!-- language="plain" -->
> > + *   pci:slot=0000:01:00.0
> > + *   ]|
> > + *
> >   *   As order the on PCI bus doesn't change (unless you'll add new device or
> >   *   reorder existing one) device selection using this filter will always
> >   *   return you same device regardless the order of enumeration.
> > @@ -1138,6 +1144,7 @@ struct filter {
> >  		char *vendor;
> >  		char *device;
> >  		char *card;
> > +		char *slot;
> >  		char *drm;
> >  		char *driver;
> >  	} data;
> > @@ -1154,6 +1161,7 @@ static void fill_filter_data(struct filter *filter, const
> char *key, const char
> >  	__fill_key(vendor);
> >  	__fill_key(device);
> >  	__fill_key(card);
> > +	__fill_key(slot);
> >  	__fill_key(drm);
> >  	__fill_key(driver);
> >  #undef __fill_key
> > @@ -1271,6 +1279,10 @@ static struct igt_list_head *filter_pci(const struct
> filter_class *fcls,
> >  		if (!is_pci_subsystem(dev))
> >  			continue;
> >
> > +		/* Skip if 'slot' doesn't match */
> > +		if (filter->data.slot && !strequal(filter->data.slot, dev-
> >pci_slot_name))
> > +			continue;
> > +
> 
> selector - pci:[vendor=%04x/name][,device=%04x][,card=%d] |
> [slot=%04x:%02x:%02x.%x]
> 
> According to pci selector I assume you want to have:
> 
> 1. pci:vendor=...,device=...
> 
> or
> 
> 2. pci:slot=xxxx:xx:xx.x
> 
> So slot should be exclusive and no vendor/device/card parameters should be
> allowed and vice versa.
> 
> Now I'm able to pass:
> 
> pci:slot=0000:00:02.0,vendor=8081
> 
> but I expect vendor check shouldn't be processed here (passing slot gives
> you explicit device and according to selector description these two are
> exclusive).
> 
> So - disable vendor/device/card if slot is used and slot when
> vendor/device/card are used. I think you even should exit with an error in
> such case.

Thanks for comments. I've fixed it and sent next revision.

--
Łukasz
---------------------------------------------------------------------
Intel Technology Poland sp. z o.o.
ul. Sowackiego 173 | 80-298 Gdask | Sd Rejonowy Gdask Pnoc | VII Wydzia Gospodarczy Krajowego Rejestru Sdowego - KRS 101882 | NIP 957-07-52-316 | Kapita zakadowy 200.000 PLN.
Ta wiadomo wraz z zacznikami jest przeznaczona dla okrelonego adresata i moe zawiera informacje poufne. W razie przypadkowego otrzymania tej wiadomoci, prosimy o powiadomienie nadawcy oraz trwae jej usunicie; jakiekolwiek przegldanie lub rozpowszechnianie jest zabronione.
This e-mail and any attachments may contain confidential material for the sole use of the intended recipient(s). If you are not the intended recipient, please contact the sender and delete all copies; any review or distribution by others is strictly prohibited.
 
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

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

end of thread, other threads:[~2021-01-05 13:43 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-01-04  9:34 [igt-dev] [PATCH i-g-t] lib/igt_device_scan: Add slot selector Łukasz Łaguna
2021-01-04 10:21 ` [igt-dev] ✓ Fi.CI.BAT: success for " Patchwork
2021-01-04 11:55 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
2021-01-04 13:33 ` [igt-dev] [PATCH i-g-t] " Zbigniew Kempczyński
2021-01-05 13:43   ` Laguna, Lukasz

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.