Commit 278d4f42 authored by Francois Dugast's avatar Francois Dugast
Browse files

drm/xe/gt_pagefault: Change vma_pagefault unit to kilobyte 



Increase the amount of bytes that can be counted before the counter
overflows, while not losing information as the VMA is not expected
to have sub-kilobyte size.

Suggested-by: default avatarMatthew Auld <matthew.auld@intel.com>
Reviewed-by: default avatarMatthew Auld <matthew.auld@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20250225195902.1247100-3-francois.dugast@intel.com


Signed-off-by: default avatarFrancois Dugast <francois.dugast@intel.com>
parent 4f109b06

drivers/gpu/drm/xe/xe_gt_pagefault.c

+1 −1
Original line number Diff line number Diff line
@@ -137,7 +137,7 @@ static int handle_vma_pagefault(struct xe_gt *gt, struct pagefault *pf, 
	bool atomic;



	xe_gt_stats_incr(gt, XE_GT_STATS_ID_VMA_PAGEFAULT_COUNT, 1);

	xe_gt_stats_incr(gt, XE_GT_STATS_ID_VMA_PAGEFAULT_BYTES, xe_vma_size(vma));

	xe_gt_stats_incr(gt, XE_GT_STATS_ID_VMA_PAGEFAULT_KB, xe_vma_size(vma) / 1024);



	trace_xe_vma_pagefault(vma);

	atomic = access_is_atomic(pf->access_type);

drivers/gpu/drm/xe/xe_gt_stats.c

+1 −1
Original line number Diff line number Diff line
@@ -29,7 +29,7 @@ void xe_gt_stats_incr(struct xe_gt *gt, const enum xe_gt_stats_id id, int incr) 
static const char *const stat_description[__XE_GT_STATS_NUM_IDS] = {

	"tlb_inval_count",

	"vma_pagefault_count",

	"vma_pagefault_bytes",

	"vma_pagefault_kb",

};



/**

drivers/gpu/drm/xe/xe_gt_stats_types.h

+1 −1
Original line number Diff line number Diff line
@@ -9,7 +9,7 @@ 
enum xe_gt_stats_id {

	XE_GT_STATS_ID_TLB_INVAL,

	XE_GT_STATS_ID_VMA_PAGEFAULT_COUNT,

	XE_GT_STATS_ID_VMA_PAGEFAULT_BYTES,

	XE_GT_STATS_ID_VMA_PAGEFAULT_KB,

	/* must be the last entry */

	__XE_GT_STATS_NUM_IDS,

};