Re: Problems running DAMON kunit tests with spinlock debugging enabled

[SeongJae Park <sj@kernel.org>]

On Thu, 5 Sep 2024 11:04:51 -0700 Guenter Roeck <linux@roeck-us.net> wrote:

> On 9/5/24 10:35, SeongJae Park wrote:
> > Hi Guenter,
> > 
> > On Wed, 4 Sep 2024 14:04:03 -0700 Guenter Roeck <linux@roeck-us.net> wrote:
> > 
> > [...]
> >> Here is another problem. With all other problems fixed, when running on a 32-bit system:
[...]
> > As you also mentioned, this function may better to be significantly re-written
> > with overflows in mind.  However, I have no enough time and setup to
> > sufficiently test the fix for now, due to some personal reasons.  Also, the
> > issue is important, but not time-critical for now, in my humble opinion.
> > Please let me know if someone has different opinions.
> > 
> > So, I made one possible fix off the top of my head, and posted as an RFC patch:
> > https://lore.kernel.org/20240905172405.46995-1-sj@kernel.org
> > 
> > I only tested that against the kunit test on my 64bit machine.  After I get
> > some time and setup (maybe 1-2 weeks later), I will take more time to review
> > the code, do more tests, and post RFC-dropped version.
> > 
> 
> No worries. I queued the RFC patch up in my testbed, so we should get results
> for all targets later today (or maybe tomorrow, the testbed is busy with stable
> release tests).

No problem, please take your time :)


Thanks,
SJ

> 
> Guenter