netdev - Re: [PATCH 2.6.24 1/5]S2io: Enable all the error and alarm indications

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Fri, 31 Aug 2007 09:01:13 -0400
From:	Jeff Garzik <jeff@...zik.org>
To:	ram.vepa@...erion.com
CC:	netdev@...r.kernel.org, support@...erion.com
Subject: Re: [PATCH 2.6.24 1/5]S2io: Enable all the error and alarm indications

Ramkrishna Vepa wrote:
> - Added support to unmask entire set of device errors and alarms.
> 
> Signed-off-by: Sivakumar Subramani <sivakumar.subramani@...erion.com>
> Signed-off-by: Santosh Rastapur <santosh.rastapur@...erion.com>
> Signed-off-by: Ramkrishna Vepa <ram.vepa@...erion.com>
> ---
> diff -Nurp orig/drivers/net/s2io.c patch1/drivers/net/s2io.c
> --- orig/drivers/net/s2io.c	2007-08-15 08:04:06.000000000 -0700
> +++ patch1/drivers/net/s2io.c	2007-08-15 08:55:53.000000000 -0700
> @@ -892,8 +892,9 @@ static void free_shared_mem(struct s2io_
>  					k++;
>  				}
>  				kfree(mac_control->rings[i].ba[j]);
> -				nic->mac_control.stats_info->sw_stat.mem_freed 				+= (sizeof(struct buffAdd) * 
> -				(rxd_count[nic->rxd_mode] + 1));
> +				nic->mac_control.stats_info->sw_stat.mem_freed += 
> +					(sizeof(struct buffAdd) * 
> +					(rxd_count[nic->rxd_mode] + 1));
>  			}
>  			kfree(mac_control->rings[i].ba);
>  			nic->mac_control.stats_info->sw_stat.mem_freed += 
> @@ -1732,6 +1733,365 @@ static int s2io_link_fault_indication(st
>  		return MAC_RMAC_ERR_TIMER;
>  }
>  
> +void en_dis_err_alarms(struct s2io_nic *nic, u16 mask, int flag)
> +{
> +	struct XENA_dev_config __iomem *bar0 = nic->bar0;
> +	register u64 val64 = 0, temp64 = 0, gen_int_mask = 0;
> +
> +	if (mask & TX_DMA_INTR) {
> +		gen_int_mask |= TXDMA_INT_M;
> +
> +		if (flag == ENABLE_INTRS) {
> +
> +			val64 = TXDMA_TDA_INT|TXDMA_PFC_INT|TXDMA_PCC_INT
> +				|TXDMA_TTI_INT|TXDMA_LSO_INT|TXDMA_TPA_INT
> +				|TXDMA_SM_INT;
> +			temp64 = readq(&bar0->txdma_int_mask);
> +			temp64 &= ~((u64) val64);
> +			writeq(temp64, &bar0->txdma_int_mask);


reduce the size of this huge functions by creating a helper function 
whose use would look something like

	writebits(TXDMA_TDA_INT|TXDMA_PFC_INT|TXDMA_PCC_INT|
		  TXDMA_TTI_INT|TXDMA_LSO_INT|TXDMA_TPA_INT|
		  TXDMA_SM_INT,
		  flag,
		  &bar0->txdma_int_mask);

That eliminates the wholesale duplication of code based on the 
'flags==ENABLE_INTRS' test, and overall increases readability and 
decreases code size

-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html