Re: [PATCH] intel-gpu-tools: Change type of variable from unsigned to uint64_t in gem_stress

[Daniel Vetter <daniel@ffwll.ch>]

On Fri, Aug 01, 2014 at 12:10:44PM +0700, Pavel Popov wrote:
> To run gem_stress with the correct number of buffers even if aperture size >= 4GB.
> 
> Signed-off-by: Pavel Popov <pavel.e.popov@intel.com>
> ---
>  tests/gem_stress.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tests/gem_stress.c b/tests/gem_stress.c
> index 3bbe487..313a82a 100644
> --- a/tests/gem_stress.c
> +++ b/tests/gem_stress.c
> @@ -737,7 +737,7 @@ static int parse_options(int opt, int opt_index)
>  static void init(void)
>  {
>  	int i;
> -	unsigned tmp;
> +	uint64_t tmp;

We have tons of places which use gem_aperture_size. Have you reviewed them
all?
-Daniel

>  
>  	if (options.num_buffers == 0) {
>  		tmp = gem_aperture_size(drm_fd);
> -- 
> 1.9.1
> 
> 
> --------------------------------------------------------------------
> Closed Joint Stock Company Intel A/O
> Registered legal address: Krylatsky Hills Business Park, 
> 17 Krylatskaya Str., Bldg 4, Moscow 121614, 
> Russian Federation
> 
> This e-mail and any attachments may contain confidential material for
> the sole use of the intended recipient(s). Any review or distribution
> by others is strictly prohibited. If you are not the intended
> recipient, please contact the sender and delete all copies.
> 
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx

-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch