Linux-mm Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Kiryl Shutsemau <kirill@shutemov.name>
To: Alexander Gordeev <agordeev@linux.ibm.com>
Cc: akpm@linux-foundation.org, david@kernel.org, ljs@kernel.org,
	 nico.pache@linux.dev, baolin.wang@linux.alibaba.com,
	baohua@kernel.org,  dev.jain@arm.com, hughd@google.com,
	lance.yang@linux.dev, liam@infradead.org,  mhocko@suse.com,
	rppt@kernel.org, ryan.roberts@arm.com, shuah@kernel.org,
	 surenb@google.com, usama.arif@linux.dev, vbabka@kernel.org,
	ziy@nvidia.com,  usama.anjum@arm.com, linux-mm@kvack.org,
	linux-kselftest@vger.kernel.org,  linux-kernel@vger.kernel.org
Subject: Re: [PATCH v2 00/17] selftests/mm: improve khugepaged coverage
Date: Tue, 11 Aug 2026 17:58:48 +0100	[thread overview]
Message-ID: <antUso_Mfdh3EiHk@thinkstation> (raw)
In-Reply-To: <f9e0fe38-47de-4577-b638-1142d4ffbe6c-agordeev@linux.ibm.com>

On Mon, Aug 10, 2026 at 02:58:43PM +0200, Alexander Gordeev wrote:
> On Fri, Aug 07, 2026 at 12:36:30PM +0100, Kiryl Shutsemau wrote:
> 
> Hi Kiryl,
> 
> ...
> >  - Dropped the swap-skip patch, an equivalent has landed
> >    (Alexander Gordeev).
> 
> And dropped [1] following David feedback [2]. Would you pursue the proper fix?

Yes.  v3 will handle the failure rather than predict it.

is_swap_enabled() already skips these cases when no swap is active, so what
is left is swap that exists but cannot take what the case asks for.  The
series already has swapout_range(), which reissues MADV_PAGEOUT for up to two
seconds and reports whether the count arrived; I will make the two cases skip
when it does not.  That covers swap too small, swap full, a memcg limit and a
folio still under writeback, without reading /proc/meminfo for any of them.

An error from madvise() itself stays a hard failure -- that is the kernel
refusing, not the machine being short of swap.

-- 
  Kiryl Shutsemau / Kirill A. Shutemov


  reply	other threads:[~2026-08-11 16:58 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-08-07 11:36 [PATCH v2 00/17] selftests/mm: improve khugepaged coverage Kiryl Shutsemau
2026-08-07 11:36 ` [PATCH v2 01/17] selftests/mm: skip collapse_compound_extreme where the PMD is too large Kiryl Shutsemau
2026-08-07 11:36 ` [PATCH v2 02/17] selftests/mm: scale khugepaged's collapse wait with the PMD size Kiryl Shutsemau
2026-08-07 11:36 ` [PATCH v2 03/17] selftests/mm: skip khugepaged page cache cases without a PMD folio Kiryl Shutsemau
2026-08-07 11:36 ` [PATCH v2 04/17] selftests/mm: retry the swapout the khugepaged swap cases rely on Kiryl Shutsemau
2026-08-07 11:36 ` [PATCH v2 05/17] selftests/mm: move is_backed_by_folio() into vm_util Kiryl Shutsemau
2026-08-07 11:36 ` [PATCH v2 06/17] selftests/mm: add folio-order check for address ranges Kiryl Shutsemau
2026-08-07 11:36 ` [PATCH v2 07/17] selftests/mm: add folio-order detection self-check Kiryl Shutsemau
2026-08-07 11:36 ` [PATCH v2 08/17] selftests/mm: add khugepaged completion barrier helper Kiryl Shutsemau
2026-08-07 11:36 ` [PATCH v2 09/17] selftests/mm: add order-parameterized khugepaged collapse cases Kiryl Shutsemau
2026-08-07 11:36 ` [PATCH v2 10/17] selftests/mm: parameterize the mixed-source collapse case by source order Kiryl Shutsemau
2026-08-07 11:36 ` [PATCH v2 11/17] selftests/mm: cover a shared-source collapse write race Kiryl Shutsemau
2026-08-07 11:36 ` [PATCH v2 12/17] selftests/mm: run every supported collapse order by default Kiryl Shutsemau
2026-08-07 11:36 ` [PATCH v2 13/17] selftests/mm: verify synchronous khugepaged driving is attributable Kiryl Shutsemau
2026-08-07 11:36 ` [PATCH v2 14/17] selftests/mm: add khugepaged race harness Kiryl Shutsemau
2026-08-07 11:36 ` [PATCH v2 15/17] selftests/mm: race collapse of windows with holes Kiryl Shutsemau
2026-08-07 11:36 ` [PATCH v2 16/17] selftests/mm: add memory-pressure threads to the khugepaged race harness Kiryl Shutsemau
2026-08-07 11:36 ` [PATCH v2 17/17] selftests/mm: zap whole PTE tables in " Kiryl Shutsemau
2026-08-10 12:58 ` [PATCH v2 00/17] selftests/mm: improve khugepaged coverage Alexander Gordeev
2026-08-11 16:58   ` Kiryl Shutsemau [this message]
2026-08-11 11:40 ` Muhammad Usama Anjum
2026-08-11 16:59   ` Kiryl Shutsemau

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=antUso_Mfdh3EiHk@thinkstation \
    --to=kirill@shutemov.name \
    --cc=agordeev@linux.ibm.com \
    --cc=akpm@linux-foundation.org \
    --cc=baohua@kernel.org \
    --cc=baolin.wang@linux.alibaba.com \
    --cc=david@kernel.org \
    --cc=dev.jain@arm.com \
    --cc=hughd@google.com \
    --cc=lance.yang@linux.dev \
    --cc=liam@infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-kselftest@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=ljs@kernel.org \
    --cc=mhocko@suse.com \
    --cc=nico.pache@linux.dev \
    --cc=rppt@kernel.org \
    --cc=ryan.roberts@arm.com \
    --cc=shuah@kernel.org \
    --cc=surenb@google.com \
    --cc=usama.anjum@arm.com \
    --cc=usama.arif@linux.dev \
    --cc=vbabka@kernel.org \
    --cc=ziy@nvidia.com \
    /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
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox