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:	Tue, 23 Jan 2007 00:40:46 -0500
From:	Jeff Garzik <jeff@...zik.org>
To:	Ananda Raju <Ananda.Raju@...erion.com>
CC:	netdev@...r.kernel.org, leonid.grossman@...erion.com,
	alicia.pena@...erion.com, ramkrishna.vepa@...erion.com,
	sivakumar.subramani@...erion.com, sreenivasa.honnur@...erion.com,
	sriram.rapuru@...erion.com
Subject: Re: [PATCH 2.6.20 1/5] s2io: Making LRO and UFO as module loadable
 parameter.

Ananda Raju wrote:
> This patch adds two load parameters napi and ufo. Previously NAPI was
> compilation option with these changes wan enable disable NAPI using load
> parameter. Also we are introducing ufo load parameter to enable/disable
> ufo feature
> 
> Signed-off-by: Sivakumar Subramani <sivakumar.subramani@...erion.com>
> ---
> diff -urpN orig/drivers/net/s2io.c patch1/drivers/net/s2io.c
> --- orig/drivers/net/s2io.c	2006-12-21 10:06:58.000000000 +0530
> +++ patch1/drivers/net/s2io.c	2007-01-08 11:56:23.000000000 +0530
> @@ -401,9 +401,10 @@ S2IO_PARM_INT(lro, 0);
>   * aggregation happens until we hit max IP pkt size(64K)
>   */
>  S2IO_PARM_INT(lro_max_pkts, 0xFFFF);
> -#ifndef CONFIG_S2IO_NAPI
>  S2IO_PARM_INT(indicate_max_pkts, 0);
> -#endif
> +
> +S2IO_PARM_INT(napi, 1);
> +S2IO_PARM_INT(ufo, 0);
>  
>  static unsigned int tx_fifo_len[MAX_TX_FIFOS] =
>      {DEFAULT_FIFO_0_LEN, [1 ...(MAX_TX_FIFOS - 1)] = DEFAULT_FIFO_1_7_LEN};
> @@ -2275,9 +2276,7 @@ static int fill_rx_buffers(struct s2io_n
>  	struct config_param *config;
>  	u64 tmp;
>  	buffAdd_t *ba;
> -#ifndef CONFIG_S2IO_NAPI
>  	unsigned long flags;
> -#endif
>  	RxD_t *first_rxdp = NULL;
>  
>  	mac_control = &nic->mac_control;
> @@ -2321,12 +2320,15 @@ static int fill_rx_buffers(struct s2io_n
>  			DBG_PRINT(INTR_DBG, "%s: Next block at: %p\n",
>  				  dev->name, rxdp);
>  		}
> -#ifndef CONFIG_S2IO_NAPI
> -		spin_lock_irqsave(&nic->put_lock, flags);
> -		mac_control->rings[ring_no].put_pos =
> -		    (block_no * (rxd_count[nic->rxd_mode] + 1)) + off;
> -		spin_unlock_irqrestore(&nic->put_lock, flags);
> -#endif
> +		if(!napi) {
> +			spin_lock_irqsave(&nic->put_lock, flags);
> +			mac_control->rings[ring_no].put_pos =
> +		    	(block_no * (rxd_count[nic->rxd_mode] + 1)) + off;
> +			spin_unlock_irqrestore(&nic->put_lock, flags);
> +		} else {
> +			mac_control->rings[ring_no].put_pos =
> +			(block_no * (rxd_count[nic->rxd_mode] + 1)) + off;
> +		}	
>  		if ((rxdp->Control_1 & RXD_OWN_XENA) &&
>  			((nic->rxd_mode >= RXD_MODE_3A) &&
>  				(rxdp->Control_2 & BIT(0)))) {
> @@ -2569,7 +2571,7 @@ static void free_rx_buffers(struct s2io_
>   * 0 on success and 1 if there are No Rx packets to be processed.
>   */
>  
> -#if defined(CONFIG_S2IO_NAPI)
> +#if defined(HAVE_NETDEV_POLL)

ACK the entire patch except for the above change:  don't bother with 
HAVE_NETDEV_POLL.  In current kernels you may assume we have it.

	Jeff



-
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

Powered by blists - more mailing lists