<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Feb 6, 2015 at 4:01 PM, Francisco Jerez <span dir="ltr"><<a href="mailto:currojerez@riseup.net" target="_blank">currojerez@riseup.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hey Matt,<br>
<span class=""><br>
Matt Turner <<a href="mailto:mattst88@gmail.com">mattst88@gmail.com</a>> writes:<br>
<br>
> On Fri, Feb 6, 2015 at 6:42 AM, Francisco Jerez <<a href="mailto:currojerez@riseup.net">currojerez@riseup.net</a>> wrote:<br>
>> MRFs cannot be read from anyway so they cannot possibly be a valid<br>
>> source of LOAD_PAYLOAD.<br>
>> ---<br>
><br>
> The function only seems to test inst->dst.file == MRF. I don't see any<br>
> code for handling MRF sources. What am I missing?<br>
<br>
</span>That test is for "handling" MRF sources -- More precisely, it's<br>
collecting the writemask and half flags for MRF writes, which can only<br>
possibly be useful if we're going to use them later on to read something<br>
out of an MRF into a payload, which we shouldn't be doing in the first<br>
place.<br>
<br>
Aside from simplifying the function somewhat, that allows us to drop the<br>
16 register gap reserved for MRFs at register offset zero, what will<br>
allow us to drop the vgrf_to_reg[] offset calculation completely (also<br>
in split_virtual_grfs()) in a future patch (not sent for review yet).<br></blockquote><div><br></div><div>No, we do read from MRF's sort-of...  Send messages have an implicit "read" from an MRF.  This was written precicely so that we could use LOAD_PAYLOAD to build MRF payloads.  We do on pre-GEN6.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">_______________________________________________<br>
mesa-dev mailing list<br>
<a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/mesa-dev" target="_blank">http://lists.freedesktop.org/mailman/listinfo/mesa-dev</a><br>
<br></blockquote></div><br></div></div>