Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Imran Khan <imran.f.khan@oracle.com>
To:     tj@kernel.org, gregkh@linuxfoundation.org, viro@zeniv.linux.org.uk
Cc:     linux-kernel@vger.kernel.org
Subject: [PATCH v7 3/8] kernfs: Introduce interface to access kernfs_open_node_lock.
Date:   Fri, 25 Feb 2022 16:21:11 +1100
Message-Id: <20220225052116.1243150-4-imran.f.khan@oracle.com>
In-Reply-To: <20220225052116.1243150-1-imran.f.khan@oracle.com>
References: <20220225052116.1243150-1-imran.f.khan@oracle.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?ARRnQsWlZeG8hDwPx26Lp2/AYR5EhdpbmqbrgYYEOCzSeXApXuwPW17oAvAx?=
 =?us-ascii?Q?UiGcpY0gkIXZlqoxtBNsRLvy57lli7K2p8e7DzGRaxl8m2TYQVjpFeMvKc4Z?=
 =?us-ascii?Q?NkuuIDse2/rY/vInx/wb2z02rH5QRtVKtogE699WhG3P2bKX3kxC2sz8WdNp?=
 =?us-ascii?Q?pyAjUrB/MiLHOd++a1s6SHcyyWVjzKpYKnOAC8YNiM5BComyn9bGnpcTkqCL?=
 =?us-ascii?Q?nu+ZmQwLkdLFhK+o92njpPd7njasax827wCWhbwZjQ5CRzMiVjfePTOzbXIg?=
 =?us-ascii?Q?4VrYA8Mg9E+sI5Y6oUCoSCXGPUyeKe3cqtCu5Um1OzMMoAh88SouQP6rozEl?=
 =?us-ascii?Q?jgEtYMXteUyfLtwCDGTengQwGK0L5GE0Br7zHbqQ7bVwmghybbG+XWum0n7x?=
 =?us-ascii?Q?OvvcgYwTvdnUk5Dzc0l/NylbqfHVTk4uI7/Gq9w4fUjem9rQJkKLhRU8bqp/?=
 =?us-ascii?Q?/vOf+Ti16iFQuRac6NNp4jQLoZ9ULal744Sxf+A0n48ogP8/6GzA5SwVoiAL?=
 =?us-ascii?Q?gRZfgxpx7hWQlHLgQMY5X5pm43kRCb11eeOMGEtonIu6/Vc8hvhUljggrCs0?=
 =?us-ascii?Q?osKEnAUu3seyMcv3Kql/6aY/no61J/nRjwM0uiA4Hyq5+EMe/Ech91QYMeCz?=
 =?us-ascii?Q?kXIe/C21oX0rElhBVZVOxkgkIjxsrKeG+OkiANBYarFfge+HqHrtNxMFpzRf?=
 =?us-ascii?Q?kH3PKIECMyNNfeahf1OcgImZqR+vk9loss+dPOyoYXbXUcWFCTREi8FE+yZ1?=
 =?us-ascii?Q?vSidGsyW1K0gqLzJlw4IJzAeDI+4mxwfGrCaEEiTcz+KmQS46D7PnGCYG+iy?=
 =?us-ascii?Q?QwDMb28fdyCAdZx22/ij8/JED4zU/nVAbcq1Ub/9tYiAYFX322cOexxvxdRB?=
 =?us-ascii?Q?OF4SSTfTVP5REf9ktf4f2ucxbxXT6MalnNbSfYT9WalwmSJY3X7Wq3Jlx17a?=
 =?us-ascii?Q?GHTV+qrK1PUSrDyq/DxCddkkwTtpcYanf+wyt0U+/RJ/ktZ9w93CmPya2pNF?=
 =?us-ascii?Q?Jn4XwUua3CJYefKLF1kc+nl/0kp/Xv5BzCyu6RmVQrK9nnDgARTI/UOKnJ9T?=
 =?us-ascii?Q?+KTIBVqCQpP0J6cZdh54T6xw1BohY+38yuM12fNnmG3JWZ8ElS9CLzGFy7Tb?=
 =?us-ascii?Q?KzNKZtz5DtRVE253r8iX0r3a2OIi/j5TtuXMNO+bKSD+W2iD9q2DMrsUQCoV?=
 =?us-ascii?Q?Ykj/0T3ql2EcAcaHwOIC2qimZzqdVJfMrkhMZjwNP8FlrkmcSE/iH3vs5HE0?=
 =?us-ascii?Q?xP5g4cZyNYrb2EQ41vjqLY0G9SI1blbyI/IBaBdjh5sluAsq4glkcwsvs14L?=
 =?us-ascii?Q?+eEIgoPpwjqptRtkoUhJk/kT9BufTga2WnB19SG9VHMAxGhPqsM/SO6rEsGq?=
 =?us-ascii?Q?LD00atvtDwU2VIDZs54hFWKMMYevS09rkWqMajAaQx9t0PSmuqyNFMH/heyJ?=
 =?us-ascii?Q?4tjrKG9Upo4xGi3e5R98ev7senH9nvxHmN8FezsNFjBEboXw3dKl/djwZ53Q?=
 =?us-ascii?Q?3ZiZNOg3tSXFTBb6BPmtp9maEFqLTnUpP8BH6zZ4sa4PnvlGY+REp/revMVv?=
 =?us-ascii?Q?wo7IZDlb21CFEVmSY2JilSHQDJpZwxaCJhIVoVdRAu81lyoOxgy9M2mXjrsb?=
 =?us-ascii?Q?A2OP5T3z1cflTcGAilJ+zZY=3D?=
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-Network-Message-Id: bc7a619e-7d9f-4749-cb8e-08d9f81eaf59
X-MS-Exchange-CrossTenant-AuthSource: CO1PR10MB4468.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 25 Feb 2022 05:21:32.7052
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 4e2c6054-71cb-48f1-bd6c-3a9705aca71b
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: /irrbPyDets1HF+27Nd7ggrKMVW9JdzJ2ibe7K4/kdAkZ35xoecKQimZ3XOh4c6JrOZQTNaFNG7ArKeSfD0BZg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MWHPR10MB1677
X-Proofpoint-Virus-Version: vendor=nai engine=6300 definitions=10268 signatures=684655
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 bulkscore=0 suspectscore=0 mlxscore=0
 spamscore=0 mlxlogscore=999 adultscore=0 phishscore=0 malwarescore=0
 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2201110000
 definitions=main-2202250027
X-Proofpoint-GUID: 2ywVi-D39Ir_gM4I27GcVYzbcssRoOld
X-Proofpoint-ORIG-GUID: 2ywVi-D39Ir_gM4I27GcVYzbcssRoOld
X-Spam-Status: No, score=-2.8 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_LOW,
        RCVD_IN_MSPIKE_H5,RCVD_IN_MSPIKE_WL,SPF_HELO_NONE,SPF_NONE,
        T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Having an interface allows to change the underlying locking mechanism
without needing to change the user of the lock. For example next patch
modifies this interface to make use of hashed spinlocks in place of global
kernfs_open_node_lock.

Signed-off-by: Imran Khan <imran.f.khan@oracle.com>
---
 fs/kernfs/file.c            | 23 ++++++++++++++---------
 fs/kernfs/kernfs-internal.h | 18 ++++++++++++++++++
 2 files changed, 32 insertions(+), 9 deletions(-)

diff --git a/fs/kernfs/file.c b/fs/kernfs/file.c
index 8996b00568c38..1658bfa048df3 100644
--- a/fs/kernfs/file.c
+++ b/fs/kernfs/file.c
@@ -25,7 +25,7 @@
  * filp->private_data points to seq_file whose ->private points to
  * kernfs_open_file.
  */
-static DEFINE_SPINLOCK(kernfs_open_node_lock);
+DEFINE_SPINLOCK(kernfs_open_node_lock);
 
 struct kernfs_open_node {
 	atomic_t		refcnt;
@@ -515,10 +515,11 @@ static int kernfs_get_open_node(struct kernfs_node *kn,
 {
 	struct kernfs_open_node *on, *new_on = NULL;
 	struct mutex *mutex = NULL;
+	spinlock_t *lock = NULL;
 
  retry:
 	mutex = kernfs_open_file_mutex_lock(kn);
-	spin_lock_irq(&kernfs_open_node_lock);
+	lock = kernfs_open_node_spinlock(kn);
 
 	if (!kn->attr.open && new_on) {
 		kn->attr.open = new_on;
@@ -531,7 +532,7 @@ static int kernfs_get_open_node(struct kernfs_node *kn,
 		list_add_tail(&of->list, &on->files);
 	}
 
-	spin_unlock_irq(&kernfs_open_node_lock);
+	spin_unlock_irq(lock);
 	mutex_unlock(mutex);
 
 	if (on) {
@@ -567,10 +568,13 @@ static void kernfs_put_open_node(struct kernfs_node *kn,
 {
 	struct kernfs_open_node *on = kn->attr.open;
 	struct mutex *mutex = NULL;
+	spinlock_t *lock = NULL;
 	unsigned long flags;
 
 	mutex = kernfs_open_file_mutex_lock(kn);
-	spin_lock_irqsave(&kernfs_open_node_lock, flags);
+	lock = kernfs_open_node_spinlock_ptr(kn);
+
+	spin_lock_irqsave(lock, flags);
 
 	if (of)
 		list_del(&of->list);
@@ -580,7 +584,7 @@ static void kernfs_put_open_node(struct kernfs_node *kn,
 	else
 		on = NULL;
 
-	spin_unlock_irqrestore(&kernfs_open_node_lock, flags);
+	spin_unlock_irqrestore(lock, flags);
 	mutex_unlock(mutex);
 
 	kfree(on);
@@ -763,15 +767,16 @@ void kernfs_drain_open_files(struct kernfs_node *kn)
 	struct kernfs_open_node *on;
 	struct kernfs_open_file *of;
 	struct mutex *mutex = NULL;
+	spinlock_t *lock = NULL;
 
 	if (!(kn->flags & (KERNFS_HAS_MMAP | KERNFS_HAS_RELEASE)))
 		return;
 
-	spin_lock_irq(&kernfs_open_node_lock);
+	lock = kernfs_open_node_spinlock(kn);
 	on = kn->attr.open;
 	if (on)
 		atomic_inc(&on->refcnt);
-	spin_unlock_irq(&kernfs_open_node_lock);
+	spin_unlock_irq(lock);
 	if (!on)
 		return;
 
@@ -916,13 +921,13 @@ void kernfs_notify(struct kernfs_node *kn)
 		return;
 
 	/* kick poll immediately */
-	spin_lock_irqsave(&kernfs_open_node_lock, flags);
+	spin_lock_irqsave(kernfs_open_node_spinlock_ptr(kn), flags);
 	on = kn->attr.open;
 	if (on) {
 		atomic_inc(&on->event);
 		wake_up_interruptible(&on->poll);
 	}
-	spin_unlock_irqrestore(&kernfs_open_node_lock, flags);
+	spin_unlock_irqrestore(kernfs_open_node_spinlock_ptr(kn), flags);
 
 	/* schedule work to kick fsnotify */
 	spin_lock_irqsave(&kernfs_notify_lock, flags);
diff --git a/fs/kernfs/kernfs-internal.h b/fs/kernfs/kernfs-internal.h
index 03e983953eda4..ef5b04d43ef1b 100644
--- a/fs/kernfs/kernfs-internal.h
+++ b/fs/kernfs/kernfs-internal.h
@@ -152,6 +152,8 @@ extern const struct inode_operations kernfs_symlink_iops;
  */
 extern struct kernfs_global_locks *kernfs_locks;
 
+extern spinlock_t kernfs_open_node_lock;
+
 static inline struct mutex *kernfs_open_file_mutex_ptr(struct kernfs_node *kn)
 {
 	int idx = hash_ptr(kn, NR_KERNFS_LOCK_BITS);
@@ -170,4 +172,20 @@ static inline struct mutex *kernfs_open_file_mutex_lock(struct kernfs_node *kn)
 	return lock;
 }
 
+static inline spinlock_t *kernfs_open_node_spinlock_ptr(struct kernfs_node *kn)
+{
+	return &kernfs_open_node_lock;
+}
+
+static inline spinlock_t *kernfs_open_node_spinlock(struct kernfs_node *kn)
+{
+	spinlock_t *lock;
+
+	lock = kernfs_open_node_spinlock_ptr(kn);
+
+	spin_lock_irq(lock);
+
+	return lock;
+}
+
 #endif	/* __KERNFS_INTERNAL_H */
-- 
2.30.2