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 0B1B7C02180 for ; Mon, 13 Jan 2025 23:05:32 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 823A66B0083; Mon, 13 Jan 2025 18:05:31 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 7ABF46B0089; Mon, 13 Jan 2025 18:05:31 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 624816B008A; Mon, 13 Jan 2025 18:05:31 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0012.hostedemail.com [216.40.44.12]) by kanga.kvack.org (Postfix) with ESMTP id 375956B0083 for ; Mon, 13 Jan 2025 18:05:31 -0500 (EST) Received: from smtpin15.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay04.hostedemail.com (Postfix) with ESMTP id EF6741A046B for ; Mon, 13 Jan 2025 23:05:30 +0000 (UTC) X-FDA: 83003962020.15.81EEA21 Received: from mail-pl1-f170.google.com (mail-pl1-f170.google.com [209.85.214.170]) by imf22.hostedemail.com (Postfix) with ESMTP id E412CC000A for ; Mon, 13 Jan 2025 23:05:28 +0000 (UTC) Authentication-Results: imf22.hostedemail.com; dkim=pass header.d=gmail.com header.s=20230601 header.b=nYSVDREG; spf=pass (imf22.hostedemail.com: domain of groeck7@gmail.com designates 209.85.214.170 as permitted sender) smtp.mailfrom=groeck7@gmail.com; dmarc=none ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1736809529; h=from:from:sender: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:dkim-signature; bh=VFqUUj0QdJm5qDnw1Ljz6cwuPJ9cjiThS0ltMVoX+yw=; b=ULdOuUsukARqXJ9ASwW0iYQXDnUd6Xflykpx0rzlLetRAI3bg/iMG+pcoYj3Sn1xsi2QFh Oco/MhrXpwZb/wJ+JjR1GaHKDH1NGyuOG/RoeTIbXfz/lchDCTiVxDHRRshMc8K6sLYN7e xCRd9WzeQwZlE6WfAW7qSIcNmOYkzgE= ARC-Authentication-Results: i=1; imf22.hostedemail.com; dkim=pass header.d=gmail.com header.s=20230601 header.b=nYSVDREG; spf=pass (imf22.hostedemail.com: domain of groeck7@gmail.com designates 209.85.214.170 as permitted sender) smtp.mailfrom=groeck7@gmail.com; dmarc=none ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1736809529; a=rsa-sha256; cv=none; b=TbUXv5yVdSXTzpvBEGc4s24f77Q8KkStys/DtbWuWDo2Gu7w83tN6rQA6fU+Shlx+KPiOf XFwRgd8kCPpBP5bOzEUqi02QHhspqZtPFbA0pM8rlnJuMcPMyACoX13VbVVwkmBdWzUhn/ QnZjVTlO7COIcUD4MYlBr7v5k7tk9ro= Received: by mail-pl1-f170.google.com with SMTP id d9443c01a7336-219f8263ae0so79240525ad.0 for ; Mon, 13 Jan 2025 15:05:28 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1736809527; x=1737414327; darn=kvack.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:sender:from:to:cc:subject:date:message-id :reply-to; bh=VFqUUj0QdJm5qDnw1Ljz6cwuPJ9cjiThS0ltMVoX+yw=; b=nYSVDREGkjdX7NmRDQ6ZxoeH49HSlqHt3qaCSjYkhdWwuhHBLpZ2Nr6H4l6hJehTbi 9DfoefzN0DtykY755VTABXtwrnbX8UgNHp9sxwV2WZ07X+W4dFr47c3FJz2bH4wrtnTu Q3Lb4FqEPG5zC0K3TEEI/1ilr6vD2T7vvpJlbURsz1Od5ClsOcWErJThRC/WiX5+QUii ZtA2PtJokoou/MvgvfDZbnA1vatUa9uWRYWxo3erNH7EnrCQVK/FmXlT4W5lTSQivWyr rGnK7gdM0h+694/yzxW07gDFbc8xDvSkOC37AHKGStFcMgyp1D+8tuVh2IoJMFm/G87T Upaw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1736809527; x=1737414327; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:sender:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=VFqUUj0QdJm5qDnw1Ljz6cwuPJ9cjiThS0ltMVoX+yw=; b=c4vP6FUleBrD9bQbj9j9EErze46uJMKcf6kd232vi2IYVwbZ7hrx+EThLLYT1jm4/+ jTzhOi4ZTLHAUIIVuokRCLuG1lleSSTNtmdOEQEDpInLm4co/0jlqmNI7c0knNWeHxSs e0ABy5FqWmvIoLQ4Z0whXI4C6YZP1Y/5gicEeY5kwMX2l95tf8CTL8Us78/081D2N2W5 4CEM3IVzmUGktcDyYloSWM1vtlQeAytUyU6Dmz4eaZmeRt4NEHEd0tkYKw1CBh/QvNT/ doWpD0ns1lAo5ZM/rOMjG0sIovQBmtjBfpfObqeTE483zj8skq0Jxfh+GkA8/JpFzzn4 q6Gw== X-Forwarded-Encrypted: i=1; AJvYcCU0j7INBQ6tnYTfsGFvSk72FP3M6mIS4KaVOaBca8/ynuSVD+C35jSpsufc/SDNCYFLkYwGM57HdA==@kvack.org X-Gm-Message-State: AOJu0YwjkM92vy5xbx7liaIcNnWgHfRkfudCF3ZP63mSHz5MEtEw+/Hl OKN9ymFjbqYHS3eYessgCm5CGeBbQ9h1FDzDPjgvOkmqGqCQDXGZ2F7vLg== X-Gm-Gg: ASbGncsuqy37NbqsAJ81bwy/MIjoyF2EZGvzhTRg6oyqrmKmKuuwwlBKp0OEC/MOk4H M975yWQ8W7Iwy7vHqylUQLolK1zRNYGqIN8d6jF+1+nS8HLFceqJarYUMcur01oPd4ucPwEa3kP 903cNsiexRKISVkwkGGyHcWaTO4T/odz2cox16S7L1bjDt7pHNSxksEABfykMwbXSraNE3fXl7F LdoamYgjXZ1infm1IYXvkMg8viaYdJWARgyrot+1wE+ZMvNTKw0kRk9rVsNnMeW9x3DmA== X-Google-Smtp-Source: AGHT+IHidRxLfx9LKAF+iL7/QFF6Aokz8jo+zl8xaP4Z/8iOx+uE5qFFxQWorjB4unFdbZwqR7d1sg== X-Received: by 2002:a05:6a00:39a6:b0:724:e75b:22d1 with SMTP id d2e1a72fcca58-72d21ff524emr31835408b3a.16.1736809527288; Mon, 13 Jan 2025 15:05:27 -0800 (PST) Received: from server.roeck-us.net ([2600:1700:e321:62f0:da43:aeff:fecc:bfd5]) by smtp.gmail.com with ESMTPSA id d2e1a72fcca58-72d406a5a32sm6671191b3a.173.2025.01.13.15.05.26 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 13 Jan 2025 15:05:26 -0800 (PST) Date: Mon, 13 Jan 2025 15:05:25 -0800 From: Guenter Roeck To: Jim Zhao Cc: akpm@linux-foundation.org, jack@suse.cz, willy@infradead.org, linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org, linux-mm@kvack.org Subject: Re: [PATCH] mm/page-writeback: Consolidate wb_thresh bumping logic into __wb_calc_thresh Message-ID: References: <20241121100539.605818-1-jimzhao.ai@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20241121100539.605818-1-jimzhao.ai@gmail.com> X-Rspamd-Queue-Id: E412CC000A X-Rspamd-Server: rspam12 X-Stat-Signature: t6gkynnxxx3yku8ptfxajyg8qqwmrwai X-Rspam-User: X-HE-Tag: 1736809528-996340 X-HE-Meta: U2FsdGVkX18jNZPlAQPLoja0s9oPtKXAM7wU0UY5FdELCtqM6W6l2oerjarK2mk9cLG8K2J7lHO/6ntykX+ZL5qaMmi8mRA5y7G4H5QA1rMm8Pd6zDJJSi7KAdDNXdxRp8JZ9RJjz+dkGBQrLWUehcA9c3t3ZztQPiAMheO6VdBzcYRBGO/9i9j3n5Gy8Dsrz+s+q0Kx7da/WvlZnxlgWAf18vYHTlZ/yQxpHSwwrr9fPlz0JOMbv3kVzRHfXJEmpgHcjOIOhp054mphwr4AFHfqc8yIY7HBDzezzDaRAqvY1KbSQzirOMewIEcOF9UeRWuCrVQtAJ0C2HDgdEOE6cd57NkGrdMTXFk4GO3HylTs+M2Zjw9LoSmeSXM0O607PBAn7mgimRE7eRTAXMqvaVB0Wciw5G5CZlzmyy4euqXj24+Y8V1w+Y6WlH/tXmmCWYHxTJb/xnOzAUJwyQUUwqQ14ZGRy0sN+P3nghhR18L/pLMV2ZCDxSmgDQxWJN32knjdnJTRC0hnej/6CmQZ9p1UCrJzxbvuQF6lwpeUgelbCV8SJmgl4jK7Uov2AMZQkqZzRLxFiTPERMNcSuzes9Z+l9GrKIGsJDnpxb1hBipKbxAL0+x9lK9DuafYLAxM4+rSbbbkd8PPDjRbnKLlM5XZONV+hTAesyn3Ot3Bpqtd5y1OwLJn36h8w1hRq6b3K6NdbXxzPE6DF97lwTmvKn0Qpy6TCJ11Qc8yTXXkqU0I2dxeAuqZHe9J419oTeChMt5yqPMBpby7oRgA+bgpWL4We6ayV/8wxMqoffsUlLmlG5vMOwrQ4cuiEPBuBVmrIkyJmRCd6RMHXnHyD1F1Zrb6/D49qpwAspxSw2nbhUJW1AB6gYqOOA03B8RKh3FzmvE2EK3v5beBabhZgqnX3l9pZkuVnQMhe6/9+gnQ6l9vBJEgs3G3jCbS7mDwQS4fpubd7eylMvtrWlTqmeh dvLIqDE7 oW4lapNITmkfhRWZsFhlUn+caGsdJTk+e/UWSIi6CYTSi9RCof44596RBsd8XgWU+0UBMYFvcBao1jTC5PnJ4IJIxwg7adk/VVDhY5U40yJGDno4Y3xgERpvIuslayqzDmavrgdwY9wszyDelMxelsmqqwxWVLzcjvoHoZ7pYZ0Y4UNTN2JQJQi1Kw785i9QjdrDAzWk0oY0u4BJHVEbk6FZXzzllogajeQu23YA7HRqR9/WzwKy9VxikuxWdvEeAlcn+quegQ/cc7dKs0z89NSnnTPOYveW1k9DgrehWhTTw6cXKD1ty/uQ21oa02SrGG4M5CAL5o99RI/MTvZW8HYRYFn/0ESCpwU5bNxRe7lJXLs9gpSGSoVLBDREJ6wfe4DtzJmY1kjk6g4QK0jZFxpKUMN4j5TVfVo893ZY9HGRvNbXy5dUaXOsz6LuuJwIE4z4b3qG89akvkOrf+RIKvba/AcCqtzNuoBUbtiwtHn0JDAvq3/vcHbrShiRFS4Q/3PsueswpZHuQVX7jkNxBXBU0MI2vQQ27X5nXbkoSxV64H8EwUQEar1VRdDBa/ggS6PbXsHvvlsNKfgsgO/SZlVP7N2WrRgt770Dyi9WmkdEZQzDMNXjpBBTjTX+MQ+WzPVPG 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: Hi, On Thu, Nov 21, 2024 at 06:05:39PM +0800, Jim Zhao wrote: > Address the feedback from "mm/page-writeback: raise wb_thresh to prevent > write blocking with strictlimit"(39ac99852fca98ca44d52716d792dfaf24981f53). > The wb_thresh bumping logic is scattered across wb_position_ratio, > __wb_calc_thresh, and wb_update_dirty_ratelimit. For consistency, > consolidate all wb_thresh bumping logic into __wb_calc_thresh. > > Reviewed-by: Jan Kara > Signed-off-by: Jim Zhao This patch triggers a boot failure with one of my 'sheb' boot tests. It is seen when trying to boot from flash (mtd). The log says ... Starting network: 8139cp 0000:00:02.0 eth0: link down udhcpc: started, v1.33.0 EXT2-fs (mtdblock3): error: ext2_check_folio: bad entry in directory #363: : directory entry across blocks - offset=0, inode=27393, rec_len=3072, name_len=2 udhcpc: sending discover udhcpc: sending discover udhcpc: sending discover EXT2-fs (mtdblock3): error: ext2_check_folio: bad entry in directory #363: : directory entry across blocks - offset=0, inode=27393, rec_len=3072, name_len=2 udhcpc: no lease, failing FAIL /etc/init.d/S55runtest: line 29: awk: Permission denied Found console ttySC1 /etc/init.d/S55runtest: line 45: awk: Permission denied can't run '/sbin/getty': No such device or address and boot stalls from there. There is no obvious kernel log message associated with the problem. Reverting this patch fixes the problem. Bisect log is attached for reference. Guenter --- # bad: [37136bf5c3a6f6b686d74f41837a6406bec6b7bc] Add linux-next specific files for 20250113 # good: [9d89551994a430b50c4fffcb1e617a057fa76e20] Linux 6.13-rc6 git bisect start 'HEAD~1' 'v6.13-rc6' # bad: [25dcaaf9b3bdaa117b8eb722ebde76ec9ed30038] Merge branch 'main' of git://git.kernel.org/pub/scm/linux/kernel/git/netdev/net-next.git git bisect bad 25dcaaf9b3bdaa117b8eb722ebde76ec9ed30038 # bad: [435091688c6715e614213d84b1426dfb86fb1c11] Merge branch 'clk-next' of git://git.kernel.org/pub/scm/linux/kernel/git/clk/linux.git git bisect bad 435091688c6715e614213d84b1426dfb86fb1c11 # bad: [51bdf4c7090c8ab260e3a7e7ddaa5d9a7303f541] Merge branch 'perf-tools-next' of git://git.kernel.org/pub/scm/linux/kernel/git/perf/perf-tools-next.git git bisect bad 51bdf4c7090c8ab260e3a7e7ddaa5d9a7303f541 # bad: [972ccc84da30e3fbab197c91caa10fbb04e487c8] foo git bisect bad 972ccc84da30e3fbab197c91caa10fbb04e487c8 # bad: [243dd93e678dd4638b1005a13b085c3c5439447c] mm, swap: simplify percpu cluster updating git bisect bad 243dd93e678dd4638b1005a13b085c3c5439447c # bad: [09a3762697e810e311af23d10d79587da440e9dd] mm/hugetlb: support FOLL_FORCE|FOLL_WRITE git bisect bad 09a3762697e810e311af23d10d79587da440e9dd # bad: [6976848d304ef960e5ac5032611cfd8a9b1b6b01] docs: tmpfs: update the large folios policy for tmpfs and shmem git bisect bad 6976848d304ef960e5ac5032611cfd8a9b1b6b01 # good: [1d3d61aef8467ce44ab3a06e6a0e3fcd930590a7] mailmap, docs: update email to carlos.bilbao@kernel.org git bisect good 1d3d61aef8467ce44ab3a06e6a0e3fcd930590a7 # good: [4d9d1429f6deb91c66591074bbd8ca6aa4cba4dc] mm/page_alloc: move set_page_refcounted() to end of __alloc_pages() git bisect good 4d9d1429f6deb91c66591074bbd8ca6aa4cba4dc # good: [c1bc8fd460ebce85d7a768d8226861438e28bd53] mm: pgtable: make ptep_clear() non-atomic git bisect good c1bc8fd460ebce85d7a768d8226861438e28bd53 # bad: [f8db55561f1b5c70ba2dd260206f295ffee9d1c9] kasan: make kasan_record_aux_stack_noalloc() the default behaviour git bisect bad f8db55561f1b5c70ba2dd260206f295ffee9d1c9 # bad: [a82412684eaeda1ef8201472107de6a40843beec] mm: change type of cma_area_count to unsigned int git bisect bad a82412684eaeda1ef8201472107de6a40843beec # bad: [fcd31b7c35949323434d50416f896bc881a5c397] mm/page-writeback: consolidate wb_thresh bumping logic into __wb_calc_thresh git bisect bad fcd31b7c35949323434d50416f896bc881a5c397 # first bad commit: [fcd31b7c35949323434d50416f896bc881a5c397] mm/page-writeback: consolidate wb_thresh bumping logic into __wb_calc_thresh