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

[Anoop Vijay <anoop.c.vijay@intel.com>]

On 12-05-2026 10:34, Tauro, Riana wrote:
>
> 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

SoC re-mapper and sysctrl are explicitly disabled for SR‑IOV VFs.

Thanks,

Anoop

>
>> 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);
>>>> +}