[PATCH i-g-t v2 6/8] tests/intel/kms_dirtyfb: pass the bo_size

Matthew Auld matthew.auld at intel.com
Tue Feb 13 10:01:23 UTC 2024


Looks possible for the derived bo_size to not be page aligned leading to
broken behaviour when vm_binding it with xe. Pass through the bo_size,
instead of leaving as zero. This will be enforced in the next patch.

Signed-off-by: Matthew Auld <matthew.auld at intel.com>
Cc: Zbigniew Kempczyński <zbigniew.kempczynski at intel.com>
Reviewed-by: Zbigniew Kempczyński <zbigniew.kempczynski at intel.com>
---
 tests/intel/kms_dirtyfb.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tests/intel/kms_dirtyfb.c b/tests/intel/kms_dirtyfb.c
index 7bf49cb15..26b82e50a 100644
--- a/tests/intel/kms_dirtyfb.c
+++ b/tests/intel/kms_dirtyfb.c
@@ -254,7 +254,7 @@ static void run_test(data_t *data)
 				    igt_drm_format_to_bpp(data->fbs[1].drm_format),
 				    0,
 				    igt_fb_mod_to_tiling(data->fbs[1].modifier),
-				    0, 0, 0, is_xe_device(data->drm_fd) ?
+				    0, data->fbs[1].size, 0, is_xe_device(data->drm_fd) ?
 				    system_memory(data->drm_fd) : 0,
 				    intel_get_pat_idx_uc(data->drm_fd));
 	dst = intel_buf_create_full(data->bops, data->fbs[2].gem_handle,
@@ -262,7 +262,7 @@ static void run_test(data_t *data)
 				    data->fbs[2].height,
 				    igt_drm_format_to_bpp(data->fbs[2].drm_format),
 				    0, igt_fb_mod_to_tiling(data->fbs[2].modifier),
-				    0, 0, 0, is_xe_device(data->drm_fd) ?
+				    0, data->fbs[2].size, 0, is_xe_device(data->drm_fd) ?
 				    system_memory(data->drm_fd) : 0,
 				    intel_get_pat_idx_uc(data->drm_fd));
 	ibb = intel_bb_create(data->drm_fd, PAGE_SIZE);
-- 
2.43.0



More information about the igt-dev mailing list