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;
Message-ID: <9f8bb458-439d-a10d-dde4-077bb28c702b@oracle.com>
Date:   Wed, 6 Jul 2022 13:26:59 -0600
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101
 Thunderbird/91.9.1
Subject: Re: [PATCH v2 0/9] Add support for shared PTEs across processes
Content-Language: en-US
To:     Andrew Morton <akpm@linux-foundation.org>
Cc:     willy@infradead.org, aneesh.kumar@linux.ibm.com, arnd@arndb.de,
        21cnbao@gmail.com, corbet@lwn.net, dave.hansen@linux.intel.com,
        david@redhat.com, ebiederm@xmission.com, hagen@jauu.net,
        jack@suse.cz, keescook@chromium.org, kirill@shutemov.name,
        kucharsk@gmail.com, linkinjeon@kernel.org,
        linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org,
        linux-mm@kvack.org, longpeng2@huawei.com, luto@kernel.org,
        markhemm@googlemail.com, pcc@google.com, rppt@kernel.org,
        sieberf@amazon.com, sjpark@amazon.de, surenb@google.com,
        tst@schoebel-theuer.de, yzaikin@google.com
References: <cover.1656531090.git.khalid.aziz@oracle.com>
 <20220701212403.77ab8139b6e1aca87fae119e@linux-foundation.org>
From:   Khalid Aziz <khalid.aziz@oracle.com>
In-Reply-To: <20220701212403.77ab8139b6e1aca87fae119e@linux-foundation.org>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?RjVxLzVidEt0eGVMb1MyaGZNSEJ5dFRVa0lkM2NGcDhvekluQW9IeFJzVkpp?=
 =?utf-8?B?Zm91bEtOU01PTEVJRFN0SGlESlNZcERyZTRtZ2J3WTIyUEJ1WVhSUkhOYndE?=
 =?utf-8?B?UDc5K2pzZE05MjJLMlk5OXd5NHJqL3NHUUtyVzR5eHZzcktlWW1QZHdPR1JT?=
 =?utf-8?B?SlZ2NXpWazBhelR6ZzAvN0tDM0FyWkNFYkJxUm55Qk9IVGtPY2twcG5sUitu?=
 =?utf-8?B?NEZjN3B4SVltRWtiSmMxTkZoUC82N3Q5WTNUcDJESDdSRGlNOUJPeS9wMk5W?=
 =?utf-8?B?bG51ODh6NnZrTWthdzBFc3Q2bGQzTTVGSlBnbWZDVm02TmxFV1pUaTRtWDFU?=
 =?utf-8?B?UXlyY1IvT3lRMHdWSXlNNTRjNkdLc2ZMT3grL1FXZDFDbUd3NVVIYUdibFp4?=
 =?utf-8?B?NjJNeWxERXFNbm1TOHpPTXVjMUlFaEJFL0dLaTZKTSs2YStXRS8xcHpVSy9q?=
 =?utf-8?B?bERtckI0akl6VjZzeXVoWlZERDUwelB3VDhHWDZnUTloWTVvN2NzUWJiRXVL?=
 =?utf-8?B?MzhEck01bGVxblA5eTJDaGs0WitNMmJKcllvY0E4R1FmdXF1M3Z0YUV2R0Ra?=
 =?utf-8?B?ekMxNVV4UmpTRE1jeERVSWJmcjdCVWtKUHBzVkYwSjhFUkNKNFM0NWdLTDJh?=
 =?utf-8?B?aHZSblRrN00zMU1GMWVHQnFDaUhNN2k3QjF0bTE0bGxuVWZXSGROeGlLV3V5?=
 =?utf-8?B?WGozcUdjY3dzQ2xyT3ZkdU1FSGhjV3ZBcWFlVUlaanBrdXNOSkRyWGgzVjNN?=
 =?utf-8?B?R08xTFNKNzJyZ1FCTVVxV282Q1ZTQ3VNYjV2TzFzaXlzbmphT3BEU0gyRnNk?=
 =?utf-8?B?QWkvUTh3RTJKQk5mWEc1OHQ5RlhpdDlPREFoTTA2V0NVYjdwUTJRaUVzTEpB?=
 =?utf-8?B?RlpxS1dHS21qWlZTdnBLVEQ3azkxQjM2OTdTc1MyRE5OcVNuRGhJYXh6b2Nz?=
 =?utf-8?B?bG5XbGI5SWk0Nkd4UWhpcGFxWS9yVnI5MjV3SkhLNXlueWlwZ2E1S1lBRE0y?=
 =?utf-8?B?dXhpQTJ4S3BMdFlBQlBHc1VreUNkR3owWmFGSGRzRHdMSCtJV3hkb3hRMkEx?=
 =?utf-8?B?bWdKNXpmWEVsUHJrcWttNnczOXdNZlduc0JvV0owWTRXN0EwQnhFNzA2YTNa?=
 =?utf-8?B?RGY1azJ1YzMrMnF1dTczWThoTzZGQXhwdzhvcGt3UTVKNGNUTk1jQXVHb3pC?=
 =?utf-8?B?QURLUVZLeG5iaUh5cndJMk1TS05FQjZmWnlRYlNPRVJId1daOVlwbHJzS0JB?=
 =?utf-8?B?K2hsTEgwNkpuZU5pd01DWmcxZmxKRG5PdlluaDNTMDRGNEE1TnRGbzlTK0VB?=
 =?utf-8?B?NTZOZFVkbUw5Qisrb1dEbmUvaFpsay94NHJUSFphMzFNVDBuSTI5ZXd3Y003?=
 =?utf-8?B?b0tZam9EZzA0blhKbXBSUDFZV0NTUTRVRUR1QU1ZeHdKaW1zS0ptSElrdnpG?=
 =?utf-8?B?MVhuNXNvMVE2aEh5ZkZQQzh4WTZ6VmV4NHFZSzNCbjJRYzFJc2JlU3VFNmdj?=
 =?utf-8?B?NE5UVURMTWlEMmRYUDF0QTVkR1ZqQWM5ZHhlZGUzTExOeDh2V0dhY3ZrVE5t?=
 =?utf-8?B?czJMdmFvY3pDQXpaZ3dXVEtxVzZ4R0xPS2pCY3gvZEI0eHhXdUNFZVN4akgv?=
 =?utf-8?B?UjJWbDgvYnM1WU4yQ2NoT2Eydi8ycDdRVlRTbys0VnBUVG00eTdnc2U3bE1S?=
 =?utf-8?B?WGszbkhMQTN2VVI4eE9wWW8wc1k2VldDZkZSS3o4aUVmNzBmdU1mWUlDZENm?=
 =?utf-8?B?clVMeENhcTFJM3dBT1NXWStFUWx5Zzc1aVR0RzVyaU9ZeHk5ZGtqR2pRb3RQ?=
 =?utf-8?B?TVl0SjBubDJnaVZ0UUhYR0htbGRkMjU2bWVYSVM4T296WDJVM0hwRTRlRU5K?=
 =?utf-8?B?VkVWRzJtVmw4TSthYlB1Wlk4ZWxKWU5zWGNGMFY5Nml1K0k1aTIwU01ZeDJG?=
 =?utf-8?B?R1JYL1BhRGdIdmRHcXFkUmZUVUMxY1ZRVFl3b2JxZ1UxSTlLdlY3eHpINnF2?=
 =?utf-8?B?R2VZclhuZTJCSEpXR0lEbkRkOTI4WGRURUU4elZtTG9WRW5iVjZHcmkyMTF6?=
 =?utf-8?B?Y0lIUmRWRTNnRTFid0tuKzlJNnVnVVpJWnF1T2RURW9uYzRmdERlUkc4UTQv?=
 =?utf-8?Q?BtJ2OuLFzJJKktpalAl4fffwH?=
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 4c9434c1-4247-4c7e-25cf-08da5f8582d5
X-MS-Exchange-CrossTenant-AuthSource: BN8PR10MB3220.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 06 Jul 2022 19:27:05.9611
 (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: mxUrCXT2I67MmjBOB48alKNlBXTdvXlDTEBkQtEj9B5Jru8pQa/iVvKPv/7WM5z6Jk0B4wKM/Ac8nQXY+G4XLg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SA2PR10MB4634
X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10434:6.0.517,18.0.883
 definitions=2022-07-06_11:2022-06-28,2022-07-06 signatures=0
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 bulkscore=0 suspectscore=0 phishscore=0
 adultscore=0 malwarescore=0 mlxscore=0 spamscore=0 mlxlogscore=898
 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2206140000
 definitions=main-2207060075
X-Proofpoint-ORIG-GUID: 3EMInjdqasQ4SJr1GBwJOm9MrmhRQPsV
X-Proofpoint-GUID: 3EMInjdqasQ4SJr1GBwJOm9MrmhRQPsV
X-Spam-Status: No, score=-2.8 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,RCVD_IN_DNSWL_LOW,
        RCVD_IN_MSPIKE_H2,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

On 7/1/22 22:24, Andrew Morton wrote:
> On Wed, 29 Jun 2022 16:53:51 -0600 Khalid Aziz <khalid.aziz@oracle.com> wrote:
> 
>> This patch series implements a mechanism in kernel to allow
>> userspace processes to opt into sharing PTEs. It adds a new
>> in-memory filesystem - msharefs.
> 
> Dumb question: why do we need a new filesystem for this?  Is it not
> feasible to permit PTE sharing for mmaps of tmpfs/xfs/ext4/etc files?

Hi Andrew,

The new filesystem is meant to provide only the control files for sharing PTE. It contains a file that provides 
alignment/size requirement. Other files are created as named objects to represent shared regions and these files provide 
information about the size and virtual address for each shared regions when the file is read. Actual shared data is not 
hosted on msharefs. Actual data is mmap'ed using anonymous pages, ext4/xfs/btfrfs/etc files.

Thanks,
Khalid