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 X-Spam-Level: X-Spam-Status: No, score=-12.8 required=3.0 tests=BAYES_00,DKIM_INVALID, DKIM_SIGNED,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE, SPF_PASS,USER_AGENT_GIT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 464EAC55179 for ; Thu, 5 Nov 2020 17:02:58 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id 8816E221FE for ; Thu, 5 Nov 2020 17:02:57 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=fail reason="signature verification failed" (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="SsicXe73" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 8816E221FE Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id DA0446B0149; Thu, 5 Nov 2020 12:02:56 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id D2A516B014A; Thu, 5 Nov 2020 12:02:56 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id BF2BC6B014B; Thu, 5 Nov 2020 12:02:56 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0235.hostedemail.com [216.40.44.235]) by kanga.kvack.org (Postfix) with ESMTP id 874436B0149 for ; Thu, 5 Nov 2020 12:02:56 -0500 (EST) Received: from smtpin11.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay05.hostedemail.com (Postfix) with ESMTP id 1BE4E181AC9CB for ; Thu, 5 Nov 2020 17:02:56 +0000 (UTC) X-FDA: 77450984352.11.sleep09_310b5dc272cb Received: from filter.hostedemail.com (10.5.16.251.rfc1918.com [10.5.16.251]) by smtpin11.hostedemail.com (Postfix) with ESMTP id ECAA1180F8B80 for ; Thu, 5 Nov 2020 17:02:55 +0000 (UTC) X-HE-Tag: sleep09_310b5dc272cb X-Filterd-Recvd-Size: 5559 Received: from mail-pf1-f195.google.com (mail-pf1-f195.google.com [209.85.210.195]) by imf09.hostedemail.com (Postfix) with ESMTP for ; Thu, 5 Nov 2020 17:02:55 +0000 (UTC) Received: by mail-pf1-f195.google.com with SMTP id z3so1835079pfb.10 for ; Thu, 05 Nov 2020 09:02:55 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=sender:from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=MFo6Y7rYb87Eb88yyF9Bn0rmWzJw6Hl9bFsypbTQVcY=; b=SsicXe73212qOk3V2lmG298fU6pI6n8NPV5V83gOgrX2NYmDvjMmXX1oMK7aaz9UYY J3NhFT+8ZM7p8/B5DYIqkAMY4OzBqRCWDljXx0IPDzxt6SmPGWFkVQtbwrwOHMwrenVA BQDlA54czjuSjNJrnXW+ovEvH0/WXPKCCbLcvb5eeyNb+Uf5x3vwci6jrgBhgQMl/6bD iHu2i7q88SMGpWQH59nvzEuu1nJN5s8FbTK09fJQd0tDfeQMKKQ7QaMHkm/3s04Qx1xY 9t2pxYyp65HYTwEa0xNc20qyhA8mtLEXPCybpu4UJ92KWrADaJ/Q++gC4rUwOyiQX2b4 dSTg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:sender:from:to:cc:subject:date:message-id :mime-version:content-transfer-encoding; bh=MFo6Y7rYb87Eb88yyF9Bn0rmWzJw6Hl9bFsypbTQVcY=; b=B5tvv1R096TPaqV1TWkufq4GHn78ISzW6zHZ0BrF/jHin+Wc3hTV9FX1kptTBNTlCO yaEItlteQsrGf3Of8AcHMF5f/s/sE3GPV+VSth2Bw2IGvnOrjJddKXpjh1K2RzHzbnG4 lRHHA+Ee/NLt75XcwLcmiCGYIqcFm8pcR1m+HqXet4HvsWr+9Y7RNArks9Vkgy1KYQcI sAYhnoeEBCg56FZerVFf3MsivJ8Eq44/4F5rYF93sM0fUf3K78UQxxcqhGV0zBWVVucq 3CVWCOlBnvnWWB3QfoebfELjzAyK0fEzTQ5L3QXoFyvxLLbj//xedNU3ghh6jmYrVq/B ZR3Q== X-Gm-Message-State: AOAM533UIQNWPo6TtmB31zfkFJk+4GtzvQt4vae8naPsgGFFeE1J5dHY WDrFkajyGQaTMwXfV5Bm5jc= X-Google-Smtp-Source: ABdhPJwE7qwJZo1qbYdWfUdjZNFKU/vsS216FqHbSWwFFHJIQ/6Dy/foejFEnMsWpMoV6D7q8QaVrQ== X-Received: by 2002:a63:c945:: with SMTP id y5mr3274005pgg.118.1604595774281; Thu, 05 Nov 2020 09:02:54 -0800 (PST) Received: from bbox-1.mtv.corp.google.com ([2620:15c:211:201:7220:84ff:fe09:5e58]) by smtp.gmail.com with ESMTPSA id b185sm2972745pgc.68.2020.11.05.09.02.52 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 05 Nov 2020 09:02:53 -0800 (PST) From: Minchan Kim To: Andrew Morton Cc: linux-kernel@vger.kernel.org, linux-mm , Minchan Kim , "Aneesh Kumar K . V" , Harish Sriram , stable@vger.kernel.org Subject: [PATCH] Revert "mm/vunmap: add cond_resched() in vunmap_pmd_range" Date: Thu, 5 Nov 2020 09:02:49 -0800 Message-Id: <20201105170249.387069-1-minchan@kernel.org> X-Mailer: git-send-email 2.29.1.341.ge80a0c044ae-goog MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable 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: This reverts commit e47110e90584a22e9980510b00d0dfad3a83354e. While I was doing zram testing, I found sometimes decompression failed since the compression buffer was corrupted. With investigation, I found below commit calls cond_resched unconditionally so it could make a problem in atomic context if the task is reschedule. Revert the original commit for now. [ 55.109012] BUG: sleeping function called from invalid context at mm/v= malloc.c:108^M [ 55.110774] in_atomic(): 1, irqs_disabled(): 0, non_block: 0, pid: 946= , name: memhog^M [ 55.111973] 3 locks held by memhog/946:^M [ 55.112807] #0: ffff9d01d4b193e8 (&mm->mmap_lock#2){++++}-{4:4}, at: = __mm_populate+0x103/0x160^M [ 55.114151] #1: ffffffffa3d53de0 (fs_reclaim){+.+.}-{0:0}, at: __allo= c_pages_slowpath.constprop.0+0xa98/0x1160^M [ 55.115848] #2: ffff9d01d56b8110 (&zspage->lock){.+.+}-{3:3}, at: zs_= map_object+0x8e/0x1f0^M [ 55.118947] CPU: 0 PID: 946 Comm: memhog Not tainted 5.9.3-00011-gc5bf= c0287345-dirty #316^M [ 55.121265] Hardware name: QEMU Standard PC (i440FX + PIIX, 1996), BIO= S 1.13.0-1 04/01/2014^M [ 55.122540] Call Trace:^M [ 55.122974] dump_stack+0x8b/0xb8^M [ 55.123588] ___might_sleep.cold+0xb6/0xc6^M [ 55.124328] unmap_kernel_range_noflush+0x2eb/0x350^M [ 55.125198] unmap_kernel_range+0x14/0x30^M [ 55.125920] zs_unmap_object+0xd5/0xe0^M [ 55.126604] zram_bvec_rw.isra.0+0x38c/0x8e0^M [ 55.127462] zram_rw_page+0x90/0x101^M [ 55.128199] bdev_write_page+0x92/0xe0^M [ 55.128957] ? swap_slot_free_notify+0xb0/0xb0^M [ 55.129841] __swap_writepage+0x94/0x4a0^M [ 55.130636] ? do_raw_spin_unlock+0x4b/0xa0^M [ 55.131462] ? _raw_spin_unlock+0x1f/0x30^M [ 55.132261] ? page_swapcount+0x6c/0x90^M [ 55.133038] pageout+0xe3/0x3a0^M [ 55.133702] shrink_page_list+0xb94/0xd60^M [ 55.134626] shrink_inactive_list+0x158/0x460^M Cc: Aneesh Kumar K.V Cc: Harish Sriram Cc: Signed-off-by: Minchan Kim --- mm/vmalloc.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/mm/vmalloc.c b/mm/vmalloc.c index 6ae491a8b210..a4f1d39ce710 100644 --- a/mm/vmalloc.c +++ b/mm/vmalloc.c @@ -102,8 +102,6 @@ static void vunmap_pmd_range(pud_t *pud, unsigned lon= g addr, unsigned long end, if (pmd_none_or_clear_bad(pmd)) continue; vunmap_pte_range(pmd, addr, next, mask); - - cond_resched(); } while (pmd++, addr =3D next, addr !=3D end); } =20 --=20 2.29.1.341.ge80a0c044ae-goog