On Mon, Jun 8, 2026 at 4:56 AM Adrian Moreno via dev <
[email protected]> wrote:

> Updating the netdev's information on demand or via stats is pretty much
> the same thing. Remove duplicated code.
>
> Signed-off-by: Adrian Moreno <[email protected]>
> ---
>  lib/netdev-linux.c | 41 +++--------------------------------------
>  1 file changed, 3 insertions(+), 38 deletions(-)
>
> diff --git a/lib/netdev-linux.c b/lib/netdev-linux.c
> index 7ce3ee5ba..48ea21334 100644
> --- a/lib/netdev-linux.c
> +++ b/lib/netdev-linux.c
> @@ -6917,6 +6917,7 @@ get_ifindex(const struct netdev *netdev_, int
> *ifindexp)
>
>  static int
>  netdev_linux_update_via_netlink(struct netdev_linux *netdev)
> +    OVS_REQUIRES(netdev->mutex)
>

This function is called by netdev_linux_get_mtu__ which is in turn called
by a number of other functions, some of which don't have locks or
annotations.

-M
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to