<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Sep 21, 2016 at 10:29 PM, Jason Ekstrand <span dir="ltr"><<a href="mailto:jason@jlekstrand.net" target="_blank">jason@jlekstrand.net</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 class="gmail_extra"><div class="gmail_quote"><span class="">On Wed, Sep 21, 2016 at 3:40 PM, Nicolas Koch <span dir="ltr"><<a href="mailto:nioko1337@gmail.com" target="_blank">nioko1337@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hey Jason, thanks for reviewing.<br>
<br>
This is my first mesa contribution, and I'm not really familiar with<br>
the mailing-list workflow.<br></blockquote><div><br></div></span><div>Thanks for the contribution and welcome to mesa!<br></div><span class=""><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Is there anything else I need to do to get this merged? Or will<br>
someone just do it in the near future?<br></blockquote><div><br></div></span><div>Other than bugging one of the reviewers to push it for you, no.  Seeing as you've already done that... I'll push it some time today. :-)</div></div></div></div></blockquote><div><br></div><div>A day later than advertised, bug<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><div class="h5"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div>
On Tue, Sep 20, 2016 at 7:09 PM, Jason Ekstrand <<a href="mailto:jason@jlekstrand.net" target="_blank">jason@jlekstrand.net</a>> wrote:<br>
> Seems reasonable<br>
><br>
> Reviewed-by: Jason Ekstrand <<a href="mailto:jason@jlekstrand.net" target="_blank">jason@jlekstrand.net</a>><br>
><br>
> This reminds me that we really need to convert CmdFillBuffer over to using<br>
> blorp...<br>
><br>
><br>
> On Sep 20, 2016 6:37 PM, "Nicolas Koch" <<a href="mailto:nioko1337@gmail.com" target="_blank">nioko1337@gmail.com</a>> wrote:<br>
>><br>
>> Vulkan spec:<br>
>> Size is the number of bytes to fill, and must be either a multiple of 4,<br>
>> or VK_WHOLE_SIZE to fill the range from offset to the end of the buffer.<br>
>> If VK_WHOLE_SIZE is used and the remaining size of the buffer is not a<br>
>> multiple of 4, then the nearest smaller multiple is used.<br>
>> ---<br>
>>  src/intel/vulkan/anv_meta_clea<wbr>r.c | 6 ++++++<br>
>>  1 file changed, 6 insertions(+)<br>
>><br>
>> diff --git a/src/intel/vulkan/anv_meta_cl<wbr>ear.c<br>
>> b/src/intel/vulkan/anv_meta_cl<wbr>ear.c<br>
>> index ed4d1db..fd0797f 100644<br>
>> --- a/src/intel/vulkan/anv_meta_cl<wbr>ear.c<br>
>> +++ b/src/intel/vulkan/anv_meta_cl<wbr>ear.c<br>
>> @@ -1012,6 +1012,12 @@ void anv_CmdFillBuffer(<br>
>><br>
>>     meta_clear_begin(&saved_<wbr>state, cmd_buffer);<br>
>><br>
>> +   if (fillSize == VK_WHOLE_SIZE) {<br>
>> +      fillSize = dst_buffer->size - dstOffset;<br>
>> +      /* Make sure fillSize is a multiple of 4 */<br>
>> +      fillSize -= fillSize & 3;<br>
>> +   }<br>
>> +<br>
>>     VkFormat format;<br>
>>     int bs;<br>
>>     if ((fillSize & 15) == 0 && (dstOffset & 15) == 0) {<br>
>> --<br>
>> 2.10.0<br>
>><br>
>> ______________________________<wbr>_________________<br>
>> mesa-dev mailing list<br>
>> <a href="mailto:mesa-dev@lists.freedesktop.org" target="_blank">mesa-dev@lists.freedesktop.org</a><br>
>> <a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev" rel="noreferrer" target="_blank">https://lists.freedesktop.org/<wbr>mailman/listinfo/mesa-dev</a><br>
</div></div></blockquote></div></div></div><br></div></div>
</blockquote></div><br></div></div>