[Mesa-dev] [PATCH] i965/gen9: Don't use fast copy blit in case of non power of 2 cpp

[Anuj Phogat]

On Mon, Aug 10, 2015 at 1:59 PM, Anuj Phogat <anuj.phogat at gmail.com> wrote:
>
>
> On Mon, Aug 10, 2015 at 11:17 AM, Anuj Phogat <anuj.phogat at gmail.com> wrote:
>>
>> Many piglit tests (if using fast copy blit in Mesa) failed earlier
>> because I missed adding this condition. Fast copy blit is currently
>> enabled for use only with Yf/Ys tiling.
>>
>> Signed-off-by: Anuj Phogat <anuj.phogat at gmail.com>
>> ---
>>  src/mesa/drivers/dri/i965/intel_blit.c | 6 ++++--
>>  1 file changed, 4 insertions(+), 2 deletions(-)
>>
>> diff --git a/src/mesa/drivers/dri/i965/intel_blit.c
>> b/src/mesa/drivers/dri/i965/intel_blit.c
>> index 6d92580..bab7d90 100644
>> --- a/src/mesa/drivers/dri/i965/intel_blit.c
>> +++ b/src/mesa/drivers/dri/i965/intel_blit.c
>> @@ -431,8 +431,10 @@ can_fast_copy_blit(struct brw_context *brw,
>>     if ((dst_offset | src_offset) & 63)
>>        return false;
>>
>> -   /* Color depth greater than 128 bits not supported. */
>> -   if (cpp > 16)
>> +   /* Color depths which are not power of 2 or greater than 128 bits are
>> +    * not supported.
>> +    */
>> +   if (!is_power_of_two(cpp) || cpp > 16)
>
>  This function got deleted on master. I'll change it to _mesa_is_pow_two().
>>
>>        return false;
>>
>>     /* For Fast Copy Blits the pitch cannot be a negative number. So, bit
>> 15
>> --
>> 2.4.3
>>
>

Bump.