lkml.org 
[lkml]   [2019]   [Jul]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    From
    SubjectRe: [PATCH v9 10/10] selftests: add openat2(2) selftests
    Date
    Hi Aleksa,

    A few minor comments below.

    Aleksa Sarai <cyphar@cyphar.com> writes:
    > diff --git a/tools/testing/selftests/openat2/Makefile b/tools/testing/selftests/openat2/Makefile
    > new file mode 100644
    > index 000000000000..8235a49928f6
    > --- /dev/null
    > +++ b/tools/testing/selftests/openat2/Makefile
    > @@ -0,0 +1,12 @@
    > +# SPDX-License-Identifier: GPL-2.0
    > +
    > +CFLAGS += -Wall -O2 -g
    > +TEST_GEN_PROGS := linkmode_test resolve_test rename_attack_test
    > +
    > +include ../lib.mk
    > +
    > +$(OUTPUT)/linkmode_test: linkmode_test.c helpers.o
    > +$(OUTPUT)/rename_attack_test: rename_attack_test.c helpers.o
    > +$(OUTPUT)/resolve_test: resolve_test.c helpers.o

    You don't need to tell make that foo depends on foo.c.

    Also if you make the dependency be on helpers.c then you won't get an
    intermediate helpers.o, and then you don't need to clean it.

    So the above three lines could just be:

    $(TEST_GEN_PROGS): helpers.c

    > +EXTRA_CLEAN = helpers.o $(wildcard /tmp/ksft-openat2-*)

    If you follow my advice above you don't need helpers.o in there.

    Deleting things from /tmp is also a bit fishy on shared machines, ie. it
    will error if those files happen to be owned by another user.

    cheers

    \
     
     \ /
      Last update: 2019-07-08 03:16    [W:4.296 / U:0.040 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site