Re: [PATCH 1/3] tools/vm: fix Makefile multi-targets

2015-11-11 Thread Sergey Senozhatsky
On (11/11/15 12:34), David Rientjes wrote: [..] > > No, I have no objection to removing -O2. I'd prefer that the rationale be > included in the commit description, however. yes, agree. thanks. -ss -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of

Re: [PATCH 1/3] tools/vm: fix Makefile multi-targets

2015-11-11 Thread David Rientjes
On Wed, 11 Nov 2015, Sergey Senozhatsky wrote: > > This doesn't purport to explain why -O2 was added or why it's needed for← > > these tools. > > Hm, that was a sem-automatic action I saw no issues with (besides, I > considered this to be too small for a separate patch). `slabinfo' can > be used

Re: [PATCH 1/3] tools/vm: fix Makefile multi-targets

2015-11-11 Thread Sergey Senozhatsky
Hello, On (11/10/15 17:20), David Rientjes wrote: >>On (11/10/15 22:32), Sergey Senozhatsky wrote: >> >> Build all of the $(TARGETS), not just the first one. >> >> Without the patch (make -n) >> == >> >> make -C ../lib/api >> gcc -Wall -Wextra -I../lib/ -o page-types

Re: [PATCH 1/3] tools/vm: fix Makefile multi-targets

2015-11-10 Thread Christoph Lameter
On Tue, 10 Nov 2015, Sergey Senozhatsky wrote: > Build all of the $(TARGETS), not just the first one. Reviewed-by: Christoph Lameter -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vge

[PATCH 1/3] tools/vm: fix Makefile multi-targets

2015-11-10 Thread Sergey Senozhatsky
Build all of the $(TARGETS), not just the first one. Without the patch (make -n) == make -C ../lib/api gcc -Wall -Wextra -I../lib/ -o page-types page-types.c ... Only 'page-types' target was built. With the patch (make -n) === make -C ../lib/