[PATCH i-g-t 1/6] lib/igt_fb: fix intel modifiers for fb copying on xe driver
Juha-Pekka Heikkila
juhapekka.heikkila at gmail.com
Mon Apr 22 20:44:36 UTC 2024
mc ccs need to go to vebox copy, blitter doesn't do mc ccs hence
on all platforms with ccs modifiers use engine copy for those.
Signed-off-by: Juha-Pekka Heikkila <juhapekka.heikkila at gmail.com>
---
lib/igt_fb.c | 18 +++++++++++++-----
1 file changed, 13 insertions(+), 5 deletions(-)
diff --git a/lib/igt_fb.c b/lib/igt_fb.c
index cc70cb91c..a77920fff 100644
--- a/lib/igt_fb.c
+++ b/lib/igt_fb.c
@@ -2508,9 +2508,9 @@ static bool blitter_ok(const struct igt_fb *fb)
if (!is_intel_device(fb->fd))
return false;
- if ((is_ccs_modifier(fb->modifier) &&
- !HAS_FLATCCS(intel_get_drm_devid(fb->fd))) ||
- is_gen12_mc_ccs_modifier(fb->modifier))
+ if ((!blt_uses_extended_block_copy(fb->fd) &&
+ is_ccs_modifier(fb->modifier)) ||
+ is_gen12_mc_ccs_modifier(fb->modifier))
return false;
if (is_xe_device(fb->fd))
@@ -3062,7 +3062,12 @@ static void free_linear_mapping(struct fb_blit_upload *blit)
igt_nouveau_delete_bo(&linear->fb);
} else if (is_xe_device(fd)) {
gem_munmap(linear->map, linear->fb.size);
- blitcopy(fb, &linear->fb);
+
+ if (blit->ibb)
+ copy_with_engine(blit, fb, &linear->fb);
+ else
+ blitcopy(fb, &linear->fb);
+
gem_close(fd, linear->fb.gem_handle);
} else {
gem_munmap(linear->map, linear->fb.size);
@@ -3142,7 +3147,10 @@ static void setup_linear_mapping(struct fb_blit_upload *blit)
linear->map = igt_nouveau_mmap_bo(&linear->fb, PROT_READ | PROT_WRITE);
} else if (is_xe_device(fd)) {
- blitcopy(&linear->fb, fb);
+ if (blit->ibb)
+ copy_with_engine(blit, &linear->fb, fb);
+ else
+ blitcopy(&linear->fb, fb);
linear->map = xe_bo_mmap_ext(fd, linear->fb.gem_handle,
linear->fb.size, PROT_READ | PROT_WRITE);
--
2.25.1
More information about the igt-dev
mailing list