mm/madvise: cleanup requests validations and classifications

[PATCH 0/4] mm/madvise: cleanup requests validations and classifications

Posted by SeongJae Park 9 months, 1 week ago

Cleanup madvise entry level code for cleaner request validations and
classifications.

Note that this series was a part of another one[1], and separated to
this one following a reviewer's nice comment[2].

Changes from the original series
(https://lore.kernel.org/20250310172318.653630-1-sj@kernel.org)
- Separate from the tlb flushes batching part
- Collect Reviewed-by: tags from Lorenzo
- Fix typos and wordsmith commit messages and comments
- Rebase on latest mm-unstable

[1] https://lore.kernel.org/20250310172318.653630-1-sj@kernel.org
[2] https://lore.kernel.org/0f90d56e-5960-4478-803e-1054696c0cde@lucifer.local

SeongJae Park (4):
  mm/madvise: use is_memory_failure() from madvise_do_behavior()
  mm/madvise: split out populate behavior check logic
  mm/madvise: deduplicate madvise_do_behavior() skip case handlings
  mm/madvise: remove len parameter of madvise_do_behavior()

 mm/madvise.c | 128 +++++++++++++++++++++++++++++----------------------
 1 file changed, 74 insertions(+), 54 deletions(-)


base-commit: 733c75c2264b9de72a463f731507e674a32ff094
-- 
2.39.5

Re: [PATCH 0/4] mm/madvise: cleanup requests validations and classifications

Posted by Lorenzo Stoakes 9 months, 1 week ago

On Wed, Mar 12, 2025 at 09:47:46AM -0700, SeongJae Park wrote:
> Cleanup madvise entry level code for cleaner request validations and
> classifications.
>
> Note that this series was a part of another one[1], and separated to
> this one following a reviewer's nice comment[2].

Thanks :>)

>
> Changes from the original series
> (https://lore.kernel.org/20250310172318.653630-1-sj@kernel.org)
> - Separate from the tlb flushes batching part
> - Collect Reviewed-by: tags from Lorenzo
> - Fix typos and wordsmith commit messages and comments
> - Rebase on latest mm-unstable
>
> [1] https://lore.kernel.org/20250310172318.653630-1-sj@kernel.org
> [2] https://lore.kernel.org/0f90d56e-5960-4478-803e-1054696c0cde@lucifer.local
>
> SeongJae Park (4):
>   mm/madvise: use is_memory_failure() from madvise_do_behavior()
>   mm/madvise: split out populate behavior check logic
>   mm/madvise: deduplicate madvise_do_behavior() skip case handlings
>   mm/madvise: remove len parameter of madvise_do_behavior()
>
>  mm/madvise.c | 128 +++++++++++++++++++++++++++++----------------------
>  1 file changed, 74 insertions(+), 54 deletions(-)
>
>
> base-commit: 733c75c2264b9de72a463f731507e674a32ff094
> --
> 2.39.5