Micro-optimize __load_ioengine()
authorBart Van Assche <bvanassche@acm.org>
Wed, 1 Jan 2020 00:31:26 +0000 (16:31 -0800)
committerBart Van Assche <bvanassche@acm.org>
Wed, 1 Jan 2020 16:58:50 +0000 (08:58 -0800)
commit044be36e333b2ee3bfbacd3e9d54fa68571b13b3
tree61f2d17c4b19062d96dbf00413b42ee3ded02fa8
parentaae515f4e1cb0b3c003e127200d344d807032a79
Micro-optimize __load_ioengine()

Instead of copying the I/O engine name, use a pointer to the I/O engine
name. This patch suppresses the following false positive Valgrind complaint:

Conditional jump or move depends on uninitialised value(s)
   at 0x41D451: __load_ioengine (ioengines.c:133)
   by 0x41D451: load_ioengine (ioengines.c:161)
   by 0x41FA59: ioengine_load (init.c:1126)
   by 0x423B64: parse_cmd_line (init.c:2670)
   by 0x4241A3: parse_options (init.c:2965)
   by 0x40F587: main (fio.c:42)

The code for copying I/O engine names was introduced by commit 2866c82d598e
("[PATCH] Separate io engines into separate loadable objects").

Signed-off-by: Bart Van Assche <bvanassche@acm.org>
ioengines.c