[PATCH v11 1/5] drm/xe/guc: Prepare GuC register list and update ADS size for error capture

Lucas De Marchi lucas.demarchi at intel.com
Fri Jun 28 16:05:24 UTC 2024


On Mon, Jun 24, 2024 at 02:54:00PM GMT, Zhanjun Dong wrote:
>@@ -398,6 +401,12 @@ int xe_guc_ads_init(struct xe_guc_ads *ads)
> 	struct xe_bo *bo;
>
> 	ads->golden_lrc_size = calculate_golden_lrc_size(ads);
>+	/*
>+	 * At time of ads init, hwconfig was not loaded, engine info like
>+	 * engine mask is not ready.

pass by comment: this never really mattered and was just artificially
done due to delaying the engine_mask initialization. engine_mask comes
from fuses and possible engines from platform, not from hwconfig.
This is gone since commit d69c3d4b5382 ("drm/xe/ads: Use flexible-array")

For ADS initialization I believe we still want size 0 when loading guc
for hwconfig and only calculate it when it's for real. This guc double-load
is something we may get rid of in future.

Lucas De Marchi


More information about the Intel-xe mailing list