Re: [PATCH] mm/damon/vaddr: preserve migration weights above UINT_MAX

[SJ Park <sj@kernel.org>]

Apparently Bijan's micron email is no more working.  I added Bijan's gmail to
Cc list.

On Wed, 28 Oct 2026 08:57:06 +0900 CharSyam <charsyam@gmail.com> wrote:

> Hi, SJ.
> 
> Thanks for your reply.

You're welcome!

> 
> 1. I found this by code inspection, not from a real-world bug report.
> I consider it a correctness issue for valid inputs.

Thank you for clarifying.

> 2. I would prefer a u64 total over rejecting those inputs, since the
> interface does not document a limit on the sum of weights.

But I doubt the value of documenting it.  I wonder if real users will find the
behavior at all.  I think it would be not too late to consider documenting it
after we get an issue report from a real world user.

Even just keeping the current behavior looks not really bad to me, as long as
it is easy to test and only shoot the foot of users who didn't run test for
their weird inputs.  We can revisit after a real issue report is delivered.

What do you think?

> 3. Given the lack of a reported real-world impact, I do not think Cc:
> stable is needed.

Makes sense, I agree.

Please avoid top-posting [1] from the next time.

[1] https://subspace.kernel.org/etiquette.html#do-not-top-post-when-replying


Thanks,
SJ

[...]