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=-7.8 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,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 EF704C432BE for ; Wed, 18 Aug 2021 17:04:58 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id D529C6102A for ; Wed, 18 Aug 2021 17:04:58 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231652AbhHRRFc (ORCPT ); Wed, 18 Aug 2021 13:05:32 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:58834 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229784AbhHRRFb (ORCPT ); Wed, 18 Aug 2021 13:05:31 -0400 Received: from mail-ej1-x635.google.com (mail-ej1-x635.google.com [IPv6:2a00:1450:4864:20::635]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 4BC42C061764 for ; Wed, 18 Aug 2021 10:04:56 -0700 (PDT) Received: by mail-ej1-x635.google.com with SMTP id b10so6397509eju.9 for ; Wed, 18 Aug 2021 10:04:56 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc:content-transfer-encoding; bh=Si7U6HwgT/s1/0DUykgf8Mxxnbx+qV+oZXkcnS5dKHQ=; b=AChYjDKZR7xEN/R81QQ9I2hRsbNGAI7NqftHe4v8mPniSSGnl/OrNvA5bEeVQApm+O ibB1uQgvIRSxkzQx4SECL93cgAfhppiDnrNRpIl1zsFnwkqprUOkuVE5Z0bsNwqKy1DK AQpCFTY1JVxMWMxVOUFVrPpc/+axmg4D1Mp12sDjducva8Umbc439ZhN5lsokrQEfN+V wCjzQ268htspW9T9IGu5U9JiHo+mrVh/6a43zfQdeG2u+2zlni0wJ81QYsD/oM5TJ0tx LCAn5PA/Rl5fV175bfdLlDt3KCgvLpCyvpNcj5pFtAsB1cTNaA5If82yuAZE8eBRavgZ wdug== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc:content-transfer-encoding; bh=Si7U6HwgT/s1/0DUykgf8Mxxnbx+qV+oZXkcnS5dKHQ=; b=g+Yiolw4Zmmn3/uS9R0urnIPJYuVR8V98xcBrakRRLw5SindvTNh3d4qVlUI/ifNdG zM3zLq0P3jY5vNuMWmLY9zzCvcPlyzM3af3xjwk5WO+T/NhBdaScyZOwVjbY0Wbg3Jwo qgbEEadZJ/dgRIB/TSObVbCj8b04+bQwhaj/AcLoZrVM83KoSLa7ZQ9P33oEvidzxTeS gxTYoRCi/2TgpAu8xOWyyIH6Yi2I41mDjARrB8ozwr20yqG2S+qLZE/XB99wJl2Rhrzr fQ5FUFQNCS86v0PF5DbHBB+GZdqgDw915FOMqLsIxNc+rBBpJB310dSY9gNk82/9C9+e rJWw== X-Gm-Message-State: AOAM532oAL7wvMD70HJi6hnoBu808nNnSnyMLRo8K9kF/Akue+DZ6T06 GNLMdjb1yKdqwgz/JB5NZCGxZgAjU97InIAeeF0= X-Google-Smtp-Source: ABdhPJwsmFeTJicZf4gc/8UKoIhnK1UkHukA04WmnpY7zZI030kGbN9o3u27fYTEth+eye5/qhxeuEakJIo/vBLukbg= X-Received: by 2002:a17:906:b890:: with SMTP id hb16mr10503469ejb.383.1629306294582; Wed, 18 Aug 2021 10:04:54 -0700 (PDT) MIME-Version: 1.0 References: <20210816180909.3603-1-shy828301@gmail.com> <20210818063042.GA2310427@u2004> <20210818075316.GB501393@hori.linux.bs1.fc.nec.co.jp> In-Reply-To: From: Yang Shi Date: Wed, 18 Aug 2021 10:04:42 -0700 Message-ID: Subject: Re: [PATCH 1/2] mm: hwpoison: don't drop slab caches for offlining non-LRU page To: David Hildenbrand Cc: =?UTF-8?B?SE9SSUdVQ0hJIE5BT1lBKOWggOWPoyDnm7TkuZ8p?= , Naoya Horiguchi , "osalvador@suse.de" , "tdmackey@twitter.com" , "akpm@linux-foundation.org" , "corbet@lwn.net" , "linux-mm@kvack.org" , "linux-kernel@vger.kernel.org" Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Aug 18, 2021 at 12:55 AM David Hildenbrand wrote= : > > On 18.08.21 09:53, HORIGUCHI NAOYA(=E5=A0=80=E5=8F=A3 =E7=9B=B4=E4=B9=9F)= wrote: > > On Wed, Aug 18, 2021 at 09:24:01AM +0200, David Hildenbrand wrote: > > ... > >> > >> Might we want to add a TODO in the code? We have a similar one in > >> mm/page_isolation.c:set_migratetype_isolate() and it's certainly a rem= inder > >> that something of value is missing. > > > > Yes, that will be helpful. The below's what's in my mind, but if someo= ne > > has better idea, that's fine. > > > > @@ -296,11 +296,9 @@ void shake_page(struct page *p, int access) > > } > > > > /* > > - * Only call shrink_node_slabs here (which would also shrink > > - * other caches) if access is not potentially fatal. > > + * TODO: Could shrink slab caches here if a lightweight range-bas= ed > > + * shrinker will be available. > > */ > > - if (access) > > - drop_slab_node(page_to_nid(p)); > > } > > EXPORT_SYMBOL_GPL(shake_page); > > Just what I had in mind, thanks! Fine to me, will add this in v2. > > > -- > Thanks, > > David / dhildenb > >