<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">
<meta name="Generator" content="Microsoft Word 15 (filtered medium)">
<style><!--
/* Font Definitions */
@font-face
{font-family:"Cambria Math";
panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
{font-family:Calibri;
panose-1:2 15 5 2 2 2 4 3 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
{margin:0in;
font-size:11.0pt;
font-family:"Calibri",sans-serif;}
a:link, span.MsoHyperlink
{mso-style-priority:99;
color:#0563C1;
text-decoration:underline;}
p.MsoPlainText, li.MsoPlainText, div.MsoPlainText
{mso-style-priority:99;
mso-style-link:"Plain Text Char";
margin:0in;
font-size:11.0pt;
font-family:"Calibri",sans-serif;}
span.PlainTextChar
{mso-style-name:"Plain Text Char";
mso-style-priority:99;
mso-style-link:"Plain Text";
font-family:"Calibri",sans-serif;}
span.EmailStyle21
{mso-style-type:personal-reply;
font-family:"Calibri",sans-serif;
color:windowtext;}
.MsoChpDefault
{mso-style-type:export-only;
font-size:10.0pt;}
@page WordSection1
{size:8.5in 11.0in;
margin:1.0in 1.0in 1.0in 1.0in;}
div.WordSection1
{page:WordSection1;}
--></style><!--[if gte mso 9]><xml>
<o:shapedefaults v:ext="edit" spidmax="1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext="edit">
<o:idmap v:ext="edit" data="1" />
</o:shapelayout></xml><![endif]-->
</head>
<body lang="EN-US" link="#0563C1" vlink="#954F72" style="word-wrap:break-word">
<div class="WordSection1">
<p class="MsoPlainText">On 2023-04-18 at 09:10 PM, Bhanuprakash Modem wrote:<o:p></o:p></p>
<p class="MsoPlainText">Looks good to me.<o:p></o:p></p>
<p class="MsoPlainText">Reviewed-by: Nidhi Gupta <<a href="mailto:nidhi1.gupta@intel.com">nidhi1.gupta@intel.com</a>><o:p></o:p></p>
<p class="MsoNormal">>diff --git a/tests/kms_universal_plane.c b/tests/kms_universal_plane.c<o:p></o:p></p>
<p class="MsoNormal">>index 114918d97..ddd356eda 100644<o:p></o:p></p>
<p class="MsoNormal">>--- a/tests/kms_universal_plane.c<o:p></o:p></p>
<p class="MsoNormal">>+++ b/tests/kms_universal_plane.c<o:p></o:p></p>
<p class="MsoNormal">>@@ -576,13 +576,18 @@ cursor_leak_test_fini(data_t *data,<o:p></o:p></p>
<p class="MsoNormal">>}<o:p></o:p></p>
<p class="MsoNormal">><o:p> </o:p></p>
<p class="MsoNormal">>static int<o:p></o:p></p>
<p class="MsoNormal">>-i915_gem_fb_count(data_t *data)<o:p></o:p></p>
<p class="MsoNormal">>+intel_gem_fb_count(data_t *data)<o:p></o:p></p>
<p class="MsoNormal">>{<o:p></o:p></p>
<p class="MsoNormal">> char buf[1024];<o:p></o:p></p>
<p class="MsoNormal">> FILE *fp;<o:p></o:p></p>
<p class="MsoNormal">> int fd;<o:p></o:p></p>
<p class="MsoNormal">> int count = 0;<o:p></o:p></p>
<p class="MsoNormal">><o:p> </o:p></p>
<p class="MsoNormal">>+ /*<o:p></o:p></p>
<p class="MsoNormal">>+ * FIXME: As of now, XE's debugfs is using i915 namespace. Once Kernel<o:p></o:p></p>
<p class="MsoNormal">>+ * changes got landed, please update this logic to use XE specific<o:p></o:p></p>
<p class="MsoNormal">>+ * debugfs.<o:p></o:p></p>
<p class="MsoNormal">>+ */<o:p></o:p></p>
<p class="MsoNormal">> fd = igt_debugfs_open(data->drm_fd, "i915_gem_framebuffer", O_RDONLY);<o:p></o:p></p>
<p class="MsoNormal">> fp = fdopen(fd, "r");<o:p></o:p></p>
<p class="MsoNormal">> igt_require(fp);<o:p></o:p></p>
<p class="MsoNormal">>@@ -617,7 +622,7 @@ cursor_leak_test_pipe(data_t *data, enum pipe pipe, igt_output_t *output)<o:p></o:p></p>
<p class="MsoNormal">> mode = igt_output_get_mode(output);<o:p></o:p></p>
<p class="MsoNormal">><o:p> </o:p></p>
<p class="MsoNormal">> /* Count GEM framebuffers before creating our cursor FB's */<o:p></o:p></p>
<p class="MsoNormal">>- count1 = i915_gem_fb_count(data);<o:p></o:p></p>
<p class="MsoNormal">>+ count1 = intel_gem_fb_count(data);<o:p></o:p></p>
<p class="MsoNormal">><o:p> </o:p></p>
<p class="MsoNormal">> /* Black background FB */<o:p></o:p></p>
<p class="MsoNormal">> igt_create_color_fb(data->drm_fd, mode->hdisplay, mode->vdisplay,<o:p></o:p></p>
<p class="MsoNormal">>@@ -675,7 +680,7 @@ cursor_leak_test_pipe(data_t *data, enum pipe pipe, igt_output_t *output)<o:p></o:p></p>
<p class="MsoNormal">> cursor_leak_test_fini(data, output, &background_fb, cursor_fb);<o:p></o:p></p>
<p class="MsoNormal">><o:p> </o:p></p>
<p class="MsoNormal">> /* We should be back to the same framebuffer count as when we started */<o:p></o:p></p>
<p class="MsoNormal">>- count2 = i915_gem_fb_count(data);<o:p></o:p></p>
<p class="MsoNormal">>+ count2 = intel_gem_fb_count(data);<o:p></o:p></p>
<p class="MsoNormal">><o:p> </o:p></p>
<p class="MsoNormal">> igt_assert_eq(count1, count2);<o:p></o:p></p>
<p class="MsoNormal">>}<o:p></o:p></p>
<p class="MsoNormal">>@@ -737,7 +742,7 @@ pageflip_win_test_pipe(data_t *data, enum pipe pipe, igt_output_t *output)<o:p></o:p></p>
<p class="MsoNormal">><o:p> </o:p></p>
<p class="MsoNormal">> int ret = 0;<o:p></o:p></p>
<p class="MsoNormal">><o:p> </o:p></p>
<p class="MsoNormal">>- igt_skip_on(is_i915_device(data->drm_fd) && data->display_ver < 9);<o:p></o:p></p>
<p class="MsoNormal">>+ igt_skip_on(is_intel_device(data->drm_fd) && data->display_ver < 9);<o:p></o:p></p>
<p class="MsoNormal">> igt_require_pipe(&data->display, pipe);<o:p></o:p></p>
<p class="MsoNormal">><o:p> </o:p></p>
<p class="MsoNormal">> igt_info("Using (pipe %s + %s) to run the subtest.\n",<o:p></o:p></p>
<p class="MsoNormal">>@@ -895,7 +900,7 @@ igt_main<o:p></o:p></p>
<p class="MsoNormal">><o:p> </o:p></p>
<p class="MsoNormal">> igt_fixture {<o:p></o:p></p>
<p class="MsoNormal">> data.drm_fd = drm_open_driver_master(DRIVER_ANY);<o:p></o:p></p>
<p class="MsoNormal">>- if (is_i915_device(data.drm_fd))<o:p></o:p></p>
<p class="MsoNormal">>+ if (is_intel_device(data.drm_fd))<o:p></o:p></p>
<p class="MsoNormal">> data.display_ver = intel_display_ver(intel_get_drm_devid(data.drm_fd));<o:p></o:p></p>
<p class="MsoNormal">><o:p> </o:p></p>
<p class="MsoNormal">> kmstest_set_vt_graphics_mode();<o:p></o:p></p>
</div>
</body>
</html>