From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756621Ab2HNPRQ (ORCPT ); Tue, 14 Aug 2012 11:17:16 -0400 Received: from mga11.intel.com ([192.55.52.93]:25364 "EHLO mga11.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756508Ab2HNPRN (ORCPT ); Tue, 14 Aug 2012 11:17:13 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.77,766,1336374000"; d="scan'208";a="200758916" Date: Tue, 14 Aug 2012 23:17:04 +0800 From: Fengguang Wu To: David Howells Cc: Koichi Yasutake , "Eric W. Biederman" , linux-am33-list@redhat.com, linux-kernel@vger.kernel.org Subject: [PATCH] mn10300: select generic atomic64_t support Message-ID: <20120814151704.GA17668@localhost> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Rationals from Eric: So I just looked a little deeper and it appears architectures that do not support atomic64_t are broken. The generic atomic64 support came in 2009 to support the perf subsystem with the expectation that all architectures would implement atomic64 support. Furthermore upon inspection of the kernel atomic64_t is used in a fair number of places beyond the performance counters: block/blk-cgroup.c drivers/acpi/apei/ drivers/block/rbd.c drivers/crypto/nx/nx.h drivers/gpu/drm/radeon/radeon.h drivers/infiniband/hw/ipath/ drivers/infiniband/hw/qib/ drivers/staging/octeon/ fs/xfs/ include/linux/perf_event.h include/net/netfilter/nf_conntrack_acct.h kernel/events/ kernel/trace/ net/mac80211/key.h net/rds/ The block control group, infiniband, xfs, crypto, 802.11, netfilter. Nothing quite so fundamental as fs/namespace.c but definitely in multiplatform-code that should work, and is already broken on those architecutres. Looking at the implementation of atomic64_add_return in lib/atomic64.c the code looks as efficient as these kinds of things get. Which leads me to the conclusion that we need atomic64 support on all architectures. CC: "Eric W. Biederman" Signed-off-by: Fengguang Wu --- arch/mn10300/Kconfig | 1 + 1 file changed, 1 insertion(+) --- linux.orig/arch/mn10300/Kconfig 2012-08-03 15:36:21.391560778 +0800 +++ linux/arch/mn10300/Kconfig 2012-08-14 22:53:14.751795832 +0800 @@ -8,6 +8,7 @@ config MN10300 select HAVE_ARCH_KGDB select HAVE_NMI_WATCHDOG if MN10300_WD_TIMER select GENERIC_CLOCKEVENTS + select GENERIC_ATOMIC64 config AM33_2 def_bool n