Page MenuHomeFreeBSD

Remove the relatively unused us_keg field. Store the zone along withthe slab in the page for malloc.
ClosedPublic

Authored by jeff on Tue, Nov 26, 10:22 PM.

Details

Summary

This hasn't been necessary since multi keg zones were removed.

Catch up to other NOOBJ page users and add uma to the union for clarity. Malloc still wants the zone to free to but conveniently we have space for another pointer.

Diff Detail

Repository
rS FreeBSD src repository
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

jeff created this revision.Tue, Nov 26, 10:22 PM
jeff retitled this revision from Remove the relatively unused us_keg field. Store the zone along with the slab in the page for malloc. to Remove the relatively unused us_keg field. Store the zone along withthe slab in the page for malloc..Tue, Nov 26, 10:26 PM
jeff edited the summary of this revision. (Show Details)
jeff added reviewers: glebius, markj, rlibby.
markj accepted this revision.Wed, Nov 27, 4:22 PM
markj added inline comments.
sys/vm/uma_core.c
4185 ↗(On Diff #64919)

Fix indentation while here?

sys/vm/vm_page.h
223 ↗(On Diff #64919)

Is this field used anymore?

This revision is now accepted and ready to land.Wed, Nov 27, 4:22 PM
jeff added inline comments.Wed, Nov 27, 8:50 PM
sys/vm/vm_page.h
223 ↗(On Diff #64919)

You are right this is unused AFAICT.