kubeovn / kube-ovn / 23479304925

27%

Build Type

push

github

Committed by web-flow

Commit Message

perf(daemon): avoid full netlink dump in getIfaceByIP (#6513)

* perf(daemon): replace LinkList+AddrList loop with single AddrList dump in getIfaceByIP

Replace the N+1 netlink dump pattern (LinkList + per-link AddrList) with
a single AddrList(nil, FAMILY_ALL) dump followed by a targeted
LinkByIndex lookup. This reduces netlink dump operations from N+1 to 2,
lowering the risk of NLM_F_DUMP_INTR errors on busy nodes.

Also replaces string-based IP comparison with net.IP.Equal() to avoid
redundant parsing and allocation.

Signed-off-by: Mengxin Liu <liumengxinfly@gmail.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Signed-off-by: Mengxin Liu <liumengxinfly@gmail.com>

* fix(daemon): validate IP before netlink dump in getIfaceByIP

Add early validation of the IP address parameter to return a clear
error for invalid input instead of a misleading "interface not found"
message.

Signed-off-by: Mengxin Liu <liumengxinfly@gmail.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Signed-off-by: Mengxin Liu <liumengxinfly@gmail.com>

---------

Signed-off-by: Mengxin Liu <liumengxinfly@gmail.com>
Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Coverage Stats

0 of 12 new or added lines in 1 file covered. (0.0%)

1 existing line in 1 file now uncovered.

13138 of 55159 relevant lines covered (23.82%)

0.28 hits per line