Messages in this thread | | | Date | Sat, 28 Nov 2009 08:28:39 +0100 | From | Ingo Molnar <> | Subject | Re: [PATCH v3 00/27] writable limits |
| |
* Jiri Slaby <jirislaby@gmail.com> wrote:
> Hi, > > I broke the threading to not mess up with the long thread. > > In this version I got rid of the rlim access_only ugliness.
Thanks Jiri - this series looks very clean already and the helpers make the various usage sites easier to understand as well.
One (very small) naming suggestion, please rename:
rlim_get_cur() rlim_get_max() task_rlim_get_cur() task_rlim_get_max()
To a more natural sounding scheme like:
rlimit() rlimit_max() task_rlimit() task_rlimit_max()
Reasons:
- 'cur' is a misnomer that came a decade+ ago from an ABI and there's no need to continue that bad tradition in helper functions. It can also be confused with 'curr' which we often use for the current task.
- 'rlimit' is general meme, plus there's no need to say it out aloud that we 'get' it - rlimit() is obvious enough.
Beyond solving these issues it makes it shorter and more logical as well.
Thanks,
Ingo
| |