Re: [LTP] [PATCH v3] partrt_nohz_full: Introducing a new test case

[chrubis@suse.cz]

Hi!
> This test case is intended to test the nohz_full kernel feature. See file
> testcases/kernel/partrt/README for more information.

This version looks good to me. My last concern is about the names of the
partrt tools.

> --- a/utils/Makefile
> +++ b/utils/Makefile
> @@ -24,6 +24,10 @@ include $(top_srcdir)/include/mk/env_pre.mk
>  
>  MAKE_TARGETS         += ffsb
>  
> +ifneq ($(wildcard rt-tools),)
> +MAKE_TARGETS         += partrt list2mask count_ticks
> +endif
> +
>  FFSBDIR			:= ffsb-6.0-rc2
>  FILTER_OUT_DIRS		:= $(FFSBDIR)
>  FFSB			:= $(FFSBDIR)/ffsb
> @@ -35,6 +39,9 @@ $(FFSB): $(abs_srcdir)/$(FFSBDIR)
>  ffsb: $(FFSB)
>  	cp $(FFSB) ffsb
>  
> +partrt list2mask count_ticks:
> +	cp rt-tools/install/bin/$@ $@

As it is now, upon the installation LTP copies all executables into one
directory and we are trying to avoid collisions by adding sufixes or
prefixes to the binaries.

Can we add a rt to all the binaries (list2maskrt, count_ticksrt) or
something similar while we install them?

-- 
Cyril Hrubis
chrubis@suse.cz

------------------------------------------------------------------------------
Learn Graph Databases - Download FREE O'Reilly Book
"Graph Databases" is the definitive new guide to graph databases and their 
applications. Written by three acclaimed leaders in the field, 
this first edition is now available. Download your free book today!
http://p.sf.net/sfu/NeoTech
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list