From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757261AbcIULW2 (ORCPT ); Wed, 21 Sep 2016 07:22:28 -0400 Received: from mail-lf0-f67.google.com ([209.85.215.67]:32927 "EHLO mail-lf0-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752679AbcIULWZ (ORCPT ); Wed, 21 Sep 2016 07:22:25 -0400 Date: Wed, 21 Sep 2016 13:22:15 +0200 From: Daniel Vetter To: SF Markus Elfring Cc: dri-devel@lists.freedesktop.org, Daniel Vetter , David Airlie , Julia Lawall , kernel-janitors@vger.kernel.org, LKML Subject: Re: [PATCH 4/5] GPU-DRM: Replace a kzalloc() call by kcalloc() in drm_legacy_addbufs_sg() Message-ID: <20160921112215.GJ4714@dvetter-linux.ger.corp.intel.com> Mail-Followup-To: SF Markus Elfring , dri-devel@lists.freedesktop.org, David Airlie , Julia Lawall , kernel-janitors@vger.kernel.org, LKML References: <566ABCD9.1060404@users.sourceforge.net> <2fd2da19-d57e-5017-0594-fd52e78493fa@users.sourceforge.net> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: X-Operating-System: Linux dvetter-linux.ger.corp.intel.com 4.6.7-300.fc24.x86_64 User-Agent: Mutt/1.7.0 (2016-08-17) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Sep 19, 2016 at 05:56:49PM +0200, SF Markus Elfring wrote: > From: Markus Elfring > Date: Mon, 19 Sep 2016 17:30:31 +0200 > > The script "checkpatch.pl" can point information out like the following. > > WARNING: Prefer kcalloc over kzalloc with multiply > > Thus fix the affected source code place. > > Signed-off-by: Markus Elfring Merged patches 1-4. I agree with Jani that changing jump labels is a pure bikeshed, so didn't apply them. Also I'll repeat that imo checkpatch patches are great to get started, but it's much better to do more involved work. Both since that tends to be more interesting, and fixing all the checkpatch issues will rob the next newbies of some great starting opportunity. Which means from now on I'll only selectively apply your checkpatch patches. We have todo list with some ideas at: https://www.x.org/wiki/DRMJanitors/ Thanks, Daniel > --- > drivers/gpu/drm/drm_bufs.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/drivers/gpu/drm/drm_bufs.c b/drivers/gpu/drm/drm_bufs.c > index 36dd685..adb1dd7 100644 > --- a/drivers/gpu/drm/drm_bufs.c > +++ b/drivers/gpu/drm/drm_bufs.c > @@ -1117,8 +1117,7 @@ static int drm_legacy_addbufs_sg(struct drm_device *dev, > return -EINVAL; > } > > - entry->buflist = kzalloc(count * sizeof(*entry->buflist), > - GFP_KERNEL); > + entry->buflist = kcalloc(count, sizeof(*entry->buflist), GFP_KERNEL); > if (!entry->buflist) { > mutex_unlock(&dev->struct_mutex); > atomic_dec(&dev->buf_alloc); > -- > 2.10.0 > > _______________________________________________ > dri-devel mailing list > dri-devel@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch