Received-SPF: pass (google.com: domain of linux-kernel+bounces-211154-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
From: "zhai.he" <zhai.he@nxp.com>
To: akpm@linux-foundation.org
Cc: sboyd@kernel.org,
	linux-mm@kvack.org,
	linux-kernel@vger.kernel.org,
	stable@vger.kernel.org,
	zhipeng.wang_1@nxp.com,
	jindong.yue@nxp.com
Subject: [PATCH v2] Supports to use the default CMA when the device-specified CMA memory is not enough.
Date: Wed, 12 Jun 2024 16:12:16 +0800
Message-Id: <20240612081216.1319089-1-zhai.he@nxp.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?us-ascii?Q?yD8XECzhXPd0WOn7zc/1+X3uwVSScC70j4TFwwfp5YqGZ9NCJpm1gIlWTOGw?=
 =?us-ascii?Q?6UWrRnjYY2ve1ltYoCM+ycBZ66j1hX4gJTw+4umjEcuaULmeuzquHZaWcBZz?=
 =?us-ascii?Q?TPMLLk0gLIYBK1igf4wMdZwLyEraFR0J3qwW05K957SfDBPOGZ2kiwzhuudA?=
 =?us-ascii?Q?fKZoIwfPhKHlTwG17/1EZHUJQo7Ymj4JRZwHpdWuC5cd17IjENFoW0GRw2Tq?=
 =?us-ascii?Q?XunqfUp3nyPlp5r+s64doZh/ZmlcDTfayZXTZTrMoIJz+MZiCytSjOzj5Qjy?=
 =?us-ascii?Q?RNC+ElmHVLB6QVmFbV2E4HYFxtVtNPwffd63Qec8oDQhyLi2PEn3JHja8raN?=
 =?us-ascii?Q?k6bpoT3kKE/ecCHbjHO3EO/rOph9MwVOdmyyTdOAIrDzKUjy4i/KOxCVWity?=
 =?us-ascii?Q?m85c4AhD/gm9ZQALTyW8amJ3BmCDNT78i/LmYmo89008uRQcSj0uAXM0WoIS?=
 =?us-ascii?Q?8DAnJkN7Hgu1nRZjDb/tyvR3t/IMa70AfE92Sr68UppOhBr+YQVwzAyjw0Ng?=
 =?us-ascii?Q?tf8oHs/z64v++js+Bs7Yz+eI9SGsWz9ZIc8ybdehyc7zrG2oxnNNE4rqThsP?=
 =?us-ascii?Q?z5q89+zzh9byMhhgar52w9VYxZTGZMBEVnuML4a0fWdFsjf8NzWgdJW+u6RT?=
 =?us-ascii?Q?VwyrXpEVTAUS+QPCikc51iyHEy9xrEiG4mQ6DxwQbC2jYKvOxA6lOtu0N2L2?=
 =?us-ascii?Q?7fvzbes0vWlAe3yRBBmC8GbqxSsN6mvT6gU/Ntm0KZstelCMU6k0k/1LFtYc?=
 =?us-ascii?Q?CZXMm0JU3vGMMTznE0RlxqD4VaaZJlsMA5APeWN4M2vb++7emTKf0nNfrQK4?=
 =?us-ascii?Q?4xruSU8eL50RnctP2WeWOevV6QH2dJ7yNID2Ncv2yIWDYm6XZcT90RR8DzUi?=
 =?us-ascii?Q?PjzKpKD4smFmoVKcfJ7RIFmgwhMNH5jvJhtoXWsYW8TaWA4Hv66EV9X8TKbY?=
 =?us-ascii?Q?utJm0xMQ3kQJHw6esjMZirt53gPe128gevyDe+gMGOqIFCpEILSq2A3yZjLO?=
 =?us-ascii?Q?wqFZlarv1QYaUZOQ4QHG9hLv4WkAOxN7CjnQ6E9ISGrFXdEzPG75CzjgQPAF?=
 =?us-ascii?Q?nNIXxXB4n7wiY6LCiNfzpH6zoET5WWhCEOTiQ4unQ+Jndf80opeIembw9Aju?=
 =?us-ascii?Q?RFlyl3/T9sksEvrJkfs4HT2vm0p7r8G86BSeOEcNOCl/rKDFRGQgomSKPNAE?=
 =?us-ascii?Q?HqVNLB/qy/y4PMdcsCzQpl9f15nfET+uaVahfHM4sBJn/HpB9dVAKMPEvxDq?=
 =?us-ascii?Q?mA2m2XCUVII+PXUxlGDL4rzliju/vhIb/Rv2OeiuCYfPqSWD0L7bMrN4vvqe?=
 =?us-ascii?Q?KY0YeVv6XT2h7Rz+EEcbKRTWoR6OPigefXdcgHl6DNsL9ORp+j1w5GNZ1lTL?=
 =?us-ascii?Q?LoHByMKnlgaBGXtJ3lgjHqE5ECE2zAMNVxeTf7KJ/cX7g3MaQhS50HH+C202?=
 =?us-ascii?Q?oCpxY+G4d0mfjm8MAoSOIf4KxON0EWjpX9QRUyvvyGMVcstfrJAzBQDysQcB?=
 =?us-ascii?Q?rKMZTqiiK+dUTNKPMfUq/qOst1R7uogXwWdnn68BvBoECffe49udT9L5qwnl?=
 =?us-ascii?Q?jo427GT5NJPWmTTC57uv9T1/J/l3bU7KetRtuRNQ?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: a748f26d-1c89-4c9a-b1e3-08dc8ab768f1
X-MS-Exchange-CrossTenant-AuthSource: AS1PR04MB9559.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 12 Jun 2024 08:12:32.5876
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: OnyQf4renci8OuXWPZZA4001VYJ2a0EmM6TMuUC6bXDrubbYp/MIoSKpN6PL4OKZ
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM7PR04MB6904

From: He Zhai <zhai.he@nxp.com>

In the current code logic, if the device-specified CMA memory
allocation fails, memory will not be allocated from the default CMA area.
This patch will use the default cma region when the device's
specified CMA is not enough.

In addition, the log level of allocation failure is changed to debug.
Because these logs will be printed when memory allocation from the
device specified CMA fails, but if the allocation fails, it will be
allocated from the default cma area. It can easily mislead developers'
judgment.

Signed-off-by: He Zhai <zhai.he@nxp.com>
---
 kernel/dma/contiguous.c | 11 +++++++++--
 mm/cma.c                |  4 ++--
 2 files changed, 11 insertions(+), 4 deletions(-)

diff --git a/kernel/dma/contiguous.c b/kernel/dma/contiguous.c
index 055da410ac71..e45cfb24500f 100644
--- a/kernel/dma/contiguous.c
+++ b/kernel/dma/contiguous.c
@@ -357,8 +357,13 @@ struct page *dma_alloc_contiguous(struct device *dev, size_t size, gfp_t gfp)
 	/* CMA can be used only in the context which permits sleeping */
 	if (!gfpflags_allow_blocking(gfp))
 		return NULL;
-	if (dev->cma_area)
-		return cma_alloc_aligned(dev->cma_area, size, gfp);
+	if (dev->cma_area) {
+		struct page *page = NULL;
+
+		page = cma_alloc_aligned(dev->cma_area, size, gfp);
+		if (page)
+			return page;
+	}
 	if (size <= PAGE_SIZE)
 		return NULL;
 
@@ -406,6 +411,8 @@ void dma_free_contiguous(struct device *dev, struct page *page, size_t size)
 	if (dev->cma_area) {
 		if (cma_release(dev->cma_area, page, count))
 			return;
+		if (cma_release(dma_contiguous_default_area, page, count))
+			return;
 	} else {
 		/*
 		 * otherwise, page is from either per-numa cma or default cma
diff --git a/mm/cma.c b/mm/cma.c
index 3e9724716bad..6e12faf1bea7 100644
--- a/mm/cma.c
+++ b/mm/cma.c
@@ -495,8 +495,8 @@ struct page *cma_alloc(struct cma *cma, unsigned long count,
 	}
 
 	if (ret && !no_warn) {
-		pr_err_ratelimited("%s: %s: alloc failed, req-size: %lu pages, ret: %d\n",
-				   __func__, cma->name, count, ret);
+		pr_debug("%s: alloc failed, req-size: %lu pages, ret: %d, try to use default cma\n",
+			    cma->name, count, ret);
 		cma_debug_show_areas(cma);
 	}
 
-- 
2.34.1