[PATCH 68] drivers/net/s2io.c: kmalloc + memset conversion to k[cz]alloc

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: <linux-kernel@...>
Cc: <kernel-janitors@...>, Andrew Morton <akpm@...>, <netdev@...>, <jgarzik@...>
Date: Tuesday, July 31, 2007 - 5:56 pm

Signed-off-by: Mariusz Kozlowski <m.kozlowski@tuxland.pl>

 drivers/net/s2io.c | 235587 -> 235340 (-247 bytes)
 drivers/net/s2io.o | 460768 -> 460120 (-648 bytes)

 drivers/net/s2io.c |   14 +++++---------
 1 file changed, 5 insertions(+), 9 deletions(-)

--- linux-2.6.23-rc1-mm1-a/drivers/net/s2io.c	2007-07-26 13:07:42.000000000 +0200
+++ linux-2.6.23-rc1-mm1-b/drivers/net/s2io.c	2007-07-31 13:44:02.000000000 +0200
@@ -531,7 +531,7 @@ static int init_shared_mem(struct s2io_n
 	for (i = 0; i < config->tx_fifo_num; i++) {
 		int fifo_len = config->tx_cfg[i].fifo_len;
 		int list_holder_size = fifo_len * sizeof(struct list_info_hold);
-		mac_control->fifos[i].list_info = kmalloc(list_holder_size,
+		mac_control->fifos[i].list_info = kzalloc(list_holder_size,
 							  GFP_KERNEL);
 		if (!mac_control->fifos[i].list_info) {
 			DBG_PRINT(INFO_DBG,
@@ -539,7 +539,6 @@ static int init_shared_mem(struct s2io_n
 			return -ENOMEM;
 		}
 		mem_allocated += list_holder_size;
-		memset(mac_control->fifos[i].list_info, 0, list_holder_size);
 	}
 	for (i = 0; i < config->tx_fifo_num; i++) {
 		int page_num = TXD_MEM_PAGE_CNT(config->tx_cfg[i].fifo_len,
@@ -3788,9 +3787,9 @@ static int s2io_enable_msi_x(struct s2io
 	u16 msi_control; /* Temp variable */
 	int ret, i, j, msix_indx = 1;

-	nic->entries = kmalloc(MAX_REQUESTED_MSI_X * sizeof(struct msix_entry),
+	nic->entries = kcalloc(MAX_REQUESTED_MSI_X, sizeof(struct msix_entry),
 			       GFP_KERNEL);
-	if (nic->entries == NULL) {
+	if (!nic->entries) {
 		DBG_PRINT(INFO_DBG, "%s: Memory allocation failed\n", \
 			__FUNCTION__);
 		nic->mac_control.stats_info->sw_stat.mem_alloc_fail_cnt++;
@@ -3798,12 +3797,11 @@ static int s2io_enable_msi_x(struct s2io
 	}
 	nic->mac_control.stats_info->sw_stat.mem_allocated
 		+= (MAX_REQUESTED_MSI_X * sizeof(struct msix_entry));
-	memset(nic->entries, 0,MAX_REQUESTED_MSI_X * sizeof(struct msix_entry));

 	nic->s2io_entries =
-		kmalloc(MAX_REQUESTED_MSI_X * sizeof(struct s2io_msix_entry),
+		kcalloc(MAX_REQUESTED_MSI_X, sizeof(struct s2io_msix_entry),
 				   GFP_KERNEL);
-	if (nic->s2io_entries == NULL) {
+	if (!nic->s2io_entries) {
 		DBG_PRINT(INFO_DBG, "%s: Memory allocation failed\n",
 			__FUNCTION__);
 		nic->mac_control.stats_info->sw_stat.mem_alloc_fail_cnt++;
@@ -3814,8 +3812,6 @@ static int s2io_enable_msi_x(struct s2io
 	}
 	 nic->mac_control.stats_info->sw_stat.mem_allocated
 		+= (MAX_REQUESTED_MSI_X * sizeof(struct s2io_msix_entry));
-	memset(nic->s2io_entries, 0,
-	       MAX_REQUESTED_MSI_X * sizeof(struct s2io_msix_entry));

 	for (i=0; i< MAX_REQUESTED_MSI_X; i++) {
 		nic->entries[i].entry = i;
-
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[KJ PATCHES] mostly kmalloc + memset conversion to k[cz]alloc, Mariusz Kozlowski, (Tue Jul 31, 12:45 pm)
[PATCH 80] fs/ext4/xattr.c: kmalloc + memset conversion to k..., Mariusz Kozlowski, (Tue Jul 31, 6:18 pm)
[PATCH 81] drivers/net/wireless/zd1201.c: kmalloc + memset c..., Mariusz Kozlowski, (Tue Jul 31, 6:19 pm)
[PATCH 82] drivers/scsi/zorro7xx.c: kmalloc + memset convers..., Mariusz Kozlowski, (Tue Jul 31, 6:21 pm)
[PATCH 79] arch/i386/mach-voyager/voyager_cat.c: kmalloc + m..., Mariusz Kozlowski, (Tue Jul 31, 6:16 pm)
[PATCH 78] fs/proc/vmcore.c: kmalloc + memset conversion to ..., Mariusz Kozlowski, (Tue Jul 31, 6:14 pm)
[PATCH 76] drivers/net/via-velocity.c: mostly kmalloc + mems..., Mariusz Kozlowski, (Tue Jul 31, 6:11 pm)
[PATCH 77] sound/oss/via82cxxx_audio.c: kmalloc + memset con..., Mariusz Kozlowski, (Tue Jul 31, 6:13 pm)
[PATCH 74] include/linux/textsearch.h: kmalloc + memset conv..., Mariusz Kozlowski, (Tue Jul 31, 6:08 pm)
[PATCH 75] drivers/macintosh/therm_adt746x.c: kmalloc + mems..., Mariusz Kozlowski, (Tue Jul 31, 6:10 pm)
[PATCH 73] drivers/parport/share.c: kmalloc + memset convers..., Mariusz Kozlowski, (Tue Jul 31, 6:07 pm)
[PATCH 71] kernel/sched.c: kmalloc + memset conversion to kc..., Mariusz Kozlowski, (Tue Jul 31, 6:03 pm)
[PATCH 72] drivers/mmc/core/sdio_bus.c: kmalloc + memset con..., Mariusz Kozlowski, (Tue Jul 31, 6:05 pm)
[PATCH 70] drivers/mtd/ubi/scan.c: kmalloc + memset conversi..., Mariusz Kozlowski, (Tue Jul 31, 6:02 pm)
[PATCH 69] drivers/net/sb1250-mac.c: kmalloc + memset conver..., Mariusz Kozlowski, (Tue Jul 31, 5:58 pm)
[PATCH 68] drivers/net/s2io.c: kmalloc + memset conversion t..., Mariusz Kozlowski, (Tue Jul 31, 5:56 pm)
[PATCH 67] net/ipv4/route.c: mostly kmalloc + memset convers..., Mariusz Kozlowski, (Tue Jul 31, 5:55 pm)
[PATCH 66] net/ipv4/raw.c: kmalloc + memset conversion to kz..., Mariusz Kozlowski, (Tue Jul 31, 5:54 pm)
[PATCH 64] drivers/scsi/qla2xxx/qla_init.c: mostly kmalloc +..., Mariusz Kozlowski, (Tue Jul 31, 5:51 pm)
[PATCH 65] drivers/mtd/maps/pmcmsp-flash.c: kmalloc + memset..., Mariusz Kozlowski, (Tue Jul 31, 5:49 pm)
[PATCH 63] drivers/scsi/pluto.c: mostly kmalloc + memset con..., Mariusz Kozlowski, (Tue Jul 31, 5:46 pm)
[PATCH 62] include/asm-avr32/pgalloc.h: kmalloc + memset con..., Mariusz Kozlowski, (Tue Jul 31, 5:41 pm)
Re: [PATCH 62] include/asm-avr32/pgalloc.h: kmalloc + memset..., Haavard Skinnemoen, (Fri Aug 3, 7:42 am)
[PATCH 61] drivers/scsi/osst.c: kmalloc + memset conversion ..., Mariusz Kozlowski, (Tue Jul 31, 5:37 pm)
[PATCH 59] drivers/video/offb.c: kmalloc + memset conversion..., Mariusz Kozlowski, (Tue Jul 31, 5:33 pm)
[PATCH 60] drivers/net/wireless/prism54/oid_mgt.c: kmalloc +..., Mariusz Kozlowski, (Tue Jul 31, 5:34 pm)
Re: [PATCH 60] drivers/net/wireless/prism54/oid_mgt.c: kmall..., Luis R. Rodriguez, (Tue Jul 31, 9:12 pm)
[PATCH 58] net/ipv4/netfilter/nf_conntrack_l3proto_ipv4_comp..., Mariusz Kozlowski, (Tue Jul 31, 5:32 pm)
[PATCH 57] net/netfilter/nf_conntrack_expect.c: kmalloc + me..., Mariusz Kozlowski, (Tue Jul 31, 5:31 pm)
[PATCH 56] drivers/scsi/NCR_D700.c: kmalloc + memset convers..., Mariusz Kozlowski, (Tue Jul 31, 5:29 pm)
[PATCH 55] drivers/char/n_hdlc.c: kmalloc + memset conversio..., Mariusz Kozlowski, (Tue Jul 31, 5:27 pm)
[PATCH 54] drivers/scsi/mvme16x_scsi.c: kmalloc + memset con..., Mariusz Kozlowski, (Tue Jul 31, 5:23 pm)
[PATCH 53] Documentation/DocBook/mtdnand.tmpl: kmalloc + mem..., Mariusz Kozlowski, (Tue Jul 31, 5:22 pm)
[PATCH 52] include/asm-m32r/thread_info.h: kmalloc + memset ..., Mariusz Kozlowski, (Tue Jul 31, 3:18 pm)
[PATCH 51] drivers/media/video/msp3400-driver.c: kmalloc + m..., Mariusz Kozlowski, (Tue Jul 31, 3:14 pm)
[PATCH 50] drivers/message/fusion/mptctl.c: mostly kmalloc +..., Mariusz Kozlowski, (Tue Jul 31, 2:54 pm)
[PATCH 49] drivers/mmc/core/mmc_ops.c: kmalloc + memset conv..., Mariusz Kozlowski, (Tue Jul 31, 2:50 pm)
[PATCH 48] include/asm-mips/thread_info.h: kmalloc + memset ..., Mariusz Kozlowski, (Tue Jul 31, 2:48 pm)
[PATCH 47] drivers/scsi/megaraid.c: kmalloc + memset convers..., Mariusz Kozlowski, (Tue Jul 31, 2:34 pm)
[PATCH 46] drivers/scsi/lpfc/lpfc_scsi.c: kmalloc + memset c..., Mariusz Kozlowski, (Tue Jul 31, 2:27 pm)
[PATCH 45] drivers/scsi/lpfc/lpfc_init.c: kmalloc + memset c..., Mariusz Kozlowski, (Tue Jul 31, 2:26 pm)
[PATCH 44] drivers/scsi/lpfc/lpfc_debugfs.c: kmalloc + memse..., Mariusz Kozlowski, (Tue Jul 31, 2:25 pm)
[PATCH 41] drivers/net/irda/irda-usb.c: mostly kmalloc + mem..., Mariusz Kozlowski, (Tue Jul 31, 2:20 pm)
Re: [PATCH 41] drivers/net/irda/irda-usb.c: mostly kmalloc +..., Mariusz Kozlowski, (Tue Jul 31, 5:17 pm)
[PATCH 43] fs/reiser4/ktxnmgrd.c: kmalloc + memset conversio..., Mariusz Kozlowski, (Tue Jul 31, 2:24 pm)
[PATCH 42] fs/jbd2/journal.c: kmalloc + memset conversion to..., Mariusz Kozlowski, (Tue Jul 31, 2:22 pm)
[PATCH 39] net/ipv4/ip_options.c: kmalloc + memset conversio..., Mariusz Kozlowski, (Tue Jul 31, 2:16 pm)
Re: [PATCH 39] net/ipv4/ip_options.c: kmalloc + memset conve..., Mariusz Kozlowski, (Tue Jul 31, 5:15 pm)
[PATCH 40] drivers/atm/iphase.c: mostly kmalloc + memset con..., Mariusz Kozlowski, (Tue Jul 31, 2:18 pm)
[PATCH 38] drivers/char/ip2/ip2main.c: kmalloc + memset conv..., Mariusz Kozlowski, (Tue Jul 31, 2:13 pm)
[PATCH 37] arch/i386/kernel/io_apic.c: kmalloc + memset conv..., Mariusz Kozlowski, (Tue Jul 31, 2:09 pm)
[PATCH 35] fs/reiser4/plugin/inode_ops_rename.c: kmalloc + m..., Mariusz Kozlowski, (Tue Jul 31, 2:06 pm)
[PATCH 36] arch/x86_64/kernel/io_apic.c: kmalloc + memset co..., Mariusz Kozlowski, (Tue Jul 31, 2:08 pm)
[PATCH 34] fs/autofs4/inode.c: kmalloc + memset conversion t..., Mariusz Kozlowski, (Tue Jul 31, 2:05 pm)
PATCH 32] drivers/mtd/inftlmount.c: kmalloc + memset convers..., Mariusz Kozlowski, (Tue Jul 31, 2:03 pm)
[PATCH 33] fs/reiser4/init_super.c: kmalloc + memset convers..., Mariusz Kozlowski, (Tue Jul 31, 2:04 pm)
[PATCH 31] drivers/scsi/ide-scsi.c: kmalloc + memset convers..., Mariusz Kozlowski, (Tue Jul 31, 2:01 pm)
Re: [PATCH 31] drivers/scsi/ide-scsi.c: kmalloc + memset con..., Bartlomiej Zolnierkiewicz..., (Wed Aug 1, 5:02 pm)
[PATCH 30] drivers/ide/arm/icside.c: kmalloc + memset conver..., Mariusz Kozlowski, (Tue Jul 31, 1:58 pm)
Re: [PATCH 30] drivers/ide/arm/icside.c: kmalloc + memset co..., Bartlomiej Zolnierkiewicz..., (Wed Aug 1, 5:01 pm)
[PATCH 29] drivers/mmc/core/host.c: kmalloc + memset convers..., Mariusz Kozlowski, (Tue Jul 31, 1:53 pm)
[PATCH 28] drivers/net/wan/hdlc_fr.c: kmalloc + memset conve..., Mariusz Kozlowski, (Tue Jul 31, 1:49 pm)
[PATCH 27] drivers/input/serio/gscps2.c: kmalloc + memset co..., Mariusz Kozlowski, (Tue Jul 31, 1:48 pm)
[PATCH 26] drivers/scsi/gdth.c: kmalloc + memset conversion ..., Mariusz Kozlowski, (Tue Jul 31, 1:46 pm)
[PATCH 24] include/asm-frv/thread_info.h: kmalloc + memset c..., Mariusz Kozlowski, (Tue Jul 31, 1:40 pm)
[PATCH 25] drivers/usb/misc/ftdi-elan.c: kmalloc + memset co..., Mariusz Kozlowski, (Tue Jul 31, 1:42 pm)
Re: [PATCH 25] drivers/usb/misc/ftdi-elan.c: kmalloc + memse..., Christoph Lameter, (Tue Jul 31, 7:46 pm)
[PATCH 23] doc firmware_sample_firmware_class.c: kmalloc + m..., Mariusz Kozlowski, (Tue Jul 31, 1:39 pm)
[PATCH 22] drivers/net/e1000new/ethtool.c: kmalloc + memset ..., Mariusz Kozlowski, (Tue Jul 31, 1:36 pm)
[PATCH 21] drivers/scsi/dpt_i2o.c: kmalloc + memset conversi..., Mariusz Kozlowski, (Tue Jul 31, 1:35 pm)
[PATCH 20] net/decnet/dn_route.c: kmalloc + memset conversio..., Mariusz Kozlowski, (Tue Jul 31, 1:33 pm)
[PATCH 19] drivers/md/dm-target.c: kmalloc + memset conversi..., Mariusz Kozlowski, (Tue Jul 31, 1:29 pm)
[PATCH 18] drivers/md/dm-table.c: kmalloc + memset conversio..., Mariusz Kozlowski, (Tue Jul 31, 1:26 pm)
[PATCH 17] drivers/md/dm-path-selector.c: kmalloc + memset c..., Mariusz Kozlowski, (Tue Jul 31, 1:25 pm)
[PATCH 16] drivers/md/dm-hw-handler.c: kmalloc + memset conv..., Mariusz Kozlowski, (Tue Jul 31, 1:23 pm)
[PATCH 13] kmalloc + memset conversion to kzalloc, Mariusz Kozlowski, (Tue Jul 31, 1:17 pm)
[PATCH 14] kmalloc + memset conversion to kzalloc, Mariusz Kozlowski, (Tue Jul 31, 1:19 pm)
[PATCH 15] kmalloc + memset conversion to kzalloc, Mariusz Kozlowski, (Tue Jul 31, 1:21 pm)
[PATCH 12] fs/reiser4/plugin/file/cryptcompress.c: kmalloc +..., Mariusz Kozlowski, (Tue Jul 31, 1:16 pm)
[PATCH 11] drivers/pci/hotplug/cpqphp_ctrl.c: kmalloc + mems..., Mariusz Kozlowski, (Tue Jul 31, 1:14 pm)
[PATCH 10] drivers/block/cpqarray.c: better error handling a..., Mariusz Kozlowski, (Tue Jul 31, 1:12 pm)
RE: [PATCH 10] drivers/block/cpqarray.c: better error handli..., Miller, Mike (OS Dev), (Tue Jul 31, 4:16 pm)
[PATCH 09] drivers/char/consolemap.c: kmalloc + memset conve..., Mariusz Kozlowski, (Tue Jul 31, 1:08 pm)
[PATCH 08] fs/cifs/connect.c: kmalloc + memset conversion to..., Mariusz Kozlowski, (Tue Jul 31, 1:05 pm)
[PATCH 07] drivers/block/cciss.c: kmalloc + memset conversio..., Mariusz Kozlowski, (Tue Jul 31, 1:03 pm)
RE: [PATCH 07] drivers/block/cciss.c: kmalloc + memset conve..., Miller, Mike (OS Dev), (Tue Jul 31, 1:14 pm)
[PATCH 06] drivers/scsi/bvme6000_scsi.c: kmalloc + memset co..., Mariusz Kozlowski, (Tue Jul 31, 1:01 pm)
[PATCH 04] drivers/sbus/char/bbc_i2c.c: kmalloc + memset con..., Mariusz Kozlowski, (Tue Jul 31, 12:58 pm)
[PATCH 05] drivers/mmc/core/bus.c: kmalloc + memset conversi..., Mariusz Kozlowski, (Tue Jul 31, 12:59 pm)
[PATCH 03] drivers/sbus/char/bbc_envctrl.c: kmalloc + memset..., Mariusz Kozlowski, (Tue Jul 31, 12:56 pm)
[PATCH 02] kmalloc + memset conversion to kzalloc, Mariusz Kozlowski, (Tue Jul 31, 12:52 pm)
[PATCH 01] kmalloc + memset conversion co kzalloc, Mariusz Kozlowski, (Tue Jul 31, 12:50 pm)