All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Matthew Wilcox (Oracle)" <willy@infradead.org>
To: akpm@linuxfoundation.org
Cc: "Matthew Wilcox (Oracle)" <willy@infradead.org>,
	linux-mm@kvack.org, Heiko Carstens <hca@linux.ibm.com>,
	Rafael Aquini <aquini@redhat.com>,
	Vlastimil Babka <vbabka@suse.cz>, Yu Zhao <yuzhao@google.com>,
	Vladimir Davydov <vdavydov.dev@gmail.com>,
	kirill.shutemov@linux.intel.com
Subject: [PATCH] mm: Mark idle page tracking as BROKEN
Date: Sat, 12 Jun 2021 01:07:14 +0100	[thread overview]
Message-ID: <20210612000714.775825-1-willy@infradead.org> (raw)

In discussion with other MM developers around how idle page tracking
should be fixed for transparent huge pages, several expressed the opinion
that it should be removed as it is inefficient at accomplishing the
job that it is supposed to, and we have better mechanisms (eg uffd) for
accomplishing the same goals these days.

Mark the feature as BROKEN for now and we can remove it entirely in a
few months if nobody complains.  It is not enabled by Android, ChromeOS,
Debian, Fedora or SUSE.  Red Hat enabled it with RHEL-8.1 and UEK followed
suit, but I have been unable to find why RHEL enabled it.

Signed-off-by: Matthew Wilcox (Oracle) <willy@infradead.org>
---
 mm/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/mm/Kconfig b/mm/Kconfig
index 02d44e3420f5..311b50bb92ce 100644
--- a/mm/Kconfig
+++ b/mm/Kconfig
@@ -772,7 +772,7 @@ config DEFERRED_STRUCT_PAGE_INIT
 
 config IDLE_PAGE_TRACKING
 	bool "Enable idle page tracking"
-	depends on SYSFS && MMU
+	depends on SYSFS && MMU && BROKEN
 	select PAGE_EXTENSION if !64BIT
 	help
 	  This feature allows to estimate the amount of user pages that have
-- 
2.30.2



             reply	other threads:[~2021-06-12  0:08 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-06-12  0:07 Matthew Wilcox (Oracle) [this message]
2021-06-12  3:14 ` Yu Zhao
2021-06-14 11:08 ` Kirill A. Shutemov
2021-06-14 13:49 ` SeongJae Park
2021-06-15  2:04   ` Andrew Morton
2021-06-15  6:40     ` SeongJae Park
2021-06-15  7:41 ` David Hildenbrand
2021-06-16  2:55   ` Matthew Wilcox
2021-06-16  6:22     ` Yu Zhao
2021-06-16  8:36       ` Vlastimil Babka
2021-06-16  8:43         ` David Hildenbrand
2021-06-16 19:23           ` Yu Zhao
2021-06-18 12:48             ` David Hildenbrand

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20210612000714.775825-1-willy@infradead.org \
    --to=willy@infradead.org \
    --cc=akpm@linuxfoundation.org \
    --cc=aquini@redhat.com \
    --cc=hca@linux.ibm.com \
    --cc=kirill.shutemov@linux.intel.com \
    --cc=linux-mm@kvack.org \
    --cc=vbabka@suse.cz \
    --cc=vdavydov.dev@gmail.com \
    --cc=yuzhao@google.com \
    --subject='Re: [PATCH] mm: Mark idle page tracking as BROKEN' \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.