bpf.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v2] selftests/bpf: Remove test_align leftovers
@ 2020-08-19 16:07 Veronika Kabatova
  2020-08-19 16:17 ` Jesper Dangaard Brouer
  0 siblings, 1 reply; 3+ messages in thread
From: Veronika Kabatova @ 2020-08-19 16:07 UTC (permalink / raw)
  To: bpf; +Cc: sdf, andriin, skozina, brouer, yhs, Veronika Kabatova

Calling generic selftests "make install" fails as rsync expects all
files from TEST_GEN_PROGS to be present. The binary is not generated
anymore (commit 3b09d27cc93d) so we can safely remove it from there
and also from gitignore.

Fixes: 3b09d27cc93d ("selftests/bpf: Move test_align under test_progs")
Signed-off-by: Veronika Kabatova <vkabatov@redhat.com>
---
 tools/testing/selftests/bpf/.gitignore | 1 -
 tools/testing/selftests/bpf/Makefile   | 2 +-
 2 files changed, 1 insertion(+), 2 deletions(-)

diff --git a/tools/testing/selftests/bpf/.gitignore b/tools/testing/selftests/bpf/.gitignore
index 1bb204cee853..9a0946ddb705 100644
--- a/tools/testing/selftests/bpf/.gitignore
+++ b/tools/testing/selftests/bpf/.gitignore
@@ -6,7 +6,6 @@ test_lpm_map
 test_tag
 FEATURE-DUMP.libbpf
 fixdep
-test_align
 test_dev_cgroup
 /test_progs*
 test_tcpbpf_user
diff --git a/tools/testing/selftests/bpf/Makefile b/tools/testing/selftests/bpf/Makefile
index a83b5827532f..fc946b7ac288 100644
--- a/tools/testing/selftests/bpf/Makefile
+++ b/tools/testing/selftests/bpf/Makefile
@@ -32,7 +32,7 @@ LDLIBS += -lcap -lelf -lz -lrt -lpthread
 
 # Order correspond to 'make run_tests' order
 TEST_GEN_PROGS = test_verifier test_tag test_maps test_lru_map test_lpm_map test_progs \
-	test_align test_verifier_log test_dev_cgroup test_tcpbpf_user \
+	test_verifier_log test_dev_cgroup test_tcpbpf_user \
 	test_sock test_btf test_sockmap get_cgroup_id_user test_socket_cookie \
 	test_cgroup_storage \
 	test_netcnt test_tcpnotify_user test_sock_fields test_sysctl \
-- 
2.26.2


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH v2] selftests/bpf: Remove test_align leftovers
  2020-08-19 16:07 [PATCH v2] selftests/bpf: Remove test_align leftovers Veronika Kabatova
@ 2020-08-19 16:17 ` Jesper Dangaard Brouer
  2020-08-20 21:26   ` Alexei Starovoitov
  0 siblings, 1 reply; 3+ messages in thread
From: Jesper Dangaard Brouer @ 2020-08-19 16:17 UTC (permalink / raw)
  To: Veronika Kabatova; +Cc: bpf, sdf, andriin, skozina, yhs, brouer

On Wed, 19 Aug 2020 18:07:10 +0200
Veronika Kabatova <vkabatov@redhat.com> wrote:

> Calling generic selftests "make install" fails as rsync expects all
> files from TEST_GEN_PROGS to be present. The binary is not generated
> anymore (commit 3b09d27cc93d) so we can safely remove it from there
> and also from gitignore.
> 
> Fixes: 3b09d27cc93d ("selftests/bpf: Move test_align under test_progs")
> Signed-off-by: Veronika Kabatova <vkabatov@redhat.com>
> ---
>  tools/testing/selftests/bpf/.gitignore | 1 -
>  tools/testing/selftests/bpf/Makefile   | 2 +-
>  2 files changed, 1 insertion(+), 2 deletions(-)

Acked-by: Jesper Dangaard Brouer <brouer@redhat.com>

-- 
Best regards,
  Jesper Dangaard Brouer
  MSc.CS, Principal Kernel Engineer at Red Hat
  LinkedIn: http://www.linkedin.com/in/brouer


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH v2] selftests/bpf: Remove test_align leftovers
  2020-08-19 16:17 ` Jesper Dangaard Brouer
@ 2020-08-20 21:26   ` Alexei Starovoitov
  0 siblings, 0 replies; 3+ messages in thread
From: Alexei Starovoitov @ 2020-08-20 21:26 UTC (permalink / raw)
  To: Jesper Dangaard Brouer
  Cc: Veronika Kabatova, bpf, Stanislav Fomichev, Andrii Nakryiko,
	Stanislav Kozina, Yonghong Song

On Wed, Aug 19, 2020 at 9:17 AM Jesper Dangaard Brouer
<brouer@redhat.com> wrote:
>
> On Wed, 19 Aug 2020 18:07:10 +0200
> Veronika Kabatova <vkabatov@redhat.com> wrote:
>
> > Calling generic selftests "make install" fails as rsync expects all
> > files from TEST_GEN_PROGS to be present. The binary is not generated
> > anymore (commit 3b09d27cc93d) so we can safely remove it from there
> > and also from gitignore.
> >
> > Fixes: 3b09d27cc93d ("selftests/bpf: Move test_align under test_progs")
> > Signed-off-by: Veronika Kabatova <vkabatov@redhat.com>
> > ---
> >  tools/testing/selftests/bpf/.gitignore | 1 -
> >  tools/testing/selftests/bpf/Makefile   | 2 +-
> >  2 files changed, 1 insertion(+), 2 deletions(-)
>
> Acked-by: Jesper Dangaard Brouer <brouer@redhat.com>

Applied to bpf tree. Thanks

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2020-08-20 21:26 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-08-19 16:07 [PATCH v2] selftests/bpf: Remove test_align leftovers Veronika Kabatova
2020-08-19 16:17 ` Jesper Dangaard Brouer
2020-08-20 21:26   ` Alexei Starovoitov

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).