[PATCH 2/2] drm/scheduler: Remove obsolete spinlock.

[Andrey Grodzovsky]

Yea, I might need to give another thought to whether this  spinlock can 
actually be removed.

Andrey

On 05/15/2018 03:38 PM, Alex Deucher wrote:
> On Tue, May 15, 2018 at 3:31 PM, Andrey Grodzovsky
> <andrey.grodzovsky at amd.com> wrote:
>> Signed-off-by: Andrey Grodzovsky <andrey.grodzovsky at amd.com>
> Please provide a better patch description.
>
> Alex
>
>> ---
>>   drivers/gpu/drm/scheduler/gpu_scheduler.c | 4 ----
>>   include/drm/gpu_scheduler.h               | 1 -
>>   2 files changed, 5 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/scheduler/gpu_scheduler.c b/drivers/gpu/drm/scheduler/gpu_scheduler.c
>> index 1f1dd70..2569a63 100644
>> --- a/drivers/gpu/drm/scheduler/gpu_scheduler.c
>> +++ b/drivers/gpu/drm/scheduler/gpu_scheduler.c
>> @@ -140,7 +140,6 @@ int drm_sched_entity_init(struct drm_gpu_scheduler *sched,
>>          entity->last_scheduled = NULL;
>>
>>          spin_lock_init(&entity->rq_lock);
>> -       spin_lock_init(&entity->queue_lock);
>>          spsc_queue_init(&entity->job_queue);
>>
>>          atomic_set(&entity->fence_seq, 0);
>> @@ -424,11 +423,8 @@ void drm_sched_entity_push_job(struct drm_sched_job *sched_job,
>>
>>          trace_drm_sched_job(sched_job, entity);
>>
>> -       spin_lock(&entity->queue_lock);
>>          first = spsc_queue_push(&entity->job_queue, &sched_job->queue_node);
>>
>> -       spin_unlock(&entity->queue_lock);
>> -
>>          /* first job wakes up scheduler */
>>          if (first) {
>>                  /* Add the entity to the run queue */
>> diff --git a/include/drm/gpu_scheduler.h b/include/drm/gpu_scheduler.h
>> index 350a62c..683eb65 100644
>> --- a/include/drm/gpu_scheduler.h
>> +++ b/include/drm/gpu_scheduler.h
>> @@ -56,7 +56,6 @@ struct drm_sched_entity {
>>          spinlock_t                      rq_lock;
>>          struct drm_gpu_scheduler        *sched;
>>
>> -       spinlock_t                      queue_lock;
>>          struct spsc_queue               job_queue;
>>
>>          atomic_t                        fence_seq;
>> --
>> 2.7.4
>>
>> _______________________________________________
>> dri-devel mailing list
>> dri-devel at lists.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/dri-devel