On 10/28/2025 4:06 AM, Tian, Kevin wrote:
>> From: Winiarski, Michal <[email protected]>
>> Sent: Wednesday, October 22, 2025 6:41 AM
>>
>> +int xe_gt_sriov_pf_control_trigger_save_vf(struct xe_gt *gt, unsigned int
>> vfid)
>
> the prefix is too long. xe_gt_sriov_trigger_save_vf() or
> xe_gt_trigger_save_vf() is sufficient.
on the Xe driver we name functions based on the sub-component name
xe_sriov_vfio.c
= xe|sriov|vfio
= Xe driver | SR-IOV feature | VFIO interface
xe_sriov_pf_control.c
= xe|sriov|pf|control
= Xe driver | SR-IOV feature | PF specific | control
xe_gt_sriov_pf_control.c
= xe|gt|sriov|pf|control
= Xe driver | GT-related | SR-IOV feature | PF specific | control
and only functions from the xe|sriov|vfio component will be exported
for use by the xe vfio driver (hence the vfio tag in their names) and
other functions will be internal to the Xe driver