From: "Paul E. McKenney" <paulmck@linux.vnet.ibm.com>
To: Akira Yokosawa <akiyks@gmail.com>
Cc: perfbook@vger.kernel.org
Subject: Re: [PATCH 0/5] CodeSamples: Cleanups and Makefile updates
Date: Fri, 9 Jun 2017 15:21:15 -0700 [thread overview]
Message-ID: <20170609222115.GO3721@linux.vnet.ibm.com> (raw)
In-Reply-To: <39761e77-17b7-7e05-d47b-07b89ff9b4b4@gmail.com>
On Sat, Jun 10, 2017 at 12:22:53AM +0900, Akira Yokosawa wrote:
> >From b31141981b13d40fbbe5d20fa339d714ecba084c Mon Sep 17 00:00:00 2001
> From: Akira Yokosawa <akiyks@gmail.com>
> Date: Fri, 9 Jun 2017 23:59:11 +0900
> Subject: [PATCH 0/5] CodeSamples: Cleanups and Makefile updates
>
> Hi Paul,
>
> Removal of gprof-helper.c and linux/list.h has become a fairly large
> patch set.
>
> Patch 1 is simple removal.
> Patch 2 is an improvement to make the "-jN" option to propagate to
> subdirectories.
> Patch 3 removes linux/list.h, with refactoring of recipes.
> Patch 4 adds subdirectories that can be built by "make" in the
> CodeSamples/ directory.
> Patch 5 improves build behavior for unsupported architectures.
> You can test the behavior by overriding "arch" variable by defining
> it as an environment variable, e.g.:
> $ export arch=unsupported
> $ make clean; make
> After the test,
> $ unset arch
> will restore the normal behavior.
>
> Thoughts?
Seems to work, applied and pushed, thank you!
Thanx, Paul
> Thanks, Akira
> --
>
> Akira Yokosawa (5):
> CodeSamples: Remove gprof-helper.c
> CodeSamples: Use $(MAKE) for recursive make
> CodeSamples: Makefile: Remove linux/list.h
> CodeSamples: Makefile: Add subdirectories
> CodeSamples: Makefile: Distinguish arch-independent targets
>
> CodeSamples/.gitignore | 1 -
> CodeSamples/Makefile | 125 +-----
> CodeSamples/SMPdesign/Makefile | 19 +-
> CodeSamples/advsync/Makefile | 11 +-
> CodeSamples/count/Makefile | 11 +-
> CodeSamples/datastruct/Issaquah/Makefile | 11 +-
> CodeSamples/datastruct/hash/Makefile | 11 +-
> CodeSamples/datastruct/log/Makefile | 11 +-
> CodeSamples/datastruct/skiplist/Makefile | 11 +-
> CodeSamples/defer/Makefile | 18 +-
> CodeSamples/depends.mk | 14 +-
> CodeSamples/gprof-helper.c | 118 ------
> CodeSamples/intro/Makefile | 15 +-
> CodeSamples/linux/list.h | 700 -------------------------------
> CodeSamples/locking/Makefile | 15 +-
> CodeSamples/recipes.mk | 10 +
> CodeSamples/toolsoftrade/Makefile | 1 +
> 17 files changed, 139 insertions(+), 963 deletions(-)
> delete mode 100644 CodeSamples/gprof-helper.c
> delete mode 100644 CodeSamples/linux/list.h
> create mode 100644 CodeSamples/recipes.mk
>
> --
> 2.7.4
>
prev parent reply other threads:[~2017-06-09 22:21 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-06-09 15:22 [PATCH 0/5] CodeSamples: Cleanups and Makefile updates Akira Yokosawa
2017-06-09 15:24 ` [PATCH 1/5] CodeSamples: Remove gprof-helper.c Akira Yokosawa
2017-06-09 15:25 ` [PATCH 2/5] CodeSamples: Use $(MAKE) for recursive make Akira Yokosawa
2017-06-09 15:27 ` [PATCH 3/5] CodeSamples: Makefile: Remove linux/list.h Akira Yokosawa
2017-06-09 15:28 ` [PATCH 4/5] CodeSamples: Makefile: Add subdirectories Akira Yokosawa
2017-06-09 15:30 ` [PATCH 5/5] CodeSamples: Makefile: Distinguish arch-independent targets Akira Yokosawa
2017-06-09 22:21 ` Paul E. McKenney [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20170609222115.GO3721@linux.vnet.ibm.com \
--to=paulmck@linux.vnet.ibm.com \
--cc=akiyks@gmail.com \
--cc=perfbook@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox