Commit ada51219 authored by Matthew Brost's avatar Matthew Brost
Browse files

drm/xe: Create ordered workqueue for GT TLB invalidation jobs 



No sense to schedule GT TLB invalidation jobs in parallel which target
the same GT given these all contend on the same lock, create ordered
workqueue for GT TLB invalidation jobs.

v3:
 - Fix type in commmit message (Stuart)

Signed-off-by: default avatarMatthew Brost <matthew.brost@intel.com>
Reviewed-by: default avatarStuart Summers <stuart.summers@intel.com>
Link: https://lore.kernel.org/r/20250724191216.4076566-4-matthew.brost@intel.com
parent 69f187d4

drivers/gpu/drm/xe/xe_gt_tlb_invalidation.c

+8 −0
Original line number Diff line number Diff line
@@ -3,6 +3,8 @@ 
 * Copyright © 2023 Intel Corporation

 */



#include <drm/drm_managed.h>



#include "xe_gt_tlb_invalidation.h"



#include "abi/guc_actions_abi.h"
@@ -123,6 +125,12 @@ int xe_gt_tlb_invalidation_init_early(struct xe_gt *gt) 
	INIT_DELAYED_WORK(&gt->tlb_invalidation.fence_tdr,

			  xe_gt_tlb_fence_timeout);



	gt->tlb_invalidation.job_wq =

		drmm_alloc_ordered_workqueue(&gt_to_xe(gt)->drm, "gt-tbl-inval-job-wq",

					     WQ_MEM_RECLAIM);

	if (IS_ERR(gt->tlb_invalidation.job_wq))

		return PTR_ERR(gt->tlb_invalidation.job_wq);



	return 0;

}

drivers/gpu/drm/xe/xe_gt_types.h

+2 −0
Original line number Diff line number Diff line
@@ -210,6 +210,8 @@ struct xe_gt { 
		 * xe_gt_tlb_fence_timeout after the timeut interval is over.

		 */

		struct delayed_work fence_tdr;

		/** @wtlb_invalidation.wq: schedules GT TLB invalidation jobs */

		struct workqueue_struct *job_wq;

		/** @tlb_invalidation.lock: protects TLB invalidation fences */

		spinlock_t lock;

	} tlb_invalidation;