* [PATCH] drm/scheduler: remove incorrect param documentation @ 2018-03-25 11:09 Nayan Deshmukh 2018-03-25 11:14 ` Christian König 0 siblings, 1 reply; 16+ messages in thread From: Nayan Deshmukh @ 2018-03-25 11:09 UTC (permalink / raw) To: dri-devel; +Cc: Nayan Deshmukh, ckoenig.leichtzumerken, christian.koenig Signed-off-by: Nayan Deshmukh <nayan26deshmukh@gmail.com> --- drivers/gpu/drm/scheduler/gpu_scheduler.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/scheduler/gpu_scheduler.c b/drivers/gpu/drm/scheduler/gpu_scheduler.c index 0d95888ccc3e..27fdda1264f7 100644 --- a/drivers/gpu/drm/scheduler/gpu_scheduler.c +++ b/drivers/gpu/drm/scheduler/gpu_scheduler.c @@ -117,7 +117,6 @@ drm_sched_rq_select_entity(struct drm_sched_rq *rq) * @sched The pointer to the scheduler * @entity The pointer to a valid drm_sched_entity * @rq The run queue this entity belongs - * @kernel If this is an entity for the kernel * @jobs The max number of jobs in the job queue * * return 0 if succeed. negative error code on failure -- 2.14.3 _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply related [flat|nested] 16+ messages in thread
* Re: [PATCH] drm/scheduler: remove incorrect param documentation 2018-03-25 11:09 [PATCH] drm/scheduler: remove incorrect param documentation Nayan Deshmukh @ 2018-03-25 11:14 ` Christian König 2018-03-25 11:21 ` Nayan Deshmukh 0 siblings, 1 reply; 16+ messages in thread From: Christian König @ 2018-03-25 11:14 UTC (permalink / raw) To: Nayan Deshmukh, dri-devel; +Cc: ckoenig.leichtzumerken Am 25.03.2018 um 13:09 schrieb Nayan Deshmukh: > Signed-off-by: Nayan Deshmukh <nayan26deshmukh@gmail.com> > --- > drivers/gpu/drm/scheduler/gpu_scheduler.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/drivers/gpu/drm/scheduler/gpu_scheduler.c b/drivers/gpu/drm/scheduler/gpu_scheduler.c > index 0d95888ccc3e..27fdda1264f7 100644 > --- a/drivers/gpu/drm/scheduler/gpu_scheduler.c > +++ b/drivers/gpu/drm/scheduler/gpu_scheduler.c > @@ -117,7 +117,6 @@ drm_sched_rq_select_entity(struct drm_sched_rq *rq) > * @sched The pointer to the scheduler > * @entity The pointer to a valid drm_sched_entity > * @rq The run queue this entity belongs > - * @kernel If this is an entity for the kernel > * @jobs The max number of jobs in the job queue If I'm not completely mistaken the jobs parameter is unused and we also have a @guilty parameter as well. The description for the @guilty should be something like "atomic_t set to 1 when a job on this queue is found to be guilty causing a timeout". Regards, Christian. > * > * return 0 if succeed. negative error code on failure _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH] drm/scheduler: remove incorrect param documentation 2018-03-25 11:14 ` Christian König @ 2018-03-25 11:21 ` Nayan Deshmukh 2018-03-25 11:29 ` Christian König 0 siblings, 1 reply; 16+ messages in thread From: Nayan Deshmukh @ 2018-03-25 11:21 UTC (permalink / raw) To: Christian König; +Cc: ckoenig.leichtzumerken, dri-devel On Sun, Mar 25, 2018 at 4:44 PM, Christian König <christian.koenig@amd.com> wrote: > Am 25.03.2018 um 13:09 schrieb Nayan Deshmukh: >> >> Signed-off-by: Nayan Deshmukh <nayan26deshmukh@gmail.com> >> --- >> drivers/gpu/drm/scheduler/gpu_scheduler.c | 1 - >> 1 file changed, 1 deletion(-) >> >> diff --git a/drivers/gpu/drm/scheduler/gpu_scheduler.c >> b/drivers/gpu/drm/scheduler/gpu_scheduler.c >> index 0d95888ccc3e..27fdda1264f7 100644 >> --- a/drivers/gpu/drm/scheduler/gpu_scheduler.c >> +++ b/drivers/gpu/drm/scheduler/gpu_scheduler.c >> @@ -117,7 +117,6 @@ drm_sched_rq_select_entity(struct drm_sched_rq *rq) >> * @sched The pointer to the scheduler >> * @entity The pointer to a valid drm_sched_entity >> * @rq The run queue this entity belongs >> - * @kernel If this is an entity for the kernel >> * @jobs The max number of jobs in the job queue > > > If I'm not completely mistaken the jobs parameter is unused and we also have > a @guilty parameter as well. > Yes. But I am not sure what should be done with the jobs parameter. Should I remove it entirely? > The description for the @guilty should be something like "atomic_t set to 1 > when a job on this queue is found to be guilty causing a timeout". > > Regards, > Christian. > > >> * >> * return 0 if succeed. negative error code on failure > > _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH] drm/scheduler: remove incorrect param documentation 2018-03-25 11:21 ` Nayan Deshmukh @ 2018-03-25 11:29 ` Christian König 2018-03-26 15:21 ` [PATCH] drm/scheduler: fix " Nayan Deshmukh 2018-03-26 15:29 ` [PATCH] drm/scheduler: remove incorrect " Nayan Deshmukh 0 siblings, 2 replies; 16+ messages in thread From: Christian König @ 2018-03-25 11:29 UTC (permalink / raw) To: Nayan Deshmukh; +Cc: ckoenig.leichtzumerken, dri-devel Am 25.03.2018 um 13:21 schrieb Nayan Deshmukh: > On Sun, Mar 25, 2018 at 4:44 PM, Christian König > <christian.koenig@amd.com> wrote: >> Am 25.03.2018 um 13:09 schrieb Nayan Deshmukh: >>> Signed-off-by: Nayan Deshmukh <nayan26deshmukh@gmail.com> >>> --- >>> drivers/gpu/drm/scheduler/gpu_scheduler.c | 1 - >>> 1 file changed, 1 deletion(-) >>> >>> diff --git a/drivers/gpu/drm/scheduler/gpu_scheduler.c >>> b/drivers/gpu/drm/scheduler/gpu_scheduler.c >>> index 0d95888ccc3e..27fdda1264f7 100644 >>> --- a/drivers/gpu/drm/scheduler/gpu_scheduler.c >>> +++ b/drivers/gpu/drm/scheduler/gpu_scheduler.c >>> @@ -117,7 +117,6 @@ drm_sched_rq_select_entity(struct drm_sched_rq *rq) >>> * @sched The pointer to the scheduler >>> * @entity The pointer to a valid drm_sched_entity >>> * @rq The run queue this entity belongs >>> - * @kernel If this is an entity for the kernel >>> * @jobs The max number of jobs in the job queue >> >> If I'm not completely mistaken the jobs parameter is unused and we also have >> a @guilty parameter as well. >> > Yes. But I am not sure what should be done with the jobs parameter. > Should I remove it entirely? Yes, probably a leftover from when we used a kfifo for the queue. Regards, Christian. > >> The description for the @guilty should be something like "atomic_t set to 1 >> when a job on this queue is found to be guilty causing a timeout". >> >> Regards, >> Christian. >> >> >>> * >>> * return 0 if succeed. negative error code on failure >> _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH] drm/scheduler: fix param documentation 2018-03-25 11:29 ` Christian König @ 2018-03-26 15:21 ` Nayan Deshmukh 2018-03-26 16:57 ` Christian König 2018-03-27 8:17 ` Daniel Vetter 2018-03-26 15:29 ` [PATCH] drm/scheduler: remove incorrect " Nayan Deshmukh 1 sibling, 2 replies; 16+ messages in thread From: Nayan Deshmukh @ 2018-03-26 15:21 UTC (permalink / raw) To: dri-devel; +Cc: Nayan Deshmukh, christian.koenig Signed-off-by: Nayan Deshmukh <nayan26deshmukh@gmail.com> --- drivers/gpu/drm/scheduler/gpu_scheduler.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/scheduler/gpu_scheduler.c b/drivers/gpu/drm/scheduler/gpu_scheduler.c index 0d95888ccc3e..1d368bc66ac2 100644 --- a/drivers/gpu/drm/scheduler/gpu_scheduler.c +++ b/drivers/gpu/drm/scheduler/gpu_scheduler.c @@ -117,8 +117,9 @@ drm_sched_rq_select_entity(struct drm_sched_rq *rq) * @sched The pointer to the scheduler * @entity The pointer to a valid drm_sched_entity * @rq The run queue this entity belongs - * @kernel If this is an entity for the kernel * @jobs The max number of jobs in the job queue + * @guilty atomic_t set to 1 when a job on this queue + * is found to be guilty causing a timeout * * return 0 if succeed. negative error code on failure */ -- 2.14.3 _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply related [flat|nested] 16+ messages in thread
* Re: [PATCH] drm/scheduler: fix param documentation 2018-03-26 15:21 ` [PATCH] drm/scheduler: fix " Nayan Deshmukh @ 2018-03-26 16:57 ` Christian König 2018-03-26 17:16 ` Nayan Deshmukh 2018-03-27 8:17 ` Daniel Vetter 1 sibling, 1 reply; 16+ messages in thread From: Christian König @ 2018-03-26 16:57 UTC (permalink / raw) To: Nayan Deshmukh, dri-devel A bit more commit message would be nice to have, like "There is no more @kernel parameter and document the new @guilty parameter". Am 26.03.2018 um 17:21 schrieb Nayan Deshmukh: > Signed-off-by: Nayan Deshmukh <nayan26deshmukh@gmail.com> With the commit message fixed the patch is Reviewed-by: Christian König <christian.koenig@amd.com>. Christian. > --- > drivers/gpu/drm/scheduler/gpu_scheduler.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/scheduler/gpu_scheduler.c b/drivers/gpu/drm/scheduler/gpu_scheduler.c > index 0d95888ccc3e..1d368bc66ac2 100644 > --- a/drivers/gpu/drm/scheduler/gpu_scheduler.c > +++ b/drivers/gpu/drm/scheduler/gpu_scheduler.c > @@ -117,8 +117,9 @@ drm_sched_rq_select_entity(struct drm_sched_rq *rq) > * @sched The pointer to the scheduler > * @entity The pointer to a valid drm_sched_entity > * @rq The run queue this entity belongs > - * @kernel If this is an entity for the kernel > * @jobs The max number of jobs in the job queue > + * @guilty atomic_t set to 1 when a job on this queue > + * is found to be guilty causing a timeout > * > * return 0 if succeed. negative error code on failure > */ _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH] drm/scheduler: fix param documentation 2018-03-26 16:57 ` Christian König @ 2018-03-26 17:16 ` Nayan Deshmukh 0 siblings, 0 replies; 16+ messages in thread From: Nayan Deshmukh @ 2018-03-26 17:16 UTC (permalink / raw) To: dri-devel; +Cc: Nayan Deshmukh, christian.koenig There is no @kernel parameter anymore and document the @guilty parameter Signed-off-by: Nayan Deshmukh <nayan26deshmukh@gmail.com> --- drivers/gpu/drm/scheduler/gpu_scheduler.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/scheduler/gpu_scheduler.c b/drivers/gpu/drm/scheduler/gpu_scheduler.c index 0d95888ccc3e..1d368bc66ac2 100644 --- a/drivers/gpu/drm/scheduler/gpu_scheduler.c +++ b/drivers/gpu/drm/scheduler/gpu_scheduler.c @@ -117,8 +117,9 @@ drm_sched_rq_select_entity(struct drm_sched_rq *rq) * @sched The pointer to the scheduler * @entity The pointer to a valid drm_sched_entity * @rq The run queue this entity belongs - * @kernel If this is an entity for the kernel * @jobs The max number of jobs in the job queue + * @guilty atomic_t set to 1 when a job on this queue + * is found to be guilty causing a timeout * * return 0 if succeed. negative error code on failure */ -- 2.14.3 _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply related [flat|nested] 16+ messages in thread
* Re: [PATCH] drm/scheduler: fix param documentation 2018-03-26 15:21 ` [PATCH] drm/scheduler: fix " Nayan Deshmukh 2018-03-26 16:57 ` Christian König @ 2018-03-27 8:17 ` Daniel Vetter 2018-03-27 17:29 ` Nayan Deshmukh 1 sibling, 1 reply; 16+ messages in thread From: Daniel Vetter @ 2018-03-27 8:17 UTC (permalink / raw) To: Nayan Deshmukh; +Cc: christian.koenig, dri-devel On Mon, Mar 26, 2018 at 08:51:14PM +0530, Nayan Deshmukh wrote: > Signed-off-by: Nayan Deshmukh <nayan26deshmukh@gmail.com> You might want to add a kerneldoc page in Documentation/gpu/scheduler.rst, which pulls in all the nice kerneldoc you have here + has a short intro text what this is all about. Cheers, Daniel > --- > drivers/gpu/drm/scheduler/gpu_scheduler.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/scheduler/gpu_scheduler.c b/drivers/gpu/drm/scheduler/gpu_scheduler.c > index 0d95888ccc3e..1d368bc66ac2 100644 > --- a/drivers/gpu/drm/scheduler/gpu_scheduler.c > +++ b/drivers/gpu/drm/scheduler/gpu_scheduler.c > @@ -117,8 +117,9 @@ drm_sched_rq_select_entity(struct drm_sched_rq *rq) > * @sched The pointer to the scheduler > * @entity The pointer to a valid drm_sched_entity > * @rq The run queue this entity belongs > - * @kernel If this is an entity for the kernel > * @jobs The max number of jobs in the job queue > + * @guilty atomic_t set to 1 when a job on this queue > + * is found to be guilty causing a timeout > * > * return 0 if succeed. negative error code on failure > */ > -- > 2.14.3 > > _______________________________________________ > dri-devel mailing list > dri-devel@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH] drm/scheduler: fix param documentation 2018-03-27 8:17 ` Daniel Vetter @ 2018-03-27 17:29 ` Nayan Deshmukh 2018-03-29 15:26 ` Alex Deucher 0 siblings, 1 reply; 16+ messages in thread From: Nayan Deshmukh @ 2018-03-27 17:29 UTC (permalink / raw) To: Daniel Vetter; +Cc: Christian König, dri-devel On Tue, Mar 27, 2018 at 1:47 PM, Daniel Vetter <daniel@ffwll.ch> wrote: > On Mon, Mar 26, 2018 at 08:51:14PM +0530, Nayan Deshmukh wrote: >> Signed-off-by: Nayan Deshmukh <nayan26deshmukh@gmail.com> > > You might want to add a kerneldoc page in Documentation/gpu/scheduler.rst, > which pulls in all the nice kerneldoc you have here + has a short intro > text what this is all about. > Yeah Sure. I'll send a patch for this in a while. Cheers, Nayan > Cheers, Daniel > >> --- >> drivers/gpu/drm/scheduler/gpu_scheduler.c | 3 ++- >> 1 file changed, 2 insertions(+), 1 deletion(-) >> >> diff --git a/drivers/gpu/drm/scheduler/gpu_scheduler.c b/drivers/gpu/drm/scheduler/gpu_scheduler.c >> index 0d95888ccc3e..1d368bc66ac2 100644 >> --- a/drivers/gpu/drm/scheduler/gpu_scheduler.c >> +++ b/drivers/gpu/drm/scheduler/gpu_scheduler.c >> @@ -117,8 +117,9 @@ drm_sched_rq_select_entity(struct drm_sched_rq *rq) >> * @sched The pointer to the scheduler >> * @entity The pointer to a valid drm_sched_entity >> * @rq The run queue this entity belongs >> - * @kernel If this is an entity for the kernel >> * @jobs The max number of jobs in the job queue >> + * @guilty atomic_t set to 1 when a job on this queue >> + * is found to be guilty causing a timeout >> * >> * return 0 if succeed. negative error code on failure >> */ >> -- >> 2.14.3 >> >> _______________________________________________ >> dri-devel mailing list >> dri-devel@lists.freedesktop.org >> https://lists.freedesktop.org/mailman/listinfo/dri-devel > > -- > Daniel Vetter > Software Engineer, Intel Corporation > http://blog.ffwll.ch _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH] drm/scheduler: fix param documentation 2018-03-27 17:29 ` Nayan Deshmukh @ 2018-03-29 15:26 ` Alex Deucher 2018-03-29 17:08 ` Nayan Deshmukh 0 siblings, 1 reply; 16+ messages in thread From: Alex Deucher @ 2018-03-29 15:26 UTC (permalink / raw) To: Nayan Deshmukh; +Cc: Maling list - DRI developers, Christian König [-- Attachment #1: Type: text/plain, Size: 2117 bytes --] On Tue, Mar 27, 2018 at 1:29 PM, Nayan Deshmukh <nayan26deshmukh@gmail.com> wrote: > On Tue, Mar 27, 2018 at 1:47 PM, Daniel Vetter <daniel@ffwll.ch> wrote: >> On Mon, Mar 26, 2018 at 08:51:14PM +0530, Nayan Deshmukh wrote: >>> Signed-off-by: Nayan Deshmukh <nayan26deshmukh@gmail.com> >> >> You might want to add a kerneldoc page in Documentation/gpu/scheduler.rst, >> which pulls in all the nice kerneldoc you have here + has a short intro >> text what this is all about. >> > Yeah Sure. I'll send a patch for this in a while. FWIW, I started improving the documentation in the GPU scheduler last week, but ran out of time to finish it. Here's the WIP patch. Feel free to run with it. Alex > > Cheers, > Nayan >> Cheers, Daniel >> >>> --- >>> drivers/gpu/drm/scheduler/gpu_scheduler.c | 3 ++- >>> 1 file changed, 2 insertions(+), 1 deletion(-) >>> >>> diff --git a/drivers/gpu/drm/scheduler/gpu_scheduler.c b/drivers/gpu/drm/scheduler/gpu_scheduler.c >>> index 0d95888ccc3e..1d368bc66ac2 100644 >>> --- a/drivers/gpu/drm/scheduler/gpu_scheduler.c >>> +++ b/drivers/gpu/drm/scheduler/gpu_scheduler.c >>> @@ -117,8 +117,9 @@ drm_sched_rq_select_entity(struct drm_sched_rq *rq) >>> * @sched The pointer to the scheduler >>> * @entity The pointer to a valid drm_sched_entity >>> * @rq The run queue this entity belongs >>> - * @kernel If this is an entity for the kernel >>> * @jobs The max number of jobs in the job queue >>> + * @guilty atomic_t set to 1 when a job on this queue >>> + * is found to be guilty causing a timeout >>> * >>> * return 0 if succeed. negative error code on failure >>> */ >>> -- >>> 2.14.3 >>> >>> _______________________________________________ >>> dri-devel mailing list >>> dri-devel@lists.freedesktop.org >>> https://lists.freedesktop.org/mailman/listinfo/dri-devel >> >> -- >> Daniel Vetter >> Software Engineer, Intel Corporation >> http://blog.ffwll.ch > _______________________________________________ > dri-devel mailing list > dri-devel@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel [-- Attachment #2: 0001-XXX-wip-gpu-scheduler-documentation.patch --] [-- Type: text/x-patch, Size: 10685 bytes --] From 1b98337276bb86e0e6aeacffc4cf7ccfa06d7b55 Mon Sep 17 00:00:00 2001 From: Alex Deucher <alexander.deucher@amd.com> Date: Tue, 20 Mar 2018 23:03:48 -0500 Subject: [PATCH] XXX: wip gpu scheduler documentation Signed-off-by: Alex Deucher <alexander.deucher@amd.com> --- drivers/gpu/drm/scheduler/gpu_scheduler.c | 160 ++++++++++++++++++++++-------- 1 file changed, 121 insertions(+), 39 deletions(-) diff --git a/drivers/gpu/drm/scheduler/gpu_scheduler.c b/drivers/gpu/drm/scheduler/gpu_scheduler.c index 0d95888ccc3e..635aec73d5f0 100644 --- a/drivers/gpu/drm/scheduler/gpu_scheduler.c +++ b/drivers/gpu/drm/scheduler/gpu_scheduler.c @@ -39,7 +39,13 @@ static bool drm_sched_entity_is_ready(struct drm_sched_entity *entity); static void drm_sched_wakeup(struct drm_gpu_scheduler *sched); static void drm_sched_process_job(struct dma_fence *f, struct dma_fence_cb *cb); -/* Initialize a given run queue struct */ +/** + * drm_sched_rq_init - initialize a given run queue struct + * + * @rq: scheduler run queue + * + * This function initializes a scheduler runqueue. + */ static void drm_sched_rq_init(struct drm_sched_rq *rq) { spin_lock_init(&rq->lock); @@ -47,6 +53,14 @@ static void drm_sched_rq_init(struct drm_sched_rq *rq) rq->current_entity = NULL; } +/** + * drm_sched_rq_add_entity - add an entity + * + * @rq: scheduler run queue + * @entity: scheduler entity + * + * This function adds a scheduling entity to the run queue. + */ static void drm_sched_rq_add_entity(struct drm_sched_rq *rq, struct drm_sched_entity *entity) { @@ -57,6 +71,14 @@ static void drm_sched_rq_add_entity(struct drm_sched_rq *rq, spin_unlock(&rq->lock); } +/** + * drm_sched_rq_remove_entity - remove an entity + * + * @rq: scheduler run queue + * @entity: scheduler entity + * + * This function removes a scheduling entity from the run queue. + */ static void drm_sched_rq_remove_entity(struct drm_sched_rq *rq, struct drm_sched_entity *entity) { @@ -70,9 +92,9 @@ static void drm_sched_rq_remove_entity(struct drm_sched_rq *rq, } /** - * Select an entity which could provide a job to run + * drm_sched_rq_select_entity - Select an entity which could provide a job to run * - * @rq The run queue to check. + * @rq: scheduler run queue to check. * * Try to find a ready entity, returns NULL if none found. */ @@ -112,16 +134,17 @@ drm_sched_rq_select_entity(struct drm_sched_rq *rq) } /** - * Init a context entity used by scheduler when submit to HW ring. + * drm_sched_entity_init - init a context entity used by scheduler when submit to HW queue * - * @sched The pointer to the scheduler - * @entity The pointer to a valid drm_sched_entity - * @rq The run queue this entity belongs - * @kernel If this is an entity for the kernel - * @jobs The max number of jobs in the job queue + * @sched: scheduler instance + * @entity: scheduler entity + * @rq: scheduler run queue to which this entity belongs + * @kernel: if this is an entity for the kernel + * @jobs: max number of jobs in the job queue + * @guilty: the guilty marker for the context * - * return 0 if succeed. negative error code on failure -*/ + * Return 0 on succeed, negative error code on failure. + */ int drm_sched_entity_init(struct drm_gpu_scheduler *sched, struct drm_sched_entity *entity, struct drm_sched_rq *rq, @@ -148,10 +171,10 @@ int drm_sched_entity_init(struct drm_gpu_scheduler *sched, EXPORT_SYMBOL(drm_sched_entity_init); /** - * Query if entity is initialized + * drm_sched_entity_is_initialized - Query if entity is initialized * - * @sched Pointer to scheduler instance - * @entity The pointer to a valid scheduler entity + * @sched: scheduler instance + * @entity: scheduler entity * * return true if entity is initialized, false otherwise */ @@ -163,11 +186,11 @@ static bool drm_sched_entity_is_initialized(struct drm_gpu_scheduler *sched, } /** - * Check if entity is idle + * drm_sched_entity_is_idle - Check if entity is idle * - * @entity The pointer to a valid scheduler entity + * @entity: scheduler entity * - * Return true if entity don't has any unscheduled jobs. + * Return true if the entity does not have any unscheduled jobs. */ static bool drm_sched_entity_is_idle(struct drm_sched_entity *entity) { @@ -179,9 +202,9 @@ static bool drm_sched_entity_is_idle(struct drm_sched_entity *entity) } /** - * Check if entity is ready + * drm_sched_entity_is_ready - Check if entity is ready * - * @entity The pointer to a valid scheduler entity + * @entity: scheduler entity * * Return true if entity could provide a job. */ @@ -197,12 +220,12 @@ static bool drm_sched_entity_is_ready(struct drm_sched_entity *entity) } /** - * Destroy a context entity + * drm_sched_entity_fini - Destroy a context entity * - * @sched Pointer to scheduler instance - * @entity The pointer to a valid scheduler entity + * @sched: scheduler instance + * @entity: scheduler entity * - * Cleanup and free the allocated resources. + * Cleanup and free the allocated entity resources. */ void drm_sched_entity_fini(struct drm_gpu_scheduler *sched, struct drm_sched_entity *entity) @@ -457,7 +480,15 @@ static void drm_sched_job_timedout(struct work_struct *work) job->sched->ops->timedout_job(job); } -void drm_sched_hw_job_reset(struct drm_gpu_scheduler *sched, struct drm_sched_job *bad) +/** + * drm_sched_hw_job_reset - stop the scheduer if it contains the bad job + * + * @sched: scheduler instance + * @bad: bad scheduler job + * + */ +void drm_sched_hw_job_reset(struct drm_gpu_scheduler *sched, + struct drm_sched_job *bad) { struct drm_sched_job *s_job; struct drm_sched_entity *entity, *tmp; @@ -478,8 +509,8 @@ void drm_sched_hw_job_reset(struct drm_gpu_scheduler *sched, struct drm_sched_jo if (bad && bad->s_priority != DRM_SCHED_PRIORITY_KERNEL) { atomic_inc(&bad->karma); /* don't increase @bad's karma if it's from KERNEL RQ, - * becuase sometimes GPU hang would cause kernel jobs (like VM updating jobs) - * corrupt but keep in mind that kernel jobs always considered good. + * because sometimes GPU hang would cause kernel jobs (like VM updating jobs) + * to break but keep in mind that kernel jobs always considered good. */ for (i = DRM_SCHED_PRIORITY_MIN; i < DRM_SCHED_PRIORITY_KERNEL; i++ ) { struct drm_sched_rq *rq = &sched->sched_rq[i]; @@ -501,6 +532,12 @@ void drm_sched_hw_job_reset(struct drm_gpu_scheduler *sched, struct drm_sched_jo } EXPORT_SYMBOL(drm_sched_hw_job_reset); +/** + * drm_sched_job_recovery - recover jobs after a reset + * + * @sched: scheduler instance + * + */ void drm_sched_job_recovery(struct drm_gpu_scheduler *sched) { struct drm_sched_job *s_job, *tmp; @@ -548,7 +585,16 @@ void drm_sched_job_recovery(struct drm_gpu_scheduler *sched) } EXPORT_SYMBOL(drm_sched_job_recovery); -/* init a sched_job with basic field */ +/** + * drm_sched_job_init - init a scheduler job + * + * @job: scheduler job to init + * @sched: scheduler instance + * @entity: scheduler entity to use + * @owner: job owner for debugging + * + * Returns 0 for success, negative error code otherwise. + */ int drm_sched_job_init(struct drm_sched_job *job, struct drm_gpu_scheduler *sched, struct drm_sched_entity *entity, @@ -570,7 +616,11 @@ int drm_sched_job_init(struct drm_sched_job *job, EXPORT_SYMBOL(drm_sched_job_init); /** - * Return ture if we can push more jobs to the hw. + * drm_sched_ready - is the scheduler ready + * + * @sched: scheduler instance + * + * Return true if we can push more jobs to the hw, otherwise false. */ static bool drm_sched_ready(struct drm_gpu_scheduler *sched) { @@ -579,7 +629,10 @@ static bool drm_sched_ready(struct drm_gpu_scheduler *sched) } /** - * Wake up the scheduler when it is ready + * drm_sched_wakeup - Wake up the scheduler when it is ready + * + * @sched: scheduler instance + * */ static void drm_sched_wakeup(struct drm_gpu_scheduler *sched) { @@ -588,8 +641,12 @@ static void drm_sched_wakeup(struct drm_gpu_scheduler *sched) } /** - * Select next entity to process -*/ + * drm_sched_select_entity - Select next entity to process + * + * @sched: scheduler instance + * + * Returns the entity to process or NULL if none are found. + */ static struct drm_sched_entity * drm_sched_select_entity(struct drm_gpu_scheduler *sched) { @@ -609,6 +666,13 @@ drm_sched_select_entity(struct drm_gpu_scheduler *sched) return entity; } +/** + * drm_sched_process_job - process a job + * + * @dma_fence: fence + * @cb: fence callbacks + * + */ static void drm_sched_process_job(struct dma_fence *f, struct dma_fence_cb *cb) { struct drm_sched_fence *s_fence = @@ -624,6 +688,13 @@ static void drm_sched_process_job(struct dma_fence *f, struct dma_fence_cb *cb) wake_up_interruptible(&sched->wake_up_worker); } +/** + * drm_sched_blocked - check if the scheduler is blocked + * + * @sched: scheduler instance + * + * Returns true if blocked, otherwise false. + */ static bool drm_sched_blocked(struct drm_gpu_scheduler *sched) { if (kthread_should_park()) { @@ -634,6 +705,13 @@ static bool drm_sched_blocked(struct drm_gpu_scheduler *sched) return false; } +/** + * drm_sched_main - main scheduler thread + * + * @param: scheduler instance + * + * Returns 0. + */ static int drm_sched_main(void *param) { struct sched_param sparam = {.sched_priority = 1}; @@ -688,15 +766,17 @@ static int drm_sched_main(void *param) } /** - * Init a gpu scheduler instance + * drm_sched_init - Init a gpu scheduler instance * - * @sched The pointer to the scheduler - * @ops The backend operations for this scheduler. - * @hw_submissions Number of hw submissions to do. - * @name Name used for debugging + * @sched: scheduler instance + * @ops: backend operations for this scheduler + * @hw_submissions: number of hw submissions that can be in flight + * @hang_limit: number of times to allow a job to hang before dropping it + * @timeout: timeout value in jiffies for the scheduler + * @name: name used for debugging * * Return 0 on success, otherwise error code. -*/ + */ int drm_sched_init(struct drm_gpu_scheduler *sched, const struct drm_sched_backend_ops *ops, unsigned hw_submission, @@ -732,9 +812,11 @@ int drm_sched_init(struct drm_gpu_scheduler *sched, EXPORT_SYMBOL(drm_sched_init); /** - * Destroy a gpu scheduler + * drm_sched_fini - Destroy a gpu scheduler + * + * @sched: scheduler instance * - * @sched The pointer to the scheduler + * Tears down and cleans up the scheduler. */ void drm_sched_fini(struct drm_gpu_scheduler *sched) { -- 2.13.6 [-- Attachment #3: Type: text/plain, Size: 160 bytes --] _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply related [flat|nested] 16+ messages in thread
* Re: [PATCH] drm/scheduler: fix param documentation 2018-03-29 15:26 ` Alex Deucher @ 2018-03-29 17:08 ` Nayan Deshmukh 0 siblings, 0 replies; 16+ messages in thread From: Nayan Deshmukh @ 2018-03-29 17:08 UTC (permalink / raw) To: Alex Deucher; +Cc: Maling list - DRI developers, Christian König On Thu, Mar 29, 2018 at 8:56 PM, Alex Deucher <alexdeucher@gmail.com> wrote: > On Tue, Mar 27, 2018 at 1:29 PM, Nayan Deshmukh > <nayan26deshmukh@gmail.com> wrote: >> On Tue, Mar 27, 2018 at 1:47 PM, Daniel Vetter <daniel@ffwll.ch> wrote: >>> On Mon, Mar 26, 2018 at 08:51:14PM +0530, Nayan Deshmukh wrote: >>>> Signed-off-by: Nayan Deshmukh <nayan26deshmukh@gmail.com> >>> >>> You might want to add a kerneldoc page in Documentation/gpu/scheduler.rst, >>> which pulls in all the nice kerneldoc you have here + has a short intro >>> text what this is all about. >>> >> Yeah Sure. I'll send a patch for this in a while. > > FWIW, I started improving the documentation in the GPU scheduler last > week, but ran out of time to finish it. Here's the WIP patch. Feel > free to run with it. > This will be useful. I'll go over it during the weekend and probably send a patch early next week. Thanks > Alex > >> >> Cheers, >> Nayan >>> Cheers, Daniel >>> >>>> --- >>>> drivers/gpu/drm/scheduler/gpu_scheduler.c | 3 ++- >>>> 1 file changed, 2 insertions(+), 1 deletion(-) >>>> >>>> diff --git a/drivers/gpu/drm/scheduler/gpu_scheduler.c b/drivers/gpu/drm/scheduler/gpu_scheduler.c >>>> index 0d95888ccc3e..1d368bc66ac2 100644 >>>> --- a/drivers/gpu/drm/scheduler/gpu_scheduler.c >>>> +++ b/drivers/gpu/drm/scheduler/gpu_scheduler.c >>>> @@ -117,8 +117,9 @@ drm_sched_rq_select_entity(struct drm_sched_rq *rq) >>>> * @sched The pointer to the scheduler >>>> * @entity The pointer to a valid drm_sched_entity >>>> * @rq The run queue this entity belongs >>>> - * @kernel If this is an entity for the kernel >>>> * @jobs The max number of jobs in the job queue >>>> + * @guilty atomic_t set to 1 when a job on this queue >>>> + * is found to be guilty causing a timeout >>>> * >>>> * return 0 if succeed. negative error code on failure >>>> */ >>>> -- >>>> 2.14.3 >>>> >>>> _______________________________________________ >>>> dri-devel mailing list >>>> dri-devel@lists.freedesktop.org >>>> https://lists.freedesktop.org/mailman/listinfo/dri-devel >>> >>> -- >>> Daniel Vetter >>> Software Engineer, Intel Corporation >>> http://blog.ffwll.ch >> _______________________________________________ >> dri-devel mailing list >> dri-devel@lists.freedesktop.org >> https://lists.freedesktop.org/mailman/listinfo/dri-devel _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH] drm/scheduler: remove incorrect param documentation 2018-03-25 11:29 ` Christian König 2018-03-26 15:21 ` [PATCH] drm/scheduler: fix " Nayan Deshmukh @ 2018-03-26 15:29 ` Nayan Deshmukh 2018-03-26 16:59 ` Christian König 1 sibling, 1 reply; 16+ messages in thread From: Nayan Deshmukh @ 2018-03-26 15:29 UTC (permalink / raw) To: Christian König; +Cc: ckoenig.leichtzumerken, dri-devel I am not removing jobs parameters as of now as it will lead to changes in all the calls to this function and will affect the etnaviv patches. Regards, Nayan. On Sun, Mar 25, 2018 at 4:59 PM, Christian König <christian.koenig@amd.com> wrote: > Am 25.03.2018 um 13:21 schrieb Nayan Deshmukh: >> >> On Sun, Mar 25, 2018 at 4:44 PM, Christian König >> <christian.koenig@amd.com> wrote: >>> >>> Am 25.03.2018 um 13:09 schrieb Nayan Deshmukh: >>>> >>>> Signed-off-by: Nayan Deshmukh <nayan26deshmukh@gmail.com> >>>> --- >>>> drivers/gpu/drm/scheduler/gpu_scheduler.c | 1 - >>>> 1 file changed, 1 deletion(-) >>>> >>>> diff --git a/drivers/gpu/drm/scheduler/gpu_scheduler.c >>>> b/drivers/gpu/drm/scheduler/gpu_scheduler.c >>>> index 0d95888ccc3e..27fdda1264f7 100644 >>>> --- a/drivers/gpu/drm/scheduler/gpu_scheduler.c >>>> +++ b/drivers/gpu/drm/scheduler/gpu_scheduler.c >>>> @@ -117,7 +117,6 @@ drm_sched_rq_select_entity(struct drm_sched_rq *rq) >>>> * @sched The pointer to the scheduler >>>> * @entity The pointer to a valid drm_sched_entity >>>> * @rq The run queue this entity belongs >>>> - * @kernel If this is an entity for the kernel >>>> * @jobs The max number of jobs in the job queue >>> >>> >>> If I'm not completely mistaken the jobs parameter is unused and we also >>> have >>> a @guilty parameter as well. >>> >> Yes. But I am not sure what should be done with the jobs parameter. >> Should I remove it entirely? > > > Yes, probably a leftover from when we used a kfifo for the queue. > > Regards, > Christian. > > >> >>> The description for the @guilty should be something like "atomic_t set to >>> 1 >>> when a job on this queue is found to be guilty causing a timeout". >>> >>> Regards, >>> Christian. >>> >>> >>>> * >>>> * return 0 if succeed. negative error code on failure >>> >>> > _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH] drm/scheduler: remove incorrect param documentation 2018-03-26 15:29 ` [PATCH] drm/scheduler: remove incorrect " Nayan Deshmukh @ 2018-03-26 16:59 ` Christian König 2018-03-26 17:04 ` Nayan Deshmukh 0 siblings, 1 reply; 16+ messages in thread From: Christian König @ 2018-03-26 16:59 UTC (permalink / raw) To: Nayan Deshmukh; +Cc: ckoenig.leichtzumerken, dri-devel Yeah, probably good idea to have that separate. But would still be a nice cleanup and it is merged upstream after the etnaviv patches anyway. Regards, Christian. Am 26.03.2018 um 17:29 schrieb Nayan Deshmukh: > I am not removing jobs parameters as of now as it will lead to changes > in all the calls to this function and will affect the etnaviv patches. > > Regards, > Nayan. > > On Sun, Mar 25, 2018 at 4:59 PM, Christian König > <christian.koenig@amd.com> wrote: >> Am 25.03.2018 um 13:21 schrieb Nayan Deshmukh: >>> On Sun, Mar 25, 2018 at 4:44 PM, Christian König >>> <christian.koenig@amd.com> wrote: >>>> Am 25.03.2018 um 13:09 schrieb Nayan Deshmukh: >>>>> Signed-off-by: Nayan Deshmukh <nayan26deshmukh@gmail.com> >>>>> --- >>>>> drivers/gpu/drm/scheduler/gpu_scheduler.c | 1 - >>>>> 1 file changed, 1 deletion(-) >>>>> >>>>> diff --git a/drivers/gpu/drm/scheduler/gpu_scheduler.c >>>>> b/drivers/gpu/drm/scheduler/gpu_scheduler.c >>>>> index 0d95888ccc3e..27fdda1264f7 100644 >>>>> --- a/drivers/gpu/drm/scheduler/gpu_scheduler.c >>>>> +++ b/drivers/gpu/drm/scheduler/gpu_scheduler.c >>>>> @@ -117,7 +117,6 @@ drm_sched_rq_select_entity(struct drm_sched_rq *rq) >>>>> * @sched The pointer to the scheduler >>>>> * @entity The pointer to a valid drm_sched_entity >>>>> * @rq The run queue this entity belongs >>>>> - * @kernel If this is an entity for the kernel >>>>> * @jobs The max number of jobs in the job queue >>>> >>>> If I'm not completely mistaken the jobs parameter is unused and we also >>>> have >>>> a @guilty parameter as well. >>>> >>> Yes. But I am not sure what should be done with the jobs parameter. >>> Should I remove it entirely? >> >> Yes, probably a leftover from when we used a kfifo for the queue. >> >> Regards, >> Christian. >> >> >>>> The description for the @guilty should be something like "atomic_t set to >>>> 1 >>>> when a job on this queue is found to be guilty causing a timeout". >>>> >>>> Regards, >>>> Christian. >>>> >>>> >>>>> * >>>>> * return 0 if succeed. negative error code on failure >>>> _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH] drm/scheduler: remove incorrect param documentation 2018-03-26 16:59 ` Christian König @ 2018-03-26 17:04 ` Nayan Deshmukh 0 siblings, 0 replies; 16+ messages in thread From: Nayan Deshmukh @ 2018-03-26 17:04 UTC (permalink / raw) To: Christian König; +Cc: ckoenig.leichtzumerken, dri-devel I will send in the patches. On Mon, Mar 26, 2018 at 10:29 PM, Christian König <christian.koenig@amd.com> wrote: > Yeah, probably good idea to have that separate. > > But would still be a nice cleanup and it is merged upstream after the > etnaviv patches anyway. > > Regards, > Christian. > > > Am 26.03.2018 um 17:29 schrieb Nayan Deshmukh: >> >> I am not removing jobs parameters as of now as it will lead to changes >> in all the calls to this function and will affect the etnaviv patches. >> >> Regards, >> Nayan. >> >> On Sun, Mar 25, 2018 at 4:59 PM, Christian König >> <christian.koenig@amd.com> wrote: >>> >>> Am 25.03.2018 um 13:21 schrieb Nayan Deshmukh: >>>> >>>> On Sun, Mar 25, 2018 at 4:44 PM, Christian König >>>> <christian.koenig@amd.com> wrote: >>>>> >>>>> Am 25.03.2018 um 13:09 schrieb Nayan Deshmukh: >>>>>> >>>>>> Signed-off-by: Nayan Deshmukh <nayan26deshmukh@gmail.com> >>>>>> --- >>>>>> drivers/gpu/drm/scheduler/gpu_scheduler.c | 1 - >>>>>> 1 file changed, 1 deletion(-) >>>>>> >>>>>> diff --git a/drivers/gpu/drm/scheduler/gpu_scheduler.c >>>>>> b/drivers/gpu/drm/scheduler/gpu_scheduler.c >>>>>> index 0d95888ccc3e..27fdda1264f7 100644 >>>>>> --- a/drivers/gpu/drm/scheduler/gpu_scheduler.c >>>>>> +++ b/drivers/gpu/drm/scheduler/gpu_scheduler.c >>>>>> @@ -117,7 +117,6 @@ drm_sched_rq_select_entity(struct drm_sched_rq >>>>>> *rq) >>>>>> * @sched The pointer to the scheduler >>>>>> * @entity The pointer to a valid drm_sched_entity >>>>>> * @rq The run queue this entity belongs >>>>>> - * @kernel If this is an entity for the kernel >>>>>> * @jobs The max number of jobs in the job queue >>>>> >>>>> >>>>> If I'm not completely mistaken the jobs parameter is unused and we also >>>>> have >>>>> a @guilty parameter as well. >>>>> >>>> Yes. But I am not sure what should be done with the jobs parameter. >>>> Should I remove it entirely? >>> >>> >>> Yes, probably a leftover from when we used a kfifo for the queue. >>> >>> Regards, >>> Christian. >>> >>> >>>>> The description for the @guilty should be something like "atomic_t set >>>>> to >>>>> 1 >>>>> when a job on this queue is found to be guilty causing a timeout". >>>>> >>>>> Regards, >>>>> Christian. >>>>> >>>>> >>>>>> * >>>>>> * return 0 if succeed. negative error code on failure >>>>> >>>>> > _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH] drm/scheduler: fix param documentation @ 2018-08-09 9:48 Nayan Deshmukh 2018-08-09 14:15 ` Alex Deucher 0 siblings, 1 reply; 16+ messages in thread From: Nayan Deshmukh @ 2018-08-09 9:48 UTC (permalink / raw) To: dri-devel; +Cc: Nayan Deshmukh, christian.koenig We no longer have sched parameter so remove its description as well Signed-off-by: Nayan Deshmukh <nayan26deshmukh@gmail.com> --- drivers/gpu/drm/scheduler/gpu_scheduler.c | 4 ---- 1 file changed, 4 deletions(-) diff --git a/drivers/gpu/drm/scheduler/gpu_scheduler.c b/drivers/gpu/drm/scheduler/gpu_scheduler.c index bfa86d5eb8ac..da2da8d85035 100644 --- a/drivers/gpu/drm/scheduler/gpu_scheduler.c +++ b/drivers/gpu/drm/scheduler/gpu_scheduler.c @@ -281,7 +281,6 @@ static void drm_sched_entity_kill_jobs_cb(struct dma_fence *f, /** * drm_sched_entity_flush - Flush a context entity * - * @sched: scheduler instance * @entity: scheduler entity * @timeout: time to wait in for Q to become empty in jiffies. * @@ -324,7 +323,6 @@ EXPORT_SYMBOL(drm_sched_entity_flush); /** * drm_sched_entity_cleanup - Destroy a context entity * - * @sched: scheduler instance * @entity: scheduler entity * * This should be called after @drm_sched_entity_do_release. It goes over the @@ -389,7 +387,6 @@ EXPORT_SYMBOL(drm_sched_entity_fini); /** * drm_sched_entity_fini - Destroy a context entity * - * @sched: scheduler instance * @entity: scheduler entity * * Calls drm_sched_entity_do_release() and drm_sched_entity_cleanup() @@ -781,7 +778,6 @@ EXPORT_SYMBOL(drm_sched_job_recovery); * drm_sched_job_init - init a scheduler job * * @job: scheduler job to init - * @sched: scheduler instance * @entity: scheduler entity to use * @owner: job owner for debugging * -- 2.14.3 _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply related [flat|nested] 16+ messages in thread
* Re: [PATCH] drm/scheduler: fix param documentation 2018-08-09 9:48 [PATCH] drm/scheduler: fix " Nayan Deshmukh @ 2018-08-09 14:15 ` Alex Deucher 0 siblings, 0 replies; 16+ messages in thread From: Alex Deucher @ 2018-08-09 14:15 UTC (permalink / raw) To: Nayan Deshmukh; +Cc: Christian Koenig, Maling list - DRI developers On Thu, Aug 9, 2018 at 5:48 AM, Nayan Deshmukh <nayan26deshmukh@gmail.com> wrote: > We no longer have sched parameter so remove its description > as well > > Signed-off-by: Nayan Deshmukh <nayan26deshmukh@gmail.com> Applied. Thanks! Alex > --- > drivers/gpu/drm/scheduler/gpu_scheduler.c | 4 ---- > 1 file changed, 4 deletions(-) > > diff --git a/drivers/gpu/drm/scheduler/gpu_scheduler.c b/drivers/gpu/drm/scheduler/gpu_scheduler.c > index bfa86d5eb8ac..da2da8d85035 100644 > --- a/drivers/gpu/drm/scheduler/gpu_scheduler.c > +++ b/drivers/gpu/drm/scheduler/gpu_scheduler.c > @@ -281,7 +281,6 @@ static void drm_sched_entity_kill_jobs_cb(struct dma_fence *f, > /** > * drm_sched_entity_flush - Flush a context entity > * > - * @sched: scheduler instance > * @entity: scheduler entity > * @timeout: time to wait in for Q to become empty in jiffies. > * > @@ -324,7 +323,6 @@ EXPORT_SYMBOL(drm_sched_entity_flush); > /** > * drm_sched_entity_cleanup - Destroy a context entity > * > - * @sched: scheduler instance > * @entity: scheduler entity > * > * This should be called after @drm_sched_entity_do_release. It goes over the > @@ -389,7 +387,6 @@ EXPORT_SYMBOL(drm_sched_entity_fini); > /** > * drm_sched_entity_fini - Destroy a context entity > * > - * @sched: scheduler instance > * @entity: scheduler entity > * > * Calls drm_sched_entity_do_release() and drm_sched_entity_cleanup() > @@ -781,7 +778,6 @@ EXPORT_SYMBOL(drm_sched_job_recovery); > * drm_sched_job_init - init a scheduler job > * > * @job: scheduler job to init > - * @sched: scheduler instance > * @entity: scheduler entity to use > * @owner: job owner for debugging > * > -- > 2.14.3 > > _______________________________________________ > dri-devel mailing list > dri-devel@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 16+ messages in thread
end of thread, other threads:[~2018-08-09 14:15 UTC | newest] Thread overview: 16+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2018-03-25 11:09 [PATCH] drm/scheduler: remove incorrect param documentation Nayan Deshmukh 2018-03-25 11:14 ` Christian König 2018-03-25 11:21 ` Nayan Deshmukh 2018-03-25 11:29 ` Christian König 2018-03-26 15:21 ` [PATCH] drm/scheduler: fix " Nayan Deshmukh 2018-03-26 16:57 ` Christian König 2018-03-26 17:16 ` Nayan Deshmukh 2018-03-27 8:17 ` Daniel Vetter 2018-03-27 17:29 ` Nayan Deshmukh 2018-03-29 15:26 ` Alex Deucher 2018-03-29 17:08 ` Nayan Deshmukh 2018-03-26 15:29 ` [PATCH] drm/scheduler: remove incorrect " Nayan Deshmukh 2018-03-26 16:59 ` Christian König 2018-03-26 17:04 ` Nayan Deshmukh -- strict thread matches above, loose matches on Subject: below -- 2018-08-09 9:48 [PATCH] drm/scheduler: fix " Nayan Deshmukh 2018-08-09 14:15 ` Alex Deucher
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox