Add more mmap tests related to character devices.

[jhb]

Summary

• Add cdev-related tests for bad args.
• Add a simple test case for mapping /dev/zero.

[jhb]

tests/sys/vm/mmap_test.c

168

I left this nullfd test here for you to see, but as my comment above suggests, I'm inclined to remove it. It is not a reliable test. I would need some some sort of /dev/foo for which non-root mmap works.

I suppose I could add a test that used /dev/mem and was skipped if the open failed due to not being root? This would at least test when run as root. Not sure if the automated tests under jenkins run as root or a mere mortal though.

171

I used this test to verify that my recent fix for /dev/zero preserved existing behavior for the MAP_SHARED case. This failed in a pre-fo_mmap kernel and now still fails after the recent change.

183

FWIW, MAP_SHARED mappings of a read/write /dev/zero fd have the same behavior. Not sure if that is a feature we should test for or just due to the implementation being simple.

[jhb]

• Use a dedicated test for MAP_PRIVATE + cdevs with /dev/mem.

[kib]

tests/sys/vm/mmap_test.c

128

Somewhat complicated suggestion is to check whether you really get a private mapping, i.e. zero pages even if other mapping wrote something.

Oh, I found the test below.

168

Might be, consider /dev/devstat ? I did not checked it actually.

183

I see. This is the feature, and since you test for the behaviour of the private mapping, the shared mapping could be easily handled.

[alc]

tests/sys/vm/mmap_test.c

183

FWIW, MAP_SHARED mappings of a read/write /dev/zero fd have the same behavior.

I'm confused by this statement. It's the same behavior as what?

Does the mapping of /dev/zero ever behave differently from MAP_ANON?

[kib]

tests/sys/vm/mmap_test.c

183

Well, the behaviour is weird for something with mmap(fd) where fd != -1. Every new mapping gets zero content, and flags only define future sharing mode. This is in contrast of other anonymous memory, like posix shm fd.

Yes, mmap("/dev/zero") is defned to behave as mmap(MAP_ANON), but sometimes caller of mmap cannot tell since it is defined by fd.

[alc]

Yes, the semantics of mmap()ing /dev/zero have always been irregular. I just wanted to be sure that there wasn't another issue.

Anyway, I think that the proposed tests are correct.

[jhb]

• Use /dev/devstat instead of /dev/mem.
• Now that the cdev_private test doesn't require root, merge it into bad_arguments.
• Duplicate the MAP_PRIVATE /dev/zero test for MAP_SHARED.

[kib]

tests/sys/vm/mmap_test.c

114–115

>= 0 could be usefully replaced by >= devstatfd in the assertion, etc.

[jhb]

Try to force an update.