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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 3ACD0C433FE for ; Fri, 15 Apr 2022 20:18:35 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1351729AbiDOUVC (ORCPT ); Fri, 15 Apr 2022 16:21:02 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:55182 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1351798AbiDOUUz (ORCPT ); Fri, 15 Apr 2022 16:20:55 -0400 Received: from mail-vs1-xe32.google.com (mail-vs1-xe32.google.com [IPv6:2607:f8b0:4864:20::e32]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 56F6FDE932 for ; Fri, 15 Apr 2022 13:18:21 -0700 (PDT) Received: by mail-vs1-xe32.google.com with SMTP id v133so4311310vsv.7 for ; Fri, 15 Apr 2022 13:18:21 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20210112; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=Cbb1YDb5NaiuUde42DTRSKAfH6XbrCM5Doeox4W/L1U=; b=R+fWKSybpUALXVyi7dr5ULcwrcaXsKRmDx3orLXnDSP1dKdn6sslxaR84/Vt1idESV R0gbbnc2Eo5O2hvxGNOm5+O5l7zO9r5grwhRC/H1pjlfnAengg/oNlV3PjITQ52N9xvj ttmr00bahnI6zPcHkkGyVH85UyIFb3BiDQXe1wN6D7t4+4SOidckXgXl4WmZtYM9G8g/ aed3/o6NSjZAcWDOvNGsi7PFryJv2kBPX+cLbidK7S7FBT3z2r51EStnpXahSrD5q8Gs bTdzVJmEmNuyfh9W4BKstW6XQcg/tnhQUf9CLonBDa0jnHYmD9BzjVmbUocVwfvlS2tZ ESWA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=Cbb1YDb5NaiuUde42DTRSKAfH6XbrCM5Doeox4W/L1U=; b=pqOavZBdCec2Q9DCwM4X/hD6LvYbvO68mPFsVoOzEc7nHrQ6Km3WVZVCeGl80A+tc2 k/5l74hhcdsvP392PhCEHDQykQeqOaUra2+rIWU7RMeOdrwHQFrCGreNfX8NQ+yYmMDp aZlXqW5LvDUPk5HWqe0MPUEmXQBkP3AuiW5B/EX3DWui8eejmWUFxX6IhxyGzVLjiQRY hPQGBZUI6Q/mlRwFUwOoZ5DNrhs26xLQeLkEb9/F6N1iyG94xNlrVnOSEZ1qf36bw6mO 4g39K/6aQgtL2zY8G2xp2okXJvrDAgmxwV37asT3DxeHIE7Vjivh0TYCI5QkCyiE3BGw 6BIQ== X-Gm-Message-State: AOAM531DfJcsgSjIScSuDWwPOdofVr+AFlKiR8vhSb7jahJjNElK0lr1 HRNbxOHp9F9aeeFGpRyl6QscxI4FrdjMGQajJ+oKpw== X-Google-Smtp-Source: ABdhPJwO94bILh3nEx3+nxP4ZO5hEG2ftNGY6x330YGarczURLY5R/Y+L+rCRpLt0Wpji7SMuT9MzVQDZVeugH9nIdA= X-Received: by 2002:a67:de17:0:b0:32a:4007:cd86 with SMTP id q23-20020a67de17000000b0032a4007cd86mr220678vsk.22.1650053900285; Fri, 15 Apr 2022 13:18:20 -0700 (PDT) MIME-Version: 1.0 References: <20220407031525.2368067-1-yuzhao@google.com> <20220407031525.2368067-7-yuzhao@google.com> <20220414143959.0daf4534613f2511b9b27f11@linux-foundation.org> In-Reply-To: From: Yu Zhao Date: Fri, 15 Apr 2022 14:17:43 -0600 Message-ID: Subject: Re: [PATCH v10 06/14] mm: multi-gen LRU: minimal implementation To: Barry Song <21cnbao@gmail.com> Cc: Andrew Morton , Stephen Rothwell , Linux-MM , Andi Kleen , Aneesh Kumar , Catalin Marinas , Dave Hansen , Hillf Danton , Jens Axboe , Jesse Barnes , Johannes Weiner , Jonathan Corbet , Linus Torvalds , Matthew Wilcox , Mel Gorman , Michael Larabel , Michal Hocko , Mike Rapoport , Rik van Riel , Vlastimil Babka , Will Deacon , Ying Huang , LAK , Linux Doc Mailing List , LKML , Kernel Page Reclaim v2 , x86 , Brian Geffon , Jan Alexander Steffens , Oleksandr Natalenko , Steven Barrett , Suleiman Souhlal , Daniel Byrne , Donald Carr , =?UTF-8?Q?Holger_Hoffst=C3=A4tte?= , Konstantin Kharlamov , Shuang Zhai , Sofia Trinh , Vaibhav Jain Content-Type: text/plain; charset="UTF-8" Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Apr 15, 2022 at 4:16 AM Barry Song <21cnbao@gmail.com> wrote: > > On Fri, Apr 15, 2022 at 9:40 AM Andrew Morton wrote: > > > > On Thu, 14 Apr 2022 14:36:03 -0600 Yu Zhao wrote: > > > > > > or it is only something > > > > meaningful for the internal code? > > > > > > This is how swappiness is interpreted. > > > > > > > if so, can we rename it to > > > > something else? otherwise, it is quite confusing. > > > > > > Feel free to suggest something. > > > > It is confusing, swap_preference? > > seems to be much better. might also be worth using MACRO to replace those > 1, 200, 0 magic numbers to tell readers the exact meaning? How do you read "swap_preference=0"? I read it as "no (swap) preference", which means anon and file are equal. The real meaning though, is "don't swap"; and the "no preference" value is actually 100.