[Mesa-dev] [PATCH 1/2] r600: Fix some printf format warns pretaining to PRIi64

Bruno Jimenez brunojimen at gmail.com
Thu Jul 30 09:38:25 PDT 2015


Hi,

I don't know if you will believe it or not, but I have just written a
very similar patch to yours :)

Anyway, yours have come first so have my

Reviewed-by: Bruno Jiménez <brunojimen at gmail.com>

Note: I have no permissions in the git repo, you should ask someone to
push this patch for you.

- Bruno

On Thu, 2015-07-30 at 09:46 +1000, Edward O'Callaghan wrote:
> Signed-off-by: Edward O'Callaghan <eocallaghan at alterapraxis.com>
> ---
>  src/gallium/drivers/r600/compute_memory_pool.c | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/src/gallium/drivers/r600/compute_memory_pool.c 
> b/src/gallium/drivers/r600/compute_memory_pool.c
> index 413aa3d..7c5113e 100644
> --- a/src/gallium/drivers/r600/compute_memory_pool.c
> +++ b/src/gallium/drivers/r600/compute_memory_pool.c
> @@ -120,7 +120,7 @@ int64_t compute_memory_prealloc_chunk(
>  
>  	assert(size_in_dw <= pool->size_in_dw);
>  
> -	COMPUTE_DBG(pool->screen, "* compute_memory_prealloc_chunk() 
> size_in_dw = %ld\n",
> +	COMPUTE_DBG(pool->screen, "* compute_memory_prealloc_chunk() 
> size_in_dw = %"PRIi64"\n",
>  		size_in_dw);
>  
>  	LIST_FOR_EACH_ENTRY(item, pool->item_list, link) {
> @@ -151,7 +151,7 @@ struct list_head *compute_memory_postalloc_chunk(
>  	struct compute_memory_item *next;
>  	struct list_head *next_link;
>  
> -	COMPUTE_DBG(pool->screen, "* 
> compute_memory_postalloc_chunck() start_in_dw = %ld\n",
> +	COMPUTE_DBG(pool->screen, "* 
> compute_memory_postalloc_chunck() start_in_dw = %"PRIi64"\n",
>  		start_in_dw);
>  
>  	/* Check if we can insert it in the front of the list */
> @@ -568,7 +568,7 @@ void compute_memory_free(struct 
> compute_memory_pool* pool, int64_t id)
>  	struct pipe_screen *screen = (struct pipe_screen *)pool
> ->screen;
>  	struct pipe_resource *res;
>  
> -	COMPUTE_DBG(pool->screen, "* compute_memory_free() id + %ld 
> \n", id);
> +	COMPUTE_DBG(pool->screen, "* compute_memory_free() id + 
> %"PRIi64" \n", id);
>  
>  	LIST_FOR_EACH_ENTRY_SAFE(item, next, pool->item_list, link) 
> {
>  
> @@ -628,7 +628,7 @@ struct compute_memory_item* compute_memory_alloc(
>  {
>  	struct compute_memory_item *new_item = NULL;
>  
> -	COMPUTE_DBG(pool->screen, "* compute_memory_alloc() 
> size_in_dw = %ld (%ld bytes)\n",
> +	COMPUTE_DBG(pool->screen, "* compute_memory_alloc() 
> size_in_dw = %"PRIi64" (%"PRIi64" bytes)\n",
>  			size_in_dw, 4 * size_in_dw);
>  
>  	new_item = (struct compute_memory_item *)


More information about the mesa-dev mailing list