On Mon, Jul 25, 2022 at 04:35:37PM +0000, David Matlack wrote: > This series renames the perf_test_util to memstress. patch 1 renames the files > perf_test_util.[ch] to memstress.[ch], and patch 2 replaces the perf_test_ > prefix on symbols with memstress_. > > The reason for this rename, as with any rename, is to improve readability. > perf_test_util is too generic and does not describe at all what the library > does, other than being used for perf tests. > > I considered a lot of different names (naming is hard) and eventually settled > on memstress for a few reasons: > > - "memstress" better describes the functionality proveded by this library, > which is to run a VM that reads/writes to memory from all vCPUs in parallel > (i.e. stressing VM memory). > > - "memstress" contains the same number of characters as "perf_test", making > it a drop in replacement in symbols wihout changing line lengths. > > - The lack of underscore between "mem" and "stress" makes it clear "memstress" > is a noun, avoiding confusion in function names. Naming is hard, acking rename patches that have good justifications is easy. For the series, Acked-by: Andrew Jones <andrew.jones@xxxxxxxxx>
