From mboxrd@z Thu Jan 1 00:00:00 1970 From: Martin Peres Subject: Re: [RFC PATCH 5/5] drm/nouveau: gpu lockup recovery Date: Mon, 23 Apr 2012 18:46:41 +0200 Message-ID: <4F958771.6040008@free.fr> References: <1335133112-8008-5-git-send-email-marcin.slusarz@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1"; Format="flowed" Content-Transfer-Encoding: quoted-printable Return-path: Received: from smtp6-g21.free.fr (smtp6-g21.free.fr [212.27.42.6]) by gabe.freedesktop.org (Postfix) with ESMTP id 466C79F373 for ; Mon, 23 Apr 2012 09:50:47 -0700 (PDT) In-Reply-To: <1335133112-8008-5-git-send-email-marcin.slusarz@gmail.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: dri-devel-bounces+sf-dri-devel=m.gmane.org@lists.freedesktop.org Errors-To: dri-devel-bounces+sf-dri-devel=m.gmane.org@lists.freedesktop.org To: dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org Hey, Just a minor mistake spotted while skimming through the patch. Le 23/04/2012 00:18, Marcin Slusarz a =E9crit : > +static inline uint64_t nv_timeout(struct drm_device *dev) > +{ > + uint64_t tm =3D 2000000000ULL; > + if (nouveau_gpu_reset_in_progress(dev)) > + tm /=3D 40; /* 50ms */ This will cause a problem on 32 bit kernels. You should use do_div. Martin