From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mgamail.intel.com (mgamail.intel.com [192.198.163.18]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id A10583E2ADA for ; Fri, 10 Jul 2026 21:55:06 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=192.198.163.18 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1783720509; cv=none; b=uSSq/1bd4xpPhaZEFxuRzpMW2N718KOxS7u0HUqZb4pxx/WdgfGKVjKw7I1KGpxwW26YxWeoaGXk5N6SkXMzaeOarJ9sOtLKPDBuLMyjSE3iL7v/Rle7NlodxVqkGYoWdLkhHBfUy/VRo8Jlq5eFekYWFQu/GWP7AJ+xBtNheJA= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1783720509; c=relaxed/simple; bh=KMs46OPL0Vngl0tZUFulMtAp5PmjK7zLdDVOXrYesaM=; h=From:To:Cc:Subject:Date:Message-Id:In-Reply-To:References: MIME-Version:Content-Type; b=WdRgHjWPPump7pckjv6YcsNlorBBgCyP9y5+dsec+FcpzlPKLExRqTc2fSBILx+5u80JLWVFAfSwFX5JXIaZkPey86Ryne/bxP8IaLXfS+JfB2b8vDahcYFgLH1iNwJT0DNYHCSB4PPz0jccSs3AFCiHVVnYJBWrs6gwAiRkYfE= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=intel.com; spf=pass smtp.mailfrom=intel.com; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b=fzfxFYim; arc=none smtp.client-ip=192.198.163.18 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=intel.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=intel.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b="fzfxFYim" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1783720507; x=1815256507; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=KMs46OPL0Vngl0tZUFulMtAp5PmjK7zLdDVOXrYesaM=; b=fzfxFYim9yy8QDxVYEzErAOr0VUDCCv3EGVT6vVsVNRpxqESBe5KxaAV e+fNML+TDvxNn718N7FMwHPRD2Tq/r33cBNt8x/7MGRXvaiQFOdk3YiZ9 FsV+unOBWYa5YeKm+P7+IcAzD/NTxo4PG+SFjEIvf0+/zEZsSB+st7Qzu OzFUy93lcOXgWcfTXyet/1QyXYC/xAPBoy+s/Nnzn6P8RyHsoRTKT+0c/ pHvq/waaqEOEacYAozqWWWAzScL6BjPst+YPSP+VwqCmBOlGLPkYWXj4a 46U0pIfSajgcZM505s3tZ7sj6of8hd9dpUu9T1nmAgC/k3DcfR5lFM9PE w==; X-CSE-ConnectionGUID: P356UOFBSmiKAyUOYUe22Q== X-CSE-MsgGUID: v8zUJBzVRiyxugxa1sHphQ== X-IronPort-AV: E=McAfee;i="6800,10657,11841"; a="83543209" X-IronPort-AV: E=Sophos;i="6.25,154,1779174000"; d="scan'208";a="83543209" Received: from fmviesa003.fm.intel.com ([10.60.135.143]) by fmvoesa112.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 10 Jul 2026 14:54:55 -0700 X-CSE-ConnectionGUID: XyjAOcA7SYSFJU7j86NXBA== X-CSE-MsgGUID: AfHXwFlDTk6q7ZBbBu82KQ== X-ExtLoop1: 1 Received: from gsse-cloud1.jf.intel.com ([10.54.39.91]) by fmviesa003-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 10 Jul 2026 14:54:55 -0700 From: Matthew Brost To: intel-xe@lists.freedesktop.org, dri-devel@lists.freedesktop.org Cc: Carlos Santa , Ryan Neph , Christian Koenig , Huang Rui , Matthew Auld , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , David Airlie , Simona Vetter , linux-kernel@vger.kernel.org, =?UTF-8?q?Thomas=20Hellstr=C3=B6m?= Subject: [PATCH v2 24/33] drm/xe: Add defrag profiling tracepoints Date: Fri, 10 Jul 2026 14:54:33 -0700 Message-Id: <20260710215442.2444235-25-matthew.brost@intel.com> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20260710215442.2444235-1-matthew.brost@intel.com> References: <20260710215442.2444235-1-matthew.brost@intel.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Add two tracepoints to profile the page defragmentation worker: - xe_bo_defrag_worker: emitted on worker entry and exit with the current defrag list count and the bytes processed in the run. - xe_bo_defrag_one: emitted per BO with the run budget, the prealloc fill ratio (got/want) and the lock-held, lock-wait and prealloc durations, plus the bytes consumed and the result. These break down the cost of a defrag attempt into its locked vs unlocked phases, making it possible to confirm that the expensive beneficial-order allocation and cpu cache mode change happen outside the BO dma-resv lock and that the residual lock-held time is copy-bound (with the tail dominated by lock-wait behind active rendering). Cc: Carlos Santa Cc: Ryan Neph Cc: Christian Koenig Cc: Huang Rui Cc: Matthew Auld Cc: Maarten Lankhorst Cc: Maxime Ripard Cc: Thomas Zimmermann Cc: David Airlie Cc: Simona Vetter Cc: dri-devel@lists.freedesktop.org Cc: linux-kernel@vger.kernel.org Cc: Thomas Hellström Assisted-by: GitHub_Copilot:claude-opus-4.8 Signed-off-by: Matthew Brost --- drivers/gpu/drm/xe/xe_bo.c | 18 +++++++++- drivers/gpu/drm/xe/xe_trace_bo.h | 59 ++++++++++++++++++++++++++++++++ 2 files changed, 76 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/xe/xe_bo.c b/drivers/gpu/drm/xe/xe_bo.c index 2f014fc9f988..a01004265349 100644 --- a/drivers/gpu/drm/xe/xe_bo.c +++ b/drivers/gpu/drm/xe/xe_bo.c @@ -1254,7 +1254,9 @@ static int xe_bo_defrag_one(struct xe_device *xe, struct xe_bo *bo, struct ttm_placement placement; struct ttm_place place; enum ttm_caching tt_caching; - unsigned int order, want = 0; + unsigned int order, want = 0, got = 0; + u64 lock_ns = 0, prealloc_ns = 0, lockwait_ns = 0; + ktime_t t0; int ret = 0; *consumed = 0; @@ -1284,6 +1286,7 @@ static int xe_bo_defrag_one(struct xe_device *xe, struct xe_bo *bo, /* Phase 2: preallocate outside the lock; */ if (want) { + t0 = ktime_get(); ret = ttm_pool_prealloc_fill(pool, tt_caching, &pp, want); if (ret || !pp.count) { ret = ret ?: -ENOMEM; @@ -1291,12 +1294,16 @@ static int xe_bo_defrag_one(struct xe_device *xe, struct xe_bo *bo, XE_GT_STATS_ID_DEFRAG_FAILED_COUNT, 1); goto out_err; } + prealloc_ns = ktime_to_ns(ktime_sub(ktime_get(), t0)); } /* * Phase 3: re-take the lock, re-check, and validate using the prealloc. */ + t0 = ktime_get(); xe_bo_lock(bo, false); + lockwait_ns = ktime_to_ns(ktime_sub(ktime_get(), t0)); + t0 = ktime_get(); if (!xe_bo_needs_defrag(bo)) { xe_bo_defrag_remove(bo); @@ -1350,9 +1357,13 @@ static int xe_bo_defrag_one(struct xe_device *xe, struct xe_bo *bo, unlock: xe_bo_unlock(bo); + lock_ns = ktime_to_ns(ktime_sub(ktime_get(), t0)); + got = pp.used; out_err: ttm_pool_prealloc_fini(pool, &pp); out: + trace_xe_bo_defrag_one(bo, budget, want, got, lock_ns, prealloc_ns, + lockwait_ns, *consumed, ret); return ret; } @@ -1373,6 +1384,9 @@ static void xe_bo_defrag_worker(struct work_struct *w) return; } + trace_xe_bo_defrag_worker(xe, 0, atomic_read(&xe->mem.defrag.count), + true); + /* * Process at most XE_BO_DEFRAG_SIZE_LIMIT bytes of newly (re)allocated * backing per run rather than draining the whole list in one go. Only @@ -1479,6 +1493,8 @@ static void xe_bo_defrag_worker(struct work_struct *w) xe_pm_runtime_put(xe); drm_dev_exit(idx); + trace_xe_bo_defrag_worker(xe, defrag_bytes, + atomic_read(&xe->mem.defrag.count), false); } static int xe_bo_move(struct ttm_buffer_object *ttm_bo, bool evict, diff --git a/drivers/gpu/drm/xe/xe_trace_bo.h b/drivers/gpu/drm/xe/xe_trace_bo.h index 86323cf3be2c..a169a322464b 100644 --- a/drivers/gpu/drm/xe/xe_trace_bo.h +++ b/drivers/gpu/drm/xe/xe_trace_bo.h @@ -85,6 +85,65 @@ TRACE_EVENT(xe_bo_move, __get_str(new_placement_name), __get_str(device_id)) ); +TRACE_EVENT(xe_bo_defrag_worker, + TP_PROTO(struct xe_device *xe, u64 defrag_bytes, int count, + bool enter), + TP_ARGS(xe, defrag_bytes, count, enter), + TP_STRUCT__entry( + __string(dev, dev_name(xe->drm.dev)) + __field(u64, defrag_bytes) + __field(int, count) + __field(bool, enter) + ), + TP_fast_assign( + __assign_str(dev); + __entry->defrag_bytes = defrag_bytes; + __entry->count = count; + __entry->enter = enter; + ), + TP_printk("dev=%s %s defrag_bytes=%llu count=%d", + __get_str(dev), __entry->enter ? "enter" : "exit", + __entry->defrag_bytes, __entry->count) +); + +TRACE_EVENT(xe_bo_defrag_one, + TP_PROTO(struct xe_bo *bo, u64 budget, unsigned int prealloc_want, + unsigned int prealloc_got, u64 lock_ns, u64 prealloc_ns, + u64 lockwait_ns, u64 consumed, int ret), + TP_ARGS(bo, budget, prealloc_want, prealloc_got, lock_ns, + prealloc_ns, lockwait_ns, consumed, ret), + TP_STRUCT__entry( + __string(dev, __dev_name_bo(bo)) + __field(size_t, size) + __field(u64, budget) + __field(unsigned int, prealloc_want) + __field(unsigned int, prealloc_got) + __field(u64, lock_ns) + __field(u64, prealloc_ns) + __field(u64, lockwait_ns) + __field(u64, consumed) + __field(int, ret) + ), + TP_fast_assign( + __assign_str(dev); + __entry->size = xe_bo_size(bo); + __entry->budget = budget; + __entry->prealloc_want = prealloc_want; + __entry->prealloc_got = prealloc_got; + __entry->lock_ns = lock_ns; + __entry->prealloc_ns = prealloc_ns; + __entry->lockwait_ns = lockwait_ns; + __entry->consumed = consumed; + __entry->ret = ret; + ), + TP_printk("dev=%s size=%zu budget=%llu prealloc=%u/%u lock_us=%llu lockwait_us=%llu prealloc_us=%llu consumed=%llu ret=%d", + __get_str(dev), __entry->size, __entry->budget, + __entry->prealloc_got, __entry->prealloc_want, + __entry->lock_ns / 1000, __entry->lockwait_ns / 1000, + __entry->prealloc_ns / 1000, __entry->consumed, + __entry->ret) +); + DECLARE_EVENT_CLASS(xe_vma, TP_PROTO(struct xe_vma *vma), TP_ARGS(vma), -- 2.34.1