Skip to content

Commit

Permalink
[JFFS2] Use memset(struct) instead of nulling struct members one by one
Browse files Browse the repository at this point in the history
Signed-off-by: Artem B. Bityutskiy <[email protected]>
Signed-off-by: Thomas Gleixner <[email protected]>
  • Loading branch information
dedekind authored and Thomas Gleixner committed Nov 6, 2005
1 parent 2f0077e commit d55849a
Showing 1 changed file with 7 additions and 10 deletions.
17 changes: 7 additions & 10 deletions fs/jffs2/build.c
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@
*
* For licensing information, see the file 'LICENCE' in this directory.
*
* $Id: build.c,v 1.83 2005/09/21 15:52:33 dedekind Exp $
* $Id: build.c,v 1.84 2005/09/27 13:40:49 dedekind Exp $
*
*/

Expand Down Expand Up @@ -309,28 +309,25 @@ int jffs2_do_mount_fs(struct jffs2_sb_info *c)
{
int ret;
int i;
int size;

c->free_size = c->flash_size;
c->nr_blocks = c->flash_size / c->sector_size;
size = sizeof(struct jffs2_eraseblock) * c->nr_blocks;
#ifndef __ECOS
if (jffs2_blocks_use_vmalloc(c))
c->blocks = vmalloc(sizeof(struct jffs2_eraseblock) * c->nr_blocks);
c->blocks = vmalloc(size);
else
#endif
c->blocks = kmalloc(sizeof(struct jffs2_eraseblock) * c->nr_blocks, GFP_KERNEL);
c->blocks = kmalloc(size, GFP_KERNEL);
if (!c->blocks)
return -ENOMEM;

memset(c->blocks, 0, size);
for (i=0; i<c->nr_blocks; i++) {
INIT_LIST_HEAD(&c->blocks[i].list);
c->blocks[i].offset = i * c->sector_size;
c->blocks[i].free_size = c->sector_size;
c->blocks[i].dirty_size = 0;
c->blocks[i].wasted_size = 0;
c->blocks[i].unchecked_size = 0;
c->blocks[i].used_size = 0;
c->blocks[i].first_node = NULL;
c->blocks[i].last_node = NULL;
c->blocks[i].bad_count = 0;
}

INIT_LIST_HEAD(&c->clean_list);
Expand Down

0 comments on commit d55849a

Please sign in to comment.