diff options
| author |   Damjan Marion <damarion@cisco.com> | 2022-01-06 21:14:08 +0100 |
|---|---|---|
| committer | Damjan Marion <damarion@cisco.com> | 2022-01-17 12:41:24 +0100 |
| commit | 88a9c0e02ab919cadd4e035133995a6afb4d1c32 (patch) | |
| tree | 15741e8bbe6c9d7077fa2a62b6ac369a17836cfb /src/vnet/devices/af_packet/af_packet.c | |
| parent | 81bb6fc611d321a92ad2218e1b852db67980768a (diff) | |
interface: improve MTU handling
- per hw-interface-class handlers
- ethernet set_mtu callback
- driver can now refuse MTU change
Type: improvement
Change-Id: I3d37c9129930ebec7bb70caf4263025413873048
Signed-off-by: Damjan Marion <damarion@cisco.com>
Diffstat (limited to 'src/vnet/devices/af_packet/af_packet.c')
| -rw-r--r-- | src/vnet/devices/af_packet/af_packet.c | 37 |
1 files changed, 16 insertions, 21 deletions
diff --git a/src/vnet/devices/af_packet/af_packet.c b/src/vnet/devices/af_packet/af_packet.c index cf32e92c94c..cf4f91e2fcd 100644 --- a/src/vnet/devices/af_packet/af_packet.c +++ b/src/vnet/devices/af_packet/af_packet.c @@ -56,31 +56,26 @@ unsigned int if_nametoindex (const char *ifname); typedef struct tpacket_req tpacket_req_t; -static u32 -af_packet_eth_flag_change (vnet_main_t * vnm, vnet_hw_interface_t * hi, - u32 flags) +static clib_error_t * +af_packet_eth_set_mtu (vnet_main_t *vnm, vnet_hw_interface_t *hi, u32 mtu) { - clib_error_t *error; + clib_error_t *error, *rv; af_packet_main_t *apm = &af_packet_main; - af_packet_if_t *apif = - pool_elt_at_index (apm->interfaces, hi->dev_instance); + af_packet_if_t *apif = pool_elt_at_index (apm->interfaces, hi->dev_instance); - if (flags == ETHERNET_INTERFACE_FLAG_MTU) - { - error = - vnet_netlink_set_link_mtu (apif->host_if_index, hi->max_packet_bytes); + error = vnet_netlink_set_link_mtu (apif->host_if_index, mtu); - if (error) - { - vlib_log_err (apm->log_class, "netlink failed to change MTU: %U", - format_clib_error, error); - clib_error_free (error); - return VNET_API_ERROR_SYSCALL_ERROR_1; - } - else - apif->host_mtu = hi->max_packet_bytes; + if (error) + { + vlib_log_err (apm->log_class, "netlink failed to change MTU: %U", + format_clib_error, error); + rv = vnet_error (VNET_ERR_SYSCALL_ERROR_1, "netlink error: %U", + format_clib_error, error); + clib_error_free (error); + return rv; } - + else + apif->host_mtu = mtu; return 0; } @@ -403,7 +398,7 @@ af_packet_create_if (af_packet_create_if_arg_t *arg) eir.dev_class_index = af_packet_device_class.index; eir.dev_instance = if_index; eir.address = hw_addr; - eir.cb.flag_change = af_packet_eth_flag_change; + eir.cb.set_mtu = af_packet_eth_set_mtu; apif->hw_if_index = vnet_eth_register_interface (vnm, &eir); } else |