Re: [PATCH v5 5/6] drm/xe/xe_ras: Move xe drm_ras registration

["Tauro, Riana" <riana.tauro@intel.com>]

On 5/4/2026 9:52 PM, Raag Jadav wrote:
> + Michal
>
> On Mon, May 04, 2026 at 04:23:14PM +0530, Tauro, Riana wrote:
>> On 5/4/2026 12:26 PM, Riana Tauro wrote:
>>> Move xe drm_ras registration to RAS initialization flow and keep
>>> hardware error initialization for processing errors reported
>>> via irq.
> ...
>
>>> +void xe_ras_init(struct xe_device *xe)
>>> +{
>>> +	int ret;
>>> +
>> Needs an SRIOV_VF check here. Will add it in next rev
> Rather, reuse vf_update_device_info() (which I think also should've been
> done for other sysctrl flags).

sysctrl might have other functionality which might be applicable for VF.
@Anoop Will sysctrl be used for any functionality in VF? Are we okay to
disable sysctrl flag for vf?

Thanks
Riana

> Raag
>
>>> +	if (xe->info.platform != XE_PVC)
>>> +		return;
>>> +
>>> +	ret = xe_drm_ras_init(xe);
>>> +	if (ret)
>>> +		drm_err(&xe->drm, "Failed to initialize xe_drm_ras %d\n", ret);
>>> +}