* [PATCH] Handling of unused variable 'do-numainfo on compilation time
@ 2012-02-03 8:43 Geunsik Lim
2012-02-03 13:39 ` Johannes Weiner
0 siblings, 1 reply; 6+ messages in thread
From: Geunsik Lim @ 2012-02-03 8:43 UTC (permalink / raw)
To: Johannes Weiner, KAMEZAWA Hiroyuki
Cc: Andrew Morton, linux-kernel, Daisuke Nishimura, linux-mm
Actually, Usage of the variable 'do_numainfo'is not suitable for gcc compiler.
Declare the variable 'do_numainfo' if the number of NUMA nodes > 1.
Signed-off-by: Geunsik Lim <geunsik.lim@samsung.com>
---
mm/memcontrol.c | 5 ++++-
1 files changed, 4 insertions(+), 1 deletions(-)
diff --git a/mm/memcontrol.c b/mm/memcontrol.c
index 556859f..4e17ac5 100644
--- a/mm/memcontrol.c
+++ b/mm/memcontrol.c
@@ -776,7 +776,10 @@ static void memcg_check_events(struct mem_cgroup *memcg, struct page *page)
/* threshold event is triggered in finer grain than soft limit */
if (unlikely(mem_cgroup_event_ratelimit(memcg,
MEM_CGROUP_TARGET_THRESH))) {
- bool do_softlimit, do_numainfo;
+ bool do_softlimit;
+#if MAX_NUMNODES > 1
+ bool do_numainfo;
+#endif
do_softlimit = mem_cgroup_event_ratelimit(memcg,
MEM_CGROUP_TARGET_SOFTLIMIT);
--
1.7.8.1
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org. For more info on Linux MM,
see: http://www.linux-mm.org/ .
Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] Handling of unused variable 'do-numainfo on compilation time
2012-02-03 8:43 [PATCH] Handling of unused variable 'do-numainfo on compilation time Geunsik Lim
@ 2012-02-03 13:39 ` Johannes Weiner
2012-02-03 14:53 ` Michal Hocko
0 siblings, 1 reply; 6+ messages in thread
From: Johannes Weiner @ 2012-02-03 13:39 UTC (permalink / raw)
To: Michal Hocko
Cc: KAMEZAWA Hiroyuki, Andrew Morton, linux-kernel, Daisuke Nishimura,
Geunsik Lim, linux-mm
Michal, this keeps coming up, please decide between the proposed
solutions ;-)
On Fri, Feb 03, 2012 at 05:43:47PM +0900, Geunsik Lim wrote:
> Actually, Usage of the variable 'do_numainfo'is not suitable for gcc compiler.
> Declare the variable 'do_numainfo' if the number of NUMA nodes > 1.
>
> Signed-off-by: Geunsik Lim <geunsik.lim@samsung.com>
> ---
> mm/memcontrol.c | 5 ++++-
> 1 files changed, 4 insertions(+), 1 deletions(-)
>
> diff --git a/mm/memcontrol.c b/mm/memcontrol.c
> index 556859f..4e17ac5 100644
> --- a/mm/memcontrol.c
> +++ b/mm/memcontrol.c
> @@ -776,7 +776,10 @@ static void memcg_check_events(struct mem_cgroup *memcg, struct page *page)
> /* threshold event is triggered in finer grain than soft limit */
> if (unlikely(mem_cgroup_event_ratelimit(memcg,
> MEM_CGROUP_TARGET_THRESH))) {
> - bool do_softlimit, do_numainfo;
> + bool do_softlimit;
> +#if MAX_NUMNODES > 1
> + bool do_numainfo;
> +#endif
>
> do_softlimit = mem_cgroup_event_ratelimit(memcg,
> MEM_CGROUP_TARGET_SOFTLIMIT);
> --
> 1.7.8.1
>
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org. For more info on Linux MM,
see: http://www.linux-mm.org/ .
Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] Handling of unused variable 'do-numainfo on compilation time
2012-02-03 13:39 ` Johannes Weiner
@ 2012-02-03 14:53 ` Michal Hocko
2012-02-03 15:36 ` Geunsik Lim
2012-02-03 16:06 ` Johannes Weiner
0 siblings, 2 replies; 6+ messages in thread
From: Michal Hocko @ 2012-02-03 14:53 UTC (permalink / raw)
To: Johannes Weiner
Cc: KAMEZAWA Hiroyuki, Andrew Morton, linux-kernel, Daisuke Nishimura,
Geunsik Lim, linux-mm
On Fri 03-02-12 14:39:50, Johannes Weiner wrote:
> Michal, this keeps coming up, please decide between the proposed
> solutions ;-)
Hmm, I thought we already sorted this out https://lkml.org/lkml/2012/1/26/25 ?
>
> On Fri, Feb 03, 2012 at 05:43:47PM +0900, Geunsik Lim wrote:
> > Actually, Usage of the variable 'do_numainfo'is not suitable for gcc compiler.
> > Declare the variable 'do_numainfo' if the number of NUMA nodes > 1.
> >
> > Signed-off-by: Geunsik Lim <geunsik.lim@samsung.com>
> > ---
> > mm/memcontrol.c | 5 ++++-
> > 1 files changed, 4 insertions(+), 1 deletions(-)
> >
> > diff --git a/mm/memcontrol.c b/mm/memcontrol.c
> > index 556859f..4e17ac5 100644
> > --- a/mm/memcontrol.c
> > +++ b/mm/memcontrol.c
> > @@ -776,7 +776,10 @@ static void memcg_check_events(struct mem_cgroup *memcg, struct page *page)
> > /* threshold event is triggered in finer grain than soft limit */
> > if (unlikely(mem_cgroup_event_ratelimit(memcg,
> > MEM_CGROUP_TARGET_THRESH))) {
> > - bool do_softlimit, do_numainfo;
> > + bool do_softlimit;
> > +#if MAX_NUMNODES > 1
> > + bool do_numainfo;
> > +#endif
> >
> > do_softlimit = mem_cgroup_event_ratelimit(memcg,
> > MEM_CGROUP_TARGET_SOFTLIMIT);
> > --
> > 1.7.8.1
> >
--
Michal Hocko
SUSE Labs
SUSE LINUX s.r.o.
Lihovarska 1060/12
190 00 Praha 9
Czech Republic
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org. For more info on Linux MM,
see: http://www.linux-mm.org/ .
Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] Handling of unused variable 'do-numainfo on compilation time
2012-02-03 14:53 ` Michal Hocko
@ 2012-02-03 15:36 ` Geunsik Lim
2012-02-03 16:04 ` Michal Hocko
2012-02-03 16:06 ` Johannes Weiner
1 sibling, 1 reply; 6+ messages in thread
From: Geunsik Lim @ 2012-02-03 15:36 UTC (permalink / raw)
To: Michal Hocko
Cc: Johannes Weiner, KAMEZAWA Hiroyuki, Andrew Morton, linux-kernel,
Daisuke Nishimura, linux-mm
[-- Attachment #1: Type: text/plain, Size: 2031 bytes --]
On Fri, Feb 3, 2012 at 11:53 PM, Michal Hocko <mhocko@suse.cz> wrote:
> On Fri 03-02-12 14:39:50, Johannes Weiner wrote:
> > Michal, this keeps coming up, please decide between the proposed
> > solutions ;-)
>
> Hmm, I thought we already sorted this out
> https://lkml.org/lkml/2012/1/26/25 ?
>
> I don't know previous history about this variable.
Is it same? Please, adjust this patch or fix the unsuitable
variable 'do_numainfo' as I mentioned.
> >
> > On Fri, Feb 03, 2012 at 05:43:47PM +0900, Geunsik Lim wrote:
> > > Actually, Usage of the variable 'do_numainfo'is not suitable for gcc
> compiler.
> > > Declare the variable 'do_numainfo' if the number of NUMA nodes > 1.
> > >
> > > Signed-off-by: Geunsik Lim <geunsik.lim@samsung.com>
> > > ---
> > > mm/memcontrol.c | 5 ++++-
> > > 1 files changed, 4 insertions(+), 1 deletions(-)
> > >
> > > diff --git a/mm/memcontrol.c b/mm/memcontrol.c
> > > index 556859f..4e17ac5 100644
> > > --- a/mm/memcontrol.c
> > > +++ b/mm/memcontrol.c
> > > @@ -776,7 +776,10 @@ static void memcg_check_events(struct mem_cgroup
> *memcg, struct page *page)
> > > /* threshold event is triggered in finer grain than soft limit */
> > > if (unlikely(mem_cgroup_event_ratelimit(memcg,
> > >
> MEM_CGROUP_TARGET_THRESH))) {
> > > - bool do_softlimit, do_numainfo;
> > > + bool do_softlimit;
> > > +#if MAX_NUMNODES > 1
> > > + bool do_numainfo;
> > > +#endif
> > >
> > > do_softlimit = mem_cgroup_event_ratelimit(memcg,
> > >
> MEM_CGROUP_TARGET_SOFTLIMIT);
> > > --
> > > 1.7.8.1
> > >
>
> --
> Michal Hocko
> SUSE Labs
> SUSE LINUX s.r.o.
> Lihovarska 1060/12
> 190 00 Praha 9
> Czech Republic
>
--
----
Best regards,
Geunsik Lim, Samsung Electronics
http://leemgs.fedorapeople.org
----
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[-- Attachment #2: Type: text/html, Size: 3589 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] Handling of unused variable 'do-numainfo on compilation time
2012-02-03 15:36 ` Geunsik Lim
@ 2012-02-03 16:04 ` Michal Hocko
0 siblings, 0 replies; 6+ messages in thread
From: Michal Hocko @ 2012-02-03 16:04 UTC (permalink / raw)
To: Geunsik Lim, Andrew Morton
Cc: Johannes Weiner, KAMEZAWA Hiroyuki, linux-kernel,
Daisuke Nishimura, linux-mm
On Sat 04-02-12 00:36:36, Geunsik Lim wrote:
> On Fri, Feb 3, 2012 at 11:53 PM, Michal Hocko <mhocko@suse.cz> wrote:
>
> > On Fri 03-02-12 14:39:50, Johannes Weiner wrote:
> > > Michal, this keeps coming up, please decide between the proposed
> > > solutions ;-)
> >
> > Hmm, I thought we already sorted this out
> > https://lkml.org/lkml/2012/1/26/25 ?
> >
> I don't know previous history about this variable.
> Is it same? Please, adjust this patch or fix the unsuitable
> variable 'do_numainfo' as I mentioned.
The patch (I guess the author is Andrew) just silence the compiler
warning which is the easiest fix in this case because we know it will be
used only for MAX_NUMNODES > 1.
Your patch fixes it as well but it adds an ugly ifdef around the
variable.
Andrew, could you pick up this one, please?
---
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] Handling of unused variable 'do-numainfo on compilation time
2012-02-03 14:53 ` Michal Hocko
2012-02-03 15:36 ` Geunsik Lim
@ 2012-02-03 16:06 ` Johannes Weiner
1 sibling, 0 replies; 6+ messages in thread
From: Johannes Weiner @ 2012-02-03 16:06 UTC (permalink / raw)
To: Michal Hocko
Cc: KAMEZAWA Hiroyuki, Andrew Morton, linux-kernel, Daisuke Nishimura,
Geunsik Lim, linux-mm
On Fri, Feb 03, 2012 at 03:53:04PM +0100, Michal Hocko wrote:
> On Fri 03-02-12 14:39:50, Johannes Weiner wrote:
> > Michal, this keeps coming up, please decide between the proposed
> > solutions ;-)
>
> Hmm, I thought we already sorted this out https://lkml.org/lkml/2012/1/26/25 ?
Hah. Sorry, I missed that.
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org. For more info on Linux MM,
see: http://www.linux-mm.org/ .
Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2012-02-03 16:06 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-02-03 8:43 [PATCH] Handling of unused variable 'do-numainfo on compilation time Geunsik Lim
2012-02-03 13:39 ` Johannes Weiner
2012-02-03 14:53 ` Michal Hocko
2012-02-03 15:36 ` Geunsik Lim
2012-02-03 16:04 ` Michal Hocko
2012-02-03 16:06 ` Johannes Weiner
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).