linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] scripts/tags.sh: add regular expression replacement pattern for memcg
@ 2014-04-10  8:11 Jianyu Zhan
  0 siblings, 0 replies; 6+ messages in thread
From: Jianyu Zhan @ 2014-04-10  8:11 UTC (permalink / raw)
  To: mmarek; +Cc: akpm, michael.opdenacker, tkhai, linux-kernel, nasa4836

Currently, no regular expression replacement pattern for PageCgroug*
defined, which causes quite a few pain while reading code. Just get
rid of it.

Also, add a comment to inform people to add pattern for whatever new
flags they add.

Signed-off-by: Jianyu Zhan <nasa4836@gmail.com>
---
 include/linux/page_cgroup.h | 6 ++++++
 scripts/tags.sh             | 8 ++++++++
 2 files changed, 14 insertions(+)

diff --git a/include/linux/page_cgroup.h b/include/linux/page_cgroup.h
index 777a524..16b4f15 100644
--- a/include/linux/page_cgroup.h
+++ b/include/linux/page_cgroup.h
@@ -44,6 +44,12 @@ static inline void __init page_cgroup_init(void)
 struct page_cgroup *lookup_page_cgroup(struct page *page);
 struct page *lookup_cgroup_page(struct page_cgroup *pc);
 
+/*
+ * Note, whenever you add a new flag here, please add a
+ * regular expression replacement pattern in scripts/tags.sh
+ * to inform our friendly and smart tags generator.
+ */
+
 #define TESTPCGFLAG(uname, lname)			\
 static inline int PageCgroup##uname(struct page_cgroup *pc)	\
 	{ return test_bit(PCG_##lname, &pc->flags); }
diff --git a/scripts/tags.sh b/scripts/tags.sh
index 58c4559..c0d52d1 100755
--- a/scripts/tags.sh
+++ b/scripts/tags.sh
@@ -171,12 +171,16 @@ exuberant()
 	--regex-c++='/PAGEFLAG\(([^,)]*).*/ClearPage\1/'		\
 	--regex-c++='/TESTSETFLAG\(([^,)]*).*/TestSetPage\1/'		\
 	--regex-c++='/TESTPAGEFLAG\(([^,)]*).*/Page\1/'			\
+	--regex-c++='/TESTPCGFLAG\(([^,)]*).*/PageCgroup\1/'		\
 	--regex-c++='/SETPAGEFLAG\(([^,)]*).*/SetPage\1/'		\
 	--regex-c++='/__SETPAGEFLAG\(([^,)]*).*/__SetPage\1/'		\
+	--regex-c++='/SETPCGFLAG\(([^,)]*).*/SetPageCgroup\1/'		\
 	--regex-c++='/TESTCLEARFLAG\(([^,)]*).*/TestClearPage\1/'	\
 	--regex-c++='/__TESTCLEARFLAG\(([^,)]*).*/TestClearPage\1/'	\
+	--regex-c++='/TESTCLEARPCGFLAG\(([^,)]*).*/TestClearPageCgroup\1/' \
 	--regex-c++='/CLEARPAGEFLAG\(([^,)]*).*/ClearPage\1/'		\
 	--regex-c++='/__CLEARPAGEFLAG\(([^,)]*).*/__ClearPage\1/'	\
+	--regex-c++='/CLEARPCGFLAG\(([^,)]*).*/ClearPageCgroup\1/'	\
 	--regex-c++='/__PAGEFLAG\(([^,)]*).*/__SetPage\1/'		\
 	--regex-c++='/__PAGEFLAG\(([^,)]*).*/__ClearPage\1/'		\
 	--regex-c++='/PAGEFLAG_FALSE\(([^,)]*).*/Page\1/'		\
@@ -229,12 +233,16 @@ emacs()
 	--regex='/PAGEFLAG(\([^,)]*\).*/ClearPage\1/'		\
 	--regex='/TESTSETFLAG(\([^,)]*\).*/TestSetPage\1/'	\
 	--regex='/TESTPAGEFLAG(\([^,)]*\).*/Page\1/'		\
+	--regex='/TESTPCGFLAG\(([^,)]*).*/PageCgroup\1/'	\
 	--regex='/SETPAGEFLAG(\([^,)]*\).*/SetPage\1/'		\
 	--regex='/__SETPAGEFLAG(\([^,)]*\).*/__SetPage\1/'	\
+	--regex='/SETPCGFLAG\(([^,)]*).*/SetPageCgroup\1/'	\
 	--regex='/TESTCLEARFLAG(\([^,)]*\).*/TestClearPage\1/'	\
 	--regex='/__TESTCLEARFLAG(\([^,)]*\).*/TestClearPage\1/'	\
+	--regex='/TESTCLEARPCGFLAG\(([^,)]*).*/TestClearPageCgroup\1/' \
 	--regex='/CLEARPAGEFLAG(\([^,)]*\).*/ClearPage\1/'	\
 	--regex='/__CLEARPAGEFLAG(\([^,)]*\).*/__ClearPage\1/'	\
+	--regex='/CLEARPCGFLAG\(([^,)]*).*/ClearPageCgroup\1/'	\
 	--regex='/__PAGEFLAG(\([^,)]*\).*/__SetPage\1/'		\
 	--regex='/__PAGEFLAG(\([^,)]*\).*/__ClearPage\1/'	\
 	--regex='/PAGEFLAG_FALSE(\([^,)]*\).*/Page\1/'		\
-- 
1.9.0.GIT


^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [PATCH] scripts/tags.sh: add regular expression replacement pattern for memcg
  2014-05-13 13:26 Jianyu Zhan
@ 2014-05-14 11:50 ` Michal Marek
  0 siblings, 0 replies; 6+ messages in thread
From: Michal Marek @ 2014-05-14 11:50 UTC (permalink / raw)
  To: Jianyu Zhan; +Cc: akpm, prarit, michael.opdenacker, tkhai, linux-kernel

On 2014-05-13 15:26, Jianyu Zhan wrote:
> Currently, while using ctags to read code, we would get stumbled on
> PageCgroup* symbols: no definition found. And it is quite dull to
> manually dig it out.
> 
> This patch adds regular expression replacement pattern for such symbols,
> like what have done for the PageXXX flag. It will teach ctags to find
> out the definition for us.
> 
> Signed-off-by: Jianyu Zhan <nasa4836@gmail.com>

Applied to kbuild.git#misc, thanks.

Michal

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] scripts/tags.sh: add regular expression replacement pattern for memcg
@ 2014-05-13 13:26 Jianyu Zhan
  2014-05-14 11:50 ` Michal Marek
  0 siblings, 1 reply; 6+ messages in thread
From: Jianyu Zhan @ 2014-05-13 13:26 UTC (permalink / raw)
  To: mmarek, akpm, prarit, michael.opdenacker, tkhai, nasa4836; +Cc: linux-kernel

Hi, Prarit,

>> Currently, no regular expression replacement pattern for PageCgroug*
>
>^^^ *Cgroup*
 
  Sorry for the typo, fixed. Thanks.

>> defined, which causes quite a few pain while reading code. Just get 
>> rid of it. 
>>
>
>IMO the patch looks fine but can you fixup your changelog a little bit?  I think
>the new "standard" has become an explanation and example of the problem, an
>explanation of what is wrong, and what you did to fix it (including an example
>of output if possible).  That way we know what was changed and why :)

 Yep, you are right. Rewrote the wording.
 Would you mind signing your Reviewed-by if it is Ok? Thanks.

---<8---
Currently, while using ctags to read code, we would get stumbled on
PageCgroup* symbols: no definition found. And it is quite dull to
manually dig it out.

This patch adds regular expression replacement pattern for such symbols,
like what have done for the PageXXX flag. It will teach ctags to find
out the definition for us.

Signed-off-by: Jianyu Zhan <nasa4836@gmail.com>
---
 scripts/tags.sh | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/scripts/tags.sh b/scripts/tags.sh
index f2c5b00..7d3ebfe 100755
--- a/scripts/tags.sh
+++ b/scripts/tags.sh
@@ -187,6 +187,10 @@ exuberant()
 	--regex-c++='/TESTCLEARFLAG_FALSE\(([^,)]*).*/TestClearPage\1/' \
 	--regex-c++='/__TESTCLEARFLAG_FALSE\(([^,)]*).*/__TestClearPage\1/' \
 	--regex-c++='/_PE\(([^,)]*).*/PEVENT_ERRNO__\1/'		\
+	--regex-c++='/TESTPCGFLAG\(([^,)]*).*/PageCgroup\1/'		\
+	--regex-c++='/SETPCGFLAG\(([^,)]*).*/SetPageCgroup\1/'		\
+	--regex-c++='/CLEARPCGFLAG\(([^,)]*).*/ClearPageCgroup\1/'	\
+	--regex-c++='/TESTCLEARPCGFLAG\(([^,)]*).*/TestClearPageCgroup\1/' \
 	--regex-c='/PCI_OP_READ\((\w*).*[1-4]\)/pci_bus_read_config_\1/' \
 	--regex-c='/PCI_OP_WRITE\((\w*).*[1-4]\)/pci_bus_write_config_\1/' \
 	--regex-c='/DEFINE_(MUTEX|SEMAPHORE|SPINLOCK)\((\w*)/\2/v/'	\
@@ -244,6 +248,10 @@ emacs()
 	--regex='/__CLEARPAGEFLAG_NOOP(\([^,)]*\).*/__ClearPage\1/' \
 	--regex='/TESTCLEARFLAG_FALSE(\([^,)]*\).*/TestClearPage\1/' \
 	--regex='/__TESTCLEARFLAG_FALSE(\([^,)]*\).*/__TestClearPage\1/' \
+	--regex='/TESTPCGFLAG\(([^,)]*).*/PageCgroup\1/'	\
+	--regex='/SETPCGFLAG\(([^,)]*).*/SetPageCgroup\1/'	\
+	--regex='/CLEARPCGFLAG\(([^,)]*).*/ClearPageCgroup\1/'	\
+	--regex='/TESTCLEARPCGFLAG\(([^,)]*).*/TestClearPageCgroup\1/' \
 	--regex='/_PE(\([^,)]*\).*/PEVENT_ERRNO__\1/'		\
 	--regex='/PCI_OP_READ(\([a-z]*[a-z]\).*[1-4])/pci_bus_read_config_\1/' \
 	--regex='/PCI_OP_WRITE(\([a-z]*[a-z]\).*[1-4])/pci_bus_write_config_\1/'
-- 
2.0.0-rc1


^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [PATCH] scripts/tags.sh: add regular expression replacement pattern for memcg
@ 2014-04-17 13:37 Jianyu Zhan
  0 siblings, 0 replies; 6+ messages in thread
From: Jianyu Zhan @ 2014-04-17 13:37 UTC (permalink / raw)
  To: mmarek, akpm, prarit, michael.opdenacker, tkhai, nasa4836; +Cc: linux-kernel

Hi, Marek.

> It does not look like new macros are being defined here on a weekly basis...

  Hmm, I think so. I've removed this comment.

> Please do not interleave the new defines with the <linux/page-flags.h>
> defines.

  Ok.

Here is the renewed patch:
---


Currently, no regular expression replacement pattern for PageCgroug*
defined, which causes quite a few pain while reading code. Just get
rid of it.

Signed-off-by: Jianyu Zhan <nasa4836@gmail.com>
---
 scripts/tags.sh | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/scripts/tags.sh b/scripts/tags.sh
index f2c5b00..7d3ebfe 100755
--- a/scripts/tags.sh
+++ b/scripts/tags.sh
@@ -187,6 +187,10 @@ exuberant()
 	--regex-c++='/TESTCLEARFLAG_FALSE\(([^,)]*).*/TestClearPage\1/' \
 	--regex-c++='/__TESTCLEARFLAG_FALSE\(([^,)]*).*/__TestClearPage\1/' \
 	--regex-c++='/_PE\(([^,)]*).*/PEVENT_ERRNO__\1/'		\
+	--regex-c++='/TESTPCGFLAG\(([^,)]*).*/PageCgroup\1/'		\
+	--regex-c++='/SETPCGFLAG\(([^,)]*).*/SetPageCgroup\1/'		\
+	--regex-c++='/CLEARPCGFLAG\(([^,)]*).*/ClearPageCgroup\1/'	\
+	--regex-c++='/TESTCLEARPCGFLAG\(([^,)]*).*/TestClearPageCgroup\1/' \
 	--regex-c='/PCI_OP_READ\((\w*).*[1-4]\)/pci_bus_read_config_\1/' \
 	--regex-c='/PCI_OP_WRITE\((\w*).*[1-4]\)/pci_bus_write_config_\1/' \
 	--regex-c='/DEFINE_(MUTEX|SEMAPHORE|SPINLOCK)\((\w*)/\2/v/'	\
@@ -244,6 +248,10 @@ emacs()
 	--regex='/__CLEARPAGEFLAG_NOOP(\([^,)]*\).*/__ClearPage\1/' \
 	--regex='/TESTCLEARFLAG_FALSE(\([^,)]*\).*/TestClearPage\1/' \
 	--regex='/__TESTCLEARFLAG_FALSE(\([^,)]*\).*/__TestClearPage\1/' \
+	--regex='/TESTPCGFLAG\(([^,)]*).*/PageCgroup\1/'	\
+	--regex='/SETPCGFLAG\(([^,)]*).*/SetPageCgroup\1/'	\
+	--regex='/CLEARPCGFLAG\(([^,)]*).*/ClearPageCgroup\1/'	\
+	--regex='/TESTCLEARPCGFLAG\(([^,)]*).*/TestClearPageCgroup\1/' \
 	--regex='/_PE(\([^,)]*\).*/PEVENT_ERRNO__\1/'		\
 	--regex='/PCI_OP_READ(\([a-z]*[a-z]\).*[1-4])/pci_bus_read_config_\1/' \
 	--regex='/PCI_OP_WRITE(\([a-z]*[a-z]\).*[1-4])/pci_bus_write_config_\1/'
-- 
1.9.0.GIT


^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [PATCH] scripts/tags.sh: add regular expression replacement pattern for memcg
  2014-04-10 12:58 Jianyu Zhan
@ 2014-04-16 21:24 ` Michal Marek
  0 siblings, 0 replies; 6+ messages in thread
From: Michal Marek @ 2014-04-16 21:24 UTC (permalink / raw)
  To: Jianyu Zhan; +Cc: akpm, michael.opdenacker, tkhai, linux-kernel, cgroups

Dne 10.4.2014 14:58, Jianyu Zhan napsal(a):
> Sorry, the comment in previous patch is somewhat misguiding, this patch
> replaces that one.
>  
> Currently, no regular expression replacement patterns for PageCgroup*
> flag-ops are defined, which causes quite a few pain while reading code. 
> Just get rid of it.
> 
> Also, add a comment to inform people to add pattern for whatever new
> flag-ops they add.
> 
> Signed-off-by: Jianyu Zhan <nasa4836@gmail.com>
> ---
>  include/linux/page_cgroup.h | 6 ++++++
>  scripts/tags.sh             | 8 ++++++++
>  2 files changed, 14 insertions(+)
> 
> diff --git a/include/linux/page_cgroup.h b/include/linux/page_cgroup.h
> index 777a524..9c60b77 100644
> --- a/include/linux/page_cgroup.h
> +++ b/include/linux/page_cgroup.h
> @@ -44,6 +44,12 @@ static inline void __init page_cgroup_init(void)
>  struct page_cgroup *lookup_page_cgroup(struct page *page);
>  struct page *lookup_cgroup_page(struct page_cgroup *pc);
>  
> +/*
> + * Note, whenever you add a *PCGFLAG operation here, please add a
> + * regular expression replacement pattern in scripts/tags.sh
> + * to inform our friendly and smart tags generator.
> + */
> +
>  #define TESTPCGFLAG(uname, lname)			\

Cgroup hackers - OK to add the above comment? It does not look like new
macros are being defined here on a weekly basis...


>  static inline int PageCgroup##uname(struct page_cgroup *pc)	\
>  	{ return test_bit(PCG_##lname, &pc->flags); }
> diff --git a/scripts/tags.sh b/scripts/tags.sh
> index 58c4559..c0d52d1 100755
> --- a/scripts/tags.sh
> +++ b/scripts/tags.sh
> @@ -171,12 +171,16 @@ exuberant()
>  	--regex-c++='/PAGEFLAG\(([^,)]*).*/ClearPage\1/'		\
>  	--regex-c++='/TESTSETFLAG\(([^,)]*).*/TestSetPage\1/'		\
>  	--regex-c++='/TESTPAGEFLAG\(([^,)]*).*/Page\1/'			\
> +	--regex-c++='/TESTPCGFLAG\(([^,)]*).*/PageCgroup\1/'		\
>  	--regex-c++='/SETPAGEFLAG\(([^,)]*).*/SetPage\1/'		\
>  	--regex-c++='/__SETPAGEFLAG\(([^,)]*).*/__SetPage\1/'		\
> +	--regex-c++='/SETPCGFLAG\(([^,)]*).*/SetPageCgroup\1/'		\
>  	--regex-c++='/TESTCLEARFLAG\(([^,)]*).*/TestClearPage\1/'	\
>  	--regex-c++='/__TESTCLEARFLAG\(([^,)]*).*/TestClearPage\1/'	\
> +	--regex-c++='/TESTCLEARPCGFLAG\(([^,)]*).*/TestClearPageCgroup\1/' \
>  	--regex-c++='/CLEARPAGEFLAG\(([^,)]*).*/ClearPage\1/'		\
>  	--regex-c++='/__CLEARPAGEFLAG\(([^,)]*).*/__ClearPage\1/'	\
> +	--regex-c++='/CLEARPCGFLAG\(([^,)]*).*/ClearPageCgroup\1/'	\
>  	--regex-c++='/__PAGEFLAG\(([^,)]*).*/__SetPage\1/'		\
>  	--regex-c++='/__PAGEFLAG\(([^,)]*).*/__ClearPage\1/'		\
>  	--regex-c++='/PAGEFLAG_FALSE\(([^,)]*).*/Page\1/'		\

Please do not interleave the new defines with the <linux/page-flags.h>
defines.

Michal

> @@ -229,12 +233,16 @@ emacs()
>  	--regex='/PAGEFLAG(\([^,)]*\).*/ClearPage\1/'		\
>  	--regex='/TESTSETFLAG(\([^,)]*\).*/TestSetPage\1/'	\
>  	--regex='/TESTPAGEFLAG(\([^,)]*\).*/Page\1/'		\
> +	--regex='/TESTPCGFLAG\(([^,)]*).*/PageCgroup\1/'	\
>  	--regex='/SETPAGEFLAG(\([^,)]*\).*/SetPage\1/'		\
>  	--regex='/__SETPAGEFLAG(\([^,)]*\).*/__SetPage\1/'	\
> +	--regex='/SETPCGFLAG\(([^,)]*).*/SetPageCgroup\1/'	\
>  	--regex='/TESTCLEARFLAG(\([^,)]*\).*/TestClearPage\1/'	\
>  	--regex='/__TESTCLEARFLAG(\([^,)]*\).*/TestClearPage\1/'	\
> +	--regex='/TESTCLEARPCGFLAG\(([^,)]*).*/TestClearPageCgroup\1/' \
>  	--regex='/CLEARPAGEFLAG(\([^,)]*\).*/ClearPage\1/'	\
>  	--regex='/__CLEARPAGEFLAG(\([^,)]*\).*/__ClearPage\1/'	\
> +	--regex='/CLEARPCGFLAG\(([^,)]*).*/ClearPageCgroup\1/'	\
>  	--regex='/__PAGEFLAG(\([^,)]*\).*/__SetPage\1/'		\
>  	--regex='/__PAGEFLAG(\([^,)]*\).*/__ClearPage\1/'	\
>  	--regex='/PAGEFLAG_FALSE(\([^,)]*\).*/Page\1/'		\
> 


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] scripts/tags.sh: add regular expression replacement pattern for memcg
@ 2014-04-10 12:58 Jianyu Zhan
  2014-04-16 21:24 ` Michal Marek
  0 siblings, 1 reply; 6+ messages in thread
From: Jianyu Zhan @ 2014-04-10 12:58 UTC (permalink / raw)
  To: mmarek; +Cc: akpm, michael.opdenacker, tkhai, linux-kernel, nasa4836

Sorry, the comment in previous patch is somewhat misguiding, this patch
replaces that one.
 
Currently, no regular expression replacement patterns for PageCgroup*
flag-ops are defined, which causes quite a few pain while reading code. 
Just get rid of it.

Also, add a comment to inform people to add pattern for whatever new
flag-ops they add.

Signed-off-by: Jianyu Zhan <nasa4836@gmail.com>
---
 include/linux/page_cgroup.h | 6 ++++++
 scripts/tags.sh             | 8 ++++++++
 2 files changed, 14 insertions(+)

diff --git a/include/linux/page_cgroup.h b/include/linux/page_cgroup.h
index 777a524..9c60b77 100644
--- a/include/linux/page_cgroup.h
+++ b/include/linux/page_cgroup.h
@@ -44,6 +44,12 @@ static inline void __init page_cgroup_init(void)
 struct page_cgroup *lookup_page_cgroup(struct page *page);
 struct page *lookup_cgroup_page(struct page_cgroup *pc);
 
+/*
+ * Note, whenever you add a *PCGFLAG operation here, please add a
+ * regular expression replacement pattern in scripts/tags.sh
+ * to inform our friendly and smart tags generator.
+ */
+
 #define TESTPCGFLAG(uname, lname)			\
 static inline int PageCgroup##uname(struct page_cgroup *pc)	\
 	{ return test_bit(PCG_##lname, &pc->flags); }
diff --git a/scripts/tags.sh b/scripts/tags.sh
index 58c4559..c0d52d1 100755
--- a/scripts/tags.sh
+++ b/scripts/tags.sh
@@ -171,12 +171,16 @@ exuberant()
 	--regex-c++='/PAGEFLAG\(([^,)]*).*/ClearPage\1/'		\
 	--regex-c++='/TESTSETFLAG\(([^,)]*).*/TestSetPage\1/'		\
 	--regex-c++='/TESTPAGEFLAG\(([^,)]*).*/Page\1/'			\
+	--regex-c++='/TESTPCGFLAG\(([^,)]*).*/PageCgroup\1/'		\
 	--regex-c++='/SETPAGEFLAG\(([^,)]*).*/SetPage\1/'		\
 	--regex-c++='/__SETPAGEFLAG\(([^,)]*).*/__SetPage\1/'		\
+	--regex-c++='/SETPCGFLAG\(([^,)]*).*/SetPageCgroup\1/'		\
 	--regex-c++='/TESTCLEARFLAG\(([^,)]*).*/TestClearPage\1/'	\
 	--regex-c++='/__TESTCLEARFLAG\(([^,)]*).*/TestClearPage\1/'	\
+	--regex-c++='/TESTCLEARPCGFLAG\(([^,)]*).*/TestClearPageCgroup\1/' \
 	--regex-c++='/CLEARPAGEFLAG\(([^,)]*).*/ClearPage\1/'		\
 	--regex-c++='/__CLEARPAGEFLAG\(([^,)]*).*/__ClearPage\1/'	\
+	--regex-c++='/CLEARPCGFLAG\(([^,)]*).*/ClearPageCgroup\1/'	\
 	--regex-c++='/__PAGEFLAG\(([^,)]*).*/__SetPage\1/'		\
 	--regex-c++='/__PAGEFLAG\(([^,)]*).*/__ClearPage\1/'		\
 	--regex-c++='/PAGEFLAG_FALSE\(([^,)]*).*/Page\1/'		\
@@ -229,12 +233,16 @@ emacs()
 	--regex='/PAGEFLAG(\([^,)]*\).*/ClearPage\1/'		\
 	--regex='/TESTSETFLAG(\([^,)]*\).*/TestSetPage\1/'	\
 	--regex='/TESTPAGEFLAG(\([^,)]*\).*/Page\1/'		\
+	--regex='/TESTPCGFLAG\(([^,)]*).*/PageCgroup\1/'	\
 	--regex='/SETPAGEFLAG(\([^,)]*\).*/SetPage\1/'		\
 	--regex='/__SETPAGEFLAG(\([^,)]*\).*/__SetPage\1/'	\
+	--regex='/SETPCGFLAG\(([^,)]*).*/SetPageCgroup\1/'	\
 	--regex='/TESTCLEARFLAG(\([^,)]*\).*/TestClearPage\1/'	\
 	--regex='/__TESTCLEARFLAG(\([^,)]*\).*/TestClearPage\1/'	\
+	--regex='/TESTCLEARPCGFLAG\(([^,)]*).*/TestClearPageCgroup\1/' \
 	--regex='/CLEARPAGEFLAG(\([^,)]*\).*/ClearPage\1/'	\
 	--regex='/__CLEARPAGEFLAG(\([^,)]*\).*/__ClearPage\1/'	\
+	--regex='/CLEARPCGFLAG\(([^,)]*).*/ClearPageCgroup\1/'	\
 	--regex='/__PAGEFLAG(\([^,)]*\).*/__SetPage\1/'		\
 	--regex='/__PAGEFLAG(\([^,)]*\).*/__ClearPage\1/'	\
 	--regex='/PAGEFLAG_FALSE(\([^,)]*\).*/Page\1/'		\
-- 
1.9.0.GIT


^ permalink raw reply related	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2014-05-14 11:50 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-04-10  8:11 [PATCH] scripts/tags.sh: add regular expression replacement pattern for memcg Jianyu Zhan
2014-04-10 12:58 Jianyu Zhan
2014-04-16 21:24 ` Michal Marek
2014-04-17 13:37 Jianyu Zhan
2014-05-13 13:26 Jianyu Zhan
2014-05-14 11:50 ` Michal Marek

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).