Page MenuHomeFreeBSD
Differential D20628

Use vm_map_entry_t field prev less
ClosedPublic
Actions

Authored by dougm on Jun 13 2019, 3:04 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Oct 17, 10:19 PM
Unknown Object (File)
Fri, Oct 3, 9:14 AM
Unknown Object (File)
Fri, Oct 3, 12:49 AM
Unknown Object (File)
Sat, Sep 27, 7:31 AM
Unknown Object (File)
Wed, Sep 24, 4:32 PM
Unknown Object (File)
Sep 14 2025, 8:07 AM
Unknown Object (File)
Sep 14 2025, 2:22 AM
Unknown Object (File)
Jun 27 2025, 1:31 AM
View All 69 Files
Subscribers
imp

Details

Reviewers
kib
markj
Commits
rS349030: Avoid using the prev field of vm_map_entry_t in two functions that
Summary

This change avoids using the prev field in two functions that iterate over consecutive entries, so that that can easily just 'remember' the prev value instead of looking it up.

This is a small part of an effort to eventually eliminate the prev field.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

dougm created this revision.Jun 13 2019, 3:04 AM
kib accepted this revision.Jun 13 2019, 9:43 AM
kib added inline comments.
vm_map.c
4855 ↗(On Diff #58578)

You can remove excessive () in this line.

4856 ↗(On Diff #58578)

And there too.

4877 ↗(On Diff #58578)

Same.

This revision is now accepted and ready to land.Jun 13 2019, 9:43 AM
Closed by commit rS349030: Avoid using the prev field of vm_map_entry_t in two functions that (authored by dougm). · Explain WhyJun 14 2019, 3:16 AM
This revision was automatically updated to reflect the committed changes.
Herald added a subscriber: imp. · View Herald TranscriptJun 14 2019, 3:16 AM

Revision Contents
Changeset List

PathSize
head/
sys/
vm/
34 lines

Diff 58608

View Options

head/sys/vm/vm_map.c

Loading...