diff options
| author | Mingming Cao <cmm@us.ibm.com> | 2007-10-19 02:39:20 -0400 |
|---|---|---|
| committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2007-10-19 14:53:34 -0400 |
| commit | 8c3478a523a48470eb11a23f01249250684677d9 (patch) | |
| tree | 5818d64427bcc1128fc74cc99e8f1fd1c9a54766 /fs/jbd | |
| parent | 345225c8e4a4adad9eb261db26aebcd3b87055ad (diff) | |
JBD/ext3 cleanups: convert to kzalloc
Convert kmalloc to kzalloc() and get rid of the memset().
Signed-off-by: Mingming Cao <cmm@us.ibm.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'fs/jbd')
| -rw-r--r-- | fs/jbd/journal.c | 3 | ||||
| -rw-r--r-- | fs/jbd/transaction.c | 3 |
2 files changed, 2 insertions, 4 deletions
diff --git a/fs/jbd/journal.c b/fs/jbd/journal.c index 5d9fec0b7ebd..f810047a7195 100644 --- a/fs/jbd/journal.c +++ b/fs/jbd/journal.c | |||
| @@ -654,10 +654,9 @@ static journal_t * journal_init_common (void) | |||
| 654 | journal_t *journal; | 654 | journal_t *journal; |
| 655 | int err; | 655 | int err; |
| 656 | 656 | ||
| 657 | journal = kmalloc(sizeof(*journal), GFP_KERNEL); | 657 | journal = kzalloc(sizeof(*journal), GFP_KERNEL); |
| 658 | if (!journal) | 658 | if (!journal) |
| 659 | goto fail; | 659 | goto fail; |
| 660 | memset(journal, 0, sizeof(*journal)); | ||
| 661 | 660 | ||
| 662 | init_waitqueue_head(&journal->j_wait_transaction_locked); | 661 | init_waitqueue_head(&journal->j_wait_transaction_locked); |
| 663 | init_waitqueue_head(&journal->j_wait_logspace); | 662 | init_waitqueue_head(&journal->j_wait_logspace); |
diff --git a/fs/jbd/transaction.c b/fs/jbd/transaction.c index 9841b1e5af03..08ff6c7028cc 100644 --- a/fs/jbd/transaction.c +++ b/fs/jbd/transaction.c | |||
| @@ -96,13 +96,12 @@ static int start_this_handle(journal_t *journal, handle_t *handle) | |||
| 96 | 96 | ||
| 97 | alloc_transaction: | 97 | alloc_transaction: |
| 98 | if (!journal->j_running_transaction) { | 98 | if (!journal->j_running_transaction) { |
| 99 | new_transaction = kmalloc(sizeof(*new_transaction), | 99 | new_transaction = kzalloc(sizeof(*new_transaction), |
| 100 | GFP_NOFS|__GFP_NOFAIL); | 100 | GFP_NOFS|__GFP_NOFAIL); |
| 101 | if (!new_transaction) { | 101 | if (!new_transaction) { |
| 102 | ret = -ENOMEM; | 102 | ret = -ENOMEM; |
| 103 | goto out; | 103 | goto out; |
| 104 | } | 104 | } |
| 105 | memset(new_transaction, 0, sizeof(*new_transaction)); | ||
| 106 | } | 105 | } |
| 107 | 106 | ||
| 108 | jbd_debug(3, "New handle %p going live.\n", handle); | 107 | jbd_debug(3, "New handle %p going live.\n", handle); |
