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 1C6E7C4828D for ; Wed, 7 Feb 2024 04:34:40 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 777D46B006E; Tue, 6 Feb 2024 23:34:39 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 727606B0074; Tue, 6 Feb 2024 23:34:39 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 617086B0078; Tue, 6 Feb 2024 23:34:39 -0500 (EST) 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 52CBC6B006E for ; Tue, 6 Feb 2024 23:34:39 -0500 (EST) Received: from smtpin30.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay05.hostedemail.com (Postfix) with ESMTP id 11F7E406E2 for ; Wed, 7 Feb 2024 04:34:39 +0000 (UTC) X-FDA: 81763741878.30.305DB0D Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by imf06.hostedemail.com (Postfix) with ESMTP id 02DBD180005 for ; Wed, 7 Feb 2024 04:34:36 +0000 (UTC) Authentication-Results: imf06.hostedemail.com; dkim=none; spf=pass (imf06.hostedemail.com: domain of anshuman.khandual@arm.com designates 217.140.110.172 as permitted sender) smtp.mailfrom=anshuman.khandual@arm.com; dmarc=pass (policy=none) header.from=arm.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1707280477; 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; bh=Rh55OmiDW3osJyvFQP1JxdUiPu71ABgwdYFtA+V9n/4=; b=7bQjHyZMaRGrUDPpxwHwqaOs71ayGoxeDMh1qejvD57pVLKWAsipB7KwoOwDQRk/hD8LXW nl9GiHb7EGcE89E+ZaUZY+JxX33Ih/enrdVGQP+xmX2TzB2dKYuQ0XHNnaEJJqp0m6iJtj rvoDZJgV2U+CVWaIkaGvhRSEoaOim5A= ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1707280477; a=rsa-sha256; cv=none; b=SqbEv68Ne33osgLyZ7/hnkVU8cEs1FdPXjx3jFZQ4kuQeF21gl8bfQWyAnlGLWQvUGdJYr 0XLwZaLh/qehnl1r6kKYBGM6NfaOmEmtvBsyWlbv4Xzao4K6ip7J7YUNwZS61aJHUIW8iU h1srwQqlS9zvurpd773/xhHjXr4DqmQ= ARC-Authentication-Results: i=1; imf06.hostedemail.com; dkim=none; spf=pass (imf06.hostedemail.com: domain of anshuman.khandual@arm.com designates 217.140.110.172 as permitted sender) smtp.mailfrom=anshuman.khandual@arm.com; dmarc=pass (policy=none) header.from=arm.com Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 267511FB; Tue, 6 Feb 2024 20:35:18 -0800 (PST) Received: from [10.162.40.23] (a077893.blr.arm.com [10.162.40.23]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 6BA453F5A1; Tue, 6 Feb 2024 20:34:34 -0800 (PST) Message-ID: <8d9f2950-6805-481f-a5e7-e5ea45128fb5@arm.com> Date: Wed, 7 Feb 2024 10:04:31 +0530 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH] mm/cma: Add sysfs file 'release_pages_success' Content-Language: en-US To: Andrew Morton Cc: linux-mm@kvack.org, alexandru.elisei@arm.com, linux-kernel@vger.kernel.org References: <20240206045731.472759-1-anshuman.khandual@arm.com> <20240206093857.d834af4f96d643c53e29e02d@linux-foundation.org> From: Anshuman Khandual In-Reply-To: <20240206093857.d834af4f96d643c53e29e02d@linux-foundation.org> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-Stat-Signature: r5g14eni68ssdgs3ipwaxp1yono7bzon X-Rspamd-Server: rspam10 X-Rspamd-Queue-Id: 02DBD180005 X-Rspam-User: X-HE-Tag: 1707280476-314130 X-HE-Meta: U2FsdGVkX18G7rfQRPWSt33xRzHkq22ZEjF+Faa8KrHHjZVQoAT4VSWTqQEvDj8KkLIIL2bcJMSfXP/6i7dT19rU9pvkdOfhLkS2kNAqw824kCMw+RhuEHWu7Ai3kRIswJzSglwfRMjN9hbHgnBSR31f+7yjErPwW61chSpNEvX4Ik5XmuJ8cPCq86aJwl91milVrdZ8fWS7G65gTcpjxPMKCjZtZumACWvKVWJSaSsQ9A+PnDRq/n/AcVKPclyfCtjQozy9Yf/TOrcj/09T0eGSje/xL8kOLJJozAgw48K9NxqihOzoT78E6fy0N65TVRNwb8XnOlHNFjUDxOIA7Q8WmPKVqEme2pClAWPGZphFJV4mpiiS8vQ1WTUGm+IGUlg4T7u49G5flwcMOJaU+akroxAduaw50MIOEoyOFwXQZ3UldM9fwGOAthvI7rZqD4iVzQs8OO8T4Q+LGn001MDQIukiXMkzoASOtFW15Bt3N2KRn17W90G5rBlpBozaJ8b+ig1ULZcB/0ohhWXFvaCYuZxDUsLr3Tsn+3HhIssO85saYrXCcKOM9u79lsZ00GQpfAM/e8SJYdVzoBKlNjtvgQKmXKyfP6J9TX3TIIY3E29FY/m8z5u/ajeEtSgCjEExL4iJYFw1NjWY3Sv7vNt4mFI03i9T9HSjTRTFMmOQafDRlbedHAPlNVj5p7ZftssxGCZoJgyzS/7+c/RuPwyjulLJFWEnp2+dPNfMfpbRzxYYcke6sclQsMahP6mLodDAklCiEfSNa9Esw76c3AqOrCb/Xb6ookLtFm5o8LXaRsNiioqfPwO+NVqHAexXIcJGWzvtMZI+4V8Ec8REGf4azCbJehBOOw/WW9twiqEG7sIjZ2MVJw4uUSzCpJ+jAfYElU2W1eIMlFJjdccfMAx2OBaf7NkAGLj2ahKw5zgjLKEGAdBuuAow14eK+bwYk4789tHJdWMr6K2GWbi +D9K0TBI ro1ac7Gz4dxK+F7v/oLi1Kn/PStY3C0vJA4jgpybeMc6tiod7/9y4DVjFA9j0t8BpReV1FOETDeEBJPaZvX8gIWwAxuHotliF5JEUWZBbTwP8zu2EQ9wYMO0B3bJNSYsGTS24yPyOVyW0kwD3KK08KiX2p2wW84i6oUmegjhdb5plnmBKGExlMDfqc4htSIIPUvnT6UkmBHIPiZcsVlvxQKFtCDgpue6j1xzQ09oY5B63ZlPPSRbUf6jK08LIHww7URiPl/hINwa5cgu7mIKO8LR5lKcJkFNIT1gIZ9YPMGE+yUmVfHxSUpAMh4h12KMaX7XOl10IURekKmUL907C9X7SCg== 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: List-Subscribe: List-Unsubscribe: On 2/6/24 23:08, Andrew Morton wrote: > On Tue, 6 Feb 2024 10:27:31 +0530 Anshuman Khandual wrote: > >> This adds the following new sysfs file tracking the number of successfully >> released pages from a given CMA heap area. Also like before - this will be >> available via CONFIG_CMA_SYSFS, and help in determining active CMA pages >> available on the system. > > "like before" is mysterious. Is this referring to some other patch? > To existing code? Please be explicit and complete. This was just referring to existing sysfs files i.e 'alloc_pages_success' and 'alloc_pages_fail'. > >> /sys/kernel/mm/cma//release_pages_success >> >> It adds an element 'nr_pages_released' (with CONFIG_CMA_SYSFS config) into >> 'struct cma' which gets updated during cma_release(). > > The changelog doesn't explain why Linux needs this feature. The value > to our users. Perhaps that info is buried in the link which is buried > below the ^---$, but as this is the most important part of a changelog, > it really should be spelled out here, completely and carefully please. Does this look better ? mm/cma: Add sysfs file 'release_pages_success' This adds the following new sysfs file tracking the number of successfully released pages from a given CMA heap area. This file will be available via CONFIG_CMA_SYSFS and help in determining active CMA pages available on the CMA heap area. This adds a new 'nr_pages_released' (CONFIG_CMA_SYSFS) into 'struct cma' which gets updated during cma_release(). /sys/kernel/mm/cma//release_pages_success After this change, an user will be able to find active CMA pages available in a given CMA heap area via the following method. Active pages = alloc_pages_success - release_pages_success That's valuable information for both software designers, and system admins as it allows them to tune the number of CMA pages available in the system. This increases user visibility for allocated CMA area and its utilization. > >> --- >> This patch applies on v6.8-rc3 >> >> Some earlier relevant discussions regarding arm64 MTE dynamic tag storage >> in this regard can be found here. >> >> https://lore.kernel.org/all/ZbpKyNVHfhf1-AAv@raptor/