Re: [PATCH 3/4] drm/scheduler: add new function to get least loaded sched v2

2018-08-01 Thread zhoucm1
vsky, Andrey mailto:andrey.grodzov...@amd.com>> *Cc:* amd-...@lists.freedesktop.org <mailto:amd-...@lists.freedesktop.org>; Maling list - DRI developers mailto:dri-devel@lists.freedesktop.org>>; Koenig, Christian mailto:christian.koe...@amd.com>> *Subject:* Re

Re: [PATCH 3/4] drm/scheduler: add new function to get least loaded sched v2

2018-08-01 Thread Nayan Deshmukh
id Zhou > > > > *From:* dri-devel *On Behalf Of > *Nayan Deshmukh > *Sent:* Thursday, August 02, 2018 12:07 AM > *To:* Grodzovsky, Andrey > *Cc:* amd-...@lists.freedesktop.org; Maling list - DRI developers < > dri-devel@lists.freedesktop.org>; Koenig, Christian <

RE: [PATCH 3/4] drm/scheduler: add new function to get least loaded sched v2

2018-08-01 Thread Zhou, David(ChunMing)
/scheduler: add new function to get least loaded sched v2 Yes, that is correct. Nayan On Wed, Aug 1, 2018, 9:05 PM Andrey Grodzovsky mailto:andrey.grodzov...@amd.com>> wrote: Clarification question - if the run queues belong to different schedulers they effectively point to different ring

Re: [PATCH 3/4] drm/scheduler: add new function to get least loaded sched v2

2018-08-01 Thread Andrey Grodzovsky
Series is Acked-by: Andrey Grodzovsky Andrey On 08/01/2018 12:06 PM, Nayan Deshmukh wrote: Yes, that is correct. Nayan On Wed, Aug 1, 2018, 9:05 PM Andrey Grodzovsky mailto:andrey.grodzov...@amd.com>> wrote: Clarification question -  if the run queues belong to different schedule

Re: [PATCH 3/4] drm/scheduler: add new function to get least loaded sched v2

2018-08-01 Thread Nayan Deshmukh
Yes, that is correct. Nayan On Wed, Aug 1, 2018, 9:05 PM Andrey Grodzovsky wrote: > Clarification question - if the run queues belong to different > schedulers they effectively point to different rings, > > it means we allow to move (reschedule) a drm_sched_entity from one ring > to another -

Re: [PATCH 3/4] drm/scheduler: add new function to get least loaded sched v2

2018-08-01 Thread Andrey Grodzovsky
Clarification question -  if the run queues belong to different schedulers they effectively point to different rings, it means we allow to move (reschedule) a drm_sched_entity from one ring to another - i assume that the idea int the first place, that you have a set of HW rings and you can ut

[PATCH 3/4] drm/scheduler: add new function to get least loaded sched v2

2018-08-01 Thread Nayan Deshmukh
The function selects the run queue from the rq_list with the least load. The load is decided by the number of jobs in a scheduler. v2: avoid using atomic read twice consecutively, instead store it locally Signed-off-by: Nayan Deshmukh --- drivers/gpu/drm/scheduler/gpu_scheduler.c | 25 +

Re: [PATCH 3/4] drm/scheduler: add new function to get least loaded sched

2018-07-31 Thread Nayan Deshmukh
I will take care of that in v2. On Tue, Jul 31, 2018 at 4:57 PM Christian König < ckoenig.leichtzumer...@gmail.com> wrote: > Am 31.07.2018 um 12:37 schrieb Nayan Deshmukh: > > The function selects the run queue from the rq_list with the > > least load. The load is decided by the number of jobs in

Re: [PATCH 3/4] drm/scheduler: add new function to get least loaded sched

2018-07-31 Thread Christian König
Am 31.07.2018 um 12:37 schrieb Nayan Deshmukh: The function selects the run queue from the rq_list with the least load. The load is decided by the number of jobs in a scheduler. Signed-off-by: Nayan Deshmukh --- drivers/gpu/drm/scheduler/gpu_scheduler.c | 26 ++ 1 fil

[PATCH 3/4] drm/scheduler: add new function to get least loaded sched

2018-07-31 Thread Nayan Deshmukh
The function selects the run queue from the rq_list with the least load. The load is decided by the number of jobs in a scheduler. Signed-off-by: Nayan Deshmukh --- drivers/gpu/drm/scheduler/gpu_scheduler.c | 26 ++ 1 file changed, 26 insertions(+) diff --git a/drivers/g