From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1947235AbcHRNsc (ORCPT ); Thu, 18 Aug 2016 09:48:32 -0400 Received: from terminus.zytor.com ([198.137.202.10]:34446 "EHLO mail.zytor.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753915AbcHRNpR (ORCPT ); Thu, 18 Aug 2016 09:45:17 -0400 Date: Thu, 18 Aug 2016 06:40:31 -0700 From: tip-bot for Johannes Berg Message-ID: Cc: linux-kernel@vger.kernel.org, johannes.berg@intel.com, mingo@kernel.org, daniel.vetter@ffwll.ch, hpa@zytor.com, tglx@linutronix.de, chris@chris-wilson.co.uk, akpm@linux-foundation.org, peterz@infradead.org, torvalds@linux-foundation.org, paulmck@linux.vnet.ibm.com Reply-To: linux-kernel@vger.kernel.org, johannes.berg@intel.com, mingo@kernel.org, hpa@zytor.com, tglx@linutronix.de, daniel.vetter@ffwll.ch, chris@chris-wilson.co.uk, peterz@infradead.org, akpm@linux-foundation.org, torvalds@linux-foundation.org, paulmck@linux.vnet.ibm.com In-Reply-To: <1470909022-687-1-git-send-email-johannes@sipsolutions.net> References: <1470909022-687-1-git-send-email-johannes@sipsolutions.net> To: linux-tip-commits@vger.kernel.org Subject: [tip:locking/urgent] Revert "drm/fb-helper: Reduce READ_ONCE(master) to lockless_dereference" Git-Commit-ID: f17b3ea3d2df7c9bf3ce1dbd65b5fd7061f8e787 X-Mailer: tip-git-log-daemon Robot-ID: Robot-Unsubscribe: Contact to get blacklisted from these emails MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Content-Type: text/plain; charset=UTF-8 Content-Disposition: inline Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Commit-ID: f17b3ea3d2df7c9bf3ce1dbd65b5fd7061f8e787 Gitweb: http://git.kernel.org/tip/f17b3ea3d2df7c9bf3ce1dbd65b5fd7061f8e787 Author: Johannes Berg AuthorDate: Thu, 11 Aug 2016 11:50:21 +0200 Committer: Ingo Molnar CommitDate: Thu, 18 Aug 2016 15:36:13 +0200 Revert "drm/fb-helper: Reduce READ_ONCE(master) to lockless_dereference" This reverts commit: fa7d81bb3c269 ("drm/fb-helper: Reduce READ_ONCE(master) to lockless_dereference") As Peter explained: [...] lockless_dereference() is _stronger_ than READ_ONCE(), not weaker. [...] Also, clue is in the name: 'dereference', you don't actually dereference the pointer here, only load it. My next patch breaks the compile without this revert, because it assumes you want to deference and thus also need the struct type visible (which it isn't here), so revert it. Tested-by: Paul E. McKenney Signed-off-by: Johannes Berg Signed-off-by: Peter Zijlstra (Intel) Reviewed-by: Daniel Vetter Cc: Andrew Morton Cc: Chris Wilson Cc: Linus Torvalds Cc: Peter Zijlstra Cc: Thomas Gleixner Link: http://lkml.kernel.org/r/1470909022-687-1-git-send-email-johannes@sipsolutions.net Signed-off-by: Ingo Molnar --- drivers/gpu/drm/drm_fb_helper.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/drm_fb_helper.c b/drivers/gpu/drm/drm_fb_helper.c index ce54e98..0a06f91 100644 --- a/drivers/gpu/drm/drm_fb_helper.c +++ b/drivers/gpu/drm/drm_fb_helper.c @@ -464,7 +464,7 @@ static bool drm_fb_helper_is_bound(struct drm_fb_helper *fb_helper) /* Sometimes user space wants everything disabled, so don't steal the * display if there's a master. */ - if (lockless_dereference(dev->master)) + if (READ_ONCE(dev->master)) return false; drm_for_each_crtc(crtc, dev) {