From: "Robert P. J. Day" Simplify the few instances where a call to "get_zeroed_page()" is closely followed by an unnecessary call to memset() to clear that page. Signed-off-by: Robert P. J. Day Cc: chas williams Cc: Mauro Carvalho Chehab Cc: James Bottomley Signed-off-by: Andrew Morton --- drivers/atm/eni.c | 1 - drivers/media/video/zoran_driver.c | 1 - drivers/scsi/53c7xx.c | 1 - 3 files changed, 3 deletions(-) diff -puN drivers/atm/eni.c~get-rid-of-double-zeroing-of-allocated-pages drivers/atm/eni.c --- a/drivers/atm/eni.c~get-rid-of-double-zeroing-of-allocated-pages +++ a/drivers/atm/eni.c @@ -912,7 +912,6 @@ static int start_rx(struct atm_dev *dev) free_page((unsigned long) eni_dev->free_list); return -ENOMEM; } - memset(eni_dev->rx_map,0,PAGE_SIZE); eni_dev->rx_mult = DEFAULT_RX_MULT; eni_dev->fast = eni_dev->last_fast = NULL; eni_dev->slow = eni_dev->last_slow = NULL; diff -puN drivers/media/video/zoran_driver.c~get-rid-of-double-zeroing-of-allocated-pages drivers/media/video/zoran_driver.c --- a/drivers/media/video/zoran_driver.c~get-rid-of-double-zeroing-of-allocated-pages +++ a/drivers/media/video/zoran_driver.c @@ -562,7 +562,6 @@ jpg_fbuffer_alloc (struct file *file) jpg_fbuffer_free(file); return -ENOBUFS; } - memset((void *) mem, 0, PAGE_SIZE); fh->jpg_buffers.buffer[i].frag_tab = (u32 *) mem; fh->jpg_buffers.buffer[i].frag_tab_bus = virt_to_bus((void *) mem); diff -puN drivers/scsi/53c7xx.c~get-rid-of-double-zeroing-of-allocated-pages drivers/scsi/53c7xx.c --- a/drivers/scsi/53c7xx.c~get-rid-of-double-zeroing-of-allocated-pages +++ a/drivers/scsi/53c7xx.c @@ -3099,7 +3099,6 @@ allocate_cmd (Scsi_Cmnd *cmd) { real = get_zeroed_page(GFP_ATOMIC); if (real == 0) return NULL; - memset((void *)real, 0, 4096); cache_push(virt_to_phys((void *)real), 4096); cache_clear(virt_to_phys((void *)real), 4096); kernel_set_cachemode((void *)real, 4096, IOMAP_NOCACHE_SER); _