Page MenuHomeFreeBSD
Differential D41435

vm: Allow MAP_32BIT for all architectures
ClosedPublic
Actions

Authored by dchagin on Aug 12 2023, 9:01 AM.
Tags
None
Referenced Files
F173969008: D41435.id125941.diff
Tue, Sep 29, 5:12 PM
F173968973: D41435.id125984.diff
Tue, Sep 29, 5:12 PM
Unknown Object (File)
Tue, Sep 22, 11:10 AM
Unknown Object (File)
Mon, Sep 21, 4:40 AM
Unknown Object (File)
Sun, Sep 20, 11:53 PM
Unknown Object (File)
Sun, Sep 13, 7:24 AM
Unknown Object (File)
Thu, Sep 10, 11:39 PM
Unknown Object (File)
Thu, Sep 10, 9:51 PM
View All Files
Subscribers
imp

Details

Reviewers
kib
jhb
alc
markj
Commits
rGf3e11927dc28: vm: Allow MAP_32BIT for all architectures

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 53071
Build 49962: arc lint + arc unit

Event Timeline

dchagin created this revision.Aug 12 2023, 9:01 AM
Herald added a subscriber: imp. · View Herald TranscriptAug 12 2023, 9:01 AM
dchagin requested review of this revision.Aug 12 2023, 9:01 AM
Harbormaster completed remote builds in B53071: Diff 125930.Aug 12 2023, 9:01 AM
dchagin added a parent revision: D41434: vm: MAP_32BIT_MAX_ADDR defined in sys/mman.h.Aug 12 2023, 9:01 AM
dchagin added a reviewer: kib.Aug 12 2023, 2:57 PM
kib added inline comments.Aug 12 2023, 4:58 PM
sys/vm/vm_mmap.c
1618

I do not think this predict_true is warranted.

kib accepted this revision.Aug 12 2023, 4:58 PM
kib added reviewers: jhb, alc, markj.
This revision is now accepted and ready to land.Aug 12 2023, 4:58 PM
alc added inline comments.Aug 12 2023, 5:34 PM
sys/vm/vm_mmap.c
1618

Agreed, nor is rewriting this to an if/else, as I noted in the other review.

dchagin updated this revision to Diff 125941.Aug 12 2023, 7:52 PM

predict_true macro removed, also fixed libsysdecode mktables

This revision now requires review to proceed.Aug 12 2023, 7:52 PM
Harbormaster completed remote builds in B53079: Diff 125941.Aug 12 2023, 7:52 PM
alc accepted this revision.Aug 12 2023, 8:48 PM
This revision is now accepted and ready to land.Aug 12 2023, 8:48 PM
markj accepted this revision.Aug 14 2023, 1:47 PM
kib accepted this revision.Aug 14 2023, 4:17 PM
Closed by commit rGf3e11927dc28: vm: Allow MAP_32BIT for all architectures (authored by dchagin). · Explain WhyAug 14 2023, 5:21 PM
This revision was automatically updated to reflect the committed changes.
dchagin added a commit: rGf3e11927dc28: vm: Allow MAP_32BIT for all architectures.

Revision Contents
Changeset List

PathSize
lib/
libc/
sys/
3 lines
libsysdecode/
9 lines
sys/
kern/
8 lines
sys/
2 lines
vm/
26 lines
tests/
sys/
vm/
5 lines
1 line

Diff 125930

View Options

lib/libc/sys/mmap.2

Loading...
View Options

lib/libsysdecode/flags.c

Loading...
View Options

sys/kern/uipc_shm.c

Loading...
View Options

sys/sys/mman.h

Loading...
View Options

sys/vm/vm_mmap.c

Loading...
View Options

tests/sys/vm/Makefile

Loading...
View Options

tests/sys/vm/mmap_map_32bit_helper.c

Loading...