Re: [PATCH 3/5] mm/vmscan: Throttle reclaim when no progress is being made

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: "Mel Gorman" <mgorman@xxxxxxxxxxxxxxxxxxx>
Subject: Re: [PATCH 3/5] mm/vmscan: Throttle reclaim when no progress is being made
From: "NeilBrown" <neilb@xxxxxxx>
Date: Tue, 21 Sep 2021 09:31:30 +1000
Cc: "Linux-MM" <linux-mm@xxxxxxxxx>, "Theodore Ts'o" <tytso@xxxxxxx>, "Andreas Dilger" <adilger.kernel@xxxxxxxxx>, "Darrick J . Wong" <djwong@xxxxxxxxxx>, "Matthew Wilcox" <willy@xxxxxxxxxxxxx>, "Michal Hocko" <mhocko@xxxxxxxx>, "Dave Chinner" <david@xxxxxxxxxxxxx>, "Rik van Riel" <riel@xxxxxxxxxxx>, "Vlastimil Babka" <vbabka@xxxxxxx>, "Johannes Weiner" <hannes@xxxxxxxxxxx>, "Jonathan Corbet" <corbet@xxxxxxx>, "Linux-fsdevel" <linux-fsdevel@xxxxxxxxxxxxxxx>, "LKML" <linux-kernel@xxxxxxxxxxxxxxx>, "Mel Gorman" <mgorman@xxxxxxxxxxxxxxxxxxx>
In-reply-to: <20210920085436.20939-4-mgorman@techsingularity.net>
References: <20210920085436.20939-1-mgorman@techsingularity.net>, <20210920085436.20939-4-mgorman@techsingularity.net>

On Mon, 20 Sep 2021, Mel Gorman wrote:
> +
> +		reclaim_throttle(pgdat, VMSCAN_THROTTLE_NOPROGRESS, HZ/10);

We always seem to pass "HZ/10" to reclaim_throttle().  Should we just
hard-code that in the one place inside reclaim_throttle() itself?

NeilBrown

Follow-Ups:
- Re: [PATCH 3/5] mm/vmscan: Throttle reclaim when no progress is being made
  - From: Mel Gorman

References:
- [RFC PATCH 0/5] Remove dependency on congestion_wait in mm/
  - From: Mel Gorman
- [PATCH 3/5] mm/vmscan: Throttle reclaim when no progress is being made
  - From: Mel Gorman

Prev by Date: Re: [PATCH v3] mm: fs: invalidate bh_lrus for only cold path
Next by Date: Re: [PATCH 1/6] MM: Support __GFP_NOFAIL in alloc_pages_bulk_*() and improve doco
Previous by thread: [PATCH 3/5] mm/vmscan: Throttle reclaim when no progress is being made
Next by thread: Re: [PATCH 3/5] mm/vmscan: Throttle reclaim when no progress is being made
Index(es):
- Date
- Thread

[Index of Archives] [Linux ARM Kernel] [Linux ARM] [Linux Omap] [Fedora ARM] [IETF Annouce] [Bugtraq] [Linux OMAP] [Linux MIPS] [eCos] [Asterisk Internet PBX] [Linux API]