Re: Re: [RFC v4 2/7] mm/damon: Account age of target regions

[SeongJae Park]

Hello Rick,

Thank you for question :)

On Wed, 04 Mar 2020 10:21:29 -0500 Rik van Riel <riel@xxxxxxxxxxx> wrote:

> [-- Attachment #1: Type: text/plain, Size: 558 bytes --]
> 
> On Tue, 2020-03-03 at 13:14 +0100, SeongJae Park wrote:
> > From: SeongJae Park <sjpark@xxxxxxxxx>
> 
> > --- a/mm/damon.c
> > +++ b/mm/damon.c
> > @@ -87,6 +87,10 @@ static struct damon_region
> > *damon_new_region(struct damon_ctx *ctx,
> >  	ret->sampling_addr = damon_rand(ctx, vm_start, vm_end);
> >  	INIT_LIST_HEAD(&ret->list);
> >  
> > +	ret->age = 0;
> > +	ret->last_vm_start = vm_start;
> > +	ret->last_vm_end = vm_end;
> 
> Wait, what tree is this supposed to apply against?
> 
> I see no mm/damon.c file in current Linus upstream.

This patchset is supposed to apply against v5.5 plus DAMON patchset[1] plus a
patch from Minchan.  You can get the tree this patchset is applied via:

    $ git clone git://github.com/sjp38/linux -b damos/rfc/v4

Or, the web is also available:
https://github.com/sjp38/linux/releases/tag/damos/rfc/v4

I am posting this as a seperate RFC patchset because 1) this patchset is based
on the tree other than Linus or other maintainers' upstream trees, 2) I
want to keep the size of original patchset small for convenience of reviewers,
3) this patchset is relatively recently made and thus might unstable compared
to the DAMON patchset[1], and 4) I want to share my plan and get early
feedbacks as many as possible.

Sorry if this made you confused.  Also, if you have some opinions regarding
this seperated postings, please let me know.


[1] https://lore.kernel.org/linux-mm/20200224123047.32506-1-sjpark@xxxxxxxxxx


Thanks,
SeongJae Park

> 
> -- 
> All Rights Reversed.