tests: Add MAP_32BIT flag test

[kib]

I am curious, why don't you code this in pure C with ATF.

tests/sys/vm/mmap_map_32bit_helper.c

8

Not needed for the new files

34

[dchagin]

Done,
+ MAP_ANON

[dchagin]

In D41236#939212, @kib wrote:

I am curious, why don't you code this in pure C with ATF.

Mostly for the sake of running the test by hand, anyway the helper is needed

[markj]

tests/sys/vm/Makefile

14

Need to exclude 32-bit arm too.

tests/sys/vm/mmap_map_32bit_helper.c

15

Maybe add _Static_assert(sizeof(vm_offset_t) >= 8);

35

36

39

It's nice to check for errors from munmap() too.

41

45

46

tests/sys/vm/mmap_map_32bit_test.sh

17

28

[dchagin]

Done, thanks. Tinderboxd

[markj]

Seems ok to me, I left a few more notes. Thanks for adding tests.

tests/sys/vm/Makefile

13

A more general way to write it:

.if ${MACHINE} != "i386" && ${MACHINE} != "arm" && (${MACHINE} != "powerpc" || ${MACHINE_ARCH} != "powerpc")

tests/sys/vm/mmap_map_32bit_helper.c

39

49

[kib]

tests/sys/vm/mmap_map_32bit_helper.c

7

You do not need to explicitly include sys/cdefs.h.

35

Shouldn't it be s32 + pagesize > MAP_32BIT_MAX_ADDR? At least to allow easier change to the mapping size.

[dchagin]

Seems done, thanks.

[dchagin]

tests/sys/vm/mmap_map_32bit_helper.c

35

I agree, thank you