So let's just call malloc() directly. This also avoids hidden
doubling of default V_if_indexlim.
Details
Details
- Reviewers
bz kp melifaro - Group Reviewers
network - Commits
- rG6ff4cac2ee9e: ifnet: initial if_grow() shall always succeed
Diff Detail
Diff Detail
- Repository
- rG FreeBSD src repository
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
Comment Actions
I think your description may be misleading and that it was initially done simply to have a single point of malloc.
That indeed does not make sense anymore with the current code.
Do you want to also bump V_if_indexlim to 16 now to make it explicit and not reduce the number of initial array size?
Comment Actions
I'd like to leave it 8. First, this number seems more sane default to me. Second, the doubling was not documented as intentional in f9132cebdc924.