[PATCH v8 06/11] drm/xe/guc: Use a two stage dump for GuC logs and add more info

Julia Filipchuk julia.filipchuk at intel.com
Wed Oct 2 20:53:54 UTC 2024


On 9/19/2024 8:21 PM, John.C.Harrison at Intel.com wrote:
> +#define GUC_PMTIMESTAMP				XE_REG(0xc3e8)
This register seems to be the lower-32 bits of a pair. Does it make
sense to read the full 64-bit counter for the log?


> +	err = xe_force_wake_get(gt_to_fw(gt), XE_FW_GT);
> +	if (err) {
> +		snapshot->stamp = ~0;
> +	} else {
> +		snapshot->stamp = xe_mmio_read32(&gt->mmio, GUC_PMTIMESTAMP);
> +		xe_force_wake_put(gt_to_fw(gt), XE_FW_GT);
> +	}


More information about the Intel-xe mailing list