Re: [Regression] drm/scheduler: track GPU active time per entity

2023-04-06 Thread Luben Tuikov
On 2023-04-06 06:45, Lucas Stach wrote: > Am Donnerstag, dem 06.04.2023 um 10:27 +0200 schrieb Daniel Vetter: >> On Thu, 6 Apr 2023 at 10:22, Christian König >> wrote: >>> >>> Am 05.04.23 um 18:09 schrieb Luben Tuikov: On 2023-04-05 10:05, Danilo Krummrich wrote: > On 4/4/23 06:31, Luben

Re: [Regression] drm/scheduler: track GPU active time per entity

2023-04-06 Thread Luben Tuikov
On 2023-04-06 04:22, Christian König wrote: > Am 05.04.23 um 18:09 schrieb Luben Tuikov: >> On 2023-04-05 10:05, Danilo Krummrich wrote: >>> On 4/4/23 06:31, Luben Tuikov wrote: On 2023-03-28 04:54, Lucas Stach wrote: > Hi Danilo, > > Am Dienstag, dem 28.03.2023 um 02:57 +0200 schr

Re: [Regression] drm/scheduler: track GPU active time per entity

2023-04-06 Thread Luben Tuikov
On 2023-04-06 11:58, Lucas Stach wrote: > Am Mittwoch, dem 05.04.2023 um 16:44 -0400 schrieb Luben Tuikov: >> On 2023-04-05 13:44, Lucas Stach wrote: >>> Hi Luben, >>> >>> Am Dienstag, dem 04.04.2023 um 00:31 -0400 schrieb Luben Tuikov: On 2023-03-28 04:54, Lucas Stach wrote: > Hi Danilo,

Re: [Regression] drm/scheduler: track GPU active time per entity

2023-04-06 Thread Daniel Vetter
On Thu, Apr 06, 2023 at 07:42:44PM +0200, Daniel Vetter wrote: > But I think there's tools to make sure we don't dig a complete hole with > these it sounds like. I guess another topic for pestering the rust folks. Or to put it very bluntly: Could we make Arc at least runtime enforce (with the usua

Re: [Regression] drm/scheduler: track GPU active time per entity

2023-04-06 Thread Daniel Vetter
On Fri, Apr 07, 2023 at 02:11:32AM +0900, Asahi Lina wrote: > On 07/04/2023 00.30, Daniel Vetter wrote: > > On Thu, Apr 06, 2023 at 11:43:19PM +0900, Asahi Lina wrote: > > > On 06/04/2023 22.37, Daniel Vetter wrote: > > > > On Thu, Apr 06, 2023 at 09:21:47PM +0900, Asahi Lina wrote: > > > > > On 06

Re: [Regression] drm/scheduler: track GPU active time per entity

2023-04-06 Thread Asahi Lina
On 07/04/2023 00.30, Daniel Vetter wrote: On Thu, Apr 06, 2023 at 11:43:19PM +0900, Asahi Lina wrote: On 06/04/2023 22.37, Daniel Vetter wrote: On Thu, Apr 06, 2023 at 09:21:47PM +0900, Asahi Lina wrote: On 06/04/2023 19.09, Daniel Vetter wrote: On Thu, Apr 06, 2023 at 06:05:11PM +0900, Asahi

Re: [Regression] drm/scheduler: track GPU active time per entity

2023-04-06 Thread Daniel Vetter
On Thu, 6 Apr 2023 at 17:59, Daniel Vetter wrote: > > On Thu, Apr 06, 2023 at 05:33:18PM +0200, Christian König wrote: > > Am 06.04.23 um 17:24 schrieb Lucas Stach: > > > Am Donnerstag, dem 06.04.2023 um 16:21 +0200 schrieb Christian König: > > > > Am 06.04.23 um 12:45 schrieb Lucas Stach: > > > >

Re: [Regression] drm/scheduler: track GPU active time per entity

2023-04-06 Thread Daniel Vetter
On Thu, Apr 06, 2023 at 05:33:18PM +0200, Christian König wrote: > Am 06.04.23 um 17:24 schrieb Lucas Stach: > > Am Donnerstag, dem 06.04.2023 um 16:21 +0200 schrieb Christian König: > > > Am 06.04.23 um 12:45 schrieb Lucas Stach: > > > > Am Donnerstag, dem 06.04.2023 um 10:27 +0200 schrieb Daniel

Re: [Regression] drm/scheduler: track GPU active time per entity

2023-04-06 Thread Lucas Stach
Am Mittwoch, dem 05.04.2023 um 16:44 -0400 schrieb Luben Tuikov: > On 2023-04-05 13:44, Lucas Stach wrote: > > Hi Luben, > > > > Am Dienstag, dem 04.04.2023 um 00:31 -0400 schrieb Luben Tuikov: > > > On 2023-03-28 04:54, Lucas Stach wrote: > > > > Hi Danilo, > > > > > > > > Am Dienstag, dem 28.03

Re: [Regression] drm/scheduler: track GPU active time per entity

2023-04-06 Thread Christian König
Am 06.04.23 um 17:24 schrieb Lucas Stach: Am Donnerstag, dem 06.04.2023 um 16:21 +0200 schrieb Christian König: Am 06.04.23 um 12:45 schrieb Lucas Stach: Am Donnerstag, dem 06.04.2023 um 10:27 +0200 schrieb Daniel Vetter: On Thu, 6 Apr 2023 at 10:22, Christian König wrote: Am 05.04.23 um 18:

Re: [Regression] drm/scheduler: track GPU active time per entity

2023-04-06 Thread Daniel Vetter
On Thu, Apr 06, 2023 at 11:43:19PM +0900, Asahi Lina wrote: > On 06/04/2023 22.37, Daniel Vetter wrote: > > On Thu, Apr 06, 2023 at 09:21:47PM +0900, Asahi Lina wrote: > > > On 06/04/2023 19.09, Daniel Vetter wrote: > > > > On Thu, Apr 06, 2023 at 06:05:11PM +0900, Asahi Lina wrote: > > > > > On 06

Re: [Regression] drm/scheduler: track GPU active time per entity

2023-04-06 Thread Lucas Stach
Am Donnerstag, dem 06.04.2023 um 16:21 +0200 schrieb Christian König: > Am 06.04.23 um 12:45 schrieb Lucas Stach: > > Am Donnerstag, dem 06.04.2023 um 10:27 +0200 schrieb Daniel Vetter: > > > On Thu, 6 Apr 2023 at 10:22, Christian König > > > wrote: > > > > Am 05.04.23 um 18:09 schrieb Luben Tuik

Re: [Regression] drm/scheduler: track GPU active time per entity

2023-04-06 Thread Asahi Lina
On 06/04/2023 22.37, Daniel Vetter wrote: On Thu, Apr 06, 2023 at 09:21:47PM +0900, Asahi Lina wrote: On 06/04/2023 19.09, Daniel Vetter wrote: On Thu, Apr 06, 2023 at 06:05:11PM +0900, Asahi Lina wrote: On 06/04/2023 17.27, Daniel Vetter wrote: On Thu, 6 Apr 2023 at 10:22, Christian König w

Re: [Regression] drm/scheduler: track GPU active time per entity

2023-04-06 Thread Christian König
Am 06.04.23 um 12:45 schrieb Lucas Stach: Am Donnerstag, dem 06.04.2023 um 10:27 +0200 schrieb Daniel Vetter: On Thu, 6 Apr 2023 at 10:22, Christian König wrote: Am 05.04.23 um 18:09 schrieb Luben Tuikov: On 2023-04-05 10:05, Danilo Krummrich wrote: On 4/4/23 06:31, Luben Tuikov wrote: On 2

Re: [Regression] drm/scheduler: track GPU active time per entity

2023-04-06 Thread Daniel Vetter
On Thu, Apr 06, 2023 at 02:19:20PM +0200, Lucas Stach wrote: > Am Donnerstag, dem 06.04.2023 um 14:09 +0200 schrieb Daniel Vetter: > > On Thu, Apr 06, 2023 at 12:45:12PM +0200, Lucas Stach wrote: > > > Am Donnerstag, dem 06.04.2023 um 10:27 +0200 schrieb Daniel Vetter: > > > > On Thu, 6 Apr 2023 at

Re: [Regression] drm/scheduler: track GPU active time per entity

2023-04-06 Thread Daniel Vetter
On Thu, Apr 06, 2023 at 09:21:47PM +0900, Asahi Lina wrote: > On 06/04/2023 19.09, Daniel Vetter wrote: > > On Thu, Apr 06, 2023 at 06:05:11PM +0900, Asahi Lina wrote: > > > On 06/04/2023 17.27, Daniel Vetter wrote: > > > > On Thu, 6 Apr 2023 at 10:22, Christian König > > > > wrote: > > > > > >

Re: [Regression] drm/scheduler: track GPU active time per entity

2023-04-06 Thread Asahi Lina
On 06/04/2023 19.09, Daniel Vetter wrote: On Thu, Apr 06, 2023 at 06:05:11PM +0900, Asahi Lina wrote: On 06/04/2023 17.27, Daniel Vetter wrote: On Thu, 6 Apr 2023 at 10:22, Christian König wrote: Am 05.04.23 um 18:09 schrieb Luben Tuikov: On 2023-04-05 10:05, Danilo Krummrich wrote: On 4/4

Re: [Regression] drm/scheduler: track GPU active time per entity

2023-04-06 Thread Lucas Stach
Am Donnerstag, dem 06.04.2023 um 14:09 +0200 schrieb Daniel Vetter: > On Thu, Apr 06, 2023 at 12:45:12PM +0200, Lucas Stach wrote: > > Am Donnerstag, dem 06.04.2023 um 10:27 +0200 schrieb Daniel Vetter: > > > On Thu, 6 Apr 2023 at 10:22, Christian König > > > wrote: > > > > > > > > Am 05.04.23 u

Re: [Regression] drm/scheduler: track GPU active time per entity

2023-04-06 Thread Daniel Vetter
On Thu, Apr 06, 2023 at 12:45:12PM +0200, Lucas Stach wrote: > Am Donnerstag, dem 06.04.2023 um 10:27 +0200 schrieb Daniel Vetter: > > On Thu, 6 Apr 2023 at 10:22, Christian König > > wrote: > > > > > > Am 05.04.23 um 18:09 schrieb Luben Tuikov: > > > > On 2023-04-05 10:05, Danilo Krummrich wrot

Re: [Regression] drm/scheduler: track GPU active time per entity

2023-04-06 Thread Lucas Stach
Am Donnerstag, dem 06.04.2023 um 10:27 +0200 schrieb Daniel Vetter: > On Thu, 6 Apr 2023 at 10:22, Christian König wrote: > > > > Am 05.04.23 um 18:09 schrieb Luben Tuikov: > > > On 2023-04-05 10:05, Danilo Krummrich wrote: > > > > On 4/4/23 06:31, Luben Tuikov wrote: > > > > > On 2023-03-28 04:5

Re: [Regression] drm/scheduler: track GPU active time per entity

2023-04-06 Thread Daniel Vetter
On Thu, Apr 06, 2023 at 06:05:11PM +0900, Asahi Lina wrote: > On 06/04/2023 17.27, Daniel Vetter wrote: > > On Thu, 6 Apr 2023 at 10:22, Christian König > > wrote: > > > > > > Am 05.04.23 um 18:09 schrieb Luben Tuikov: > > > > On 2023-04-05 10:05, Danilo Krummrich wrote: > > > > > On 4/4/23 06:3

Re: [Regression] drm/scheduler: track GPU active time per entity

2023-04-06 Thread Asahi Lina
On 06/04/2023 17.27, Daniel Vetter wrote: On Thu, 6 Apr 2023 at 10:22, Christian König wrote: Am 05.04.23 um 18:09 schrieb Luben Tuikov: On 2023-04-05 10:05, Danilo Krummrich wrote: On 4/4/23 06:31, Luben Tuikov wrote: On 2023-03-28 04:54, Lucas Stach wrote: Hi Danilo, Am Dienstag, dem 28

Re: [Regression] drm/scheduler: track GPU active time per entity

2023-04-06 Thread Daniel Vetter
On Thu, 6 Apr 2023 at 10:22, Christian König wrote: > > Am 05.04.23 um 18:09 schrieb Luben Tuikov: > > On 2023-04-05 10:05, Danilo Krummrich wrote: > >> On 4/4/23 06:31, Luben Tuikov wrote: > >>> On 2023-03-28 04:54, Lucas Stach wrote: > Hi Danilo, > > Am Dienstag, dem 28.03.2023 um

Re: [Regression] drm/scheduler: track GPU active time per entity

2023-04-06 Thread Christian König
Am 05.04.23 um 18:09 schrieb Luben Tuikov: On 2023-04-05 10:05, Danilo Krummrich wrote: On 4/4/23 06:31, Luben Tuikov wrote: On 2023-03-28 04:54, Lucas Stach wrote: Hi Danilo, Am Dienstag, dem 28.03.2023 um 02:57 +0200 schrieb Danilo Krummrich: Hi all, Commit df622729ddbf ("drm/scheduler: t

Re: [Regression] drm/scheduler: track GPU active time per entity

2023-04-05 Thread Luben Tuikov
On 2023-04-05 13:44, Lucas Stach wrote: > Hi Luben, > > Am Dienstag, dem 04.04.2023 um 00:31 -0400 schrieb Luben Tuikov: >> On 2023-03-28 04:54, Lucas Stach wrote: >>> Hi Danilo, >>> >>> Am Dienstag, dem 28.03.2023 um 02:57 +0200 schrieb Danilo Krummrich: Hi all, Commit df622729ddbf

Re: [Regression] drm/scheduler: track GPU active time per entity

2023-04-05 Thread Lucas Stach
Hi Luben, Am Dienstag, dem 04.04.2023 um 00:31 -0400 schrieb Luben Tuikov: > On 2023-03-28 04:54, Lucas Stach wrote: > > Hi Danilo, > > > > Am Dienstag, dem 28.03.2023 um 02:57 +0200 schrieb Danilo Krummrich: > > > Hi all, > > > > > > Commit df622729ddbf ("drm/scheduler: track GPU active time pe

Re: [Regression] drm/scheduler: track GPU active time per entity

2023-04-05 Thread Danilo Krummrich
On 4/5/23 18:09, Luben Tuikov wrote: On 2023-04-05 10:05, Danilo Krummrich wrote: On 4/4/23 06:31, Luben Tuikov wrote: On 2023-03-28 04:54, Lucas Stach wrote: Hi Danilo, Am Dienstag, dem 28.03.2023 um 02:57 +0200 schrieb Danilo Krummrich: Hi all, Commit df622729ddbf ("drm/scheduler: track G

Re: [Regression] drm/scheduler: track GPU active time per entity

2023-04-05 Thread Luben Tuikov
On 2023-04-05 10:05, Danilo Krummrich wrote: > On 4/4/23 06:31, Luben Tuikov wrote: >> On 2023-03-28 04:54, Lucas Stach wrote: >>> Hi Danilo, >>> >>> Am Dienstag, dem 28.03.2023 um 02:57 +0200 schrieb Danilo Krummrich: Hi all, Commit df622729ddbf ("drm/scheduler: track GPU active tim

Re: [Regression] drm/scheduler: track GPU active time per entity

2023-04-05 Thread Danilo Krummrich
On 4/4/23 06:31, Luben Tuikov wrote: On 2023-03-28 04:54, Lucas Stach wrote: Hi Danilo, Am Dienstag, dem 28.03.2023 um 02:57 +0200 schrieb Danilo Krummrich: Hi all, Commit df622729ddbf ("drm/scheduler: track GPU active time per entity") tries to track the accumulated time that a job was activ

Re: [Regression] drm/scheduler: track GPU active time per entity

2023-04-03 Thread Luben Tuikov
On 2023-03-28 04:54, Lucas Stach wrote: > Hi Danilo, > > Am Dienstag, dem 28.03.2023 um 02:57 +0200 schrieb Danilo Krummrich: >> Hi all, >> >> Commit df622729ddbf ("drm/scheduler: track GPU active time per entity") >> tries to track the accumulated time that a job was active on the GPU >> writin

Re: [Regression] drm/scheduler: track GPU active time per entity

2023-03-28 Thread Christian König
The same strategy was suggested before for amdgpu and reverted as well because of running into the same problems. No idea how that slipped, Andrew reviewed it and IIRC he was also the one who reverted the initial approach for amdgpu. Instead of letting the scheduler sum that stuff up proactiv

Re: [Regression] drm/scheduler: track GPU active time per entity

2023-03-28 Thread Lucas Stach
Hi Danilo, Am Dienstag, dem 28.03.2023 um 02:57 +0200 schrieb Danilo Krummrich: > Hi all, > > Commit df622729ddbf ("drm/scheduler: track GPU active time per entity") > tries to track the accumulated time that a job was active on the GPU > writing it to the entity through which the job was deplo

[Regression] drm/scheduler: track GPU active time per entity

2023-03-27 Thread Danilo Krummrich
Hi all, Commit df622729ddbf ("drm/scheduler: track GPU active time per entity") tries to track the accumulated time that a job was active on the GPU writing it to the entity through which the job was deployed to the scheduler originally. This is done within drm_sched_get_cleanup_job() which f

[Regression] drm/scheduler: track GPU active time per entity

2023-03-27 Thread Danilo Krummrich
Hi all, Commit df622729ddbf ("drm/scheduler: track GPU active time per entity") tries to track the accumulated time that a job was active on the GPU writing it to the entity through which the job was deployed to the scheduler originally. This is done within drm_sched_get_cleanup_job() which f