From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id 1F303EB64D9 for ; Tue, 27 Jun 2023 19:20:33 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id A9F048D0002; Tue, 27 Jun 2023 15:20:32 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id A4E5C8D0001; Tue, 27 Jun 2023 15:20:32 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 8EF7D8D0002; Tue, 27 Jun 2023 15:20:32 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0012.hostedemail.com [216.40.44.12]) by kanga.kvack.org (Postfix) with ESMTP id 7C5D38D0001 for ; Tue, 27 Jun 2023 15:20:32 -0400 (EDT) Received: from smtpin08.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay04.hostedemail.com (Postfix) with ESMTP id 2C1781A096D for ; Tue, 27 Jun 2023 19:20:32 +0000 (UTC) X-FDA: 80949494304.08.A59618B Received: from madras.collabora.co.uk (madras.collabora.co.uk [46.235.227.172]) by imf27.hostedemail.com (Postfix) with ESMTP id 1213640009 for ; Tue, 27 Jun 2023 19:20:29 +0000 (UTC) Authentication-Results: imf27.hostedemail.com; dkim=pass header.d=collabora.com header.s=mail header.b=LOFsHOBX; dmarc=pass (policy=quarantine) header.from=collabora.com; spf=pass (imf27.hostedemail.com: domain of usama.anjum@collabora.com designates 46.235.227.172 as permitted sender) smtp.mailfrom=usama.anjum@collabora.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1687893630; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=BjcnIwiPxapJns6Fvhfj+Gzckwlkn68eXeMaqlAMhxY=; b=g+GxKLB7wcuACXCB0dPQBc/YFvUs0ja34smkJd/3/O/4OhaS/HkTo0s3YDzcP3eWjpLGel TLmcTP6nozxzlLsAxdGBjTn4UGs2mM0FSvbp1pJYZdPvpVGR9/HLB/7QgCr1YhnFLm/EsX Ew6oI/HcgELuyJ29q7gIfvRN104V09M= ARC-Authentication-Results: i=1; imf27.hostedemail.com; dkim=pass header.d=collabora.com header.s=mail header.b=LOFsHOBX; dmarc=pass (policy=quarantine) header.from=collabora.com; spf=pass (imf27.hostedemail.com: domain of usama.anjum@collabora.com designates 46.235.227.172 as permitted sender) smtp.mailfrom=usama.anjum@collabora.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1687893630; a=rsa-sha256; cv=none; b=cXz4cYRWr59rYyrn6vlkjy/4u49uRSqjSSGp3HibhAisKo7Uq8S40tUArljVRQoK5mDfUG vI8fFDiL77fzcG2NIW+jpCjm+dpZv9RUnUTt8VncAIGu342JmA15G/sNMos8rvs3LHa1en ltSRBfxtCIe58CTeFTENDhcSw3vnHrU= Received: from [192.168.10.54] (unknown [182.179.162.32]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits)) (No client certificate requested) (Authenticated sender: usama.anjum) by madras.collabora.co.uk (Postfix) with ESMTPSA id C722C6602B7B; Tue, 27 Jun 2023 20:20:20 +0100 (BST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=collabora.com; s=mail; t=1687893628; bh=F15IKQFG8u9IA2svUTjlj+/hsiBSNLiVRfc6Ea4qK5Y=; h=Date:Cc:Subject:To:References:From:In-Reply-To:From; b=LOFsHOBX4U4Uy7UEUZWZ9MD/sgEEQnXUPJ+PgInwntLBz8NCYLV2NWdKQC6xJSN9t 5PMAm0JXmkf5t+AuNrzjesiNU0aV8cXovWxXYwIo9CdyBjX2SF+SsDzHt7FQD6pdnq AyB4LtU3+GIGkwCpiWzVppiq+uUONypRnev2BLiFqwftfArFrV0c+va3i+VamrQceM 5PDZmAJUNG5ULcVU52WtGNB4O/sW9hbubxcey/dBnPc0TB2COuFDtfrnh0AoDm1wWF Z0Z/OPHd41D5S6/n7POqDERCDuvHs4R9ym079MJE6BQZglz28kGtEBN70vs2HGRj7s wyzaL6AVZZnLg== Message-ID: <6ac9c60e-0a6b-110a-cace-97afbd9708a0@collabora.com> Date: Wed, 28 Jun 2023 00:20:16 +0500 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.12.0 Cc: Muhammad Usama Anjum , Peter Xu , David Hildenbrand , Andrew Morton , Danylo Mocherniuk , Paul Gofman , Cyrill Gorcunov , Mike Rapoport , Nadav Amit , Alexander Viro , Shuah Khan , Christian Brauner , Yang Shi , Vlastimil Babka , "Liam R . Howlett" , Yun Zhou , Suren Baghdasaryan , Alex Sierra , Matthew Wilcox , Pasha Tatashin , Axel Rasmussen , "Gustavo A . R . Silva" , Dan Williams , linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org, linux-mm@kvack.org, linux-kselftest@vger.kernel.org, Greg KH , kernel@collabora.com Subject: Re: [PATCH v21 2/5] fs/proc/task_mmu: Implement IOCTL to get and optionally clear info about PTEs Content-Language: en-US To: =?UTF-8?B?TWljaGHFgiBNaXJvc8WCYXc=?= , Andrei Vagin References: <20230626113156.1274521-1-usama.anjum@collabora.com> <20230626113156.1274521-3-usama.anjum@collabora.com> <13ea54c0-25a3-285c-f47e-d6da11c91795@collabora.com> From: Muhammad Usama Anjum In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-Rspamd-Queue-Id: 1213640009 X-Rspam-User: X-Rspamd-Server: rspam05 X-Stat-Signature: aaad345mpeypn43u3eje31d3fsduj369 X-HE-Tag: 1687893629-672576 X-HE-Meta: U2FsdGVkX195euj19eB6hhddFdRFf89ZgJjh4jX/tQXMpSeN9wMa9lmS8J4NsbsZ9UOJMR0Y8b0/Sl6w0EVoc6EBveVWzSqjtGKaGDBpdVQeEmAS2Y3TS+7a0I2MqHq0tMnWMSZ6v/16CnH+ezh9WlZr65GFtDXsYYwJsC3EfcHaZHTSSLSzZd4qRvRQgj/6EFgsm7kUjhQQopNHtDAtTxl16eKsAoNb+qtrlV+VFwW9SDBb/2foU3PiO0f10GGHqxkT4hBx8CjYC1h8CvISJjqTip0V8yFxqfwxNeabwARmKSUlsVyt1Uo3TKK/BXQbcFhrI8boGqtIDPE4PxLeoBvZw6/YaCAnIj1L/bId5beU6lqT+MLFVApw+Kdof5VzFpV7cIwZb8O+jOiYb63E1/FlGmN2LNIzKgjJK1wts5/QseVi4ECO1P/Jx8XQn5UkGnXDLYDIE7MOd0uJIwEcwqrfvOFozUAmiDeFln3pOuajtW+s1hrIChjUOUgulFAH1BHGMA3dHb/sxkSm73SH8FvAZGAEZeKXXz/aJbZjAms+yUIyf9lYYB0IUaROukiNdlAwDHm0HPeRX7wd3LVWMyPNi8Oj4UpzFOtnWKcnUDkVCDM7wjF5v8WAxwUr7cqKRJhPUkPLIXf13b5b+3u1pVNKBqUV/dlIf6RBehaibdv3b157I/SFoIfaIHQI08O/emJmVgwGXi/SjG9Vr1DEdO90WWU4qBQ3pcq5m2krvDPCowxJGgQPEjhLw1f75/0/ps4batd2YYPD833w07hrp4sGPJsGWWjwo9qPkq7A+Vmy/o5s2jM+ORGteWNG6fSAmv1dxcZFi1E90IYOo02i6VwHbsOiqcLmT78UGGBae7llB23i6QGkq+F3qQaDDhF/Oi54sfCnXKQSm/kcNxjSA2LZdeuGgWrOhW4+zqZLz/4ah4TmwgaDR8asAnNScFA9KwaE3rTu7oBERCuLhL6 fuzHQ/q9 RRI8zD7oG5MrNLEZRFiLMIraT78vYviB+mGBTjDwmXkPCTK5bdOg7MqusG39HjSwAJUsPtzCoZIyQbg1FvSJzaiZs/3HZ8/gLadw6S/8mP4byAb50GH7EYwCJ0WppZnK4y3J7CHIP4oh3YMV7GjDDwLBpwFYn51s3OQzYpqGlHvfkP5Dfam29PuCwOzO/JTfUqzyqQ82Na7sf746Yh7Od6fQCl4YbzxggS409Y2vdCGVw1wRZBNB9qmhWlETiH6MAwJELSwQ0xR+ZYMn6ni+TCDhlhpjfZcFRRg9f5ckoAoLRLVG1djD7AET7NA== X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: Thanks Michał for replying. On 6/27/23 11:52 PM, Michał Mirosław wrote: > On Tue, 27 Jun 2023 at 11:00, Muhammad Usama Anjum > wrote: >> >> Hi Andrei and Michal, >> >> Lets resolve last two points. Please reply below. >> >> On 6/27/23 6:46 AM, Andrei Vagin wrote: > [...] >>> And we need to report an address where it stopped scanning. >>> We can do that by adding zero length vector. >> I don't want to do multiplexing the ending address in vec. Can we add >> end_addr variable in struct pm_scan_arg to always return the ending address? >> >> struct pm_scan_arg { >> ... >> _u64 end_addr; >> }; > > The idea to emit a zero-length entry for the end looks nice. This has > the disadvantage that we'd need to either reserve one entry for the > ending marker or stop the walk after the last entry is no longer > matching. This is ambiguous. > > Another solution would be to rewrite 'start' and 'len'. The caller > would be forced to use non-const `pm_scan_arg`, but I expect the `vec` > pointer would normally be written anyway (unless using only a > statically-allocated buffer). > Also, if the 'len' is replaced with 'end' that would make the ioctl > easily restartable (just call again if start != end). Nice idea. But returning ending address in len seems a bit strange. pm_scan_arg already has 11 members. Wouldn't it be okay to add one more? It would be straight forward as well. If nobody replies until tomorrow, I'll start returning ending address in len. > > Best Regards > Michał Mirosław -- BR, Muhammad Usama Anjum