From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751766AbcEANC2 (ORCPT ); Sun, 1 May 2016 09:02:28 -0400 Received: from mail-pa0-f65.google.com ([209.85.220.65]:35707 "EHLO mail-pa0-f65.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751074AbcEANC1 (ORCPT ); Sun, 1 May 2016 09:02:27 -0400 From: Muhammad Falak R Wani To: David Airlie Cc: Ben Skeggs , dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org Subject: [PATCH] drm/nouveau/fence: use RCU_INIT_POINTER() when NULLing. Date: Sun, 1 May 2016 18:30:44 +0530 Message-Id: <1462107742-9784-1-git-send-email-falakreyaz@gmail.com> X-Mailer: git-send-email 1.9.1 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org It is safe to use RCU_INIT_POINTER() to NULL, instead of rcu_assign_pointer(). This results in slightly smaller/faster code. The follwoing semantic patch was used: @@ @@ - rcu_assign_pointer + RCU_INIT_POINTER (..., NULL) Signed-off-by: Muhammad Falak R Wani --- drivers/gpu/drm/nouveau/nouveau_fence.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/nouveau/nouveau_fence.c b/drivers/gpu/drm/nouveau/nouveau_fence.c index 9a8c5b7..c842dc4 100644 --- a/drivers/gpu/drm/nouveau/nouveau_fence.c +++ b/drivers/gpu/drm/nouveau/nouveau_fence.c @@ -60,7 +60,7 @@ nouveau_fence_signal(struct nouveau_fence *fence) fence_signal_locked(&fence->base); list_del(&fence->head); - rcu_assign_pointer(fence->channel, NULL); + RCU_INIT_POINTER(fence->channel, NULL); if (test_bit(FENCE_FLAG_USER_BITS, &fence->base.flags)) { struct nouveau_fence_chan *fctx = nouveau_fctx(fence); -- 1.9.1