<html>
<head>
<base href="https://bugs.freedesktop.org/">
</head>
<body>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - [BYT] ES31-CTS.functional.copy_image.non_compressed.viewclass_96_bits.rgb32f_rgb32f"
href="https://bugs.freedesktop.org/show_bug.cgi?id=101910#c2">Comment # 2</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - [BYT] ES31-CTS.functional.copy_image.non_compressed.viewclass_96_bits.rgb32f_rgb32f"
href="https://bugs.freedesktop.org/show_bug.cgi?id=101910">bug 101910</a>
from <span class="vcard"><a class="email" href="mailto:topi.pohjolainen@intel.com" title="Topi Pohjolainen <topi.pohjolainen@intel.com>"> <span class="fn">Topi Pohjolainen</span></a>
</span></b>
<pre>Well, the difference is valign 2 vs 4. Forcing valign 4 even for linear with
RGB32F:
diff --git a/src/intel/isl/isl_gen7.c b/src/intel/isl/isl_gen7.c
index 24d411f..57751ea 100644
--- a/src/intel/isl/isl_gen7.c
+++ b/src/intel/isl/isl_gen7.c
@@ -383,7 +383,13 @@ isl_gen7_choose_image_alignment_el(const struct isl_device
*dev,
assert(!(require_valign4 && gen7_format_needs_valign2(dev, info->format)));
/* We default to VALIGN_2 because it uses the least memory. */
- const uint32_t valign = require_valign4 ? 4 : 2;
+ uint32_t valign = require_valign4 ? 4 : 2;
+
+ /* In practise it looks that even as linear tiled this has to be programmed
+ * to 4 on BayTrail.
+ */
+ if (info->format == ISL_FORMAT_R32G32B32_FLOAT && ISL_DEV_IS_BAYTRAIL(dev))
+ valign = 4;
*image_align_el = isl_extent3d(halign, valign, 1);
}
fixes the tests.</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the QA Contact for the bug.</li>
</ul>
</body>
</html>