Re: [PATCH v2 7/7] kunit, slub: add test_kfree_rcu() and test_leak_destroy()

2024-09-30 Thread Vlastimil Babka
On 9/26/24 14:54, Vlastimil Babka wrote: > On 9/25/24 14:56, Hyeonggon Yoo wrote: >> >> I don't think this was a noise :) IMO some people want to see WARNING >> during testing to catch errors, >> but not for the slub_kunit test case. I think a proper approach here >> would be suppressing >> warnin

Re: [PATCH v2 7/7] kunit, slub: add test_kfree_rcu() and test_leak_destroy()

2024-09-26 Thread Vlastimil Babka
On 9/25/24 14:56, Hyeonggon Yoo wrote: > On Sun, Sep 22, 2024 at 11:13 PM Guenter Roeck wrote: >> >> On 9/21/24 23:16, Hyeonggon Yoo wrote: >> > On Sun, Sep 22, 2024 at 6:25 AM Vlastimil Babka wrote: >> >> >> >> On 9/21/24 23:08, Guenter Roeck wrote: >> >>> On 9/21/24 13:40, Vlastimil Babka wrote

Re: [PATCH v2 7/7] kunit, slub: add test_kfree_rcu() and test_leak_destroy()

2024-09-25 Thread Hyeonggon Yoo
On Sun, Sep 22, 2024 at 11:13 PM Guenter Roeck wrote: > > On 9/21/24 23:16, Hyeonggon Yoo wrote: > > On Sun, Sep 22, 2024 at 6:25 AM Vlastimil Babka wrote: > >> > >> On 9/21/24 23:08, Guenter Roeck wrote: > >>> On 9/21/24 13:40, Vlastimil Babka wrote: > +CC kunit folks > > On 9/20/

Re: [PATCH v2 7/7] kunit, slub: add test_kfree_rcu() and test_leak_destroy()

2024-09-22 Thread Guenter Roeck
On 9/21/24 23:16, Hyeonggon Yoo wrote: On Sun, Sep 22, 2024 at 6:25 AM Vlastimil Babka wrote: On 9/21/24 23:08, Guenter Roeck wrote: On 9/21/24 13:40, Vlastimil Babka wrote: +CC kunit folks On 9/20/24 15:35, Guenter Roeck wrote: Hi, Hi, On Wed, Aug 07, 2024 at 12:31:20PM +0200, Vlastim

Re: [PATCH v2 7/7] kunit, slub: add test_kfree_rcu() and test_leak_destroy()

2024-09-21 Thread Hyeonggon Yoo
On Sun, Sep 22, 2024 at 6:25 AM Vlastimil Babka wrote: > > On 9/21/24 23:08, Guenter Roeck wrote: > > On 9/21/24 13:40, Vlastimil Babka wrote: > >> +CC kunit folks > >> > >> On 9/20/24 15:35, Guenter Roeck wrote: > >>> Hi, > >> > >> Hi, > >> > >>> On Wed, Aug 07, 2024 at 12:31:20PM +0200, Vlastimi

Re: [PATCH v2 7/7] kunit, slub: add test_kfree_rcu() and test_leak_destroy()

2024-09-21 Thread Vlastimil Babka
On 9/21/24 23:08, Guenter Roeck wrote: > On 9/21/24 13:40, Vlastimil Babka wrote: >> +CC kunit folks >> >> On 9/20/24 15:35, Guenter Roeck wrote: >>> Hi, >> >> Hi, >> >>> On Wed, Aug 07, 2024 at 12:31:20PM +0200, Vlastimil Babka wrote: Add a test that will create cache, allocate one object,

Re: [PATCH v2 7/7] kunit, slub: add test_kfree_rcu() and test_leak_destroy()

2024-09-21 Thread Guenter Roeck
On 9/21/24 13:40, Vlastimil Babka wrote: +CC kunit folks On 9/20/24 15:35, Guenter Roeck wrote: Hi, Hi, On Wed, Aug 07, 2024 at 12:31:20PM +0200, Vlastimil Babka wrote: Add a test that will create cache, allocate one object, kfree_rcu() it and attempt to destroy it. As long as the usage of

Re: [PATCH v2 7/7] kunit, slub: add test_kfree_rcu() and test_leak_destroy()

2024-09-21 Thread Vlastimil Babka
+CC kunit folks On 9/20/24 15:35, Guenter Roeck wrote: > Hi, Hi, > On Wed, Aug 07, 2024 at 12:31:20PM +0200, Vlastimil Babka wrote: >> Add a test that will create cache, allocate one object, kfree_rcu() it >> and attempt to destroy it. As long as the usage of kvfree_rcu_barrier() >> in kmem_cach

Re: [PATCH v2 7/7] kunit, slub: add test_kfree_rcu() and test_leak_destroy()

2024-09-20 Thread Guenter Roeck
Hi, On Wed, Aug 07, 2024 at 12:31:20PM +0200, Vlastimil Babka wrote: > Add a test that will create cache, allocate one object, kfree_rcu() it > and attempt to destroy it. As long as the usage of kvfree_rcu_barrier() > in kmem_cache_destroy() works correctly, there should be no warnings in > dmesg

Re: [PATCH v2 7/7] kunit, slub: add test_kfree_rcu() and test_leak_destroy()

2024-09-14 Thread Vlastimil Babka
On 9/14/24 15:22, Hyeonggon Yoo wrote: > On Wed, Aug 7, 2024 at 7:31 PM Vlastimil Babka wrote: >> >> Add a test that will create cache, allocate one object, kfree_rcu() it >> and attempt to destroy it. As long as the usage of kvfree_rcu_barrier() >> in kmem_cache_destroy() works correctly, there s

Re: [PATCH v2 7/7] kunit, slub: add test_kfree_rcu() and test_leak_destroy()

2024-09-14 Thread Hyeonggon Yoo
On Wed, Aug 7, 2024 at 7:31 PM Vlastimil Babka wrote: > > Add a test that will create cache, allocate one object, kfree_rcu() it > and attempt to destroy it. As long as the usage of kvfree_rcu_barrier() > in kmem_cache_destroy() works correctly, there should be no warnings in > dmesg and the test