From: Balbir Singh <balbir@linux.vnet.ibm.com>
To: Andrew Morton <akpm@linux-foundation.org>
Cc: YAMAMOTO Takashi <yamamoto@valinux.co.jp>,
Paul Menage <menage@google.com>,
linux-kernel@vger.kernel.org, linux-mm@kvack.org,
Balbir Singh <balbir@linux.vnet.ibm.com>,
KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>
Subject: [2/2] memrlimit fix usage of tmp as a parameter name
Date: Fri, 20 Jun 2008 20:31:52 +0530 [thread overview]
Message-ID: <20080620150152.16094.76790.sendpatchset@localhost.localdomain> (raw)
In-Reply-To: <20080620150132.16094.29151.sendpatchset@localhost.localdomain>
Fix the variable tmp being used in write_strategy. This patch replaces tmp
with val, the fact that it is an output parameter can be interpreted from
the pass by reference.
Signed-off-by: Balbir Singh <balbir@linux.vnet.ibm.com>
---
mm/memrlimitcgroup.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff -puN mm/memrlimitcgroup.c~memrlimit-cgroup-simple-cleanup mm/memrlimitcgroup.c
--- linux-2.6.26-rc5/mm/memrlimitcgroup.c~memrlimit-cgroup-simple-cleanup 2008-06-20 20:14:00.000000000 +0530
+++ linux-2.6.26-rc5-balbir/mm/memrlimitcgroup.c 2008-06-20 20:22:08.000000000 +0530
@@ -118,13 +118,13 @@ static u64 memrlimit_cgroup_read(struct
cft->private);
}
-static int memrlimit_cgroup_write_strategy(char *buf, unsigned long long *tmp)
+static int memrlimit_cgroup_write_strategy(char *buf, unsigned long long *val)
{
- *tmp = memparse(buf, &buf);
+ *val = memparse(buf, &buf);
if (*buf != '\0')
return -EINVAL;
- *tmp = PAGE_ALIGN(*tmp);
+ *val = PAGE_ALIGN(*val);
return 0;
}
_
--
Warm Regards,
Balbir Singh
Linux Technology Center
IBM, ISTL
WARNING: multiple messages have this Message-ID (diff)
From: Balbir Singh <balbir@linux.vnet.ibm.com>
To: Andrew Morton <akpm@linux-foundation.org>
Cc: YAMAMOTO Takashi <yamamoto@valinux.co.jp>,
Paul Menage <menage@google.com>,
linux-kernel@vger.kernel.org, linux-mm@kvack.org,
Balbir Singh <balbir@linux.vnet.ibm.com>,
KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>
Subject: [2/2] memrlimit fix usage of tmp as a parameter name
Date: Fri, 20 Jun 2008 20:31:52 +0530 [thread overview]
Message-ID: <20080620150152.16094.76790.sendpatchset@localhost.localdomain> (raw)
In-Reply-To: <20080620150132.16094.29151.sendpatchset@localhost.localdomain>
Fix the variable tmp being used in write_strategy. This patch replaces tmp
with val, the fact that it is an output parameter can be interpreted from
the pass by reference.
Signed-off-by: Balbir Singh <balbir@linux.vnet.ibm.com>
---
mm/memrlimitcgroup.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff -puN mm/memrlimitcgroup.c~memrlimit-cgroup-simple-cleanup mm/memrlimitcgroup.c
--- linux-2.6.26-rc5/mm/memrlimitcgroup.c~memrlimit-cgroup-simple-cleanup 2008-06-20 20:14:00.000000000 +0530
+++ linux-2.6.26-rc5-balbir/mm/memrlimitcgroup.c 2008-06-20 20:22:08.000000000 +0530
@@ -118,13 +118,13 @@ static u64 memrlimit_cgroup_read(struct
cft->private);
}
-static int memrlimit_cgroup_write_strategy(char *buf, unsigned long long *tmp)
+static int memrlimit_cgroup_write_strategy(char *buf, unsigned long long *val)
{
- *tmp = memparse(buf, &buf);
+ *val = memparse(buf, &buf);
if (*buf != '\0')
return -EINVAL;
- *tmp = PAGE_ALIGN(*tmp);
+ *val = PAGE_ALIGN(*val);
return 0;
}
_
--
Warm Regards,
Balbir Singh
Linux Technology Center
IBM, ISTL
--
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/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>
next prev parent reply other threads:[~2008-06-20 15:02 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-06-20 15:01 [0/2] memrlimit improve error handling Balbir Singh
2008-06-20 15:01 ` Balbir Singh
2008-06-20 15:01 ` [1/2] memrlimit handle attach_task() failure, add can_attach() callback Balbir Singh
2008-06-20 15:01 ` Balbir Singh
2008-06-25 23:37 ` Andrew Morton
2008-06-25 23:37 ` Andrew Morton
2008-06-25 23:52 ` Balbir Singh
2008-06-25 23:52 ` Balbir Singh
2008-06-20 15:01 ` Balbir Singh [this message]
2008-06-20 15:01 ` [2/2] memrlimit fix usage of tmp as a parameter name Balbir Singh
2008-06-25 23:41 ` Andrew Morton
2008-06-25 23:41 ` Andrew Morton
2008-06-25 23:51 ` Balbir Singh
2008-06-25 23:51 ` Balbir Singh
2008-06-24 23:11 ` [0/2] memrlimit improve error handling Hugh Dickins
2008-06-24 23:11 ` Hugh Dickins
2008-06-25 1:01 ` Balbir Singh
2008-06-25 1:01 ` Balbir Singh
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=20080620150152.16094.76790.sendpatchset@localhost.localdomain \
--to=balbir@linux.vnet.ibm.com \
--cc=akpm@linux-foundation.org \
--cc=kamezawa.hiroyu@jp.fujitsu.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=menage@google.com \
--cc=yamamoto@valinux.co.jp \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.