From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [PATCH] intel-gpu-tools: Change type of variable from unsigned to uint64_t in gem_stress Date: Mon, 4 Aug 2014 11:09:40 +0200 Message-ID: <20140804090940.GU8727@phenom.ffwll.local> References: <1406869844-6825-1-git-send-email-pavel.e.popov@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail-wi0-f176.google.com (mail-wi0-f176.google.com [209.85.212.176]) by gabe.freedesktop.org (Postfix) with ESMTP id B018D6E37D for ; Mon, 4 Aug 2014 02:09:29 -0700 (PDT) Received: by mail-wi0-f176.google.com with SMTP id bs8so4646684wib.3 for ; Mon, 04 Aug 2014 02:09:28 -0700 (PDT) Content-Disposition: inline In-Reply-To: <1406869844-6825-1-git-send-email-pavel.e.popov@intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Pavel Popov Cc: intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org 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 > --- > 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