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)
Tue, Jan 28, 11:18 PM
Unknown Object (File)
Dec 12 2024, 11:33 AM
Unknown Object (File)
Oct 30 2024, 2:28 PM
Unknown Object (File)
Sep 24 2024, 1:00 PM
Unknown Object (File)
Sep 21 2024, 5:50 AM
Unknown Object (File)
Sep 20 2024, 8:49 PM
Unknown Object (File)
Sep 20 2024, 8:49 PM
Unknown Object (File)
Sep 20 2024, 8:49 PM
View All 36 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...