From patchwork Thu Jun 21 12:13:03 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Mark Rutland X-Patchwork-Id: 139525 Delivered-To: patch@linaro.org Received: by 2002:a2e:970d:0:0:0:0:0 with SMTP id r13-v6csp1991183lji; Thu, 21 Jun 2018 05:13:35 -0700 (PDT) X-Google-Smtp-Source: ADUXVKKjPr1n323bRrHlUT3g9Zf0hQKd2xSra91q8LSlEpk1d7sKl7RAvPKnN3MRDEzHY/lEeyjw X-Received: by 2002:a62:a054:: with SMTP id r81-v6mr26972622pfe.10.1529583215786; Thu, 21 Jun 2018 05:13:35 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1529583215; cv=none; d=google.com; s=arc-20160816; b=iN34TEOr6oWYMGoh6IMBX+kZE/0cmIBdt7bvfpPGNDAngL9MTORhZKz8e6gN+LkICq YHBod1KqQe8BdpQWzBfRjXj8Q+ZYHtFLbLec5gpOwWkdc9mr6iJMXk16pI0ZgDqZ32vo OOwPskStCtEJoqzquIqOypD7rgwvZ35kz8Mjm88QJUuVrRwyKVIfq9qxcuIn8TPblVFC YO2od8kQKm/8R/2L+eLA77zkl5r/2LYAs4TyfqtnVT2Vuv0lNpszYRHOMDUpB4iCUmw7 TfbnQweM/sgeGLKn57NyzalXA/EEFKWgS2KSB3GnKFHDsMyYHMvr1VWTJc2AuiAu5i2e F1LQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:message-id:date:subject:cc:to:from :arc-authentication-results; bh=r68EoZwgwnLHwJZW3VBfLS4PTgkSYd0zm80UiwhcYVA=; b=G94kgyXG6CGub5IOf1aPlds8hgbiYYJ+p1rqOqmADLl80/EK+3LxDzwcDWIfYwPLHT ryLz1E7AAatyx/F013mYpG5JL+zOASxjWiBgKfj70k3oFuTAKTCqdINNIYo6nndx7WRx ycmhC5fIr0Wg59kXebh3XVWwq1dAovA1kS4h9i7739A3fheSfXbCUlyzn42jD/v24Ade vIc6Tcj7Ka5wg/gcIPADGsQt4ysQ7s2iTOqUpiFbMpO2Ex993HbPAIVUb1MV+p4l2500 drhX/pnVbd4FdS1n7N0yaalzhDh4Q1BcyVkYGR0B0sz5mcwysCMS+HOMFZb4C04lx5f/ Nqsg== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id q13-v6si4737622pll.72.2018.06.21.05.13.35; Thu, 21 Jun 2018 05:13:35 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933440AbeFUMNd (ORCPT + 30 others); Thu, 21 Jun 2018 08:13:33 -0400 Received: from usa-sjc-mx-foss1.foss.arm.com ([217.140.101.70]:49214 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932763AbeFUMNb (ORCPT ); Thu, 21 Jun 2018 08:13:31 -0400 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.72.51.249]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 956E51435; Thu, 21 Jun 2018 05:13:31 -0700 (PDT) Received: from lakrids.cambridge.arm.com (usa-sjc-imap-foss1.foss.arm.com [10.72.51.249]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPA id A42773F578; Thu, 21 Jun 2018 05:13:28 -0700 (PDT) From: Mark Rutland To: mingo@kernel.org, will.deacon@arm.com, peterz@infradead.org, linux-kernel@vger.kernel.org Cc: Mark Rutland , albert@sifive.com, arnd@arndb.de, benh@kernel.crashing.org, boqun.feng@gmail.com, geert@linux-m68k.org, ink@jurassic.park.msu.ru, linux@armlinux.org.uk, mattst88@gmail.com, mpe@ellerman.id.au, palmer@sifive.com, paulus@samba.org, rth@twiddle.net, vgupta@synopsys.com Subject: [PATCHv4 00/18] atomics: API cleanups Date: Thu, 21 Jun 2018 13:13:03 +0100 Message-Id: <20180621121321.4761-1-mark.rutland@arm.com> X-Mailer: git-send-email 2.11.0 Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org This series contains a few cleanups of the atomic API, fixing inconsistencies between atomic_* and atomic64_*, and minimizing repetition in arch code. This is nicer for arch code, and the improved regularity will help when generating the atomic headers in future. Since v1 [1]: * Add missing inc_not_zero #define for x86_32 * Remove newly redundant test op definitions for riscv * Remove atomic_inc_not_zero_hint() * Make inc/dec ops optional Since v2 [2]: * Make test ops return bool * Clean up ifdeffery for andnot opes * Update comments for clarity * Rebase to v4.18-rc1 * Accumulate acks * Fix typos Since v3 [3]: * Fix return type of atomic64_fetch_add_unless() for alpha * Fix return type of atomic64_fetch_add_unless() for instrumentation * Fix more typos The bulk of the patches reorganise things so architectures consistently provide _fetch_add_unless(), with atomic_fetch_add_unless() provided as a wrapper by core code. A generic fallback is provided for _fetch_add_unless(), based on _read() and _try_cmpxchg(). Other patches in the series add common fallbacks for: * atomic64_inc_not_zero() * _inc_and_test() * _dec_and_test() * _sub_and_test() * add_negative() * _*inc*() * _*dec*() ... as almost all architectures provide (near-)identical implementation of these today. Where an architecture provides a non-trivial definition, it is updated to provide a matching preprocessor symbol, and the instrumented atomics are updated correspondingly. The end result is a strongly negative diffstat, though grows by a reasonable amount. We could halve this in future by templating the various fallbacks for atomic{,64}_t with code generation. I've pushed this out to my atomics/api-cleanup branch [4] on kernel.org. Thanks, Mark. [1] https://lkml.kernel.org/r/20180523133533.1076-1-mark.rutland@arm.com [2] https://lkml.kernel.org/r/20180529154346.3168-1-mark.rutland@arm.com [3] https://lkml.kernel.org/r/20180618101919.51973-1-mark.rutland@arm.com [4] git://git.kernel.org/pub/scm/linux/kernel/git/mark/linux.git atomics/api-cleanup Mark Rutland (18): atomics/treewide: s/__atomic_add_unless/atomic_fetch_add_unless/ atomics/treewide: remove redundant atomic_inc_not_zero() definitions atomics/treewide: remove atomic_inc_not_zero_hint() atomics: make conditional ops return bool atomics/treewide: make atomic64_inc_not_zero() optional atomics/treewide: make atomic_fetch_add_unless() optional atomics: prepare for atomic64_fetch_add_unless() atomics/generic: define atomic64_fetch_add_unless() atomics/alpha: define atomic64_fetch_add_unless() atomics/arc: define atomic64_fetch_add_unless() atomics/arm: define atomic64_fetch_add_unless() atomics/powerpc: define atomic64_fetch_add_unless() atomics/riscv: define atomic64_fetch_add_unless() atomics/treewide: make atomic64_fetch_add_unless() optional atomics/treewide: make test ops optional atomics/treewide: make unconditional inc/dec ops optional atomics/treewide: make conditional inc/dec ops optional atomics/treewide: clean up andnot ifdeffery arch/alpha/include/asm/atomic.h | 56 ++-- arch/arc/include/asm/atomic.h | 86 ++----- arch/arm/include/asm/atomic.h | 55 +--- arch/arm64/include/asm/atomic.h | 47 +--- arch/h8300/include/asm/atomic.h | 15 +- arch/hexagon/include/asm/atomic.h | 18 +- arch/ia64/include/asm/atomic.h | 81 ------ arch/m68k/include/asm/atomic.h | 24 +- arch/mips/include/asm/atomic.h | 172 ------------- arch/openrisc/include/asm/atomic.h | 4 +- arch/parisc/include/asm/atomic.h | 107 -------- arch/powerpc/include/asm/atomic.h | 52 ++-- arch/riscv/include/asm/atomic.h | 149 +---------- arch/s390/include/asm/atomic.h | 65 ----- arch/sh/include/asm/atomic.h | 35 --- arch/sparc/include/asm/atomic_32.h | 24 +- arch/sparc/include/asm/atomic_64.h | 65 +---- arch/sparc/lib/atomic32.c | 4 +- arch/x86/include/asm/atomic.h | 30 +-- arch/x86/include/asm/atomic64_32.h | 61 +---- arch/x86/include/asm/atomic64_64.h | 48 +--- arch/xtensa/include/asm/atomic.h | 98 ------- drivers/block/rbd.c | 2 +- drivers/infiniband/core/rdma_core.c | 2 +- fs/afs/rxrpc.c | 2 +- include/asm-generic/atomic-instrumented.h | 71 ++++- include/asm-generic/atomic.h | 33 --- include/asm-generic/atomic64.h | 15 +- include/linux/atomic.h | 415 +++++++++++++++++++++++------- kernel/bpf/syscall.c | 4 +- lib/atomic64.c | 12 +- net/atm/pppoatm.c | 2 +- net/rxrpc/call_object.c | 2 +- net/rxrpc/conn_object.c | 4 +- net/rxrpc/local_object.c | 2 +- net/rxrpc/peer_object.c | 2 +- 36 files changed, 511 insertions(+), 1353 deletions(-) -- 2.11.0