diff options
| author | Matt Helsley <matthltc@us.ibm.com> | 2006-10-02 02:18:25 -0700 |
|---|---|---|
| committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-10-02 07:57:22 -0700 |
| commit | 2453a3062d36f39f01302f9f1ad18e7a0c54fe38 (patch) | |
| tree | f684fbca3edfdcaeca3618a0c518bcf39418c9fc | |
| parent | 5d124e99c2fee1c8f3020ecb0dff8d5617ee7991 (diff) | |
| download | lwn-2453a3062d36f39f01302f9f1ad18e7a0c54fe38.tar.gz lwn-2453a3062d36f39f01302f9f1ad18e7a0c54fe38.zip | |
[PATCH] ipc: replace kmalloc and memset in get_undo_list with kzalloc
Simplify get_undo_list() by dropping the unnecessary cast, removing the
size variable, and switching to kzalloc() instead of a kmalloc() followed
by a memset().
This cleanup was split then modified from Jes Sorenson's Task Notifiers
patches.
Signed-off-by: Matt Helsley <matthltc@us.ibm.com>
Cc: Jes Sorensen <jes@sgi.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
| -rw-r--r-- | ipc/sem.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/ipc/sem.c b/ipc/sem.c index 69edeb1e2a65..0dafcc455f92 100644 --- a/ipc/sem.c +++ b/ipc/sem.c @@ -1003,15 +1003,12 @@ static inline void unlock_semundo(void) static inline int get_undo_list(struct sem_undo_list **undo_listp) { struct sem_undo_list *undo_list; - int size; undo_list = current->sysvsem.undo_list; if (!undo_list) { - size = sizeof(struct sem_undo_list); - undo_list = (struct sem_undo_list *) kmalloc(size, GFP_KERNEL); + undo_list = kzalloc(sizeof(*undo_list), GFP_KERNEL); if (undo_list == NULL) return -ENOMEM; - memset(undo_list, 0, size); spin_lock_init(&undo_list->lock); atomic_set(&undo_list->refcnt, 1); current->sysvsem.undo_list = undo_list; |