[PATCH 0/5] PF: Add functions to save and restore VF configuration blob

[Michal Wajdeczko]

On 03.10.2024 13:05, Michał Winiarski wrote:
> On Thu, Sep 19, 2024 at 07:15:23PM GMT, Michal Wajdeczko wrote:
>> We already have support to save and restore GuC VF state, but that
>> will only work when the target VF configuration (provisioning) will
>> be exactly the same as the source VF configuration. To help with
>> assuring that both configurations match, allow to encode whole VF
>> configuration that can be saved as blob and restored later.
>>
>> In the future we may want to use such captured configuration blobs
>> as templates to make sure we provision VFs with exactly the same
>> configuration that was previously tested or recommended, or when
>> debugfs knobs are not be available.
> 
> If we end up using this for that purpose - we will need to turn the
> ordering of fields in the blob into an ABI (or at least version it).

I guess we only need to say that this blob version uses KLVs, no need to
enforce any explicit order of the KLVs; and any version data may/should
be placed in the additional header maintained by the real exporter ;)

> 
> For the series:
> Reviewed-by: Michał Winiarski <michal.winiarski at intel.com>

Thanks!

> 
> -Michał
> 
>>
>> Michal Wajdeczko (5):
>>   drm/xe/guc: Add yet another helper macro for threshold
>>   drm/xe/pf: Update success code of pf_validate_vf_config()
>>   drm/xe/pf: Allow to encode subset of VF configuration KLVs
>>   drm/xe/pf: Add functions to save and restore VF configuration blob
>>   drm/xe/pf: Allow to save and restore VF config blob from debugfs
>>
>>  drivers/gpu/drm/xe/xe_gt_sriov_pf_config.c    | 204 ++++++++++++++++--
>>  drivers/gpu/drm/xe/xe_gt_sriov_pf_config.h    |   4 +
>>  drivers/gpu/drm/xe/xe_gt_sriov_pf_debugfs.c   |  78 +++++++
>>  .../gpu/drm/xe/xe_guc_klv_thresholds_set.h    |   7 +
>>  4 files changed, 279 insertions(+), 14 deletions(-)
>>
>> -- 
>> 2.43.0
>>