From mboxrd@z Thu Jan 1 00:00:00 1970 From: Artem Savkov Subject: Re: [PATCH] drm: i915_gem_inactive_scan: insufficient nr_to_scan size Date: Tue, 23 Jul 2013 23:29:49 +0400 Message-ID: <20130723192949.GA3786@thinkpad.lan> References: <1374607086-6761-1-git-send-email-artem.savkov@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Return-path: Content-Disposition: inline In-Reply-To: <1374607086-6761-1-git-send-email-artem.savkov@gmail.com> Sender: linux-kernel-owner@vger.kernel.org To: dchinner@redhat.com, glommer@openvz.org Cc: daniel.vetter@ffwll.ch, airlied@linux.ie, intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org List-Id: dri-devel@lists.freedesktop.org On Tue, Jul 23, 2013 at 11:18:06PM +0400, Artem Savkov wrote: > nr_to_scan needs to be (unsigned) long, not int, otherwise we get negative > values on 32bit systems during shrink resulting in lots of messages like one > below. > > [ 4078.989603] shrink_slab: i915_gem_inactive_scan+0x0/0xc0 negative objects to delete nr=-289580136 > > Introduced in "drivers: convert shrinkers to new count/scan API" > (383e4f4a4cdeb2dd552e90811089bcb4d41eb32e in linux-next.git) Sorry, looks like my analysis was wrong, I can still see the issue with this patch. -- Regards, Artem