Re: [PATCH 3/3] drm/xe/vf: Don't expose privileged GT debugfs files if VF

[Michal Wajdeczko <michal.wajdeczko@intel.com>]

On 21.03.2025 17:55, Lucas De Marchi wrote:
> On Thu, Mar 20, 2025 at 10:07:38PM +0100, Michal Wajdeczko wrote:

...

> 
> Another possible idea: would it make sense to add a flag in
> drm.driver_features to note if the driver if pf/vf? That way we may
> simplify other places and then here we could just do:
> 
>     { {"steering", .show = xe_gt_debugfs_simple_show, .data = steering,
>        .driver_features = DRIVER_SRIOV_PF },
>        ...
>     }
> 
> instead of splitting the array.

tempting idea, but I'm not sure how/when this could be accepted at drm
level, and whether our single static drm_driver instance will survive
both PF and VFs instances at the same time

> 
> pass by comment since I was involved in other threads related to default
> context in PTL... Are these default_lrc_* accurate from the VF point of
> view? Are they different from the ones for PF?

it should reflect what VF uses and it should be the same as on PF