[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