<!DOCTYPE html><html><head> <meta http-equiv="Content-Type" content="text/html; charset=utf-8"> </head> <body> <div class="moz-cite-prefix">On 11/27/2024 21:03, Marcin Bernatowicz wrote: </div> <blockquote type="cite" cite="mid:20241127200302.1376594-5-marcin.bernatowicz@linux.intel.com"> <pre wrap="" class="moz-quote-pre">Introduce functions to get and set SR-IOV provisioning quota and spare attributes. These functions provide access to shared resource attributes such as GGTT, LMEM, contexts, and doorbells for both PF and VF contexts. Add returning and asserting variants to allow flexible error handling based on usage scenarios. Signed-off-by: Marcin Bernatowicz <a class="moz-txt-link-rfc2396E" href="mailto:marcin.bernatowicz@linux.intel.com"><marcin.bernatowicz@linux.intel.com></a> Cc: Adam Miszczak <a class="moz-txt-link-rfc2396E" href="mailto:adam.miszczak@linux.intel.com"><adam.miszczak@linux.intel.com></a> Cc: Jakub Kolakowski <a class="moz-txt-link-rfc2396E" href="mailto:jakub1.kolakowski@intel.com"><jakub1.kolakowski@intel.com></a> Cc: Lukasz Laguna <a class="moz-txt-link-rfc2396E" href="mailto:lukasz.laguna@intel.com"><lukasz.laguna@intel.com></a> Cc: Michał Wajdeczko <a class="moz-txt-link-rfc2396E" href="mailto:michal.wajdeczko@intel.com"><michal.wajdeczko@intel.com></a> Cc: Michał Winiarski <a class="moz-txt-link-rfc2396E" href="mailto:michal.winiarski@intel.com"><michal.winiarski@intel.com></a> Cc: Narasimha C V <a class="moz-txt-link-rfc2396E" href="mailto:narasimha.c.v@intel.com"><narasimha.c.v@intel.com></a> Cc: Piotr Piórkowski <a class="moz-txt-link-rfc2396E" href="mailto:piotr.piorkowski@intel.com"><piotr.piorkowski@intel.com></a> Cc: Satyanarayana K V P <a class="moz-txt-link-rfc2396E" href="mailto:satyanarayana.k.v.p@intel.com"><satyanarayana.k.v.p@intel.com></a> Cc: Tomasz Lis <a class="moz-txt-link-rfc2396E" href="mailto:tomasz.lis@intel.com"><tomasz.lis@intel.com></a> --- lib/xe/xe_sriov_provisioning.c | 123 +++++++++++++++++++++++++++++++++ lib/xe/xe_sriov_provisioning.h | 14 ++++ 2 files changed, 137 insertions(+) diff --git a/lib/xe/xe_sriov_provisioning.c b/lib/xe/xe_sriov_provisioning.c index 0c5cd01fd..ce06c00de 100644 --- a/lib/xe/xe_sriov_provisioning.c +++ b/lib/xe/xe_sriov_provisioning.c @@ -561,3 +561,126 @@ void xe_sriov_require_default_scheduling_attributes(int pf) } } } + +/** + * xe_sriov_shared_res_attr_name - Retrieve the attribute name for a shared resource + * @res: Shared resource type (see enum xe_sriov_shared_res) + * @vf_num: VF number (1-based) or 0 for PF + * + * Returns the attribute name corresponding to the specified + * shared resource type and VF number. For VF (vf_num > 0), the "quota" + * attribute name is returned (e.g., "contexts_quota"). For PF (vf_num == 0), + * the "spare" attribute name is returned (e.g., "contexts_spare"). + * + * Return: + * The attribute name as a string if the resource type is valid. + * NULL if the resource type is invalid. + */ +const char *xe_sriov_shared_res_attr_name(enum xe_sriov_shared_res res, + unsigned int vf_num) +{ + switch (res) { + case XE_SRIOV_SHARED_RES_CONTEXTS: + return vf_num ? "contexts_quota" : "contexts_spare"; + case XE_SRIOV_SHARED_RES_DOORBELLS: + return vf_num ? "doorbells_quota" : "doorbells_spare"; + case XE_SRIOV_SHARED_RES_GGTT: + return vf_num ? "ggtt_quota" : "ggtt_spare"; + case XE_SRIOV_SHARED_RES_LMEM: + return vf_num ? "lmem_quota" : "lmem_spare"; + } + + return NULL; +} + +/** + * __xe_sriov_pf_get_shared_res_attr - Read shared resource attribute + * @pf: PF device file descriptor + * @res: Shared resource type (see enum xe_sriov_shared_res) + * @vf_num: VF number (1-based) or 0 for PF + * @gt_num: GT number + * @value: Pointer to store the read attribute value + * + * Reads the specified shared resource attribute for the given PF device @pf, + * VF number @vf_num, and GT @gt_num. The attribute depends on @vf_num: + * - For VF (vf_num > 0), reads the "quota" attribute. + * - For PF (vf_num == 0), reads the "spare" attribute. + * + * + * Return: 0 on success, negative error code on failure. + */ +int __xe_sriov_pf_get_shared_res_attr(int pf, enum xe_sriov_shared_res res, + unsigned int vf_num, unsigned int gt_num, + uint64_t *value) +{ + return __xe_sriov_pf_debugfs_get_u64(pf, vf_num, gt_num, + xe_sriov_shared_res_attr_name(res, vf_num), + value); +} + +/** + * xe_sriov_pf_get_shared_res_attr - Read shared resource attribute + * @pf: PF device file descriptor + * @res: Shared resource type (see enum xe_sriov_shared_res) + * @vf_num: VF number (1-based) or 0 for PF + * @gt_num: GT number + * + * A throwing version of __xe_sriov_pf_get_shared_res_attr(). + * Instead of returning an error code, it returns the quota value and asserts + * in case of an error. + * + * Return: The value for the given shared resource attribute. + * Asserts in case of failure. + */ +uint64_t xe_sriov_pf_get_shared_res_attr(int pf, enum xe_sriov_shared_res res, + unsigned int vf_num, + unsigned int gt_num) +{ + uint64_t value; + + igt_fail_on(__xe_sriov_pf_get_shared_res_attr(pf, res, vf_num, gt_num, &value)); + + return value; +} + +/** + * __xe_sriov_pf_set_shared_res_attr - Set a shared resource attribute + * @pf: PF device file descriptor + * @res: Shared resource type (see enum xe_sriov_shared_res) + * @vf_num: VF number (1-based) or 0 for PF + * @gt_num: GT number + * @value: Value to set for the shared resource attribute + * + * Sets the specified shared resource attribute for the given PF device @pf, + * VF number @vf_num, and GT @gt_num. The attribute depends on @vf_num: + * - For VF (vf_num > 0), reads the "quota" attribute. + * - For PF (vf_num == 0), reads the "spare" attribute. + * + * Return: 0 on success, negative error code on failure. + */ +int __xe_sriov_pf_set_shared_res_attr(int pf, enum xe_sriov_shared_res res, + unsigned int vf_num, unsigned int gt_num, + uint64_t value) +{ + return __xe_sriov_pf_debugfs_set_u64(pf, vf_num, gt_num, + xe_sriov_shared_res_attr_name(res, vf_num), + value); +} + +/** + * xe_sriov_pf_set_shared_res_attr - Set the shared resource attribute value + * @pf: PF device file descriptor + * @res: Shared resource type (see enum xe_sriov_shared_res) + * @vf_num: VF number (1-based) or 0 for PF + * @gt_num: GT number + * @value: Value to set + * + * A throwing version of __xe_sriov_pf_set_shared_res_attr(). + * Instead of returning an error code, it asserts in case of an error. + */ +void xe_sriov_pf_set_shared_res_attr(int pf, enum xe_sriov_shared_res res, + unsigned int vf_num, unsigned int gt_num, + uint64_t value) +{ + igt_fail_on(__xe_sriov_pf_set_shared_res_attr(pf, res, vf_num, gt_num, value)); +} diff --git a/lib/xe/xe_sriov_provisioning.h b/lib/xe/xe_sriov_provisioning.h index 4ddf3149f..30e769019 100644 --- a/lib/xe/xe_sriov_provisioning.h +++ b/lib/xe/xe_sriov_provisioning.h @@ -100,5 +100,19 @@ int __xe_sriov_set_sched_priority(int pf, unsigned int vf_num, unsigned int gt_n void xe_sriov_set_sched_priority(int pf, unsigned int vf_num, unsigned int gt_num, enum xe_sriov_sched_priority value); void xe_sriov_require_default_scheduling_attributes(int pf); +const char *xe_sriov_shared_res_attr_name(enum xe_sriov_shared_res res, + unsigned int vf_num); +int __xe_sriov_pf_get_shared_res_attr(int pf, enum xe_sriov_shared_res res, + unsigned int vf_num, unsigned int gt_num, + uint64_t *value); +uint64_t xe_sriov_pf_get_shared_res_attr(int pf, enum xe_sriov_shared_res res, + unsigned int vf_num, + unsigned int gt_num); +int __xe_sriov_pf_set_shared_res_attr(int pf, enum xe_sriov_shared_res res, + unsigned int vf_num, unsigned int gt_num, + uint64_t value); +void xe_sriov_pf_set_shared_res_attr(int pf, enum xe_sriov_shared_res res, + unsigned int vf_num, unsigned int gt_num, + uint64_t value); #endif /* __XE_SRIOV_PROVISIONING_H__ */</pre> </blockquote> LGTM, Reviewed-by: Lukasz Laguna <a class="moz-txt-link-rfc2396E" href="mailto:lukasz.laguna@intel.com"><lukasz.laguna@intel.com></a> </body> </html>