Received: by 2002:a05:6358:53a8:b0:117:f937:c515 with SMTP id z40csp2064895rwe; Sat, 15 Apr 2023 10:46:58 -0700 (PDT) X-Google-Smtp-Source: AKy350aKGFgSc9dunrRHFYrqIKLk1gWs8zNzg5DyFbUBSNtHcEzBkjLYgcmWU5N+ys7O3hU+Vdlk X-Received: by 2002:a17:90a:5315:b0:23f:81c0:eadd with SMTP id x21-20020a17090a531500b0023f81c0eaddmr9543816pjh.47.1681580818016; Sat, 15 Apr 2023 10:46:58 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1681580817; cv=none; d=google.com; s=arc-20160816; b=aGnqtm+QRalbPBBtoZ0KJk1gwaLVF9OCFED/ytO61NDmeu4P93vGTzslt+538113Sd SkV4Ib/e4G+Dfh0gv25GMRIKcII4gUDMw5+jXasSuhXB3Pm7cyKwZXIpq477J7wi5eZT TWDURvCTc5Oi/JCZifeVwB7VBCrY+9z8cMDkjrV/YWMhtUiKlNTbIi540thv+zlAGpcj 0zQFEGiiT5xFBZKO3/vGwXzOt/AYSn2VHBeWbzkvZNvzMMjBpsBDJnmzWS0HqzBElMzx d1tY7UQqJUbkdurGdd7uMF8CRJIFDYvFHoYbJ6HPTNVNKd409VBiPkNGK2UDo0Bvs67k KX5w== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:in-reply-to:references :cc:to:from:subject:user-agent:mime-version:date:message-id :dkim-signature; bh=ft7n3pSsyVF2OSVjBeFR7VFYeidiVFTD3Reo30zy8Ac=; b=xlQ2gwB5xZWXfSFmPIx5O6zcERWEMJWqsHzGZCNqBkolqRBi2Y1dwtB7k0E3c1T1yM m1KGNf+sQG4sw7w8szOMnnt9iT8PbkoXb8qPvkD5iJ6HiUuMgGmiwH6meVCSvCMrNUE6 pq8VLIhWWNfKqRz6G4SDW8TOX8CFWmsgXUUzwUrP+9ws2Ssctm4lcsr5sX4p/5ggNRC6 J3jnCnFwjuPxd3mUrK8GJCAhZDe6U4lHHehwZPIrgEgnKzPJZnXLEq+T0Ey9fqeSb1XP cLIw9cxctajUcXlOZ1qngDKiuJ1o1QdTbrKULu6e4nzp+F4GhV1jlmy9wk/Qpfqt+BBa Xx2w== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@foxmail.com header.s=s201512 header.b=HGtgO6zK; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=foxmail.com Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id 4-20020a630f44000000b0050bf5a814a8si7484762pgp.403.2023.04.15.10.46.45; Sat, 15 Apr 2023 10:46:57 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@foxmail.com header.s=s201512 header.b=HGtgO6zK; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=foxmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229795AbjDORmz (ORCPT + 99 others); Sat, 15 Apr 2023 13:42:55 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:51622 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229639AbjDORmy (ORCPT ); Sat, 15 Apr 2023 13:42:54 -0400 Received: from out162-62-57-137.mail.qq.com (out162-62-57-137.mail.qq.com [162.62.57.137]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 4A34E35AB for ; Sat, 15 Apr 2023 10:42:50 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=foxmail.com; s=s201512; t=1681580567; bh=ft7n3pSsyVF2OSVjBeFR7VFYeidiVFTD3Reo30zy8Ac=; h=Date:Subject:From:To:Cc:References:In-Reply-To; b=HGtgO6zKJAtUxFOoUCsA4+U7wUpOyHDH6QahQQKpyzvGMmd6nStrDfvv22CtIKS8s poOkbGkVuDwyJuj0tXeFjwHZIrG3otWrvIE3fwFEH9ahUjn+mslPSbBcitNecgEx03 u5Q2zxI5K5ieT7QtUgQ4A8PObSyqQjWgGlhdLK2w= Received: from [192.168.31.3] ([106.92.97.36]) by newxmesmtplogicsvrszb6-0.qq.com (NewEsmtp) with SMTP id AAC39AF2; Sun, 16 Apr 2023 01:42:44 +0800 X-QQ-mid: xmsmtpt1681580564t4yz4x0k0 Message-ID: X-QQ-XMAILINFO: Mdc3TkmnJyI/yOxekswrVuNLfRHbcNl5ulLlhk/GDW8D5F7N+3a2uZT3ECAwNZ BOFiQx4+AggGxfIVOs6z5mwX67N1xVo2nMFUtYaB8KBRJW2Yal67NWwybKe16BVqVhoQcu2/N4NS v6H7FfEbykAKmriGOgh2aSoeKkEB4cE9epj6Bjx00V7WVHp6fqQJgemrEHa/aT5njBPVebcVGO9R MfSgZQuxQU64RrCDQrowMYC3o6DMw5l/hg/CUKYDqPcSk4lBc0tNHthPI67nJjY0QP0i2VEVfYTA RUJfJ3Jy4ee296fMoDM+ZaWaSwXAlPZMkLF1b39SLeDpqa1VQQRhBY84eU0HyGDGcaZ1FQTPVosh euE/EOLdjQP2akBb5twEUFtcr+i+HrIfQFsFPwwpnhumSrvpsEIoqi/e3jm9iUg6azPrMThZi4RE +9UP+hQ3YofIHHSvEJRs+aA7nas2IFg8oftdD4593jPj7hvtDHctXeaxzOEQQIMlhdv5TKdrpdYm 8oq2HgbBf+zVMKsj8dHtkVU18eESFLHExhNxDoQ7JCLxiyEMeBUlr7FzwxEMrc1GIeLJUS7xKG+M Hg5Pcaevks2do6klSHnGCBu5vyYxu+WMWkM0HittMxoxejH20Uyspp0LnXFFQS9jiL5XKE5YdH5s 1mHVKe3+XF7ljvH+EQByymhLnyfrJVE1/cfpmUqDhXeapO08Vh+x7ajPLzwUZAiad9r3monjtNk/ hYLPgRiwUAtCcHpNW8rtNPj5i+hR2kt/oy0P1TETed34rose1TIm/29UZw7wNOgvG2Kx6OtwhsIs 4jm57qegxANQroWJ9oOfDOdmIyK4yz/z2hYeVg6/WfUOQ9JNmZXsrcqNPQznBIZnkiMDGCUPHpE6 R5oguv5sb0lXo9UXl3oEwaap2hPOzzgnYOYrC/XpkLwZnPBswK6JZm1uzfVw0Qq6FPeTsVB3v3E6 sZKxY4E4oq9c7et4FSeJ363gs3vpaNUfNpTLN+yAZ1isQp8gc9ow== X-OQ-MSGID: <1a0154ae-abed-7480-762e-358d7627ca66@foxmail.com> Date: Sun, 16 Apr 2023 01:42:44 +0800 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:102.0) Gecko/20100101 Thunderbird/102.9.1 Subject: Re: [PATCH] mm: compaction: optimize compact_memory to comply with the admin-guide From: Wen Yang To: Andrew Morton Cc: Mel Gorman , Oscar Salvador , William Lam , Fu Wei , linux-mm@kvack.org, linux-kernel@vger.kernel.org References: <20230411134801.a4aadef5aba0f51e0d44bb7a@linux-foundation.org> In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit X-Spam-Status: No, score=0.8 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FORGED_MUA_MOZILLA, FREEMAIL_FROM,HELO_DYNAMIC_IPADDR,NICE_REPLY_A,RCVD_IN_DNSWL_NONE, RCVD_IN_MSPIKE_H2,RDNS_DYNAMIC,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=no autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 在 2023/4/13 00:54, Wen Yang 写道: > > 在 2023/4/12 04:48, Andrew Morton 写道: >> On Wed, 12 Apr 2023 02:24:26 +0800 wenyang.linux@foxmail.com wrote: >> >>> For the /proc/sys/vm/compact_memory file, the admin-guide states: >>> When 1 is written to the file, all zones are compacted such that free >>> memory is available in contiguous blocks where possible. This can be >>> important for example in the allocation of huge pages although >>> processes >>> will also directly compact memory as required >>> >>> But it was not strictly followed, writing any value would cause all >>> zones to be compacted. In some critical scenarios, some applications >>> operating it, such as echo 0, have caused serious problems. >> Really?  You mean someone actually did this and didn't observe the >> effect during their testing? > > Thanks for your reply. > > Since /proc/sys/vm/compact_memory has been well documented for over a > decade: > > https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/tree/Documentation/admin-guide/sysctl/vm.rst#n109 > > > it is believed that only writing 1 will trigger trigger all zones to > be compacted. > > Especially for those who write applications, they may only focus on > documentation and generally do not read kernel code.  Moreover, such > problems are not easily detected through testing on low pressure > machines. > > Writing any meaningful or meaningless values will trigger it and > affect the entire server: > > # echo 1 > /proc/sys/vm/compact_memory > # echo 0 > /proc/sys/vm/compact_memory > # echo dead > /proc/sys/vm/compact_memory > # echo "hello world" > /proc/sys/vm/compact_memory > > The implementation of this high-risk operation may require following > the admin-guides. > > -- > > Best wishes, > > Wen > > Hello, do you think it's better to optimize the sysctl_compaction_handler code or update the admin-guide document? -- Best wishes, Wen >>> It has been slightly optimized to comply with the admin-guide. >