[PATCH] drm/xe/guc: Re-word message about ADS size changes

Lucas De Marchi lucas.demarchi at intel.com
Wed Mar 26 04:31:55 UTC 2025


On Tue, Mar 25, 2025 at 01:32:11PM -0700, John.C.Harrison at Intel.com wrote:
>From: John Harrison <John.C.Harrison at Intel.com>
>
>The error capture list in the ADS is initially allocated using a
>placeholder size. When the actual size is determinied later on, there
>is a debug print about the new size. However, the wording is such that
>some people see it as an unexpected thing and therefore a potential
>problem. So re-word it to be a little less concerning.
>
>Signed-off-by: John Harrison <John.C.Harrison at Intel.com>


Reviewed-by: Lucas De Marchi <lucas.demarchi at intel.com>

thanks
Lucas De Marchi

>---
> drivers/gpu/drm/xe/xe_guc_ads.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
>diff --git a/drivers/gpu/drm/xe/xe_guc_ads.c b/drivers/gpu/drm/xe/xe_guc_ads.c
>index e7c9e095a19f..bee4e0cfe7b8 100644
>--- a/drivers/gpu/drm/xe/xe_guc_ads.c
>+++ b/drivers/gpu/drm/xe/xe_guc_ads.c
>@@ -682,8 +682,8 @@ static int guc_capture_prep_lists(struct xe_guc_ads *ads)
> 	}
>
> 	if (ads->capture_size != PAGE_ALIGN(total_size))
>-		xe_gt_dbg(gt, "ADS capture alloc size changed from %d to %d\n",
>-			  ads->capture_size, PAGE_ALIGN(total_size));
>+		xe_gt_dbg(gt, "Updated ADS capture size %d (was %d)\n",
>+			  PAGE_ALIGN(total_size), ads->capture_size);
> 	return PAGE_ALIGN(total_size);
> }
>
>-- 
>2.49.0
>


More information about the Intel-xe mailing list