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;
Date:   Thu, 9 Sep 2021 09:46:20 +0000
From:   Huang Shijie <shijie@os.amperecomputing.com>
To:     David Hildenbrand <david@redhat.com>
Cc:     viro@zeniv.linux.org.uk, akpm@linux-foundation.org,
        jlayton@kernel.org, bfields@fieldses.org,
        torvalds@linux-foundation.org, linux-fsdevel@vger.kernel.org,
        linux-kernel@vger.kernel.org, linux-mm@kvack.org,
        song.bao.hua@hisilicon.com, patches@amperecomputing.com,
        zwang@amperecomputing.com
Subject: Re: [RFC PATCH] fs/exec: Add the support for ELF program's NUMA
 replication
Message-ID: <YTnX7IyC420MNBLq@hsj>
References: <20210906161613.4249-1-shijie@os.amperecomputing.com>
 <2cb841ca-2a04-f088-cee2-6c020ecc9508@redhat.com>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <2cb841ca-2a04-f088-cee2-6c020ecc9508@redhat.com>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?YQvgZOIqmmZAVIs6TXSDMbHMPMDoZT6EP9UM+1Nf4gPEm5U6SqDwMJCCSB/p?=
 =?us-ascii?Q?SNe7a4zg3C/E0BuLMDHyq/o/169npuUgUg1/WAb2LqYw1QVe01dvO1hYw3YB?=
 =?us-ascii?Q?+XXWiUkMSLmJbqX1iuANcvSbwGvzaUHAx95v9Ez6gdHEr19Ef80TrfMh9cvC?=
 =?us-ascii?Q?N8j5vSgE1OiBBuu6Wp14w6s9e7h25gRMaOV0sL+7v5gVLxJC7fvEomjEY6Hj?=
 =?us-ascii?Q?aZGbsF/J9R/UNHwKywnGckWGZbEt6PSp7Iq8yx1R72XsECNQKSe2hfY1kEYW?=
 =?us-ascii?Q?V5IPKGoBhOfMiKvuRBml1y522U13qDBcHos3OkxegzqSnIFbj6dGfnPghko0?=
 =?us-ascii?Q?e+FlFzPlsLY3ku7dmgLaV3VBTvHFRyo3zKvQRvxNFecRFkAaiUJ9/U6zh4oa?=
 =?us-ascii?Q?svTcbxEdsL/oHGGFine7iPnuDtntzAzwJmyjPidxhOuEcW6lg4i2GFWXGaZg?=
 =?us-ascii?Q?U0wyGhAySX6zXDf2YRzOTNtRwDDbl82yUIL+P+OmjT5n8b6UZBeuJX8RH8ie?=
 =?us-ascii?Q?sTorygDf3s/ferm7qk6Bm5LcKk2rodR4a5evYlICiBjBkcpdwLHb7wv6FJYG?=
 =?us-ascii?Q?rryX52RmoTC1Fsw9W3thxC3yu5hZAvGJxj8V98sbNI5SM20uLlHvUTCZ+Mfi?=
 =?us-ascii?Q?Fmkd7GIb+miPT8N1eANTaSSPuDKnsaKM8jpbISsVxugPHdQ7m/KbAL5dGmWv?=
 =?us-ascii?Q?0914qOL9bLayXHqZzUEL02hkiovSeHJmrdS10m7OMurQN+81TxDMczOO2Xk5?=
 =?us-ascii?Q?IjBSDogptmR2WUv0EF6jVJub8SwNsfbcNee+wvp29VAZ7/Ip3NtXAok/mkb2?=
 =?us-ascii?Q?TBshjeXpog2cXGErF7C5MP9+yyqZrl1QIIFGMWorCjitcwMv35eoYEeahETS?=
 =?us-ascii?Q?pmQG1QsYD+orlhQbFHozsgnV9wJ9FUUWS9hBHx42RWqAnLDyNDjwKd+BS47y?=
 =?us-ascii?Q?ALVPjl4S8ThaY3A0VhVnNf3CCLgAwAESBiMGR7f5E7mWajU7NoImoLffcHKn?=
 =?us-ascii?Q?1Td/rDAzdMklKoIoptrsW57SxdOD5eJi46GNDRKQuOgiLGE2G/jNCadJORyC?=
 =?us-ascii?Q?8WZBJLpN6VWzU0t+SUoKa9TaVSmexIwpISDbZJoS5xap2qoCzV6aFPjbwFO4?=
 =?us-ascii?Q?qxYjxyXGwI4pb+fiYJhdsJLFM6brCXXL0VoTCXFVVzs9PyrZ7G0TWc32x2ye?=
 =?us-ascii?Q?AIuXr+oRmDepygHa95471205MeQVZJIMDl+OHAE/4kMCoLvWAuPsavQzFqGJ?=
 =?us-ascii?Q?vNcLaS1grDZ6LGC2e7mS84HTVo2H/zPa3DuHMMBa9E9J4uZ6EEbI1xP3+pIk?=
 =?us-ascii?Q?W+yvke/qvWxJZ7rjWJe4mHvs?=
X-OriginatorOrg: os.amperecomputing.com
X-MS-Exchange-CrossTenant-Network-Message-Id: aa62bd01-a3cf-4e10-1960-08d97333d832
X-MS-Exchange-CrossTenant-AuthSource: MWHPR0101MB3165.prod.exchangelabs.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 09 Sep 2021 01:47:56.4190
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 3bc2b170-fd94-476d-b0ce-4229bdc904a7
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: DCEVysz2gLB8xT3ma1wfT0k+K3eMwYSm2VHPHdHWkAyy9PqPWqKsLHlS8RnsCnAKHw+1BESLJ4xg95POAGt0v/9eSZ+FVAS+tQ0BNe548kbttmiPZ/rqQMajLfCUkc3X
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CO1PR01MB6712
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Mon, Sep 06, 2021 at 11:35:01AM +0200, David Hildenbrand wrote:
> On 06.09.21 18:16, Huang Shijie wrote:
> > This patch adds AT_NUMA_REPLICATION for execveat().
> > 
> > If this flag is set, the kernel will trigger COW(copy on write)
> > on the mmapped ELF binary. So the program will have a copied-page
> > on its NUMA node, even if the original page in page cache is
> > on other NUMA nodes.
> 
> Am I missing something important or is this just absolutely not what we
> want?

Please see the thread:
https://marc.info/?l=linux-kernel&m=163070220429222&w=2

Linus did not think it is a good choice to implement the "per-numa node page cache"

> 
> This means that for each and every invocation of the binary, we'll COW the
> complete binary -- an awful lot of wasted main memory and swap.
Only the one who cares about the performance in NUMA will use this interface.
Most of the people never use it..


Thanks
Huang Shijie