From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 19EE1CAC598 for ; Wed, 17 Sep 2025 03:46:21 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id AF00910E408; Wed, 17 Sep 2025 03:46:21 +0000 (UTC) Authentication-Results: gabe.freedesktop.org; dkim=pass (2048-bit key; unprotected) header.d=intel.com header.i=@intel.com header.b="mxyrIrqF"; dkim-atps=neutral Received: from mgamail.intel.com (mgamail.intel.com [192.198.163.9]) by gabe.freedesktop.org (Postfix) with ESMTPS id 1FE4210E409 for ; Wed, 17 Sep 2025 03:46:21 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1758080781; x=1789616781; h=from:to:subject:date:message-id:in-reply-to:references: mime-version:content-transfer-encoding; bh=L7g2+jC8QpsY4djJ6hXGv1CNwGjSDFBihdn2buXJGTw=; b=mxyrIrqFshodvXaw+DORlOXM5zb3MMlT0pGUonY1y0j0falwxrOAYXGg QiJWC01zmr3tBA8n8EjQIX98CWqZta8DQG7MRtV3NFkwEd0I5NFKmenQI nQ54hWmuMi6Xy4CbEdMe/56GY2DPEuFA5IiRy7+vspRpeXaUwggEEAkrw vOqm6aKWHh8lGbOQTQuPrF7lWgsq22RsqhQGJhA3w+Itg9Q3XYPeelfum v9BfbWhUJPcW44eZnYyUpnTqDcCZZvVOfgBornG7QmR92rejMKY+PiqLA ePfNQukmupnsndry6BTI60cBx0lUSDgHQ0lG5Vh+GGwqmwV3cNTRBCc+P g==; X-CSE-ConnectionGUID: ZvhPSZtZTAiX8LY0siw3XQ== X-CSE-MsgGUID: O0yO6E5bQ2aPncN7FiSZRA== X-IronPort-AV: E=McAfee;i="6800,10657,11555"; a="71060785" X-IronPort-AV: E=Sophos;i="6.18,270,1751266800"; d="scan'208";a="71060785" Received: from fmviesa003.fm.intel.com ([10.60.135.143]) by fmvoesa103.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 16 Sep 2025 20:46:21 -0700 X-CSE-ConnectionGUID: ZvH3S4/rR02irWZmn/LrGg== X-CSE-MsgGUID: Op5IduxhT1CjJjeTmyfXcQ== X-ExtLoop1: 1 Received: from lstrano-desk.jf.intel.com ([10.54.39.91]) by fmviesa003-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 16 Sep 2025 20:46:21 -0700 From: Matthew Brost To: intel-xe@lists.freedesktop.org Subject: [PATCH 03/28] Revert "drm/xe/vf: Post migration, repopulate ring area for pending request" Date: Tue, 16 Sep 2025 20:45:50 -0700 Message-Id: <20250917034615.3977603-4-matthew.brost@intel.com> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20250917034615.3977603-1-matthew.brost@intel.com> References: <20250917034615.3977603-1-matthew.brost@intel.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-BeenThere: intel-xe@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Intel Xe graphics driver List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-xe-bounces@lists.freedesktop.org Sender: "Intel-xe" This reverts commit a0dda25d24e636df5c30a9370464b7cebc709faf. --- drivers/gpu/drm/xe/xe_exec_queue.c | 24 ------------------------ drivers/gpu/drm/xe/xe_exec_queue.h | 3 +-- drivers/gpu/drm/xe/xe_guc_submit.c | 24 ------------------------ drivers/gpu/drm/xe/xe_guc_submit.h | 2 -- drivers/gpu/drm/xe/xe_sriov_vf.c | 1 - 5 files changed, 1 insertion(+), 53 deletions(-) diff --git a/drivers/gpu/drm/xe/xe_exec_queue.c b/drivers/gpu/drm/xe/xe_exec_queue.c index 063c89d981e5..914df8c2d323 100644 --- a/drivers/gpu/drm/xe/xe_exec_queue.c +++ b/drivers/gpu/drm/xe/xe_exec_queue.c @@ -1115,27 +1115,3 @@ int xe_exec_queue_contexts_hwsp_rebase(struct xe_exec_queue *q, void *scratch) return err; } - -/** - * xe_exec_queue_jobs_ring_restore - Re-emit ring commands of requests pending on given queue. - * @q: the &xe_exec_queue struct instance - */ -void xe_exec_queue_jobs_ring_restore(struct xe_exec_queue *q) -{ - struct xe_gpu_scheduler *sched = &q->guc->sched; - struct xe_sched_job *job; - - /* - * This routine is used within VF migration recovery. This means - * using the lock here introduces a restriction: we cannot wait - * for any GFX HW response while the lock is taken. - */ - spin_lock(&sched->base.job_list_lock); - list_for_each_entry(job, &sched->base.pending_list, drm.list) { - if (xe_sched_job_is_error(job)) - continue; - - q->ring_ops->emit_job(job); - } - spin_unlock(&sched->base.job_list_lock); -} diff --git a/drivers/gpu/drm/xe/xe_exec_queue.h b/drivers/gpu/drm/xe/xe_exec_queue.h index 15ec852e7f7e..8821ceb838d0 100644 --- a/drivers/gpu/drm/xe/xe_exec_queue.h +++ b/drivers/gpu/drm/xe/xe_exec_queue.h @@ -92,7 +92,6 @@ void xe_exec_queue_update_run_ticks(struct xe_exec_queue *q); int xe_exec_queue_contexts_hwsp_rebase(struct xe_exec_queue *q, void *scratch); -void xe_exec_queue_jobs_ring_restore(struct xe_exec_queue *q); - struct xe_lrc *xe_exec_queue_lrc(struct xe_exec_queue *q); + #endif diff --git a/drivers/gpu/drm/xe/xe_guc_submit.c b/drivers/gpu/drm/xe/xe_guc_submit.c index f2e14419d7af..68466e2335e7 100644 --- a/drivers/gpu/drm/xe/xe_guc_submit.c +++ b/drivers/gpu/drm/xe/xe_guc_submit.c @@ -845,30 +845,6 @@ guc_exec_queue_run_job(struct drm_sched_job *drm_job) return fence; } -/** - * xe_guc_jobs_ring_rebase - Re-emit ring commands of requests pending - * on all queues under a guc. - * @guc: the &xe_guc struct instance - */ -void xe_guc_jobs_ring_rebase(struct xe_guc *guc) -{ - struct xe_exec_queue *q; - unsigned long index; - - /* - * This routine is used within VF migration recovery. This means - * using the lock here introduces a restriction: we cannot wait - * for any GFX HW response while the lock is taken. - */ - mutex_lock(&guc->submission_state.lock); - xa_for_each(&guc->submission_state.exec_queue_lookup, index, q) { - if (exec_queue_killed_or_banned_or_wedged(q)) - continue; - xe_exec_queue_jobs_ring_restore(q); - } - mutex_unlock(&guc->submission_state.lock); -} - static void guc_exec_queue_free_job(struct drm_sched_job *drm_job) { struct xe_sched_job *job = to_xe_sched_job(drm_job); diff --git a/drivers/gpu/drm/xe/xe_guc_submit.h b/drivers/gpu/drm/xe/xe_guc_submit.h index 78c3f07e31a0..5b4a0a6fd818 100644 --- a/drivers/gpu/drm/xe/xe_guc_submit.h +++ b/drivers/gpu/drm/xe/xe_guc_submit.h @@ -36,8 +36,6 @@ int xe_guc_exec_queue_memory_cat_error_handler(struct xe_guc *guc, u32 *msg, int xe_guc_exec_queue_reset_failure_handler(struct xe_guc *guc, u32 *msg, u32 len); int xe_guc_error_capture_handler(struct xe_guc *guc, u32 *msg, u32 len); -void xe_guc_jobs_ring_rebase(struct xe_guc *guc); - struct xe_guc_submit_exec_queue_snapshot * xe_guc_exec_queue_snapshot_capture(struct xe_exec_queue *q); void diff --git a/drivers/gpu/drm/xe/xe_sriov_vf.c b/drivers/gpu/drm/xe/xe_sriov_vf.c index d6e2ed9b9bbc..0581b881b628 100644 --- a/drivers/gpu/drm/xe/xe_sriov_vf.c +++ b/drivers/gpu/drm/xe/xe_sriov_vf.c @@ -340,7 +340,6 @@ static int gt_vf_post_migration_fixups(struct xe_gt *gt) err = xe_guc_contexts_hwsp_rebase(>->uc.guc, buf); if (err) goto out; - xe_guc_jobs_ring_rebase(>->uc.guc); xe_guc_ct_fixup_messages_with_ggtt(>->uc.guc.ct, shift); } -- 2.34.1