Current devlink resources are designed as a thing that user could limit, but there is not much otherwise that could be done with them. Perhaps that's the reason there is no much adoption despite API being there for multiple years. Add new mode of operation, where user could allocate/assign resources (from a common pool) to specific devices. That requires "occ set" support, triggered by user. To support that mode, "occ get" is (only then) turned into a simple "get/show" operation, as opposed to "ask driver about current occupation" in the "legacy" mode. Signed-off-by: Przemek Kitszel --- RFC, Feb '25 https://lore.kernel.org/intel-wired-lan/20250219164410.35665-3-przemyslaw.kitszel@intel.com I have structured code to just choose "the mode" of operation based on the presence of .occ_set callback, instead of naming the mode, what, even if not the most clear code, avoids the need of naming the modes. --- include/net/devlink.h | 7 +++ net/devlink/resource.c | 98 +++++++++++++++++++++++++++++++++--------- 2 files changed, 85 insertions(+), 20 deletions(-) diff --git a/include/net/devlink.h b/include/net/devlink.h index 5d3a1337bfa1..9b777f02beca 100644 --- a/include/net/devlink.h +++ b/include/net/devlink.h @@ -420,6 +420,8 @@ devlink_resource_size_params_init(struct devlink_resource_size_params *size_para } typedef u64 devlink_resource_occ_get_t(void *priv); +typedef int devlink_resource_occ_set_t(u64 size, struct netlink_ext_ack *extack, + void *priv); #define DEVLINK_RESOURCE_ID_PARENT_TOP 0 @@ -1934,6 +1936,11 @@ void devl_resource_occ_get_register(struct devlink *devlink, void *occ_get_priv); void devl_resource_occ_get_unregister(struct devlink *devlink, u64 resource_id); +void devl_resource_occ_set_get_register(struct devlink *devlink, + u64 resource_id, + devlink_resource_occ_set_t *occ_set, + devlink_resource_occ_get_t *occ_get, + void *occ_priv); int devl_params_register(struct devlink *devlink, const struct devlink_param *params, size_t params_count); diff --git a/net/devlink/resource.c b/net/devlink/resource.c index 3d2f42bc2fb5..2212eff5230d 100644 --- a/net/devlink/resource.c +++ b/net/devlink/resource.c @@ -19,7 +19,8 @@ * @list: parent list * @resource_list: list of child resources * @occ_get: occupancy getter callback - * @occ_get_priv: occupancy getter callback priv + * @occ_set: occupancy setter callback + * @occ_priv: occupancy callbacks priv */ struct devlink_resource { const char *name; @@ -32,7 +33,8 @@ struct devlink_resource { struct list_head list; struct list_head resource_list; devlink_resource_occ_get_t *occ_get; - void *occ_get_priv; + devlink_resource_occ_set_t *occ_set; + void *occ_priv; }; static struct devlink_resource * @@ -137,6 +139,9 @@ int devlink_nl_resource_set_doit(struct sk_buff *skb, struct genl_info *info) if (err) return err; + if (resource->occ_set) + return resource->occ_set(size, info->extack, resource->occ_priv); + resource->size_new = size; devlink_resource_validate_children(resource); if (resource->parent) @@ -162,13 +167,40 @@ devlink_resource_size_params_put(struct devlink_resource *resource, return 0; } -static int devlink_resource_occ_put(struct devlink_resource *resource, - struct sk_buff *skb) +static int +devlink_resource_occ_size_put_legacy(struct devlink_resource *resource, + struct sk_buff *skb) { - if (!resource->occ_get) - return 0; - return devlink_nl_put_u64(skb, DEVLINK_ATTR_RESOURCE_OCC, - resource->occ_get(resource->occ_get_priv)); + if (devlink_nl_put_u64(skb, DEVLINK_ATTR_RESOURCE_SIZE, resource->size)) + goto err; + if (resource->size != resource->size_new && + devlink_nl_put_u64(skb, DEVLINK_ATTR_RESOURCE_SIZE_NEW, + resource->size_new)) + goto err; + if (resource->occ_get && + devlink_nl_put_u64(skb, DEVLINK_ATTR_RESOURCE_OCC, + resource->occ_get(resource->occ_priv))) + goto err; + if (nla_put_u8(skb, DEVLINK_ATTR_RESOURCE_SIZE_VALID, + resource->size_valid)) + goto err; + + return 0; +err: + return -EMSGSIZE; +} + +static int devlink_resource_occ_size_put(struct devlink_resource *resource, + struct sk_buff *skb) +{ + if (!resource->occ_get || !resource->occ_set) + return devlink_resource_occ_size_put_legacy(resource, skb); + + if (nla_put_u8(skb, DEVLINK_ATTR_RESOURCE_SIZE_VALID, true)) + return -EMSGSIZE; + + return devlink_nl_put_u64(skb, DEVLINK_ATTR_RESOURCE_SIZE, + resource->occ_get(resource->occ_priv)); } static int devlink_resource_put(struct devlink *devlink, struct sk_buff *skb, @@ -183,24 +215,16 @@ static int devlink_resource_put(struct devlink *devlink, struct sk_buff *skb, return -EMSGSIZE; if (nla_put_string(skb, DEVLINK_ATTR_RESOURCE_NAME, resource->name) || - devlink_nl_put_u64(skb, DEVLINK_ATTR_RESOURCE_SIZE, resource->size) || devlink_nl_put_u64(skb, DEVLINK_ATTR_RESOURCE_ID, resource->id)) goto nla_put_failure; - if (resource->size != resource->size_new && - devlink_nl_put_u64(skb, DEVLINK_ATTR_RESOURCE_SIZE_NEW, - resource->size_new)) - goto nla_put_failure; - if (devlink_resource_occ_put(resource, skb)) + if (devlink_resource_occ_size_put(resource, skb)) goto nla_put_failure; if (devlink_resource_size_params_put(resource, skb)) goto nla_put_failure; + if (list_empty(&resource->resource_list)) goto out; - if (nla_put_u8(skb, DEVLINK_ATTR_RESOURCE_SIZE_VALID, - resource->size_valid)) - goto nla_put_failure; - child_resource_attr = nla_nest_start_noflag(skb, DEVLINK_ATTR_RESOURCE_LIST); if (!child_resource_attr) @@ -634,6 +658,39 @@ int devl_resource_size_get(struct devlink *devlink, } EXPORT_SYMBOL_GPL(devl_resource_size_get); +/** + * devl_resource_occ_set_get_register - register occupancy getter and setter + * + * @devlink: devlink + * @resource_id: resource id + * @occ_set: occupancy setter callback + * @occ_get: occupancy getter callback + * @occ_priv: occupancy getter callback priv + * + * Setter will be called when the user wants to change the resource size, + * getter is called to show the user what is current size of the resource. + */ +void devl_resource_occ_set_get_register(struct devlink *devlink, + u64 resource_id, + devlink_resource_occ_set_t *occ_set, + devlink_resource_occ_get_t *occ_get, + void *occ_priv) +{ + struct devlink_resource *resource; + + lockdep_assert_held(&devlink->lock); + + resource = devlink_resource_find(devlink, NULL, resource_id); + if (WARN_ON(!resource)) + return; + WARN_ON(resource->occ_get || resource->occ_set); + + resource->occ_set = occ_set; + resource->occ_get = occ_get; + resource->occ_priv = occ_priv; +} +EXPORT_SYMBOL_GPL(devl_resource_occ_set_get_register); + /** * devl_resource_occ_get_register - register occupancy getter * @@ -657,7 +714,7 @@ void devl_resource_occ_get_register(struct devlink *devlink, WARN_ON(resource->occ_get); resource->occ_get = occ_get; - resource->occ_get_priv = occ_get_priv; + resource->occ_priv = occ_get_priv; } EXPORT_SYMBOL_GPL(devl_resource_occ_get_register); @@ -678,9 +735,10 @@ void devl_resource_occ_get_unregister(struct devlink *devlink, if (WARN_ON(!resource)) return; WARN_ON(!resource->occ_get); + WARN_ON(resource->occ_set); resource->occ_get = NULL; - resource->occ_get_priv = NULL; + resource->occ_priv = NULL; } EXPORT_SYMBOL_GPL(devl_resource_occ_get_unregister); -- 2.39.3