mirror of
https://github.com/torvalds/linux
synced 2024-10-07 03:42:55 +00:00
devlink: extend multicast filtering by port index
Expose the previously introduced notification multicast messages filtering infrastructure and allow the user to select messages using port index. Signed-off-by: Jiri Pirko <jiri@nvidia.com> Signed-off-by: Paolo Abeni <pabeni@redhat.com>
This commit is contained in:
parent
13b127d257
commit
ded6f77c05
|
@ -2264,3 +2264,4 @@ operations:
|
||||||
attributes:
|
attributes:
|
||||||
- bus-name
|
- bus-name
|
||||||
- dev-name
|
- dev-name
|
||||||
|
- port-index
|
||||||
|
|
|
@ -195,6 +195,8 @@ struct devlink_obj_desc {
|
||||||
struct rcu_head rcu;
|
struct rcu_head rcu;
|
||||||
const char *bus_name;
|
const char *bus_name;
|
||||||
const char *dev_name;
|
const char *dev_name;
|
||||||
|
unsigned int port_index;
|
||||||
|
bool port_index_valid;
|
||||||
long data[];
|
long data[];
|
||||||
};
|
};
|
||||||
|
|
||||||
|
@ -206,6 +208,13 @@ static inline void devlink_nl_obj_desc_init(struct devlink_obj_desc *desc,
|
||||||
desc->dev_name = dev_name(devlink->dev);
|
desc->dev_name = dev_name(devlink->dev);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
static inline void devlink_nl_obj_desc_port_set(struct devlink_obj_desc *desc,
|
||||||
|
struct devlink_port *devlink_port)
|
||||||
|
{
|
||||||
|
desc->port_index = devlink_port->index;
|
||||||
|
desc->port_index_valid = true;
|
||||||
|
}
|
||||||
|
|
||||||
int devlink_nl_notify_filter(struct sock *dsk, struct sk_buff *skb, void *data);
|
int devlink_nl_notify_filter(struct sock *dsk, struct sk_buff *skb, void *data);
|
||||||
|
|
||||||
static inline void devlink_nl_notify_send_desc(struct devlink *devlink,
|
static inline void devlink_nl_notify_send_desc(struct devlink *devlink,
|
||||||
|
|
|
@ -490,6 +490,7 @@ static void devlink_recover_notify(struct devlink_health_reporter *reporter,
|
||||||
enum devlink_command cmd)
|
enum devlink_command cmd)
|
||||||
{
|
{
|
||||||
struct devlink *devlink = reporter->devlink;
|
struct devlink *devlink = reporter->devlink;
|
||||||
|
struct devlink_obj_desc desc;
|
||||||
struct sk_buff *msg;
|
struct sk_buff *msg;
|
||||||
int err;
|
int err;
|
||||||
|
|
||||||
|
@ -509,7 +510,10 @@ static void devlink_recover_notify(struct devlink_health_reporter *reporter,
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
devlink_nl_notify_send(devlink, msg);
|
devlink_nl_obj_desc_init(&desc, devlink);
|
||||||
|
if (reporter->devlink_port)
|
||||||
|
devlink_nl_obj_desc_port_set(&desc, reporter->devlink_port);
|
||||||
|
devlink_nl_notify_send_desc(devlink, msg, &desc);
|
||||||
}
|
}
|
||||||
|
|
||||||
void
|
void
|
||||||
|
|
|
@ -73,8 +73,13 @@ int devlink_nl_notify_filter_set_doit(struct sk_buff *skb,
|
||||||
flt->dev_name = pos;
|
flt->dev_name = pos;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
if (attrs[DEVLINK_ATTR_PORT_INDEX]) {
|
||||||
|
flt->port_index = nla_get_u32(attrs[DEVLINK_ATTR_PORT_INDEX]);
|
||||||
|
flt->port_index_valid = true;
|
||||||
|
}
|
||||||
|
|
||||||
/* Don't attach empty filter. */
|
/* Don't attach empty filter. */
|
||||||
if (!flt->bus_name && !flt->dev_name) {
|
if (!flt->bus_name && !flt->dev_name && !flt->port_index_valid) {
|
||||||
kfree(flt);
|
kfree(flt);
|
||||||
flt = NULL;
|
flt = NULL;
|
||||||
}
|
}
|
||||||
|
@ -101,6 +106,9 @@ static bool devlink_obj_desc_match(const struct devlink_obj_desc *desc,
|
||||||
if (desc->dev_name && flt->dev_name &&
|
if (desc->dev_name && flt->dev_name &&
|
||||||
strcmp(desc->dev_name, flt->dev_name))
|
strcmp(desc->dev_name, flt->dev_name))
|
||||||
return false;
|
return false;
|
||||||
|
if (desc->port_index_valid && flt->port_index_valid &&
|
||||||
|
desc->port_index != flt->port_index)
|
||||||
|
return false;
|
||||||
return true;
|
return true;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
@ -561,9 +561,10 @@ static const struct nla_policy devlink_selftests_run_nl_policy[DEVLINK_ATTR_SELF
|
||||||
};
|
};
|
||||||
|
|
||||||
/* DEVLINK_CMD_NOTIFY_FILTER_SET - do */
|
/* DEVLINK_CMD_NOTIFY_FILTER_SET - do */
|
||||||
static const struct nla_policy devlink_notify_filter_set_nl_policy[DEVLINK_ATTR_DEV_NAME + 1] = {
|
static const struct nla_policy devlink_notify_filter_set_nl_policy[DEVLINK_ATTR_PORT_INDEX + 1] = {
|
||||||
[DEVLINK_ATTR_BUS_NAME] = { .type = NLA_NUL_STRING, },
|
[DEVLINK_ATTR_BUS_NAME] = { .type = NLA_NUL_STRING, },
|
||||||
[DEVLINK_ATTR_DEV_NAME] = { .type = NLA_NUL_STRING, },
|
[DEVLINK_ATTR_DEV_NAME] = { .type = NLA_NUL_STRING, },
|
||||||
|
[DEVLINK_ATTR_PORT_INDEX] = { .type = NLA_U32, },
|
||||||
};
|
};
|
||||||
|
|
||||||
/* Ops table for devlink */
|
/* Ops table for devlink */
|
||||||
|
@ -1243,7 +1244,7 @@ const struct genl_split_ops devlink_nl_ops[74] = {
|
||||||
.cmd = DEVLINK_CMD_NOTIFY_FILTER_SET,
|
.cmd = DEVLINK_CMD_NOTIFY_FILTER_SET,
|
||||||
.doit = devlink_nl_notify_filter_set_doit,
|
.doit = devlink_nl_notify_filter_set_doit,
|
||||||
.policy = devlink_notify_filter_set_nl_policy,
|
.policy = devlink_notify_filter_set_nl_policy,
|
||||||
.maxattr = DEVLINK_ATTR_DEV_NAME,
|
.maxattr = DEVLINK_ATTR_PORT_INDEX,
|
||||||
.flags = GENL_CMD_CAP_DO,
|
.flags = GENL_CMD_CAP_DO,
|
||||||
},
|
},
|
||||||
};
|
};
|
||||||
|
|
|
@ -507,6 +507,7 @@ static void devlink_port_notify(struct devlink_port *devlink_port,
|
||||||
enum devlink_command cmd)
|
enum devlink_command cmd)
|
||||||
{
|
{
|
||||||
struct devlink *devlink = devlink_port->devlink;
|
struct devlink *devlink = devlink_port->devlink;
|
||||||
|
struct devlink_obj_desc desc;
|
||||||
struct sk_buff *msg;
|
struct sk_buff *msg;
|
||||||
int err;
|
int err;
|
||||||
|
|
||||||
|
@ -525,7 +526,9 @@ static void devlink_port_notify(struct devlink_port *devlink_port,
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
devlink_nl_notify_send(devlink, msg);
|
devlink_nl_obj_desc_init(&desc, devlink);
|
||||||
|
devlink_nl_obj_desc_port_set(&desc, devlink_port);
|
||||||
|
devlink_nl_notify_send_desc(devlink, msg, &desc);
|
||||||
}
|
}
|
||||||
|
|
||||||
static void devlink_ports_notify(struct devlink *devlink,
|
static void devlink_ports_notify(struct devlink *devlink,
|
||||||
|
|
Loading…
Reference in a new issue