lkml.org 
[lkml]   [2012]   [Apr]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH v4 2/4] tools: Add a toplevel Makefile
    On Tue, Apr 10, 2012 at 05:20:38PM +0200, Borislav Petkov wrote:
    > From: Borislav Petkov <borislav.petkov@amd.com>
    >
    > Add a Makefile with all the targets under tools/.
    >
    > Signed-off-by: Borislav Petkov <borislav.petkov@amd.com>
    > ---
    > tools/Makefile | 31 +++++++++++++++++++++++++++++++
    > 1 file changed, 31 insertions(+)
    > create mode 100644 tools/Makefile
    >
    > diff --git a/tools/Makefile b/tools/Makefile
    > new file mode 100644
    > index 000000000000..e93b8ec3d250
    > --- /dev/null
    > +++ b/tools/Makefile
    > @@ -0,0 +1,31 @@
    > +include scripts/Makefile.include
    > +
    > +perf firewire lguest usb virtio vm: FORCE
    > + $(QUIET_SUBDIR0)$@/ $(QUIET_SUBDIR1)
    > +
    > +cpupower: FORCE
    > + $(QUIET_SUBDIR0)power/$@/ $(QUIET_SUBDIR1)
    > +
    > +turbostat x86_energy_perf_policy: FORCE
    > + $(QUIET_SUBDIR0)power/x86/$@/ $(QUIET_SUBDIR1)
    > +
    > +selftests: FORCE
    > + $(QUIET_SUBDIR0)testing/$@/ $(QUIET_SUBDIR1)
    > +
    > +firewire_clean lguest_clean perf_clean vm_clean usb_clean virtio_clean:
    > + $(QUIET_SUBDIR0)$(@:_clean=)/ $(QUIET_SUBDIR1) clean
    > +
    > +cp_clean:
    > + $(QUIET_SUBDIR0)power/cpupower/ $(QUIET_SUBDIR1) clean
    > +
    > +turbostat_clean x86_energy_perf_policy_clean:
    > + $(QUIET_SUBDIR0)power/x86/$(@:_clean=)/ $(QUIET_SUBDIR1) clean
    > +
    > +selftests_clean:
    > + $(QUIET_SUBDIR0)testing/$(@:_clean=)/ $(QUIET_SUBDIR1) clean
    > +
    > +clean: cp_clean firewire_clean lguest_clean perf_clean selftests_clean \
    > + turbostat_clean usb_clean virtio_clean \
    > + x86_energy_perf_policy_clean
    > +
    > +.PHONY: FORCE

    Things would be much simpler is all tools were adjusted to:
    a) the subdir they live in is the make target name
    b) each tool has three mandatory targets:
    1) default target => build the tool
    2) clean
    3) install

    I can see that with the layout today you cannot
    distingush between the various targets below power.
    But the current solutions is not scaleable nor maintainable.

    Even if you cannot fix power/* then testing looks easy.

    But all that can always to improved and consolidated later.

    Sam


    \
     
     \ /
      Last update: 2012-04-10 22:37    [W:3.485 / U:0.008 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site