On Sunday, October 22, 2017 1:01:40 PM PDT Jordan Justen wrote:
> This would cause the read of the metadata content to fail, which would
> prevent the linking from being skipped.
> 
> Seen on Rocket League with i965 shader cache.
> 
> Cc: Timothy Arceri <tarc...@itsqueeze.com>
> Signed-off-by: Jordan Justen <jordan.l.jus...@intel.com>
> Reviewed-by: Timothy Arceri <tarc...@itsqueeze.com>
> ---
>  src/util/disk_cache.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/util/disk_cache.c b/src/util/disk_cache.c
> index e38cacb259b..fde6e2e0974 100644
> --- a/src/util/disk_cache.c
> +++ b/src/util/disk_cache.c
> @@ -1110,7 +1110,7 @@ disk_cache_get(struct disk_cache *cache, const 
> cache_key key, size_t *size)
>         * TODO: pass the metadata back to the caller and do some basic
>         * validation.
>         */
> -      cache_item_md_size += sizeof(cache_key);
> +      cache_item_md_size += num_keys * sizeof(cache_key);
>        ret = lseek(fd, num_keys * sizeof(cache_key), SEEK_CUR);
>        if (ret == -1)
>           goto fail;
> 

Reviewed-by: Kenneth Graunke <kenn...@whitecape.org>

Attachment: signature.asc
Description: This is a digitally signed message part.

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to