]> git.ipfire.org Git - thirdparty/kernel/stable.git/commit
samples: bpf: Fix tracex7 error raised on the missing argument
authorJuhee Kang <claudiajkang@gmail.com>
Tue, 27 Jul 2021 04:10:55 +0000 (04:10 +0000)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sat, 18 Sep 2021 11:42:05 +0000 (13:42 +0200)
commit78ce0579c68e4d6e6e6731ef272a7a9e827489b5
tree6ad72246850c5b30d2e2b0eafa9cff1f89310c75
parent12bcb996518c7dcee013e7fba4c43b3c09c25149
samples: bpf: Fix tracex7 error raised on the missing argument

[ Upstream commit 7d07006f05922b95518be403f08ef8437b67aa32 ]

The current behavior of 'tracex7' doesn't consist with other bpf samples
tracex{1..6}. Other samples do not require any argument to run with, but
tracex7 should be run with btrfs device argument. (it should be executed
with test_override_return.sh)

Currently, tracex7 doesn't have any description about how to run this
program and raises an unexpected error. And this result might be
confusing since users might not have a hunch about how to run this
program.

    // Current behavior
    # ./tracex7
    sh: 1: Syntax error: word unexpected (expecting ")")
    // Fixed behavior
    # ./tracex7
    ERROR: Run with the btrfs device argument!

In order to fix this error, this commit adds logic to report a message
and exit when running this program with a missing argument.

Additionally in test_override_return.sh, there is a problem with
multiple directory(tmpmnt) creation. So in this commit adds a line with
removing the directory with every execution.

Signed-off-by: Juhee Kang <claudiajkang@gmail.com>
Signed-off-by: Andrii Nakryiko <andrii@kernel.org>
Acked-by: Yonghong Song <yhs@fb.com>
Link: https://lore.kernel.org/bpf/20210727041056.23455-1-claudiajkang@gmail.com
Signed-off-by: Sasha Levin <sashal@kernel.org>
samples/bpf/test_override_return.sh
samples/bpf/tracex7_user.c