Received: by 2002:a05:6a10:9848:0:0:0:0 with SMTP id x8csp453339pxf; Wed, 31 Mar 2021 07:33:40 -0700 (PDT) X-Google-Smtp-Source: ABdhPJyYdWggoG7jPv/K73ryK48DWmVmHuSYjaQvPqV0DS6sjpUVV1nQLCvZLdXwZxkCT+t2+rk5 X-Received: by 2002:a05:6402:1350:: with SMTP id y16mr4035462edw.309.1617201220215; Wed, 31 Mar 2021 07:33:40 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1617201220; cv=none; d=google.com; s=arc-20160816; b=sYS3yFAX2cBjus96hdLmz+hnkjBEUCYXVfyld8qTNYHf0ep8fdcbvibTErEqnWm6iu k2xpvtoLRDMEOuM4yjeNB//06Knz+zKBhY52DMeT3q+zZvcfttZyb7jODGawiEXDkKwr oLZmBXn+3FMzZUXq0lpmiG/nTq8wlKOYY649M/SAIaxpw/Roaphmr05tW4oNDLkQnCVT rhgsyxOQYnO9q6zKnOlk2T0ea7D/R85QIOBELn81J3j/Eah44pEPGvCBq5gZ2QFYuVsx 4TslbbgQUxMfRsTEYG9+XbaQrw4jpo1jFTMholwXi4bapkAppcn9WlFG7LNJ64FDKPhc +kbg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:message-id:date:subject:cc:to:from :dkim-signature; bh=ak6UnX/J/u2EhlYU06zGBNlptJIVjIKeYqN7uOxK5+8=; b=OIBNE2WnU0ajiG1B0RFXjMptUoH+Tt8Wlgso2wjgBx86Os+05GwMBeLyZsn28JKXt+ LjvkKZLxEyyPTuFWpk8f5rNha2kCPaWtnz8x/OSuiPg3SLOU40ZxlJaTnM9xjMVF3jIr nmuQUfy1l+H1EmHpxaQMrFE6Xnk0P+uBvSuQImT9iymQauZ55fwwEQthJivaor77SjlE Q95QmUfGFSxgkDqvD4lnML3KKnw/Fzqp73VWWOBxEiElckGLC5R2XyT940/wcLmBqX3Y S5HfPcqbAiGDZ6hLZhd+So5bKUk6lPEGiZ9xMOWW20W/gAjTqU0P5ayZtCkrYUbSPGb1 ciSA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b="nzY/wb74"; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id a17si1752428eju.485.2021.03.31.07.33.18; Wed, 31 Mar 2021 07:33:40 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b="nzY/wb74"; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S235835AbhCaObu (ORCPT + 99 others); Wed, 31 Mar 2021 10:31:50 -0400 Received: from mail.kernel.org ([198.145.29.99]:43552 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S235630AbhCaObo (ORCPT ); Wed, 31 Mar 2021 10:31:44 -0400 Received: by mail.kernel.org (Postfix) with ESMTPSA id 01F3760FF0; Wed, 31 Mar 2021 14:31:36 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1617201104; bh=Oq2p2++lqOERCPM++AATk5Rdw/3Ja6gaVMhRgPIzPF0=; h=From:To:Cc:Subject:Date:From; b=nzY/wb74M0jYfXHnZiSzoI4o7pq5Y3fTkzio1WdfkdRKufUohKQwkm1PBLN3tufHE 9OgAOpE32h2+T9nnF1ZCPS1coRoUrCzNA6DyKMuNSxpqkRqMIdROAGSe2CoLT6u0u0 /1h66strn7saF1NEJrq8/A+Hy2dp6ib8IsgmKIVMt9rrPFl0WEWvgcqy/N+E6U5hwm TGkr8KWwp9XlnInAw2drOXo7N61mG3GRNZ4E45pXzIx/rRrUYdKrtzVzakE1zxYrUT pBm1qXLMVF5/zB174lV3VS47Fv+LLoFhQ33oZLj5e8FJszP+LGNs4UTY3rumbNQKgY KMQvZq5datBQg== From: guoren@kernel.org To: guoren@kernel.org Cc: linux-riscv@lists.infradead.org, linux-kernel@vger.kernel.org, linux-csky@vger.kernel.org, linux-arch@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, linux-xtensa@linux-xtensa.org, openrisc@lists.librecores.org, sparclinux@vger.kernel.org, Guo Ren Subject: [PATCH v6 0/9] riscv: Add qspinlock/qrwlock Date: Wed, 31 Mar 2021 14:30:31 +0000 Message-Id: <1617201040-83905-1-git-send-email-guoren@kernel.org> X-Mailer: git-send-email 2.7.4 Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Guo Ren Current riscv is still using baby spinlock implementation. It'll cause fairness and cache line bouncing problems. Many people are involved and pay the efforts to improve it: - The first version of patch was made in 2019.1: https://lore.kernel.org/linux-riscv/20190211043829.30096-1-michaeljclark@mac.com/#r - The second version was made in 2020.11: https://lore.kernel.org/linux-riscv/1606225437-22948-2-git-send-email-guoren@kernel.org/ - A good discussion at Platform HSC.2021-03-08: https://drive.google.com/drive/folders/1ooqdnIsYx7XKor5O1XTtM6D1CHp4hc0p - A good discussion on V4 in mailling list: https://lore.kernel.org/linux-riscv/1616868399-82848-1-git-send-email-guoren@kernel.org/T/#t - Openrisc's maintainer want to implement arch_cmpxchg infrastructure. https://lore.kernel.org/linux-riscv/1616868399-82848-1-git-send-email-guoren@kernel.org/T/#m11b712fb6a4fda043811b1f4c3d61446951ed65a Hope your comments and Tested-by or Co-developed-by or Reviewed-by ... Let's kick the qspinlock into riscv right now (Also for the architecture which hasn't xchg16 atomic instruction.) Change V6: - Add ticket-lock for riscv, default is qspinlock - Keep ticket-lock for csky, default is ticketlock - Using smp_cond_load for riscv ticket-lock - Optimize csky ticketlock with smp_cond_load, store_release - Add PPC_LBARX_LWARX for powerpc Change V5: - Fixup #endif comment typo by Waiman - Remove cmpxchg coding convention patches which will get into a separate patchset later by Arnd's advice - Try to involve more architectures in the discussion Change V4: - Remove custom sub-word xchg implementation - Add ARCH_USE_QUEUED_SPINLOCKS_XCHG32 in locking/qspinlock Change V3: - Coding convention by Peter Zijlstra's advices Change V2: - Coding convention in cmpxchg.h - Re-implement short xchg - Remove char & cmpxchg implementations Guo Ren (8): locking/qspinlock: Add ARCH_USE_QUEUED_SPINLOCKS_XCHG32 riscv: locks: Introduce ticket-based spinlock implementation csky: locks: Optimize coding convention csky: Convert custom spinlock/rwlock to generic qspinlock/qrwlock openrisc: qspinlock: Add ARCH_USE_QUEUED_SPINLOCKS_XCHG32 sparc: qspinlock: Add ARCH_USE_QUEUED_SPINLOCKS_XCHG32 xtensa: qspinlock: Add ARCH_USE_QUEUED_SPINLOCKS_XCHG32 powerpc/qspinlock: Add ARCH_USE_QUEUED_SPINLOCKS_XCHG32 Michael Clark (1): riscv: Convert custom spinlock/rwlock to generic qspinlock/qrwlock arch/csky/Kconfig | 8 ++ arch/csky/include/asm/Kbuild | 2 + arch/csky/include/asm/spinlock.h | 15 +-- arch/csky/include/asm/spinlock_types.h | 4 + arch/openrisc/Kconfig | 1 + arch/powerpc/Kconfig | 1 + arch/riscv/Kconfig | 8 ++ arch/riscv/include/asm/Kbuild | 3 + arch/riscv/include/asm/spinlock.h | 158 +++++++++--------------- arch/riscv/include/asm/spinlock_types.h | 26 ++-- arch/sparc/Kconfig | 1 + arch/xtensa/Kconfig | 1 + kernel/Kconfig.locks | 3 + kernel/locking/qspinlock.c | 46 +++---- 14 files changed, 142 insertions(+), 135 deletions(-) -- 2.17.1