mirror of
				https://github.com/torvalds/linux.git
				synced 2025-11-03 18:20:25 +02:00 
			
		
		
		
	netdev-genl: Support setting per-NAPI config values
Add support to set per-NAPI defer_hard_irqs and gro_flush_timeout. Signed-off-by: Joe Damato <jdamato@fastly.com> Reviewed-by: Jakub Kicinski <kuba@kernel.org> Reviewed-by: Eric Dumazet <edumazet@google.com> Link: https://patch.msgid.link/20241011184527.16393-7-jdamato@fastly.com Signed-off-by: Jakub Kicinski <kuba@kernel.org>
This commit is contained in:
		
							parent
							
								
									86e25f40aa
								
							
						
					
					
						commit
						1287c1ae0f
					
				
					 6 changed files with 77 additions and 0 deletions
				
			
		| 
						 | 
				
			
			@ -693,6 +693,17 @@ operations:
 | 
			
		|||
        reply:
 | 
			
		||||
          attributes:
 | 
			
		||||
            - id
 | 
			
		||||
    -
 | 
			
		||||
      name: napi-set
 | 
			
		||||
      doc: Set configurable NAPI instance settings.
 | 
			
		||||
      attribute-set: napi
 | 
			
		||||
      flags: [ admin-perm ]
 | 
			
		||||
      do:
 | 
			
		||||
        request:
 | 
			
		||||
          attributes:
 | 
			
		||||
            - id
 | 
			
		||||
            - defer-hard-irqs
 | 
			
		||||
            - gro-flush-timeout
 | 
			
		||||
 | 
			
		||||
kernel-family:
 | 
			
		||||
  headers: [ "linux/list.h"]
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -201,6 +201,7 @@ enum {
 | 
			
		|||
	NETDEV_CMD_NAPI_GET,
 | 
			
		||||
	NETDEV_CMD_QSTATS_GET,
 | 
			
		||||
	NETDEV_CMD_BIND_RX,
 | 
			
		||||
	NETDEV_CMD_NAPI_SET,
 | 
			
		||||
 | 
			
		||||
	__NETDEV_CMD_MAX,
 | 
			
		||||
	NETDEV_CMD_MAX = (__NETDEV_CMD_MAX - 1)
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -22,6 +22,10 @@ static const struct netlink_range_validation netdev_a_page_pool_ifindex_range =
 | 
			
		|||
	.max	= 2147483647ULL,
 | 
			
		||||
};
 | 
			
		||||
 | 
			
		||||
static const struct netlink_range_validation netdev_a_napi_defer_hard_irqs_range = {
 | 
			
		||||
	.max	= 2147483647ULL,
 | 
			
		||||
};
 | 
			
		||||
 | 
			
		||||
/* Common nested types */
 | 
			
		||||
const struct nla_policy netdev_page_pool_info_nl_policy[NETDEV_A_PAGE_POOL_IFINDEX + 1] = {
 | 
			
		||||
	[NETDEV_A_PAGE_POOL_ID] = NLA_POLICY_FULL_RANGE(NLA_UINT, &netdev_a_page_pool_id_range),
 | 
			
		||||
| 
						 | 
				
			
			@ -87,6 +91,13 @@ static const struct nla_policy netdev_bind_rx_nl_policy[NETDEV_A_DMABUF_FD + 1]
 | 
			
		|||
	[NETDEV_A_DMABUF_QUEUES] = NLA_POLICY_NESTED(netdev_queue_id_nl_policy),
 | 
			
		||||
};
 | 
			
		||||
 | 
			
		||||
/* NETDEV_CMD_NAPI_SET - do */
 | 
			
		||||
static const struct nla_policy netdev_napi_set_nl_policy[NETDEV_A_NAPI_GRO_FLUSH_TIMEOUT + 1] = {
 | 
			
		||||
	[NETDEV_A_NAPI_ID] = { .type = NLA_U32, },
 | 
			
		||||
	[NETDEV_A_NAPI_DEFER_HARD_IRQS] = NLA_POLICY_FULL_RANGE(NLA_U32, &netdev_a_napi_defer_hard_irqs_range),
 | 
			
		||||
	[NETDEV_A_NAPI_GRO_FLUSH_TIMEOUT] = { .type = NLA_UINT, },
 | 
			
		||||
};
 | 
			
		||||
 | 
			
		||||
/* Ops table for netdev */
 | 
			
		||||
static const struct genl_split_ops netdev_nl_ops[] = {
 | 
			
		||||
	{
 | 
			
		||||
| 
						 | 
				
			
			@ -171,6 +182,13 @@ static const struct genl_split_ops netdev_nl_ops[] = {
 | 
			
		|||
		.maxattr	= NETDEV_A_DMABUF_FD,
 | 
			
		||||
		.flags		= GENL_ADMIN_PERM | GENL_CMD_CAP_DO,
 | 
			
		||||
	},
 | 
			
		||||
	{
 | 
			
		||||
		.cmd		= NETDEV_CMD_NAPI_SET,
 | 
			
		||||
		.doit		= netdev_nl_napi_set_doit,
 | 
			
		||||
		.policy		= netdev_napi_set_nl_policy,
 | 
			
		||||
		.maxattr	= NETDEV_A_NAPI_GRO_FLUSH_TIMEOUT,
 | 
			
		||||
		.flags		= GENL_ADMIN_PERM | GENL_CMD_CAP_DO,
 | 
			
		||||
	},
 | 
			
		||||
};
 | 
			
		||||
 | 
			
		||||
static const struct genl_multicast_group netdev_nl_mcgrps[] = {
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -33,6 +33,7 @@ int netdev_nl_napi_get_dumpit(struct sk_buff *skb, struct netlink_callback *cb);
 | 
			
		|||
int netdev_nl_qstats_get_dumpit(struct sk_buff *skb,
 | 
			
		||||
				struct netlink_callback *cb);
 | 
			
		||||
int netdev_nl_bind_rx_doit(struct sk_buff *skb, struct genl_info *info);
 | 
			
		||||
int netdev_nl_napi_set_doit(struct sk_buff *skb, struct genl_info *info);
 | 
			
		||||
 | 
			
		||||
enum {
 | 
			
		||||
	NETDEV_NLGRP_MGMT,
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -303,6 +303,51 @@ int netdev_nl_napi_get_dumpit(struct sk_buff *skb, struct netlink_callback *cb)
 | 
			
		|||
	return err;
 | 
			
		||||
}
 | 
			
		||||
 | 
			
		||||
static int
 | 
			
		||||
netdev_nl_napi_set_config(struct napi_struct *napi, struct genl_info *info)
 | 
			
		||||
{
 | 
			
		||||
	u64 gro_flush_timeout = 0;
 | 
			
		||||
	u32 defer = 0;
 | 
			
		||||
 | 
			
		||||
	if (info->attrs[NETDEV_A_NAPI_DEFER_HARD_IRQS]) {
 | 
			
		||||
		defer = nla_get_u32(info->attrs[NETDEV_A_NAPI_DEFER_HARD_IRQS]);
 | 
			
		||||
		napi_set_defer_hard_irqs(napi, defer);
 | 
			
		||||
	}
 | 
			
		||||
 | 
			
		||||
	if (info->attrs[NETDEV_A_NAPI_GRO_FLUSH_TIMEOUT]) {
 | 
			
		||||
		gro_flush_timeout = nla_get_uint(info->attrs[NETDEV_A_NAPI_GRO_FLUSH_TIMEOUT]);
 | 
			
		||||
		napi_set_gro_flush_timeout(napi, gro_flush_timeout);
 | 
			
		||||
	}
 | 
			
		||||
 | 
			
		||||
	return 0;
 | 
			
		||||
}
 | 
			
		||||
 | 
			
		||||
int netdev_nl_napi_set_doit(struct sk_buff *skb, struct genl_info *info)
 | 
			
		||||
{
 | 
			
		||||
	struct napi_struct *napi;
 | 
			
		||||
	unsigned int napi_id;
 | 
			
		||||
	int err;
 | 
			
		||||
 | 
			
		||||
	if (GENL_REQ_ATTR_CHECK(info, NETDEV_A_NAPI_ID))
 | 
			
		||||
		return -EINVAL;
 | 
			
		||||
 | 
			
		||||
	napi_id = nla_get_u32(info->attrs[NETDEV_A_NAPI_ID]);
 | 
			
		||||
 | 
			
		||||
	rtnl_lock();
 | 
			
		||||
 | 
			
		||||
	napi = napi_by_id(napi_id);
 | 
			
		||||
	if (napi) {
 | 
			
		||||
		err = netdev_nl_napi_set_config(napi, info);
 | 
			
		||||
	} else {
 | 
			
		||||
		NL_SET_BAD_ATTR(info->extack, info->attrs[NETDEV_A_NAPI_ID]);
 | 
			
		||||
		err = -ENOENT;
 | 
			
		||||
	}
 | 
			
		||||
 | 
			
		||||
	rtnl_unlock();
 | 
			
		||||
 | 
			
		||||
	return err;
 | 
			
		||||
}
 | 
			
		||||
 | 
			
		||||
static int
 | 
			
		||||
netdev_nl_queue_fill_one(struct sk_buff *rsp, struct net_device *netdev,
 | 
			
		||||
			 u32 q_idx, u32 q_type, const struct genl_info *info)
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -201,6 +201,7 @@ enum {
 | 
			
		|||
	NETDEV_CMD_NAPI_GET,
 | 
			
		||||
	NETDEV_CMD_QSTATS_GET,
 | 
			
		||||
	NETDEV_CMD_BIND_RX,
 | 
			
		||||
	NETDEV_CMD_NAPI_SET,
 | 
			
		||||
 | 
			
		||||
	__NETDEV_CMD_MAX,
 | 
			
		||||
	NETDEV_CMD_MAX = (__NETDEV_CMD_MAX - 1)
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
		Loading…
	
		Reference in a new issue