diff options
| author | Oleg Nesterov <oleg@redhat.com> | 2013-04-30 15:28:19 -0700 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2013-04-30 17:04:07 -0700 |
| commit | 12eaaf309a798973d215f7f21aa5a67a760ed7c8 (patch) | |
| tree | 6be40c0ec42f7cf8eb5d09c31e1f36cc21499720 /Documentation/IRQ.txt | |
| parent | 830e0fc967a7ee5013d5d1cf6a3cea71a8868466 (diff) | |
| download | linux-12eaaf309a798973d215f7f21aa5a67a760ed7c8.tar.bz2 | |
set_task_comm: kill the pointless memset() + wmb()
set_task_comm() does memset() + wmb() before strlcpy(). This buys
nothing and to add to the confusion, the comment is wrong.
- We do not need memset() to be "safe from non-terminating string
reads", the final char is always zero and we never change it.
- wmb() is paired with nothing, it cannot prevent from printing
the mixture of the old/new data unless the reader takes the lock.
Signed-off-by: Oleg Nesterov <oleg@redhat.com>
Cc: Andi Kleen <andi@firstfloor.org>
Cc: John Stultz <johnstul@us.ibm.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'Documentation/IRQ.txt')
0 files changed, 0 insertions, 0 deletions