diff options
| author | Zbigniew Kempczyński <zbigniew.kempczynski@intel.com> | 2022-08-02 12:22:57 +0200 |
|---|---|---|
| committer | Zbigniew Kempczyński <zbigniew.kempczynski@intel.com> | 2022-08-03 04:39:10 +0200 |
| commit | 7c255c236f950181b58cf6bec7a94069bb7337e9 (patch) | |
| tree | 43edae7d2cb75b8392fef88a8eca18e77f9b25ee /lib/drmtest.c | |
| parent | 541d11665dc829f60c84061422adce6b44fa2aef (diff) | |
lib/drmtest: Don't log a warning when drm driver is not loaded
Some subtests left machine with drm driver unloaded. When IGT_DEVICE is
set __get_card_for_nth_filter() is logging warning each call so we
have confusing duplicates if function is called few times. Lets remove
warning from it and migrate logging resposibility to the caller.
Fixes: https://gitlab.freedesktop.org/drm/intel/-/issues/6498
Signed-off-by: Zbigniew Kempczyński <zbigniew.kempczynski@intel.com>
Signed-off-by: Chris Wilson <chris.p.wilson@intel.com>
Cc: Petri Latvala <petri.latvala@intel.com>
Reviewed-by: Petri Latvala <petri.latvala@intel.com>
Diffstat (limited to 'lib/drmtest.c')
| -rw-r--r-- | lib/drmtest.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/lib/drmtest.c b/lib/drmtest.c index 4f9d8a11c..16e80bdfc 100644 --- a/lib/drmtest.c +++ b/lib/drmtest.c @@ -365,8 +365,6 @@ static bool __get_card_for_nth_filter(int idx, struct igt_device_card *card) igt_debug("Filter matched %s | %s\n", card->card, card->render); return true; } - - igt_warn("No card matches the filter!\n"); } return false; @@ -442,7 +440,8 @@ int __drm_open_driver_another(int idx, int chipset) } if (!found || !strlen(card.card)) - igt_warn("No card matches the filter!\n"); + igt_warn("No card matches the filter! [%s]\n", + igt_device_filter_get(idx)); else if (_is_already_opened(card.card, idx)) igt_warn("card maching filter %d is already opened\n", idx); else |