Messages in this thread |  | | | Date | Tue, 13 May 2008 20:59:52 -0500 | | From | Paul Jackson <> | | Subject | Re: [RFC/PATCH 1/8]: CGroup Files: Add locking mode to cgroups control files | |
Andrew wrote:
> As Matt observed, this is just a poorly-named variable.
How about the following code for cgroup_file_write():
char buf[64]; /* avoid kmalloc() in small cases */
char *p; /* buf[] or kmalloc'd buffer */
...
if (nbytes < sizeof(buf)) {
p = buf;
} else {
p = kmalloc(nbytes + 1, GFP_KERNEL);
if (p == NULL)
return -ENOMEM;
}
Possible advantages of above code:
* Uses short names for local variables of limited scope.
* Doesn't set p until needed, so as:
1) to highlight the symmetry of its setting, to either buf[]
(small cases) or to a kmalloc'd buffer (large cases), and
2) to avoid implying that p needs to be set in the intervening
"..." code above.
* Comments variable declarations.
--
I won't rest till it's the best ...
Programmer, Linux Scalability
Paul Jackson <pj@sgi.com> 1.940.382.4214
|  |