Received-SPF: pass (google.com: domain of linux-kernel+bounces-93375-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.48.161 as permitted sender) client-ip=147.75.48.161;
From: JonasZhou <JonasZhou-oc@zhaoxin.com>
To: <david@fromorbit.com>
CC: <CobeChen@zhaoxin.com>, <JonasZhou@zhaoxin.com>, <LouisQi@zhaoxin.com>,
	<brauner@kernel.org>, <jack@suse.cz>, <jonaszhou-oc@zhaoxin.com>,
	<linux-fsdevel@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
	<viro@zeniv.linux.org.uk>, <willy@infradead.org>
Subject: Re: [PATCH] fs/address_space: move i_mmap_rwsem to mitigate a false sharing with i_mmap.
Date: Wed, 6 Mar 2024 14:16:22 +0800
Message-ID: <20240306061622.113920-1-JonasZhou-oc@zhaoxin.com>
In-Reply-To: <ZcFsJvUOVMc8e0yO@dread.disaster.area>
References: <ZcFsJvUOVMc8e0yO@dread.disaster.area>
Precedence: bulk
MIME-Version: 1.0
Content-Transfer-Encoding: 8bit
Content-Type: text/plain

> On Mon, Feb 05, 2024 at 02:22:29PM +0800, JonasZhou wrote:
> 
> As I expected, your test is exercising the contention case rather
> than the single, uncontended case. As such, your patch is simply
> optimising the structure layout for the contended case at the
> expense of an extra cacheline miss in the uncontended case.
> 
> I'm not an mm expert, so I don't know which case we should optimise
> for.
> 
> However, the existing code is not obviously wrong, it's just that
> your micro-benchmark exercises the pathological worst case for the
> optimisation choices made for this structure. Whether the contention
> case is worth optimising is the first decision that needs to be
> made, then people can decide if hacking minor optimisations into the
> code is better than reworking the locking and/or algorithm to avoid
> the contention altogether is a better direction...

According to https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/log/?qt=grep&q=unixbench, 
many people use Unixbench and submit optimization patches to Linux based 
on its scores. So this is not my micro-benchmark exercises.

When multiple processes open the same file, such as multiple processes 
opening libc.so, there will be contention.

> -Dave.
> -- 
> Dave Chinner
> david@fromorbit.com