Speed up geom_stats_resync in the presence of many devices

[asomers]

Summary

The old code had a O(n) loop, where n is the size of /dev/devstat.
Multiply that by another O(n) loop in devstat_mmap for a total of
O(n^2).

This change adds DIOCGMEDIASIZE support to /dev/devstat so userland can
quickly determine the right amount of memory to map, eliminating the
O(n) loop in userland.

This change decreases the time to run "gstat -bI0.001" with 16,384 md
devices from 29.7s to 4.2s.

MFC after: 2 weeks
Sponsored by: Axcient

[skip ci] fix a typo in a comment in mdconfig.c Sponsored by: Axcient

Test Plan

Tested with gstat -bI0.001 with up to 16,384 md devices

[asomers]

• Speed up geom_stats_resync in the presence of many devices
• [skip ci] fix a typo in a comment in mdconfig.c

[imp]

Looks good to my eye

[mav]

lib/libgeom/geom_stats.c

74

I don't very like the use of DIOCGMEDIASIZE here, but I have no better ideas.

80

In the new code you are neither reading nor updating npages variable. You could read npages to skip mmap() call if nothing has changed. And you should update it for other library parts to work correctly.

sys/kern/subr_devstat.c

506

Double semicolon.

[imp]

lib/libgeom/geom_stats.c

74

I had the same thought... but didn't want to suggest a new one

[asomers]

• * Fix mav's npages observation

[mav]

lib/libgeom/geom_stats.c

136

I think previous order was intentional to make time more up to date of paging on access take some time.

[asomers]

lib/libgeom/geom_stats.c

136

Interesting idea. Do you have any good idea why there used to be two memset calls?

[mav]

lib/libgeom/geom_stats.c

136

Not really. Guess some form of swap pager woodoo. Something like first pass triggers swap-in, while second brings up to date TLB caches if not yet, to make following copy as coherent as possible.

[asomers]

lib/libgeom/geom_stats.c

136

So you really think both memsets are required?

[mav]

lib/libgeom/geom_stats.c

136

And you think it is a copy/paste bug?

[asomers]

lib/libgeom/geom_stats.c

136

Yes, I think it's a copy/paste bug.

[mav]

lib/libgeom/geom_stats.c

77

I haven't used mmap() that much to know. I am curios whether the lack of munmap() before used some trick not avoid extra actions.

136

I this case I'd say don't fix what's not broken. But whatever you decide. Responsibility is yours. ;)

[asomers]

lib/libgeom/geom_stats.c

77

I don't know. But the resulted in rapid accumulation of MAP ENTRY memory with every call to geom_stats_resync

136

Even though that code is very old, the original author is still active. So I'll pull it out of this review and take it up separately.