[PATCH i-g-t 2/3] lib/igt_device_scan: change device list variable visibility

[Lucas De Marchi]

On Thu, Jan 23, 2025 at 10:22:09AM +0100, Zbigniew Kempczyński wrote:
>In commit: 2e8f3e06fb (tests/xe_evict|exec_threads: Use fd reopen
>to avoid corrupting global data) I simply changed tests which
>corrupted device list global variable. i915 didn't use opening
>devices in multithreaded tests but this changed with xe.
>
>Device scan is now unconditional so we may change device list
>variable visibility to TLS. With this reverting of above commit
>should be safe (but I'm not sure is it worth to do this).
>
>Signed-off-by: Zbigniew Kempczyński <zbigniew.kempczynski at intel.com>
>Cc: Lucas De Marchi <lucas.demarchi at intel.com>
>Cc: Kamil Konieczny <kamil.konieczny at linux.intel.com>


Reviewed-by: Lucas De Marchi <lucas.demarchi at intel.com>


Lucas De Marchi

>---
> lib/igt_device_scan.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/lib/igt_device_scan.c b/lib/igt_device_scan.c
>index 304cfa17a8..c1b0e0e5ff 100644
>--- a/lib/igt_device_scan.c
>+++ b/lib/igt_device_scan.c
>@@ -247,7 +247,7 @@ struct igt_device {
> };
>
> /* Scanned devices */
>-static struct {
>+static __thread struct {
> 	struct igt_list_head all;
> 	struct igt_list_head filtered;
> 	bool devs_scanned;
>-- 
>2.34.1
>