linux-kernel - Re: [RFC PATCH 3/4] mm: add sys fs configuration for page reporting

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <64985d09-d11e-5fc0-64b0-b56ef514448a@linux.intel.com>
Date:   Mon, 13 Apr 2020 08:02:35 -0700
From:   Alexander Duyck <alexander.h.duyck@...ux.intel.com>
To:     Mel Gorman <mgorman@...hsingularity.net>, linux-mm@...ck.org,
        linux-kernel@...r.kernel.org,
        Andrea Arcangeli <aarcange@...hat.com>,
        Dan Williams <dan.j.williams@...el.com>,
        Dave Hansen <dave.hansen@...el.com>,
        David Hildenbrand <david@...hat.com>,
        Michal Hocko <mhocko@...nel.org>,
        Andrew Morton <akpm@...ux-foundation.org>,
        Alex Williamson <alex.williamson@...hat.com>
Subject: Re: [RFC PATCH 3/4] mm: add sys fs configuration for page reporting

On 4/12/2020 2:09 AM, liliangleo wrote:
> This patch add 'delay_millisecs', 'mini_order', 'batch_size',
> in '/sys/kernel/mm/page_report/'. Usage:
> 
> "delay_millisecs":
> Time delay interval between page free and work start to run.
> 
> "mini_order":
> Only pages with order equal or greater than mini_order will be
> reported.
> 
> "batch_size"
> Wake up the worker only when free pages total size are greater
> than 'batch_size'.
> 
> Cc: Alexander Duyck <alexander.h.duyck@...ux.intel.com>
> Cc: Mel Gorman <mgorman@...hsingularity.net>
> Cc: Andrea Arcangeli <aarcange@...hat.com>
> Cc: Dan Williams <dan.j.williams@...el.com>
> Cc: Dave Hansen <dave.hansen@...el.com>
> Cc: David Hildenbrand <david@...hat.com>
> Cc: Michal Hocko <mhocko@...nel.org>
> Cc: Andrew Morton <akpm@...ux-foundation.org>
> Cc: Alex Williamson <alex.williamson@...hat.com>
> Signed-off-by: liliangleo <liliangleo@...iglobal.com>

I am not really a fan of making these configurable globally. Especially 
since the existing virtio-balloon is relying on some of this being 
configured the way it is.

It would make much more sense to push these configuration options out to 
the registration interface so that the thing that is registering for 
page reporting can configure them when it is registered.