For now, copy the mbuf allocator.
Details
- Reviewers
jeff markj - Group Reviewers
manpages - Commits
- rS357547: uma: add UMA_ZONE_CONTIG, and a default contig_alloc
kyua test -k /usr/tests/sys/Kyuafile
Diff Detail
- Lint
Lint Passed - Unit
No Test Coverage - Build Status
Buildable 29143 Build 27079: arc lint + arc unit
Event Timeline
share/man/man9/zone.9 | ||
---|---|---|
345 | drop ", but only" | |
472–473 | s/yields/allocates memory is used redundantly. I would say 'if memory with special constraints such as attributes, alignment, or address ranges must be used..." | |
sys/vm/uma.h | ||
239 | I think this could be phrased better as a constraint than a statement of fact. "Objects must be physically contiguous" Although that still seems to imply that somehow multiple objects are contiguous. We really want to express that a single object will be. | |
sys/vm/uma_core.c | ||
2124 | There really shouldn't be any boot time contig allocations but I suppose it doesn't hurt to handle it. |
share/man/man9/zone.9 | ||
---|---|---|
472–473 | Okay, and then drop the VM_MEMATTR_DEFAULT example? | |
sys/vm/uma.h | ||
646–647 | This is gratuitous, but maybe related enough to sneak in here. | |
sys/vm/uma_core.c | ||
1698–1700 | Also gratuitous, but related. | |
2124 | Yes... I thought to handle it since it's easy and I think it works fine with startup_alloc. If you prefer, I can add it to the KASSERT about early boot PCPU alloc instead. |
share/man/man9/zone.9 | ||
---|---|---|
344 | Per mandoc style, sentences should start on new lines. |
sys/vm/uma.h | ||
---|---|---|
239 | Yeah, I'm having trouble phrasing this concisely with the right sense. I think your suggestion is okay. Here's another idea: "Each object must be internally physically contiguous." |
sys/vm/uma.h | ||
---|---|---|
239 | Your proposed phrasing, specifically "Each object", is less likely to be misinterpreted. No description that fits in one line will be unambiguous, If you are willing to use two lines, I would suggest: "Physical memory underlying an object must be contiguous" |
share/man/man9/zone.9 | ||
---|---|---|
345 | I would suggest: "The physical memory underlying each item in this zone must be contiguous. In particular, if an item spans a page boundary, then the underlying physical pages must be contiguous." |
sys/vm/uma.h | ||
---|---|---|
239 | Thanks, I'll use that. |
Rebase over r357392.
@jeff, note that I moved the assignment of uk_allocf = pcpu_page_alloc under the startup_alloc check, because I thought it read better that way. Please let me know if you'd like that restored, or if there was another reason for it being outside that check.