From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753024AbbIPQVp (ORCPT ); Wed, 16 Sep 2015 12:21:45 -0400 Received: from pegasos-out.vodafone.de ([80.84.1.38]:39502 "EHLO pegasos-out.vodafone.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752377AbbIPQVn (ORCPT ); Wed, 16 Sep 2015 12:21:43 -0400 X-Spam-Flag: NO X-Spam-Score: -0.054 Authentication-Results: rohrpostix2.prod.vfnet.de (amavisd-new); dkim=pass header.i=@vodafone.de X-DKIM: OpenDKIM Filter v2.6.8 pegasos-out.vodafone.de B12AC1F6CD7 Subject: Re: [PATCH 12/31] drm/amdgpu: use kmemdup rather than duplicating its implementation To: Andrzej Hajda , Alex Deucher , =?UTF-8?Q?Christian_K=c3=b6nig?= , David Airlie References: <1438934377-4922-1-git-send-email-a.hajda@samsung.com> <1438934377-4922-13-git-send-email-a.hajda@samsung.com> <55F93CDB.70505@samsung.com> Cc: Marek Szyprowski , linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, Bartlomiej Zolnierkiewicz From: =?UTF-8?Q?Christian_K=c3=b6nig?= Message-ID: <55F9970E.7090201@vodafone.de> Date: Wed, 16 Sep 2015 18:21:34 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.2.0 MIME-Version: 1.0 In-Reply-To: <55F93CDB.70505@samsung.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 16.09.2015 11:56, Andrzej Hajda wrote: > Ping. > > Regards > Andrzej > > On 08/07/2015 09:59 AM, Andrzej Hajda wrote: >> The patch was generated using fixed coccinelle semantic patch >> scripts/coccinelle/api/memdup.cocci [1]. >> >> [1]: http://permalink.gmane.org/gmane.linux.kernel/2014320 >> >> Signed-off-by: Andrzej Hajda Somehow missed that. Patch is Reviewed-by: Christian König . Regards, Christian. >> --- >> drivers/gpu/drm/amd/amdgpu/amdgpu_object.c | 4 +--- >> 1 file changed, 1 insertion(+), 3 deletions(-) >> >> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_object.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_object.c >> index 8da6424..94bd992 100644 >> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_object.c >> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_object.c >> @@ -549,12 +549,10 @@ int amdgpu_bo_set_metadata (struct amdgpu_bo *bo, void *metadata, >> if (metadata == NULL) >> return -EINVAL; >> >> - buffer = kzalloc(metadata_size, GFP_KERNEL); >> + buffer = kmemdup(metadata, metadata_size, GFP_KERNEL); >> if (buffer == NULL) >> return -ENOMEM; >> >> - memcpy(buffer, metadata, metadata_size); >> - >> kfree(bo->metadata); >> bo->metadata_flags = flags; >> bo->metadata = buffer; > _______________________________________________ > dri-devel mailing list > dri-devel@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/dri-devel