Page MenuHomeFreeBSD

vmm: Add memlock rlimit and racct limit
Needs ReviewPublic

Authored by cyril_freebsdfoundation.org on Aug 16 2021, 7:40 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Apr 10, 8:22 PM
Unknown Object (File)
Sat, Apr 6, 1:07 AM
Unknown Object (File)
Fri, Apr 5, 9:47 PM
Unknown Object (File)
Mon, Mar 25, 11:21 AM
Unknown Object (File)
Feb 22 2024, 1:18 PM
Unknown Object (File)
Feb 22 2024, 8:02 AM
Unknown Object (File)
Feb 1 2024, 10:24 AM
Unknown Object (File)
Jan 23 2024, 11:36 PM

Details

Reviewers
markj
grehan
Group Reviewers
bhyve
Summary

This revision adds the rlimit and racct limit for the memlock property to vmm. After this change, a VM's vmspace cannot have any more wired memory than is allowed for the process that creates the VM.

Note that the limit is taken from the current process, but the calculation is independent from the process' vmspace. This means that a process can theoretically use mlock to wire memory up to the process' memlock limit, and then create a VM which has wired memory up to the process' memlock limit. In short, a process can now wire twice as much memory as the limit.

Additionally, this means that rctl/racct may not report the true amount of memory wired by a process; it will only report the amount of wired memory for the process vmspace or the process-created VM's vmspace, whichever occurred latest. Which means aggregate counts are also incorrect.

In practice, I don't think there are any programs that will create a VM with wired memory and use mlock, so this discrepancy will likely never surface.

This strategy seems semantically better than the alternative, which is to add the process' vmspace and VM's vmspace in both files vmm.c and vm/vm_mmap.c, since the process doesn't keep track of the VMs that it created.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

grehan added a subscriber: grehan.

Double-accounting for memory is a never-ending issue in bhyve :(

This revision is now accepted and ready to land.Aug 17 2021, 10:26 PM

Moved a calculation out of a locked area.

This revision now requires review to proceed.Aug 18 2021, 8:14 PM