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 A87AAC433EF for ; Wed, 25 May 2022 08:41:55 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 42DE08D0002; Wed, 25 May 2022 04:41:55 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 4009F8D0001; Wed, 25 May 2022 04:41:55 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 2EDA58D0002; Wed, 25 May 2022 04:41:55 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0010.hostedemail.com [216.40.44.10]) by kanga.kvack.org (Postfix) with ESMTP id 2042E8D0001 for ; Wed, 25 May 2022 04:41:55 -0400 (EDT) Received: from smtpin05.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay02.hostedemail.com (Postfix) with ESMTP id E454A346FF for ; Wed, 25 May 2022 08:41:54 +0000 (UTC) X-FDA: 79503622548.05.49C78A7 Received: from smtp-out1.suse.de (smtp-out1.suse.de [195.135.220.28]) by imf08.hostedemail.com (Postfix) with ESMTP id D697C160031 for ; Wed, 25 May 2022 08:41:32 +0000 (UTC) Received: from imap2.suse-dmz.suse.de (imap2.suse-dmz.suse.de [192.168.254.74]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (P-521) server-digest SHA512) (No client certificate requested) by smtp-out1.suse.de (Postfix) with ESMTPS id 1E6A1219B9; Wed, 25 May 2022 08:41:53 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.de; s=susede2_rsa; t=1653468113; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc: mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=jePr5VZ+W+l9RSA8ra4ij2xbn2TevRksWLlauTlE47w=; b=HPIJ7TIVTH1Neu0j9iQY2qOxzjd3OLblvIm0qwqJJCng9cYlkniJdvDw38RBTXAWK5P+sm 9qBU95jW/qHhTLKlbtwBh7pbkEyZFvU/3paAr2n7s09ovn0JnJJn2V9cDxc6uAiZ+cepfu JI6aUTBFEeS2ph8YoSdmxS8ygvxuWUg= DKIM-Signature: v=1; a=ed25519-sha256; c=relaxed/relaxed; d=suse.de; s=susede2_ed25519; t=1653468113; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc: mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=jePr5VZ+W+l9RSA8ra4ij2xbn2TevRksWLlauTlE47w=; b=JSdQSz5MpNsONqrQkFkCm+KZ4w871BS4eF6IPX6up4upZ4OvZadxucsMQGdolrEIXPJLy5 wIT2fY2src7pEmAw== Received: from imap2.suse-dmz.suse.de (imap2.suse-dmz.suse.de [192.168.254.74]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (P-521) server-digest SHA512) (No client certificate requested) by imap2.suse-dmz.suse.de (Postfix) with ESMTPS id 0208E13ADF; Wed, 25 May 2022 08:41:51 +0000 (UTC) Received: from dovecot-director2.suse.de ([192.168.254.65]) by imap2.suse-dmz.suse.de with ESMTPSA id 4dZ5OM/rjWKRPAAAMHmgww (envelope-from ); Wed, 25 May 2022 08:41:51 +0000 Date: Wed, 25 May 2022 10:41:47 +0200 From: Oscar Salvador To: Miaohe Lin Cc: akpm@linux-foundation.org, mike.kravetz@oracle.com, naoya.horiguchi@nec.com, peterx@redhat.com, apopple@nvidia.com, ying.huang@intel.com, david@redhat.com, songmuchun@bytedance.com, hch@lst.de, dhowells@redhat.com, cl@linux.com, linux-mm@kvack.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v3 3/4] mm/migration: return errno when isolate_huge_page failed Message-ID: References: <20220525081822.53547-1-linmiaohe@huawei.com> <20220525081822.53547-4-linmiaohe@huawei.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20220525081822.53547-4-linmiaohe@huawei.com> X-Rspam-User: X-Rspamd-Queue-Id: D697C160031 X-Stat-Signature: 999x67j8uzgnsqxigwd9thre7485r5gn Authentication-Results: imf08.hostedemail.com; dkim=pass header.d=suse.de header.s=susede2_rsa header.b=HPIJ7TIV; dkim=pass header.d=suse.de header.s=susede2_ed25519 header.b=JSdQSz5M; spf=pass (imf08.hostedemail.com: domain of osalvador@suse.de designates 195.135.220.28 as permitted sender) smtp.mailfrom=osalvador@suse.de; dmarc=pass (policy=none) header.from=suse.de X-Rspamd-Server: rspam03 X-HE-Tag: 1653468092-316962 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: On Wed, May 25, 2022 at 04:18:21PM +0800, Miaohe Lin wrote: > We might fail to isolate huge page due to e.g. the page is under migration > which cleared HPageMigratable. We should return errno in this case rather > than always return 1 which could confuse the user, i.e. the caller might > think all of the memory is migrated while the hugetlb page is left behind. > We make the prototype of isolate_huge_page consistent with isolate_lru_page > as suggested by Huang Ying and rename isolate_huge_page to isolate_hugetlb > as suggested by Muchun to improve the readability. > > Fixes: e8db67eb0ded ("mm: migrate: move_pages() supports thp migration") > Suggested-by: Huang Ying > Signed-off-by: Miaohe Lin Looks good to me, one thing below though: > --- > include/linux/hugetlb.h | 6 +++--- > mm/gup.c | 2 +- > mm/hugetlb.c | 11 +++++------ > mm/memory-failure.c | 2 +- > mm/mempolicy.c | 2 +- > mm/migrate.c | 5 +++-- > 6 files changed, 14 insertions(+), 14 deletions(-) > ... > --- a/mm/migrate.c > +++ b/mm/migrate.c > @@ -1627,8 +1627,9 @@ static int add_page_for_migration(struct mm_struct *mm, unsigned long addr, > > if (PageHuge(page)) { > if (PageHead(page)) { > - isolate_huge_page(page, pagelist); > - err = 1; > + err = isolate_hugetlb(page, pagelist); > + if (!err) > + err = 1; > } We used to always return 1 which means page has been queued for migration, as we did not check isolate_huge_page() return value. Now, we either return 1 or 0 depending on isolate_hugetlb(). I guess that was fine because in the end, if isolate_huge_page() failed, the page just does not get added to 'pagelist', right? So, it is just confusing for the user because he might not get an error so he thinks the page will be migrated, and yet will not? -- Oscar Salvador SUSE Labs