<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On 4 August 2015 at 17:57, Oded Gabbay <span dir="ltr"><<a href="mailto:oded.gabbay@gmail.com" target="_blank">oded.gabbay@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div style="font-size:small"><br></div><div class="gmail_extra"><br><div class="gmail_quote"><span class="">On Mon, Aug 3, 2015 at 6:40 PM, Emil Velikov <span dir="ltr"><<a href="mailto:emil.l.velikov@gmail.com" target="_blank">emil.l.velikov@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Oded,<br>
<span><br>
On 2 August 2015 at 11:37, Oded Gabbay <<a href="mailto:oded.gabbay@gmail.com" target="_blank">oded.gabbay@gmail.com</a>> wrote:<br>
> This patch fixes a bug that is manifested in the read path of mesa when<br>
> running on big-endian machines. The effects can be seen when running<br>
> piglit sanity test and/or taking a screen capture.<br>
><br>
> The bug is caused when _mesa_format_convert receives src_format as<br>
> mesa_format, which it thens changes to mesa_array_format. During this<br>
> change, it checks for endianness and swaps the bytes accordingly.<br>
> However, because the bytes are _already_ swapped in the memory itself<br>
> (being written there by llvmpipe), and src_format value matches the<br>
> _actual_ contents of the memory, the result of the read is wrong.<br>
><br>
</span>I'm assuming that you're looked at swrast + softpipe as well - do they<br>
use the same approach or is llvmpipe the odd one out ?<br>
<br></blockquote></span><div><div style="font-size:small;display:inline">​Hi Emil,</div></div><div><div style="font-size:small;display:inline">​</div> </div><div><div style="font-size:small">​I checked it with swrast, softpipe AND llvmpipe.</div><div style="font-size:small">Without my patch, all methods fail piglit sanity on ppc64</div><div style="font-size:small">With my patch, all ​methods pass piglit sanity</div><div style="font-size:small"><br></div></div></div></div></div></blockquote><div>I've mentioned the other drivers, as your commit message explicitly mentions llvmpipe alone. Perhaps dropping that hunk or adding the other two would be ok ? Props for checking all the drivers though !</div><div><br></div><div>I'm afraid that I've never looked into the BE codepaths so don't know how useful of a review I can do here.</div><div><br></div><div>Emil</div><div><br></div></div></div></div>