Re: [PATCH] mm/damon: modified damon_call_control from static to kmalloc

[SeongJae Park <sj@kernel.org>]

On Thu, 11 Dec 2025 17:13:09 +0900 JaeJoon Jung <rgbi3307@gmail.com> wrote:

> On Thu, 11 Dec 2025 at 16:54, SeongJae Park <sj@kernel.org> wrote:
> >
> > On Thu, 11 Dec 2025 16:38:46 +0900 JaeJoon Jung <rgbi3307@gmail.com> wrote:
> >
> > > On Thu, 11 Dec 2025 at 15:04, SeongJae Park <sj@kernel.org> wrote:
> > [...]
> > > > Sounds good.  Would you mind sending a patch for that?
> > >
> > > Would you like me to send you the patch with only the comments corrected?
> >
> > I was just asking your preferrence, and seems you have no strong one.  I will
> > do that on my own unless you mind.
> 
> I have been argued the following so far:
> "I proposed unifying them into kmalloc()/kfree() methods and
> removing the dealloc_on_cancel."

Based on your previous email [1], I understand you argue this patch is better
to be merged as-is, though you agree it has the pros and the cons.  And
therefore you are letting me to make the final decision.

I was implicitly saying I show the cons more than the pros, so don't want to
merge this as is.  Sorry if it was too implicit.  Hope now my opinion is
clearly reached out to you.

I also suggested making the comment better, and you agreed it but I understand
you are not interested in making that change.  I will do it.

So, all is well and no more action from your side is required.

> I apologize for not being able to respond promptly and smoothly
> this week due to a busy my external lecture schedule.

No worry, take your time.

[1] https://lore.kernel.org/CAHOvCC6bJLf=LuBpm90n5poz8Sqhz9wjXtk5ypbPwkvhb1ptag@mail.gmail.com


Thanks,
SJ

[...]