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 F151BC3ABD8 for ; Sat, 17 May 2025 00:09:01 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 7E1816B0085; Fri, 16 May 2025 20:09:00 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 791C96B0088; Fri, 16 May 2025 20:09:00 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 5E41E6B0089; Fri, 16 May 2025 20:09:00 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0015.hostedemail.com [216.40.44.15]) by kanga.kvack.org (Postfix) with ESMTP id 3885E6B0085 for ; Fri, 16 May 2025 20:09:00 -0400 (EDT) Received: from smtpin05.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay10.hostedemail.com (Postfix) with ESMTP id C6CB6C0418 for ; Sat, 17 May 2025 00:09:00 +0000 (UTC) X-FDA: 83450464440.05.B6972DB Received: from NAM10-BN7-obe.outbound.protection.outlook.com (mail-bn7nam10on2069.outbound.protection.outlook.com [40.107.92.69]) by imf06.hostedemail.com (Postfix) with ESMTP id 09DB418000A for ; Sat, 17 May 2025 00:08:57 +0000 (UTC) Authentication-Results: imf06.hostedemail.com; dkim=pass header.d=Nvidia.com header.s=selector2 header.b=TnAvt9Yd; spf=pass (imf06.hostedemail.com: domain of ziy@nvidia.com designates 40.107.92.69 as permitted sender) smtp.mailfrom=ziy@nvidia.com; dmarc=pass (policy=reject) header.from=nvidia.com; arc=pass ("microsoft.com:s=arcselector10001:i=1") ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1747440538; 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=X6oVxWtMMQg54NZ+huvySA3grEfsBBuGJV3RAMw3Zf0=; b=5KXWdZQJ8r3Yuajo+MDQj1y2AV8hUJph23t69bBFx4xjJu2YbOWIePyeNOnlSnOB3l40bI Eviaum7nTBS/FwCMz/Jv3e3ckxBsB2TteYhl23tjPPQTLdAyPDuDaWloKapzfXaQuEZ57R DQ4VArJtD+skBT7X5kbjr2F38HJ3fBw= ARC-Authentication-Results: i=2; imf06.hostedemail.com; dkim=pass header.d=Nvidia.com header.s=selector2 header.b=TnAvt9Yd; spf=pass (imf06.hostedemail.com: domain of ziy@nvidia.com designates 40.107.92.69 as permitted sender) smtp.mailfrom=ziy@nvidia.com; dmarc=pass (policy=reject) header.from=nvidia.com; arc=pass ("microsoft.com:s=arcselector10001:i=1") ARC-Seal: i=2; s=arc-20220608; d=hostedemail.com; t=1747440538; a=rsa-sha256; cv=pass; b=RzcYCqKgW/xkGrtM0w1+Xb9T5a0ddz08uF0IxF5yRjCM1am40yXwzeJychXeyXQGb2w4mT yEPp9dnFAjH+WKtqMFOozPBBzJ8wm1IdE0p82bvVNsZB7bQOk0XSegrNHyfFD6Yp4zEJ3I Fsqv3NWHzVCnjOtk0REteHo53Kg7GDI= ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=cEzB0A6ZII4lMdW/f2ll6PT0KpVXn6aCk4qBlamarp2HusQf6KLPw+Cp3WWXoT3UF8Ypsas/eJt+jnh8OJSgi3nEl5xXqe0dyLSPLznJClzEul0EvXF5H2VLjusFW0CzwgjHCCidr5jMJIsKIz7Paw8WKvwsOwepuD6mNnLeLEs7Q8T7xJfgqKyhK6ws607F1S5/x5Fl5i3pT6EWibUxI0g3UReeNY/jFxwTE7kkPBC6+s9qvDfHC7U519ilfXP7ZHy3EG7nedCweWEwD7QOyLOFzb27+PIV2SLQ6kS1JwCx4aXYAZyjecN14z/OCjuGv0rQkLiEV5AUuJ/wI5570g== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=X6oVxWtMMQg54NZ+huvySA3grEfsBBuGJV3RAMw3Zf0=; b=qUBOsydDW6lNW3gvoKOZF/8iV7mEjaehRS3k6euCvH6VqFLfFSoYpok+VUYTHA9OsVKFDf0AIsFM0iHA1x8RsOD0L3huPderaC9mQ14fCcPlYFCIJQhU1tMnqAz3n6tlOyYJqKI3iv08eoFkWRvltrQ67o0m8e0YSeDGILyAj0dRpcvzEk9WfPGQsESYJQ9vfEIZpqGZ2dPYFSiDZF0HPJRmAu+kKEfgACFWkOTS8eGyoL6KbasGoWZFiQcNVvSmLtlsW+Nztu0XXD9DdYfq3O9vCSJ6IKdjT88odOF0CCUWxizrW147GbpQOTz5IbSJy+TEWqb7aeWvb3F0EjULRw== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=nvidia.com; dmarc=pass action=none header.from=nvidia.com; dkim=pass header.d=nvidia.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=Nvidia.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=X6oVxWtMMQg54NZ+huvySA3grEfsBBuGJV3RAMw3Zf0=; b=TnAvt9YdgZGfNYZtwOtrMccA6yZS2JZCMq4ZicKQ0WApYkIDOWdwYAXc990bSVwdreE7LAgLYPDbXpOtku03yI2GQdN5h5HtMYxs7uRcJu3hBZgbNbZU1c9IwMAOSYBV1jmrlid4A3kCst8UtfySx0j7Qkfdn8w+grGIgPMGyN4ehBUQsJjjPN0w1jjDQkAmEn6Vrb3kyxQXDeclR2qaq5Dn9g/z8A06rW11wrzskFv2dRl0ABmKIeO0FAn2BvLkNzHwXXn6oxtmHB1jLvOdCtUyqA+1f+/Oojd8CePFGzUQc4GTfip7u2wBHdt8YDpbBWXaQAfi+295uZkjpPyvdA== Received: from BN5PR12MB9464.namprd12.prod.outlook.com (2603:10b6:408:2ab::10) by DM3PR12MB9287.namprd12.prod.outlook.com (2603:10b6:8:1ac::18) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8722.31; Sat, 17 May 2025 00:08:54 +0000 Received: from BN5PR12MB9464.namprd12.prod.outlook.com ([fe80::e83f:10b6:1259:d000]) by BN5PR12MB9464.namprd12.prod.outlook.com ([fe80::e83f:10b6:1259:d000%4]) with mapi id 15.20.8722.027; Sat, 17 May 2025 00:08:54 +0000 From: Zi Yan To: David Hildenbrand Cc: linux-kernel@vger.kernel.org, linux-s390@vger.kernel.org, kvm@vger.kernel.org, linux-mm@kvack.org, Christian Borntraeger , Janosch Frank , Claudio Imbrenda , Heiko Carstens , Vasily Gorbik , Alexander Gordeev , Sven Schnelle , Thomas Huth , Matthew Wilcox , Sebastian Mitterle Subject: Re: [PATCH v1 2/3] s390/uv: always return 0 from s390_wiggle_split_folio() if successful Date: Fri, 16 May 2025 20:08:53 -0400 X-Mailer: MailMate (2.0r6255) Message-ID: In-Reply-To: <20250516123946.1648026-3-david@redhat.com> References: <20250516123946.1648026-1-david@redhat.com> <20250516123946.1648026-3-david@redhat.com> Content-Type: text/plain Content-Transfer-Encoding: quoted-printable X-ClientProxiedBy: BN0PR02CA0046.namprd02.prod.outlook.com (2603:10b6:408:e5::21) To BN5PR12MB9464.namprd12.prod.outlook.com (2603:10b6:408:2ab::10) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: BN5PR12MB9464:EE_|DM3PR12MB9287:EE_ X-MS-Office365-Filtering-Correlation-Id: 99f395e2-4869-43c6-acbb-08dd94d7030a X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|366016|7416014|376014|1800799024|7053199007; X-Microsoft-Antispam-Message-Info: =?us-ascii?Q?9o8EKEXOkzOPZrFvlj3sD6lfXP9TqJYuJEIX2ru6qH5q82q536G8/OeCW3oU?= =?us-ascii?Q?sTrmy/XDVj4lPA4r3lLkEF6T/4H3CJZmhDjg8JXCwYz7AihVd2VJKh0aXQPy?= =?us-ascii?Q?Xc15LsF6ZvW39WiV4tS1NUHCfF6X2b+D2uxs6rCZvemCI4UrMN9hgLHueZwG?= =?us-ascii?Q?dkSqaBJ84M3JCM/7FxZtyj+MKFmYdskW7Dx6MczmL3Sz2IzvKTr3orI1ARbl?= =?us-ascii?Q?e0TtelKMPlOSo99Aw0vwMnganHjDrJZwTmnQPkVaf1pLS91LL5OZZbTLKZFS?= =?us-ascii?Q?uezXBCbR3h7bImSDc32s42HGIBDAE3RYgxxNEj9doPulJmBWvpGWwMtsh7En?= =?us-ascii?Q?zoRIf1zcNa2A6wf7+Eb9JNG6YxKmu7oc+wXtD0OlaKIsWItn/Cxfz74oHIgo?= =?us-ascii?Q?7XlP4dpGWjjxDKsWz0vcPRHptBD1lhJm2BPBIoaa3B9gG1VaVKU+SfsdQAQL?= =?us-ascii?Q?NT+uYtcDapMTeOPy78K+WdtH0RFks3v1CmvvL2ITabj1+S/WiGViFubwa5d0?= =?us-ascii?Q?zxqDwbTXXL7Sg30u0jt7kFxVIv+7oGqZsGnfBmsXF36TWl5QiSi3Q9MJ+G4d?= =?us-ascii?Q?yj7c1CqZe7frXRHBDCptKOS2M6gSnoSq0zhiC1M7HgYNKlSJExyJpd0zNABF?= =?us-ascii?Q?ORx+P0LCW3ckmYVa1nlZ0mrrGiN1j0r5VYCtDnbfGG53O7t1jh8Pjil7mS/u?= =?us-ascii?Q?WO5bH4Pxxy8oft81FcFV8jpnqY3jcGbSiS85aqzqvxFTh43IM0MnurkUtUny?= =?us-ascii?Q?DUJaKOnlxTA2mJjakBuPZUIkjPwMQp+D0Becsko9EkOCb/HYnr6lJf6ISWrr?= =?us-ascii?Q?Xq6pIY7Q7pdiBOAGSTsgwNjnyCtz1UFRhZqLoYZoKw3x/uuvnua/w/m7awlr?= =?us-ascii?Q?9YgoWAUgBsOqLslLaSfYHrs4sytKrFyJv4iQRbVGG2L8NnHHiT6jzm8RAT2t?= =?us-ascii?Q?MAMvD+v8Wqkdo2W1baoxdhZCKPP3ZE/ba/SMWIBbEykG2hz5Z5mLa9NeoGaT?= =?us-ascii?Q?57Ne4wYryYeVirdYlhJAA/9gSjMMvkbKRS0kXNlC9FIjk2TP8x/t7y7AtWB4?= =?us-ascii?Q?STHZi9xAdgOz7MBTE+w8oemJBTNjE5h6Wmhaws5RI6bSIOR6uCJEmUfXAn85?= =?us-ascii?Q?rdH2gRL5GZxD7O9xHZCG6MKm97tnWhU8kUKuudpX37Y+7fUdIkmt3FhdC6nZ?= =?us-ascii?Q?VNq8NnyQ4ZARinYr2uI3IErH1HS1YSzcosGz0v/MLFBv5SsD0BleIgg4lPHw?= =?us-ascii?Q?9F19qXkcUxMvE2nL3GKFRZzsBHO1vjo3Zv3O/zLy0Iw+v8R0n2cngbPP+InM?= =?us-ascii?Q?9YSV8C5QnQT+ptCJoXwVJPluOtgJf5FXoudPPM7lrkCVBvbsIqhpAetXmoht?= =?us-ascii?Q?pHRyGjnl0+vTczwJ43eWckGX9SVyzkTzSwmdAdAaik3oI/ZWu7P9CAFSTiEN?= =?us-ascii?Q?NeAl3X6gkpQ=3D?= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:BN5PR12MB9464.namprd12.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(366016)(7416014)(376014)(1800799024)(7053199007);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?cVSY6aUK4rsndB/R/FBT8we2WqDmUGjhmFB7TPxvm7JUb/EDJjauHSKvgfLt?= =?us-ascii?Q?WYyGlOdJ6GOYwnCplLGVBqRlMVIdQK2XDcc7g7svHl25yeQgKPDpNL7haGu+?= =?us-ascii?Q?ZC43Dls8WIoKGXzc9wbZGO/KTQdgsJW9WOblddtV+DOKbyJpIxiXZTV8q24m?= =?us-ascii?Q?Ml2Z+Y6+L1lL298Ib9ktdjWpUtaUUfpXaUbRvDZzRtdNN0BTG0H/RqUJId/S?= =?us-ascii?Q?Om08u0QvgpEXUd4S/3SETS0zt+otjftJZTQqYJa8XMGKzBU+C0E1JMnK6j/r?= =?us-ascii?Q?6JW02WR8N47VOFkHvGgjltl7hpazVtlOgASjnjBIFMCDL7ZEBsmlYwa5wmof?= =?us-ascii?Q?HECGlSRVZM20rZlvaXtTXsun5WNR0KAEnyinF+o76JfxHEx+zkha8wOHDnDQ?= =?us-ascii?Q?oj2uWGQCjXKZbMva7rWkdtViV/5MHfAYFPKSBUVKSLzCAOd6pdseenpIyIeD?= =?us-ascii?Q?72khoLEsz0ENpbUSdd+FyUg0VyBYXK4lLqdQ+7LfXY2Y4XNA7rUllERioBWt?= =?us-ascii?Q?9WsPGDnzsWmZelpXdnyQllQHTnbu9nHZ+7DYxy8TqqTUzk+BHIYhYtVfhjcW?= =?us-ascii?Q?hyM6jlEwaq9dQUmLkxGI+U85PfnaCQuYLk7RzJcA89YB0K32w0Z2cqazndL3?= =?us-ascii?Q?Ncc7aFTI5SyZ8spzr1ihy03ykbkUTkPd52WfqxjxDfn9F8sho8e28/nkTmxX?= =?us-ascii?Q?QOsDce2D6MhqcOXDcrkjjr3yT/2FItiU8T10rTvLyO9YjNbjMkTptAPpbbuB?= =?us-ascii?Q?xiOLryQE4uz+FDxRmJ9Wg0KBJ2SQ44MxHQjhvKCz+OGNLWKf+0psN12ekFR9?= =?us-ascii?Q?lvWkEvjKCjpkgqc0qzFrdoiFUk8hEUusY4Jql+K63etstfZqBKVKJy2za7WS?= =?us-ascii?Q?52va+PR+89Y3Ivz4IxDn4aGumrHxr5x2SC6n0uYohUOCsx+q7ZtlzP2VtssP?= =?us-ascii?Q?6kJl9m00/BHAggCa2GXffGwVIiAxPRDwGcs1K+xPfbyNJ3sDI/E5cmhRutyG?= =?us-ascii?Q?0uRKQrjPruPfgU+slVbv3F+EEKUVG+wXZzmjdQGkxrlPi5IvHE7RiduIrOGY?= =?us-ascii?Q?OhZpDW1GcIWwXpl1Xng7eLhelp2ARA3EbUOxAfXsYyhp1YyZFwD9PP7m45Ip?= =?us-ascii?Q?k0Ztex1Q92L1ZsvmXjCGYn7M4p6u0ahbgExxzz8IFNZPLV/I6etsSv5/Olyk?= =?us-ascii?Q?jCZQlsJjo2ZYY1iL/2Lxwp/QGEEnS4E6jnKiHfLjx/zQQMKRnil2aHkpbYfR?= =?us-ascii?Q?0O8t+t5eYTPlHSzWDHS3pLfKTGOrtH4Z9272pQroxNmzD7MKCnpLcSkgnqin?= =?us-ascii?Q?hfhbdO2FqIJu4Y3648hjSb1NURFekPHTLf3jALyRwOn6JrlnlmXk9hgYsrnC?= =?us-ascii?Q?trWU8ycSiOkE6e3Cd1/rws5clgsWud/ib5PJA+cBAUodFpmqD712nroCkJx4?= =?us-ascii?Q?e1sb6AjNuCcoLMVTndPtyDS+AYHu37LlFkSE+RLtzsr5+VAiUYVQEdiInRKl?= =?us-ascii?Q?S7aJ0m0z7PlgAbQZUbTRrqrJTbYdJONHPAHbvm+uYEuybbA0vBJua5PLpm2D?= =?us-ascii?Q?PztqqNnK9yauS73hPLokCyE3jmURnE5vCvBSjYx6?= X-OriginatorOrg: Nvidia.com X-MS-Exchange-CrossTenant-Network-Message-Id: 99f395e2-4869-43c6-acbb-08dd94d7030a X-MS-Exchange-CrossTenant-AuthSource: BN5PR12MB9464.namprd12.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 17 May 2025 00:08:54.7964 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 43083d15-7273-40c1-b7db-39efd9ccc17a X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: uS+1hFEZjbv1wF7DTyDSlZyf/wEecp25cmelwVZT/ck1XtORk5fW0UkwAaNXruo2 X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM3PR12MB9287 X-Rspam-User: X-Rspamd-Queue-Id: 09DB418000A X-Rspamd-Server: rspam09 X-Stat-Signature: 6ncqb547s3ebie46u8exsoad7k3yog17 X-HE-Tag: 1747440537-588572 X-HE-Meta: U2FsdGVkX19y+OaUQDmI2Y5cvYeravKcEkZiaxnmCcNz2PtPnqRcuYhI/DfSZnyS9IL/PSvYltbP+jY8KuIlCOvME66V4xt/F3DePlaeiCsOdwn3O4fS3L9bv8qQ3kJiAIdtBQDLcBZYWZc4dbf5xG4Rp5QpKTqqDBXxeuyT8+zZhUsdqRwysvxaXXXm1r+blJ2oL6UI8xB62C+Ep5IQ2WI/M/Ud3+XU8Hq+IRXmG6U8AFp2MCLJos0JuuAw7EPdLLoNIYhSwRjNVteuNXecCCqoMJF9qVAT/hQTIx78DQBu3ElV+aKHDL6P0AxFqOaID3CaMRqnOEcPxS0Xd/NX/hvBh5ewTQKZpe4pIgGVrfuMzYE/4MAid5TzsgKiHy5lzkgssm0F0cZOPy/otvsCrbQuNQGyerGd9n3rXUiAbmHlRyT42yxWKRc0v4Dgplpx80mi1czRRy6JymBNJQ3eSYGEznerHvwGFdMt5H9LqSQEcmxqosGARdFx9RmvXAu+MK8LxzgqEj6wQNRyFfPyn2oOS8evXUoBnm/HPeR5Yh55sErZFCn34Y1BzqHWGwTQ81ykyXvopnaLkuZv4OCYozUGFd4Np0LaFbBFzio5GyAcf1quhZGbh5H5KTfYQdXLpiGcjPob7uLbr/mGPR9KfJazt58o9j0A8/wkqNgocrJxoS+t/+KirntF+7Mx3vLRCznw483Ejcb164P2L2PwIOwrSK4TzLrTCe0fjdksfHkZsRb8qc9ZNgmJxWgGoFlR+PUowqlRYbrXK88B2YRMsIC62B5Z2W3Ywb4lJBQrOWvGBrfM163EIMP7cfawplGaGTLN692djFxp97lU1ummLZRQK11vMLuQx0v2/B3bmOM3B1dzIXs/+EiFYKjQGqlhfU7cSz2vUnM6fkzlLGVZDVpi88Vih2c5e3HNDviN2S3DqEFLUjOxb1p6q9Q60zdaS2DjJIjNpRy8p97Q0Ps cr/Fislg Qaw8fnGO7SBbxyb1pI710GT1sL7R4e5TU4SyyiOv2qGjdXW6qym5cAtR4zOVfm6AUE4sb+tOjzmi8AqrqKSn8rM+fihWYBKqtOS0aIEQzE2JLxk0jAXnP7HZrjGk8YHE6czeB6jDUMc0EVRKoFme3ClbHP+05PCPm5XqTAMK0bQoHZw9qxqWCYulYJnA8kh6mZK1CW4ng9wRPaUyJdRVKPmIKho4HY/c/iGW8EpgbMytAqznjZ5wqZrbyRXUBpZGRU7JXi1+LqThkI8Sxt4k43CdSF5TWiAeMKcuBF1NJRmWwJ2OuZAehJqdYcVugbcogaalTwpstjVEq+g0TMJmevlph1EgS4y8zceDCzKK4jWum1Z+lHmqSMya5kVxphGsmX727PlUwaW2JJqymKXmjaN5++GDRi66DgXn6FZwT57sr38KEYoBlyOTuDLnOxaQjB6o0K2z+ZC9u8pM= 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 16 May 2025, at 8:39, David Hildenbrand wrote: > Let's consistently return 0 if the operation was successful, and just > detect ourselves whether splitting is required -- folio_test_large() is= > a cheap operation. > > Update the documentation. > > Should we simply always return -EAGAIN instead of 0, so we don't have > to handle it in the caller? Not sure, staring at the documentation, thi= s > way looks a bit cleaner. > > Signed-off-by: David Hildenbrand > --- > arch/s390/kernel/uv.c | 22 ++++++++++++---------- > 1 file changed, 12 insertions(+), 10 deletions(-) > > diff --git a/arch/s390/kernel/uv.c b/arch/s390/kernel/uv.c > index 2cc3b599c7fe3..f6ddb2b54032e 100644 > --- a/arch/s390/kernel/uv.c > +++ b/arch/s390/kernel/uv.c > @@ -324,34 +324,36 @@ static int make_folio_secure(struct mm_struct *mm= , struct folio *folio, struct u > } > > /** > - * s390_wiggle_split_folio() - try to drain extra references to a foli= o and optionally split. > + * s390_wiggle_split_folio() - try to drain extra references to a foli= o and > + * split the folio if it is large. > * @mm: the mm containing the folio to work on > * @folio: the folio > - * @split: whether to split a large folio > * > * Context: Must be called while holding an extra reference to the fol= io; > * the mm lock should not be held. > - * Return: 0 if the folio was split successfully; > - * -EAGAIN if the folio was not split successfully but another= attempt > - * can be made, or if @split was set to false; > - * -EINVAL in case of other errors. See split_folio(). > + * Return: 0 if the operation was successful; > + * -EAGAIN if splitting the large folio was not successful, > + * but another attempt can be made; > + * -EINVAL in case of other folio splitting errors. See split_folio= (). > */ > -static int s390_wiggle_split_folio(struct mm_struct *mm, struct folio = *folio, bool split) > +static int s390_wiggle_split_folio(struct mm_struct *mm, struct folio = *folio) > { > int rc; > > lockdep_assert_not_held(&mm->mmap_lock); > folio_wait_writeback(folio); > lru_add_drain_all(); > - if (split) { > + > + if (folio_test_large(folio)) { > folio_lock(folio); > rc =3D split_folio(folio); > folio_unlock(folio); > > if (rc !=3D -EBUSY) > return rc; > + return -EAGAIN; > } > - return -EAGAIN; > + return 0; > } I can see how this function is written to service as two purposes, trying to get rid of pcp ref of a folio and split a folio (to avoid the extra pcp ref from failing split, lru_add_drain_all() is called before split). Hope it will be refactored later. > > int make_hva_secure(struct mm_struct *mm, unsigned long hva, struct uv= _cb_header *uvcb) > @@ -394,7 +396,7 @@ int make_hva_secure(struct mm_struct *mm, unsigned = long hva, struct uv_cb_header > mmap_read_unlock(mm); > > if (rc =3D=3D -E2BIG || rc =3D=3D -EBUSY) { > - rc =3D s390_wiggle_split_folio(mm, folio, rc =3D=3D -E2BIG); > + rc =3D s390_wiggle_split_folio(mm, folio); > if (!rc) > rc =3D -EAGAIN; > } > -- = > 2.49.0 The changes look good to me. Acked-by: Zi Yan -- Best Regards, Yan, Zi