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=-3.6 required=3.0 tests=BAYES_00,DKIM_INVALID, DKIM_SIGNED,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS autolearn=no 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 8FA86C43461 for ; Thu, 17 Sep 2020 17:52:24 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id 011CD20872 for ; Thu, 17 Sep 2020 17:52:23 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=fail reason="key not found in DNS" (0-bit key) header.d=suse.com header.i=@suse.com header.b="m9qgQiSV" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 011CD20872 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=suse.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id 488258E0003; Thu, 17 Sep 2020 13:52:23 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 437098E0001; Thu, 17 Sep 2020 13:52:23 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 34C0E8E0003; Thu, 17 Sep 2020 13:52:23 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0004.hostedemail.com [216.40.44.4]) by kanga.kvack.org (Postfix) with ESMTP id 1FCF18E0001 for ; Thu, 17 Sep 2020 13:52:23 -0400 (EDT) Received: from smtpin21.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay05.hostedemail.com (Postfix) with ESMTP id C9ADF181AEF23 for ; Thu, 17 Sep 2020 17:52:22 +0000 (UTC) X-FDA: 77273297724.21.box88_330402527124 Received: from filter.hostedemail.com (10.5.16.251.rfc1918.com [10.5.16.251]) by smtpin21.hostedemail.com (Postfix) with ESMTP id A9744180442DC for ; Thu, 17 Sep 2020 17:52:22 +0000 (UTC) X-HE-Tag: box88_330402527124 X-Filterd-Recvd-Size: 3899 Received: from mx2.suse.de (mx2.suse.de [195.135.220.15]) by imf30.hostedemail.com (Postfix) with ESMTP for ; Thu, 17 Sep 2020 17:52:22 +0000 (UTC) X-Virus-Scanned: by amavisd-new at test-mx.suse.de DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.com; s=cantorsusede; t=1600365141; h=from:from:reply-to:subject:subject: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=7ACZ1i9L0eM7pFjFd6XaMh+LP19pdesYPnrbce9bvdE=; b=m9qgQiSVGElQlgS9Y1bmW4eQ8wF71jXeVpuMDzMUgwElV7P1mbCwz/Zv/UkQGU9Q0wpoUk O9BwIm4LqU4fpPS20ikdrS6Z3FNw0MALD5/aJkkPfSxSyM2DPL+8v18XO9O1uy9STJDw7W SmroQcsO1DBGXMthvakPKGk8BQimldnSyHzyXKsAPdspi96E2kWUsLTGIQOBOBi/9Em76k T/+UL+topOL0WxGchvg7SIQi3D49aiZiPWhZedfJfqFFqJJSrdwPwbiq9SnTy3+/O7SjIV dvSBzs/ORRMDWWa5zipmgYvF8xU+YoLJswYD4KVt9DV6r/lVuzgUjI/Q7rZMXA== Received: from relay2.suse.de (unknown [195.135.221.27]) by mx2.suse.de (Postfix) with ESMTP id ACFF4B0BE; Thu, 17 Sep 2020 17:52:54 +0000 (UTC) Date: Thu, 17 Sep 2020 19:52:17 +0200 From: Michal Hocko To: Vijay Balakrishna Cc: Andrew Morton , "Kirill A. Shutemov" , Oleg Nesterov , Song Liu , Andrea Arcangeli , Pavel Tatashin , Allen Pais , linux-kernel@vger.kernel.org, linux-mm@kvack.org Subject: Re: [v4] mm: khugepaged: avoid overriding min_free_kbytes set by user Message-ID: <20200917175217.GD29887@dhcp22.suse.cz> References: <1600324779-6782-1-git-send-email-vijayb@linux.microsoft.com> <20200917092805.GA29887@dhcp22.suse.cz> <21c4788c-2516-2e47-5885-486c352c2b6e@linux.microsoft.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <21c4788c-2516-2e47-5885-486c352c2b6e@linux.microsoft.com> 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 Thu 17-09-20 10:27:16, Vijay Balakrishna wrote: > > > On 9/17/2020 2:28 AM, Michal Hocko wrote: > > On Wed 16-09-20 23:39:39, Vijay Balakrishna wrote: > > > set_recommended_min_free_kbytes need to honor min_free_kbytes set by the > > > user. Post start-of-day THP enable or memory hotplug operations can > > > lose user specified min_free_kbytes, in particular when it is higher than > > > calculated recommended value. > > > > I was about to recommend a more detailed explanation when I have > > realized that this patch is not really needed after all. Unless I am > > missing something. > > > > init_per_zone_wmark_min ignores the newly calculated min_free_kbytes if > > it is lower than user_min_free_kbytes. So calculated min_free_kbytes >= > > user_min_free_kbytes. > > > > Except for value clamping when the value is reduced and this likely > > needs fixing. But set_recommended_min_free_kbytes should be fine. > > > > IIUC, after start-of-day if a user performs > - THP disable > - modifies min_free_bytes > - THP enable > above sequence currently wouldn't result in calling init_per_zone_wmark_min. I will not, but why do you think this matters? All we should care about is that auto-tuning shouldn't reduce user provided value [1] and that the memory hotplug should be consistent with the boot time heuristic. init_per_zone_wmark_min should make sure that the user value is not reduced and thp heuristic makes sure it will not reduce this value. So the property should be transitive with the existing code (modulo the problem I have highlighted). [1] one could argue that it shouldn't even increase the value strictly speaking because an admin might have a very good reason to decrease the value but this has never been the semantic and changing it now might be problematic -- Michal Hocko SUSE Labs