devlink: use DEVLINK_VAR_ATTR_TYPE_* instead of NLA_* in fmsg

Use newly introduced DEVLINK_VAR_ATTR_TYPE_* enum values instead of
internal NLA_* in fmsg health reporter code.

Signed-off-by: Jiri Pirko <jiri@nvidia.com>
Link: https://patch.msgid.link/20250505114513.53370-5-jiri@resnulli.us
Signed-off-by: Jakub Kicinski <kuba@kernel.org>

diff --git a/net/devlink/health.c b/net/devlink/health.c
index 95a7a62d85a27c36fc72443a44e9d4c7c33b58c7..b3ce8ecbb7fb18ca9a521160aad374b59789ee5b 100644 (file)
--- a/net/devlink/health.c
+++ b/net/devlink/health.c
@@ -735,7 +735,7 @@ static void devlink_fmsg_put_name(struct devlink_fmsg *fmsg, const char *name)
                return;
        }
 
-       item->nla_type = NLA_NUL_STRING;
+       item->nla_type = DEVLINK_VAR_ATTR_TYPE_NUL_STRING;
        item->len = strlen(name) + 1;
        item->attrtype = DEVLINK_ATTR_FMSG_OBJ_NAME;
        memcpy(&item->value, name, item->len);
@@ -822,32 +822,37 @@ static void devlink_fmsg_put_value(struct devlink_fmsg *fmsg,
 static void devlink_fmsg_bool_put(struct devlink_fmsg *fmsg, bool value)
 {
        devlink_fmsg_err_if_binary(fmsg);
-       devlink_fmsg_put_value(fmsg, &value, sizeof(value), NLA_FLAG);
+       devlink_fmsg_put_value(fmsg, &value, sizeof(value),
+                              DEVLINK_VAR_ATTR_TYPE_FLAG);
 }
 
 static void devlink_fmsg_u8_put(struct devlink_fmsg *fmsg, u8 value)
 {
        devlink_fmsg_err_if_binary(fmsg);
-       devlink_fmsg_put_value(fmsg, &value, sizeof(value), NLA_U8);
+       devlink_fmsg_put_value(fmsg, &value, sizeof(value),
+                              DEVLINK_VAR_ATTR_TYPE_U8);
 }
 
 void devlink_fmsg_u32_put(struct devlink_fmsg *fmsg, u32 value)
 {
        devlink_fmsg_err_if_binary(fmsg);
-       devlink_fmsg_put_value(fmsg, &value, sizeof(value), NLA_U32);
+       devlink_fmsg_put_value(fmsg, &value, sizeof(value),
+                              DEVLINK_VAR_ATTR_TYPE_U32);
 }
 EXPORT_SYMBOL_GPL(devlink_fmsg_u32_put);
 
 static void devlink_fmsg_u64_put(struct devlink_fmsg *fmsg, u64 value)
 {
        devlink_fmsg_err_if_binary(fmsg);
-       devlink_fmsg_put_value(fmsg, &value, sizeof(value), NLA_U64);
+       devlink_fmsg_put_value(fmsg, &value, sizeof(value),
+                              DEVLINK_VAR_ATTR_TYPE_U64);
 }
 
 void devlink_fmsg_string_put(struct devlink_fmsg *fmsg, const char *value)
 {
        devlink_fmsg_err_if_binary(fmsg);
-       devlink_fmsg_put_value(fmsg, value, strlen(value) + 1, NLA_NUL_STRING);
+       devlink_fmsg_put_value(fmsg, value, strlen(value) + 1,
+                              DEVLINK_VAR_ATTR_TYPE_NUL_STRING);
 }
 EXPORT_SYMBOL_GPL(devlink_fmsg_string_put);
 
@@ -857,7 +862,8 @@ void devlink_fmsg_binary_put(struct devlink_fmsg *fmsg, const void *value,
        if (!fmsg->err && !fmsg->putting_binary)
                fmsg->err = -EINVAL;
 
-       devlink_fmsg_put_value(fmsg, value, value_len, NLA_BINARY);
+       devlink_fmsg_put_value(fmsg, value, value_len,
+                              DEVLINK_VAR_ATTR_TYPE_BINARY);
 }
 EXPORT_SYMBOL_GPL(devlink_fmsg_binary_put);
 
@@ -927,36 +933,6 @@ void devlink_fmsg_binary_pair_put(struct devlink_fmsg *fmsg, const char *name,
 }
 EXPORT_SYMBOL_GPL(devlink_fmsg_binary_pair_put);
 
-static int
-devlink_fmsg_item_fill_type(struct devlink_fmsg_item *msg, struct sk_buff *skb)
-{
-       enum devlink_var_attr_type var_attr_type;
-
-       switch (msg->nla_type) {
-       case NLA_FLAG:
-               var_attr_type = DEVLINK_VAR_ATTR_TYPE_FLAG;
-               break;
-       case NLA_U8:
-               var_attr_type = DEVLINK_VAR_ATTR_TYPE_U8;
-               break;
-       case NLA_U32:
-               var_attr_type = DEVLINK_VAR_ATTR_TYPE_U32;
-               break;
-       case NLA_U64:
-               var_attr_type = DEVLINK_VAR_ATTR_TYPE_U64;
-               break;
-       case NLA_NUL_STRING:
-               var_attr_type = DEVLINK_VAR_ATTR_TYPE_NUL_STRING;
-               break;
-       case NLA_BINARY:
-               var_attr_type = DEVLINK_VAR_ATTR_TYPE_BINARY;
-               break;
-       default:
-               return -EINVAL;
-       }
-       return nla_put_u8(skb, DEVLINK_ATTR_FMSG_OBJ_VALUE_TYPE, var_attr_type);
-}
-
 static int
 devlink_fmsg_item_fill_data(struct devlink_fmsg_item *msg, struct sk_buff *skb)
 {
@@ -964,20 +940,20 @@ devlink_fmsg_item_fill_data(struct devlink_fmsg_item *msg, struct sk_buff *skb)
        u8 tmp;
 
        switch (msg->nla_type) {
-       case NLA_FLAG:
+       case DEVLINK_VAR_ATTR_TYPE_FLAG:
                /* Always provide flag data, regardless of its value */
                tmp = *(bool *)msg->value;
 
                return nla_put_u8(skb, attrtype, tmp);
-       case NLA_U8:
+       case DEVLINK_VAR_ATTR_TYPE_U8:
                return nla_put_u8(skb, attrtype, *(u8 *)msg->value);
-       case NLA_U32:
+       case DEVLINK_VAR_ATTR_TYPE_U32:
                return nla_put_u32(skb, attrtype, *(u32 *)msg->value);
-       case NLA_U64:
+       case DEVLINK_VAR_ATTR_TYPE_U64:
                return devlink_nl_put_u64(skb, attrtype, *(u64 *)msg->value);
-       case NLA_NUL_STRING:
+       case DEVLINK_VAR_ATTR_TYPE_NUL_STRING:
                return nla_put_string(skb, attrtype, (char *)&msg->value);
-       case NLA_BINARY:
+       case DEVLINK_VAR_ATTR_TYPE_BINARY:
                return nla_put(skb, attrtype, msg->len, (void *)&msg->value);
        default:
                return -EINVAL;
@@ -1011,7 +987,8 @@ devlink_fmsg_prepare_skb(struct devlink_fmsg *fmsg, struct sk_buff *skb,
                        err = nla_put_flag(skb, item->attrtype);
                        break;
                case DEVLINK_ATTR_FMSG_OBJ_VALUE_DATA:
-                       err = devlink_fmsg_item_fill_type(item, skb);
+                       err = nla_put_u8(skb, DEVLINK_ATTR_FMSG_OBJ_VALUE_TYPE,
+                                        item->nla_type);
                        if (err)
                                break;
                        err = devlink_fmsg_item_fill_data(item, skb);