Page MenuHomeFreeBSD

ifnet: initial if_grow() shall always succeed
ClosedPublic

Authored by glebius on Dec 4 2021, 9:16 PM.
Tags
None
Referenced Files
Unknown Object (File)
Feb 19 2024, 6:37 PM
Unknown Object (File)
Dec 20 2023, 3:35 AM
Unknown Object (File)
Dec 13 2023, 12:23 PM
Unknown Object (File)
Dec 12 2023, 5:51 AM
Unknown Object (File)
Nov 18 2023, 6:31 PM
Unknown Object (File)
Nov 18 2023, 6:31 PM
Unknown Object (File)
Nov 18 2023, 6:30 PM
Unknown Object (File)
Nov 18 2023, 6:16 PM
Subscribers

Details

Summary

So let's just call malloc() directly. This also avoids hidden
doubling of default V_if_indexlim.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 43140
Build 40028: arc lint + arc unit

Event Timeline

This revision is now accepted and ready to land.Dec 4 2021, 10:51 PM

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?

In D33261#752134, @bz wrote:

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?

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.