libifconfig: Fix bridge status member list
ClosedPublic
Actions

Authored by grembo on Dec 20 2023, 8:27 PM.

Details

Reviewers

Commits

rGe65182c7bc28: libifconfig: Fix bridge status member list
rGc2299997ce17: libifconfig: Fix bridge status member list
rG3d36053ca6d6: libifconfig: Fix bridge status member list

Summary

When this functionality was moved to libifconfig in 3dfbda3401abea84da9,
the end of list calculation was modified for unknown reasons, practically
limiting the number of bridge member returned to 102.

This patch changes the calculation back to what it was originally.

Test Plan

Create 200 epair interfaces and add them to bridge0, check
the number of members shown on ifconfig bridge0.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

grembo created this revision.Dec 20 2023, 8:27 PM

Herald added a subscriber: imp. · View Herald TranscriptDec 20 2023, 8:27 PM

grembo requested review of this revision.Dec 20 2023, 8:27 PM

Harbormaster completed remote builds in B55050: Diff 131615.Dec 20 2023, 8:27 PM

grembo added a reviewer: kp.Dec 20 2023, 8:29 PM

I'll try to review tomorrow.

Can I get you to add a test case for this to tests/sys/net/if_bridge_test.sh too?

Add unit test

Herald added subscribers: glebius, melifaro, ae. · View Herald TranscriptDec 20 2023, 9:17 PM

Harbormaster completed remote builds in B55052: Diff 131617.Dec 20 2023, 9:17 PM

post merge clesanup

Harbormaster completed remote builds in B55053: Diff 131618.Dec 20 2023, 9:18 PM

@kp here you go (lint error seems to be a false positive)

kp accepted this revision.Dec 21 2023, 2:52 PM

This revision is now accepted and ready to land.Dec 21 2023, 2:52 PM

This revision was landed with ongoing or failed builds.Dec 21 2023, 3:51 PM

Closed by commit rG3d36053ca6d6: libifconfig: Fix bridge status member list (authored by grembo). · Explain Why

This revision was automatically updated to reflect the committed changes.

grembo added a commit: rG3d36053ca6d6: libifconfig: Fix bridge status member list.

@kp thanks for the review

grembo added a commit: rGc2299997ce17: libifconfig: Fix bridge status member list.Jan 5 2024, 11:58 AM

grembo added a commit: rGe65182c7bc28: libifconfig: Fix bridge status member list.Jan 28 2024, 5:57 PM

Revision Contents
Changeset List

Path

Size

lib/

libifconfig/

libifconfig_bridge.c

2 lines

tests/

sys/

net/

if_bridge_test.sh

28 lines

Diff 131679

View Options

lib/libifconfig/libifconfig_bridge.c