Page MenuHomeFreeBSD

address late reviewer concerns about helper functions patch
AbandonedPublic

Authored by dougm on Oct 21 2018, 6:01 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Jan 14, 4:57 PM
Unknown Object (File)
Nov 29 2024, 12:33 AM
Unknown Object (File)
Nov 29 2024, 12:33 AM
Unknown Object (File)
Nov 29 2024, 12:32 AM
Unknown Object (File)
Nov 22 2024, 1:48 PM
Unknown Object (File)
Oct 1 2024, 5:25 AM
Unknown Object (File)
Sep 18 2024, 2:09 AM
Unknown Object (File)
Sep 14 2024, 10:56 PM
Subscribers
None

Details

Reviewers
alc
kib
markj
Summary

Make style and formatting changes recommended in a late review of D17610: Use helper functions to reduce code duplication in merging vm_entry neighbors

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

sys/vm/vm_map.c
1647

vm_map_simplify_test() so that people only see this function as a helper to vm_map_simplify_entry().

1663

vm_map_simplify_deallocate()

Add 'simplify' to 'neighbor' function names.

The concerns that this change would address will be addressed elsewhere.

So what is going on with these corrections ?

In D17639#379371, @kib wrote:

So what is going on with these corrections ?

D17635 Avoid duplicate set_max_free after unlink
will address them, once D14005 is committed.
Unless something else comes up.