net: dcb: add new apptrust attribute
authorDaniel Machon <daniel.machon@microchip.com>
Tue, 1 Nov 2022 09:48:30 +0000 (10:48 +0100)
committerPaolo Abeni <pabeni@redhat.com>
Thu, 3 Nov 2022 14:16:50 +0000 (15:16 +0100)
Add new apptrust extension attributes to the 8021Qaz APP managed object.

Two new attributes, DCB_ATTR_DCB_APP_TRUST_TABLE and
DCB_ATTR_DCB_APP_TRUST, has been added. Trusted selectors are passed in
the nested attribute DCB_ATTR_DCB_APP_TRUST, in order of precedence.

The new attributes are meant to allow drivers, whose hw supports the
notion of trust, to be able to set whether a particular app selector is
trusted - and in which order.

Signed-off-by: Daniel Machon <daniel.machon@microchip.com>
Reviewed-by: Petr Machata <petrm@nvidia.com>
Signed-off-by: Paolo Abeni <pabeni@redhat.com>
include/net/dcbnl.h
include/uapi/linux/dcbnl.h
net/dcb/dcbnl.c

index 2b2d86f..8841ab6 100644 (file)
@@ -109,6 +109,10 @@ struct dcbnl_rtnl_ops {
        /* buffer settings */
        int (*dcbnl_getbuffer)(struct net_device *, struct dcbnl_buffer *);
        int (*dcbnl_setbuffer)(struct net_device *, struct dcbnl_buffer *);
+
+       /* apptrust */
+       int (*dcbnl_setapptrust)(struct net_device *, u8 *, int);
+       int (*dcbnl_getapptrust)(struct net_device *, u8 *, int *);
 };
 
 #endif /* __NET_DCBNL_H__ */
index dc7ef96..9904722 100644 (file)
@@ -410,6 +410,7 @@ enum dcbnl_attrs {
  * @DCB_ATTR_IEEE_PEER_ETS: peer ETS configuration - get only
  * @DCB_ATTR_IEEE_PEER_PFC: peer PFC configuration - get only
  * @DCB_ATTR_IEEE_PEER_APP: peer APP tlv - get only
+ * @DCB_ATTR_DCB_APP_TRUST_TABLE: selector trust table
  */
 enum ieee_attrs {
        DCB_ATTR_IEEE_UNSPEC,
@@ -423,6 +424,7 @@ enum ieee_attrs {
        DCB_ATTR_IEEE_QCN,
        DCB_ATTR_IEEE_QCN_STATS,
        DCB_ATTR_DCB_BUFFER,
+       DCB_ATTR_DCB_APP_TRUST_TABLE,
        __DCB_ATTR_IEEE_MAX
 };
 #define DCB_ATTR_IEEE_MAX (__DCB_ATTR_IEEE_MAX - 1)
index 0c17bb2..cec0632 100644 (file)
@@ -166,6 +166,7 @@ static const struct nla_policy dcbnl_ieee_policy[DCB_ATTR_IEEE_MAX + 1] = {
        [DCB_ATTR_IEEE_QCN]         = {.len = sizeof(struct ieee_qcn)},
        [DCB_ATTR_IEEE_QCN_STATS]   = {.len = sizeof(struct ieee_qcn_stats)},
        [DCB_ATTR_DCB_BUFFER]       = {.len = sizeof(struct dcbnl_buffer)},
+       [DCB_ATTR_DCB_APP_TRUST_TABLE] = {.type = NLA_NESTED},
 };
 
 /* DCB number of traffic classes nested attributes. */
@@ -1062,9 +1063,9 @@ nla_put_failure:
 /* Handle IEEE 802.1Qaz/802.1Qau/802.1Qbb GET commands. */
 static int dcbnl_ieee_fill(struct sk_buff *skb, struct net_device *netdev)
 {
-       struct nlattr *ieee, *app;
-       struct dcb_app_type *itr;
        const struct dcbnl_rtnl_ops *ops = netdev->dcbnl_ops;
+       struct nlattr *ieee, *app, *apptrust;
+       struct dcb_app_type *itr;
        int dcbx;
        int err;
 
@@ -1166,6 +1167,30 @@ static int dcbnl_ieee_fill(struct sk_buff *skb, struct net_device *netdev)
        spin_unlock_bh(&dcb_lock);
        nla_nest_end(skb, app);
 
+       if (ops->dcbnl_getapptrust) {
+               u8 selectors[IEEE_8021QAZ_APP_SEL_MAX + 1] = {0};
+               int nselectors, i;
+
+               apptrust = nla_nest_start(skb, DCB_ATTR_DCB_APP_TRUST_TABLE);
+               if (!apptrust)
+                       return -EMSGSIZE;
+
+               err = ops->dcbnl_getapptrust(netdev, selectors, &nselectors);
+               if (!err) {
+                       for (i = 0; i < nselectors; i++) {
+                               enum ieee_attrs_app type =
+                                       dcbnl_app_attr_type_get(selectors[i]);
+                               err = nla_put_u8(skb, type, selectors[i]);
+                               if (err) {
+                                       nla_nest_cancel(skb, apptrust);
+                                       return err;
+                               }
+                       }
+               }
+
+               nla_nest_end(skb, apptrust);
+       }
+
        /* get peer info if available */
        if (ops->ieee_peer_getets) {
                struct ieee_ets ets;
@@ -1554,6 +1579,53 @@ static int dcbnl_ieee_set(struct net_device *netdev, struct nlmsghdr *nlh,
                }
        }
 
+       if (ieee[DCB_ATTR_DCB_APP_TRUST_TABLE]) {
+               u8 selectors[IEEE_8021QAZ_APP_SEL_MAX + 1] = {0};
+               struct nlattr *attr;
+               int nselectors = 0;
+               int rem;
+
+               if (!ops->dcbnl_setapptrust) {
+                       err = -EOPNOTSUPP;
+                       goto err;
+               }
+
+               nla_for_each_nested(attr, ieee[DCB_ATTR_DCB_APP_TRUST_TABLE],
+                                   rem) {
+                       enum ieee_attrs_app type = nla_type(attr);
+                       u8 selector;
+                       int i;
+
+                       if (!dcbnl_app_attr_type_validate(type) ||
+                           nla_len(attr) != 1 ||
+                           nselectors >= sizeof(selectors)) {
+                               err = -EINVAL;
+                               goto err;
+                       }
+
+                       selector = nla_get_u8(attr);
+
+                       if (!dcbnl_app_selector_validate(type, selector)) {
+                               err = -EINVAL;
+                               goto err;
+                       }
+
+                       /* Duplicate selector ? */
+                       for (i = 0; i < nselectors; i++) {
+                               if (selectors[i] == selector) {
+                                       err = -EINVAL;
+                                       goto err;
+                               }
+                       }
+
+                       selectors[nselectors++] = selector;
+               }
+
+               err = ops->dcbnl_setapptrust(netdev, selectors, nselectors);
+               if (err)
+                       goto err;
+       }
+
 err:
        err = nla_put_u8(skb, DCB_ATTR_IEEE, err);
        dcbnl_ieee_notify(netdev, RTM_SETDCB, DCB_CMD_IEEE_SET, seq, 0);