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 D640FC43334 for ; Wed, 6 Jul 2022 09:55:40 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id E500D6B0071; Wed, 6 Jul 2022 05:55:39 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id DD7A06B0073; Wed, 6 Jul 2022 05:55:39 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id C79386B0074; Wed, 6 Jul 2022 05:55:39 -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 B5B886B0071 for ; Wed, 6 Jul 2022 05:55:39 -0400 (EDT) Received: from smtpin02.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay13.hostedemail.com (Postfix) with ESMTP id 651416128E for ; Wed, 6 Jul 2022 09:55:39 +0000 (UTC) X-FDA: 79656217998.02.569854A Received: from outbound-smtp35.blacknight.com (outbound-smtp35.blacknight.com [46.22.139.218]) by imf26.hostedemail.com (Postfix) with ESMTP id 8DB35140028 for ; Wed, 6 Jul 2022 09:55:38 +0000 (UTC) Received: from mail.blacknight.com (pemlinmail04.blacknight.ie [81.17.254.17]) by outbound-smtp35.blacknight.com (Postfix) with ESMTPS id D9C7F1D4D for ; Wed, 6 Jul 2022 10:55:36 +0100 (IST) Received: (qmail 18395 invoked from network); 6 Jul 2022 09:55:36 -0000 Received: from unknown (HELO techsingularity.net) (mgorman@techsingularity.net@[84.203.198.246]) by 81.17.254.9 with ESMTPSA (AES256-SHA encrypted, authenticated); 6 Jul 2022 09:55:36 -0000 Date: Wed, 6 Jul 2022 10:55:35 +0100 From: Mel Gorman To: Oliver Sang Cc: Andrew Morton , 0day robot , LKML , linux-mm@kvack.org, lkp@lists.01.org, Nicolas Saenz Julienne , Marcelo Tosatti , Vlastimil Babka , Michal Hocko , Hugh Dickins Subject: Re: [mm/page_alloc] 2bd8eec68f: BUG:sleeping_function_called_from_invalid_context_at_mm/gup.c Message-ID: <20220706095535.GD27531@techsingularity.net> References: <20220613125622.18628-8-mgorman@techsingularity.net> <20220703132209.875b823d1cb7169a8d51d56d@linux-foundation.org> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-15 Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.10.1 (2018-07-13) ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1657101339; a=rsa-sha256; cv=none; b=rJmFVWmyJ7+TGbo3AkM97MrRR3unpxFC0ojMPhruPmXWNGNNend5ELyiS7EE2CJrlxp+WK AtrDCcAEAvMS525YAl40A1M+q5CRvtBeL8MFF5sVtDURzaLxbZ9NHFki0Z8BF44QUa9ZKj Qfr6cBk/+R3q7IxRxFocreOdcXsYBik= ARC-Authentication-Results: i=1; imf26.hostedemail.com; dkim=none; dmarc=none; spf=pass (imf26.hostedemail.com: domain of mgorman@techsingularity.net designates 46.22.139.218 as permitted sender) smtp.mailfrom=mgorman@techsingularity.net ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1657101339; 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: in-reply-to:in-reply-to:references:references; bh=pS50gD/9iwYY9N8MTFdA23ODvNCk4vkyKCmBj5bPpKA=; b=iAHt77mZIE+woyUgsQ02AEl5bUOLPaamcd1iKhDJ6D/chBVR4qP6VxaPg7KDoMPUfzCgtE SMOg5jUJJFnkq5fF+vxXIjNOyBNYTZ1FhP+sDnPn6mcEA8yf+mGmPSr2PR11ARmnHXBOK4 7fBp/1DE3j91MYjRzhMUil/6FOm+UGo= X-Rspamd-Server: rspam04 X-Rspam-User: Authentication-Results: imf26.hostedemail.com; dkim=none; dmarc=none; spf=pass (imf26.hostedemail.com: domain of mgorman@techsingularity.net designates 46.22.139.218 as permitted sender) smtp.mailfrom=mgorman@techsingularity.net X-Stat-Signature: hoqdbsipbmdsn6j87e4uyion6rga6t1e X-Rspamd-Queue-Id: 8DB35140028 X-HE-Tag: 1657101338-251389 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 Tue, Jul 05, 2022 at 09:51:25PM +0800, Oliver Sang wrote: > Hi Andrew Morton, > > On Sun, Jul 03, 2022 at 01:22:09PM -0700, Andrew Morton wrote: > > On Sun, 3 Jul 2022 17:44:30 +0800 kernel test robot wrote: > > > > > FYI, we noticed the following commit (built with gcc-11): > > > > > > commit: 2bd8eec68f740608db5ea58ecff06965228764cb ("[PATCH 7/7] mm/page_alloc: Replace local_lock with normal spinlock") > > > url: https://github.com/intel-lab-lkp/linux/commits/Mel-Gorman/Drain-remote-per-cpu-directly/20220613-230139 > > > base: https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git b13baccc3850ca8b8cccbf8ed9912dbaa0fdf7f3 > > > patch link: https://lore.kernel.org/lkml/20220613125622.18628-8-mgorman@techsingularity.net > > > > > > > Did this test include the followup patch > > mm-page_alloc-replace-local_lock-with-normal-spinlock-fix.patch? > > no, we just fetched original patch set and test upon it. > > now we applied the patch you pointed to us upon 2bd8eec68f and found the issue > still exist. > (attached dmesg FYI) > Thanks Oliver. The trace is odd in that it hits in GUP when the page allocator is no longer active and the context is a syscall. First, is this definitely the first patch the problem occurs? Second, it's possible for IRQs to be enabled and an IRQ delivered before preemption is enabled. It's not clear why that would be a problem other than lacking symmetry or how it could result in the reported BUG but might as well rule it out. This is build tested only diff --git a/mm/page_alloc.c b/mm/page_alloc.c index 934d1b5a5449..d0141e51e613 100644 --- a/mm/page_alloc.c +++ b/mm/page_alloc.c @@ -192,14 +192,14 @@ static DEFINE_MUTEX(pcp_batch_high_lock); #define pcpu_spin_unlock(member, ptr) \ ({ \ - spin_unlock(&ptr->member); \ pcpu_task_unpin(); \ + spin_unlock(&ptr->member); \ }) #define pcpu_spin_unlock_irqrestore(member, ptr, flags) \ ({ \ - spin_unlock_irqrestore(&ptr->member, flags); \ pcpu_task_unpin(); \ + spin_unlock_irqrestore(&ptr->member, flags); \ }) /* struct per_cpu_pages specific helpers. */