Page MenuHomeFreeBSD

Move phys_avail and dump_avail into MI code.

Authored by jeff on Aug 13 2019, 5:31 PM.



I need to move phys_avail and dump_avail into MI code so that I can write boot time allocators. This also eliminates a lot of confusing redundant code and definitions. Since MI is manipulating these arrays it makes sense to define them MI.

The following two revisions depend on this refactoring:

Test Plan

I have built universe and tested on amd64.

Diff Detail

Lint OK
No Unit Test Coverage
Build Status
Buildable 25846
Build 24417: arc lint + arc unit

Event Timeline

jeff created this revision.Aug 13 2019, 5:31 PM
jeff edited the summary of this revision. (Show Details)Aug 13 2019, 5:36 PM
jeff added reviewers: kib, jhb, jhibbits.
jhb added inline comments.Aug 13 2019, 5:49 PM

Maybe (PHYS_AVAIL_ENTRIES + 2) (similar to how the default for PHYS_AVAIL_ENTRIES is defined above)


Do you want to add a note here to say something like: "Initially dump_avail matches phys_avail, but phys_avail is modified during boot to provide early memory allocations before the VM system is able to allocate memory."?

jeff edited the summary of this revision. (Show Details)Aug 13 2019, 5:53 PM
jeff edited the test plan for this revision. (Show Details)
jeff added reviewers: br, mhorne, marius, markj.
jeff added inline comments.Aug 13 2019, 5:56 PM

This crept in from a different commit. I will fix before I merge.

markj accepted this revision.Aug 13 2019, 7:32 PM
markj added inline comments.

I think this can actually be PHYS_AVAIL_ENTRIES, but it doesn't really matter.


Extra newline.

This revision is now accepted and ready to land.Aug 13 2019, 7:32 PM
kib accepted this revision.Aug 15 2019, 4:37 PM

(assuming Mark and John notes are handled)

jeff closed this revision.Aug 16 2019, 4:56 PM