On Thu, 2010-07-15 at 06:32 -0700, Jing Zhang wrote:
> One of the parameters, budget, is modified to poll RX queues in fair
> manner,
> another parameter in the whole game, work_done, should be correct
> accordingly.
> 
> 
> Signed-off-by: Jing Zhang <zj.na...@gmail.com>
> ---
> 
> 
> --- linux-2.6.34/drivers/net/ixgbe/ixgbe_main.c 2010-05-17
> 05:17:36.000000000 +0800
> +++ ixgbe/ixgbe_main_01.c 2010-07-15 20:51:18.000000000 +0800
> @@ -1559,12 +1559,14 @@ static int ixgbe_clean_rxtx_many(struct 
>   budget = max(budget, 1);
>   r_idx = find_first_bit(q_vector->rxr_idx, adapter->num_rx_queues);
>   for (i = 0; i < q_vector->rxr_count; i++) {
> + int this_work = 0;
>   ring = adapter->rx_ring[r_idx];
>  #ifdef CONFIG_IXGBE_DCA
>   if (adapter->flags & IXGBE_FLAG_DCA_ENABLED)
>   ixgbe_update_rx_dca(adapter, ring);
>  #endif
> - ixgbe_clean_rx_irq(q_vector, ring, &work_done, budget);
> + ixgbe_clean_rx_irq(q_vector, ring, &this_work, budget);
> + work_done += this_work;
>   r_idx = find_next_bit(q_vector->rxr_idx, adapter->num_rx_queues,
>                        r_idx + 1);
>   }
> 
> 

These patches do not apply cleanly to net-next-2.6 (David Miller's
Networking tree) git tree.

Have you checked to see if the issue you are trying to fix is still an
issue with the latest upstream kernel?

Cheers,
Jeff Kirsher


------------------------------------------------------------------------------
The Palm PDK Hot Apps Program offers developers who use the
Plug-In Development Kit to bring their C/C++ apps to Palm for a share 
of $1 Million in cash or HP Products. Visit us here for more details:
http://ad.doubleclick.net/clk;226879339;13503038;l?
http://clk.atdmt.com/CRS/go/247765532/direct/01/
_______________________________________________
E1000-devel mailing list
E1000-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/e1000-devel
To learn more about Intel&#174; Ethernet, visit 
http://communities.intel.com/community/wired

Reply via email to