diff mbox series

[net] test/vsock: remove vsock_perf executable on `make clean`

Message ID 20230803085454.30897-1-sgarzare@redhat.com (mailing list archive)
State Accepted
Commit 3c50c8b240390907c9a33c86d25d850520db6dfa
Delegated to: Netdev Maintainers
Headers show
Series [net] test/vsock: remove vsock_perf executable on `make clean` | expand

Checks

Context Check Description
netdev/series_format success Single patches do not need cover letters
netdev/tree_selection success Clearly marked for net
netdev/fixes_present success Fixes tag present in non-next series
netdev/header_inline success No static functions without inline keyword in header files
netdev/build_32bit success Errors and warnings before: 9 this patch: 9
netdev/cc_maintainers success CCed 5 of 5 maintainers
netdev/build_clang success Errors and warnings before: 9 this patch: 9
netdev/verify_signedoff success Signed-off-by tag matches author and committer
netdev/deprecated_api success None detected
netdev/check_selftest success No net selftest shell script
netdev/verify_fixes success Fixes tag looks correct
netdev/build_allmodconfig_warn success Errors and warnings before: 9 this patch: 9
netdev/checkpatch success total: 0 errors, 0 warnings, 0 checks, 6 lines checked
netdev/kdoc success Errors and warnings before: 0 this patch: 0
netdev/source_inline success Was 0 now: 0

Commit Message

Stefano Garzarella Aug. 3, 2023, 8:54 a.m. UTC
We forgot to add vsock_perf to the rm command in the `clean`
target, so now we have a left over after `make clean` in
tools/testing/vsock.

Fixes: 8abbffd27ced ("test/vsock: vsock_perf utility")
Cc: AVKrasnov@sberdevices.ru
Signed-off-by: Stefano Garzarella <sgarzare@redhat.com>
---
 tools/testing/vsock/Makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Simon Horman Aug. 3, 2023, 3:04 p.m. UTC | #1
On Thu, Aug 03, 2023 at 10:54:54AM +0200, Stefano Garzarella wrote:
> We forgot to add vsock_perf to the rm command in the `clean`
> target, so now we have a left over after `make clean` in
> tools/testing/vsock.
> 
> Fixes: 8abbffd27ced ("test/vsock: vsock_perf utility")
> Cc: AVKrasnov@sberdevices.ru
> Signed-off-by: Stefano Garzarella <sgarzare@redhat.com>

Reviewed-by: Simon Horman <horms@kernel.org>
Tested-by: Simon Horman <horms@kernel.org> # build-tested
patchwork-bot+netdevbpf@kernel.org Aug. 3, 2023, 6:10 p.m. UTC | #2
Hello:

This patch was applied to netdev/net.git (main)
by Jakub Kicinski <kuba@kernel.org>:

On Thu,  3 Aug 2023 10:54:54 +0200 you wrote:
> We forgot to add vsock_perf to the rm command in the `clean`
> target, so now we have a left over after `make clean` in
> tools/testing/vsock.
> 
> Fixes: 8abbffd27ced ("test/vsock: vsock_perf utility")
> Cc: AVKrasnov@sberdevices.ru
> Signed-off-by: Stefano Garzarella <sgarzare@redhat.com>
> 
> [...]

Here is the summary with links:
  - [net] test/vsock: remove vsock_perf executable on `make clean`
    https://git.kernel.org/netdev/net/c/3c50c8b24039

You are awesome, thank you!
diff mbox series

Patch

diff --git a/tools/testing/vsock/Makefile b/tools/testing/vsock/Makefile
index 43a254f0e14d..21a98ba565ab 100644
--- a/tools/testing/vsock/Makefile
+++ b/tools/testing/vsock/Makefile
@@ -8,5 +8,5 @@  vsock_perf: vsock_perf.o
 CFLAGS += -g -O2 -Werror -Wall -I. -I../../include -I../../../usr/include -Wno-pointer-sign -fno-strict-overflow -fno-strict-aliasing -fno-common -MMD -U_FORTIFY_SOURCE -D_GNU_SOURCE
 .PHONY: all test clean
 clean:
-	${RM} *.o *.d vsock_test vsock_diag_test
+	${RM} *.o *.d vsock_test vsock_diag_test vsock_perf
 -include *.d