Page MenuHomeFreeBSD

vm_map: add vm_map_find_locked(9)
ClosedPublic

Authored by kib on Sun, Sep 15, 7:01 AM.
Tags
None
Referenced Files
F97454696: D46678.diff
Sun, Sep 29, 11:20 AM
Unknown Object (File)
Fri, Sep 20, 8:01 AM
Unknown Object (File)
Wed, Sep 18, 8:00 PM
Unknown Object (File)
Tue, Sep 17, 11:19 PM
Unknown Object (File)
Sun, Sep 15, 10:38 AM
Unknown Object (File)
Sun, Sep 15, 10:38 AM
Unknown Object (File)
Sun, Sep 15, 10:31 AM
Unknown Object (File)
Sun, Sep 15, 7:30 AM
Subscribers

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

kib requested review of this revision.Sun, Sep 15, 7:01 AM
This revision is now accepted and ready to land.Sun, Sep 15, 1:21 PM

Just out of curiosity, is this a first step in a larger set of changes?

sys/vm/vm_map.c
2145

Drop the "vm_map_find: " prefix?

2259–2265

Hijacking this review for the moment, I want to share an observation: If not for vm_map_alignspace() changing *addr when it fails, a restart would not be necessary here. On entry to vm_map_alignspace(), *addr already holds an address that is large enough and qualifies as ANY_SPACE.

kib marked an inline comment as done.Sun, Sep 15, 5:51 PM
In D46678#1064006, @alc wrote:

Just out of curiosity, is this a first step in a larger set of changes?

This is for D46619

This revision was automatically updated to reflect the committed changes.