Commit e4604100 authored by Lucas De Marchi's avatar Lucas De Marchi Committed by Rodrigo Vivi

drm/xe: Generalize fake device creation

Instead of requiring tests to initialize a fake device an keep it in
sync with xe_pci.c when it's platform-dependent, export a function from
xe_pci.c to be used and piggy back on the device info creation. For
simpler tests that don't need any specific platform and just need a fake
xe device to pass around, xe_pci_fake_device_init_any() can be used.
Signed-off-by: default avatarLucas De Marchi <lucas.demarchi@intel.com>
Reviewed-by: default avatarMatt Roper <matthew.d.roper@intel.com>
Link: https://lore.kernel.org/r/20230401085151.1786204-5-lucas.demarchi@intel.comSigned-off-by: default avatarRodrigo Vivi <rodrigo.vivi@intel.com>
parent 60d5c6ab
...@@ -8,6 +8,7 @@ ...@@ -8,6 +8,7 @@
#include "tests/xe_test.h" #include "tests/xe_test.h"
#include <kunit/test.h> #include <kunit/test.h>
#include <kunit/visibility.h>
struct kunit_test_data { struct kunit_test_data {
int ndevs; int ndevs;
...@@ -60,3 +61,49 @@ int xe_call_for_each_device(xe_device_fn xe_fn) ...@@ -60,3 +61,49 @@ int xe_call_for_each_device(xe_device_fn xe_fn)
return ret; return ret;
} }
int xe_pci_fake_device_init(struct xe_device *xe, enum xe_platform platform,
enum xe_subplatform subplatform)
{
const struct pci_device_id *ent = pciidlist;
const struct xe_device_desc *desc;
const struct xe_subplatform_desc *subplatform_desc;
if (platform == XE_TEST_PLATFORM_ANY) {
desc = (const void *)ent->driver_data;
subplatform_desc = NULL;
goto done;
}
for (ent = pciidlist; ent->device; ent++) {
desc = (const void *)ent->driver_data;
if (desc->platform == platform)
break;
}
if (!ent->device)
return -ENODEV;
if (subplatform == XE_TEST_SUBPLATFORM_ANY) {
subplatform_desc = desc->subplatforms;
goto done;
}
for (subplatform_desc = desc->subplatforms;
subplatform_desc && subplatform_desc->subplatform;
subplatform_desc++)
if (subplatform_desc->subplatform == subplatform)
break;
if (subplatform == XE_SUBPLATFORM_NONE && subplatform_desc)
return -ENODEV;
if (subplatform != XE_SUBPLATFORM_NONE && !subplatform_desc)
return -ENODEV;
done:
xe_info_init(xe, desc, subplatform_desc);
return 0;
}
EXPORT_SYMBOL_IF_KUNIT(xe_pci_fake_device_init);
...@@ -6,10 +6,26 @@ ...@@ -6,10 +6,26 @@
#ifndef _XE_PCI_TEST_H_ #ifndef _XE_PCI_TEST_H_
#define _XE_PCI_TEST_H_ #define _XE_PCI_TEST_H_
#include "xe_platform_types.h"
struct xe_device; struct xe_device;
/*
* Some defines just for clarity: these mean the test doesn't care about what
* platform it will get since it doesn't depend on any platform-specific bits
*/
#define XE_TEST_PLATFORM_ANY XE_PLATFORM_UNINITIALIZED
#define XE_TEST_SUBPLATFORM_ANY XE_SUBPLATFORM_UNINITIALIZED
typedef int (*xe_device_fn)(struct xe_device *); typedef int (*xe_device_fn)(struct xe_device *);
int xe_call_for_each_device(xe_device_fn xe_fn); int xe_call_for_each_device(xe_device_fn xe_fn);
int xe_pci_fake_device_init(struct xe_device *xe, enum xe_platform platform,
enum xe_subplatform subplatform);
#define xe_pci_fake_device_init_any(xe__) \
xe_pci_fake_device_init(xe__, XE_TEST_PLATFORM_ANY, \
XE_TEST_SUBPLATFORM_ANY)
#endif #endif
...@@ -423,7 +423,7 @@ static void xe_pci_remove(struct pci_dev *pdev) ...@@ -423,7 +423,7 @@ static void xe_pci_remove(struct pci_dev *pdev)
static int xe_pci_probe(struct pci_dev *pdev, const struct pci_device_id *ent) static int xe_pci_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
{ {
const struct xe_device_desc *desc = (void *)ent->driver_data; const struct xe_device_desc *desc = (const void *)ent->driver_data;
const struct xe_subplatform_desc *subplatform_desc; const struct xe_subplatform_desc *subplatform_desc;
struct xe_device *xe; struct xe_device *xe;
int err; int err;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment