Re: [PATCH] dma-buf: Take a breath during dma-fence-chain subtests

2025-07-08 Thread Christian König
On 08.07.25 10:56, Janusz Krzysztofik wrote: >> >> There is no reason to test enabling signaling each of the element in a loop. >> So there should be something like 4096 calls to the dma_fence_chain_cb >> function each jumping to the next unsignaled fence and re-installing the >> callback. > >

Re: [PATCH] dma-buf: Take a breath during dma-fence-chain subtests

2025-07-08 Thread Janusz Krzysztofik
On Monday, 7 July 2025 15:14:11 CEST Christian König wrote: > On 07.07.25 14:25, Janusz Krzysztofik wrote: > > Hi Christian, > > > > I've taken over that old issue and have a few questions to you. > > Thanks a lot for that, something really fishy seems to be going on here. > > > On Thursday, 27

Re: [PATCH] dma-buf: Take a breath during dma-fence-chain subtests

2025-07-07 Thread Christian König
On 07.07.25 14:25, Janusz Krzysztofik wrote: > Hi Christian, > > I've taken over that old issue and have a few questions to you. Thanks a lot for that, something really fishy seems to be going on here. > On Thursday, 27 February 2025 15:11:39 CEST Christian König wrote: ... >> Question is why is

Re: [PATCH] dma-buf: Take a breath during dma-fence-chain subtests

2025-07-07 Thread Janusz Krzysztofik
Hi Christian, I've taken over that old issue and have a few questions to you. On Thursday, 27 February 2025 15:11:39 CEST Christian König wrote: > Am 27.02.25 um 13:52 schrieb Andi Shyti: > > Hi Nitin, > > > > On Wed, Feb 26, 2025 at 09:25:34PM +0530, Nitin Gote wrote: > >> Give the scheduler a c

Re: [PATCH] dma-buf: Take a breath during dma-fence-chain subtests

2025-03-18 Thread Christian König
Am 18.03.25 um 11:34 schrieb Nitin Gote: > Give the scheduler a chance to breathe by calling cond_resched() > as some of the loops may take some time on old machines (apl/bsw/pnv), and > so catch the attention of the watchdogs. > > Closes: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/129

Re: [PATCH] dma-buf: Take a breath during dma-fence-chain subtests

2025-03-07 Thread Christian König
Am 06.03.25 um 16:11 schrieb Nitin Gote: > Give the scheduler a chance to breath by adding delay of 10ms > as some of the loops may take some time on old machines (like apl/bsw/pnv), > and so catch the attention of the watchdogs. > > v1: Instead of cond_resched(), use more generic call like > f

RE: [PATCH] dma-buf: Take a breath during dma-fence-chain subtests

2025-02-27 Thread Gote, Nitin R
Hi, > Am 27.02.25 um 13:52 schrieb Andi Shyti: > > Hi Nitin, > > > > On Wed, Feb 26, 2025 at 09:25:34PM +0530, Nitin Gote wrote: > >> Give the scheduler a chance to breath by calling cond_resched() as > >> some of the loops may take some time on old machines (like > >> apl/bsw/pnv), and so catch

Re: [PATCH] dma-buf: Take a breath during dma-fence-chain subtests

2025-02-27 Thread Christian König
Am 27.02.25 um 13:52 schrieb Andi Shyti: > Hi Nitin, > > On Wed, Feb 26, 2025 at 09:25:34PM +0530, Nitin Gote wrote: >> Give the scheduler a chance to breath by calling cond_resched() >> as some of the loops may take some time on old machines (like apl/bsw/pnv), >> and so catch the attention of the

Re: [PATCH] dma-buf: Take a breath during dma-fence-chain subtests

2025-02-27 Thread Andi Shyti
Hi Nitin, On Wed, Feb 26, 2025 at 09:25:34PM +0530, Nitin Gote wrote: > Give the scheduler a chance to breath by calling cond_resched() > as some of the loops may take some time on old machines (like apl/bsw/pnv), > and so catch the attention of the watchdogs. > > Closes: https://gitlab.freedeskt