Page MenuHomeFreeBSD

Gather all mbuf(9) allocation APIs in one place
AbandonedPublic

Authored by glebius on Feb 3 2016, 10:52 PM.
Tags
None
Referenced Files
Unknown Object (File)
Feb 4 2024, 5:13 PM
Unknown Object (File)
Jan 3 2024, 9:19 AM
Unknown Object (File)
Dec 15 2023, 2:33 AM
Unknown Object (File)
Dec 15 2023, 2:32 AM
Unknown Object (File)
Nov 18 2023, 11:39 AM
Unknown Object (File)
Nov 10 2023, 2:47 AM
Unknown Object (File)
Oct 7 2023, 6:11 PM
Unknown Object (File)
Sep 18 2023, 10:24 AM
Subscribers

Details

Reviewers
rrs
gnn
rwatson
Group Reviewers
transport
Summary

Before upcoming changes to mbuf(9) gather all allocation APIs
into kern_mbuf.c, and declare them in sys/mbuf.h

The uipc_mbuf.c now will have only mbuf manipulation routines.
sys/mbuf.h has only declaration and doesn't diclose internal
allocator structures.

Test Plan

Since inlining level has reduced, benchmarking is required.

Diff Detail

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

Event Timeline

glebius retitled this revision from to Gather all mbuf(9) allocation APIs in one place.
glebius updated this object.
glebius edited the test plan for this revision. (Show Details)
glebius added a reviewer: gnn.
glebius edited edge metadata.

Update wrt the most recent changes.

glebius edited edge metadata.

Back partially inline m_init().

sys/dev/cxgb/cxgb_sge.c
80 ↗(On Diff #12994)

Is there a reason these are not extern in a header file?

glebius edited edge metadata.

Back inline m_get(), m_gethdr(), m_getcl().

rrs added a reviewer: rrs.
This revision is now accepted and ready to land.Feb 10 2016, 1:54 PM
glebius edited edge metadata.

Okay, let's uninline even less.

This revision now requires review to proceed.Feb 11 2016, 12:28 AM

Committed as 5e4bc63b7cfe2b39fc02a4d0c9105da286fdfdd4