[PATCH 2/2] drm/xe/kunit: Update RTP test to check VF rules

[Lucas De Marchi]

On Fri, Jun 14, 2024 at 11:48:21PM GMT, Michal Wajdeczko wrote:
>Add basic test cases that will verify VF_READY and VF_ONLY rules.
>
>Signed-off-by: Michal Wajdeczko <michal.wajdeczko at intel.com>
>Cc: Lucas De Marchi <lucas.demarchi at intel.com>
>---
> drivers/gpu/drm/xe/tests/xe_rtp_test.c | 118 ++++++++++++++++++++++++-
> 1 file changed, 117 insertions(+), 1 deletion(-)
>
>diff --git a/drivers/gpu/drm/xe/tests/xe_rtp_test.c b/drivers/gpu/drm/xe/tests/xe_rtp_test.c
>index 06759d754783..0d8731e13217 100644
>--- a/drivers/gpu/drm/xe/tests/xe_rtp_test.c
>+++ b/drivers/gpu/drm/xe/tests/xe_rtp_test.c
>@@ -232,6 +232,109 @@ static const struct rtp_test_case cases[] = {
> 			{}
> 		},
> 	},
>+	{
>+		.name = "vf-ready",
>+		.expected_reg = REGULAR_REG1,
>+		.expected_set_bits = REG_BIT(0),
>+		.expected_clr_bits = REG_BIT(0),
>+		.expected_count = 1,
>+		/* VF-ready does not impact native/PF */

this contradicts the above expected*bits

>+		.entries = (const struct xe_rtp_entry_sr[]) {
>+			{ XE_RTP_NAME("basic-1"),
>+			  XE_RTP_RULES(FUNC(match_yes), VF_READY),

I was wondering if "VF_READY" wouldn't be more a case for entry->flags
rather than entry->rules. Do you have a concrete case for that?

Lucas De Marchi