Lucas Stach <l.stach at pengutronix.de> writes:

> From: Christian Gmeiner <christian.gmeiner at gmail.com>

> +static bool etnaviv_validate_load_state(struct etnaviv_gpu *gpu, u32 *buf,
> +     unsigned int state, unsigned int num)
> +{
> +     return true;
> +     if (0x1200 - state < num * 4)
> +             return false;
> +     if (0x1228 - state < num * 4)
> +             return false;
> +     if (0x1238 - state < num * 4)
> +             return false;
> +     if (0x1284 - state < num * 4)
> +             return false;
> +     if (0x128c - state < num * 4)
> +             return false;
> +     if (0x1304 - state < num * 4)
> +             return false;
> +     if (0x1310 - state < num * 4)
> +             return false;
> +     if (0x1318 - state < num * 4)
> +             return false;
> +     if (0x1280c - state < num * 4 + 0x0c)
> +             return false;
> +     if (0x128ac - state < num * 4 + 0x0c)
> +             return false;
> +     if (0x128cc - state < num * 4 + 0x0c)
> +             return false;
> +     if (0x1297c - state < num * 4 + 0x0c)
> +             return false;
> +     return true;
> +}

I was browsing the code, and noticed that it looks like you've got a
debugging early return in your validation function here.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 818 bytes
Desc: not available
URL: 
<http://lists.freedesktop.org/archives/dri-devel/attachments/20150916/e05ca0b4/attachment-0001.sig>

Reply via email to