Re: b44: regression in 2.6.22 (resend)

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Maximilian Engelhardt <maxi@...>
Cc: linux-kernel <linux-kernel@...>, linux-wireless <linux-wireless@...>, Stephen Hemminger <shemminger@...>, Arnaldo Carvalho de Melo <acme@...>, Jeff Garzik <jgarzik@...>, Gary Zambrano <zambrano@...>, <netdev@...>, Andrew Morton <akpm@...>
Date: Sunday, May 27, 2007 - 5:13 pm

On Sunday 27 May 2007 21:25:17 Maximilian Engelhardt wrote:


This is the diff between these two kernels.
I'm not sure why you see a much better TX throughput here.

Can you re-check to make sure it's not just some test-jitter?


--- linux-2.6.21.1/drivers/net/b44.c    2007-05-27 22:58:01.000000000 +0200
+++ linux-2.6.22-rc3/drivers/net/b44.c  2007-05-27 23:01:44.000000000 +0200
@@ -825,12 +825,11 @@
                        if (copy_skb == NULL)
                                goto drop_it_no_recycle;
 
-                       copy_skb->dev = bp->dev;
                        skb_reserve(copy_skb, 2);
                        skb_put(copy_skb, len);
                        /* DMA sync done above, copy just the actual packet */
-                       memcpy(copy_skb->data, skb->data+bp->rx_offset, len);
-
+                       skb_copy_from_linear_data_offset(skb, bp->rx_offset,
+                                                        copy_skb->data, len);
                        skb = copy_skb;
                }
                skb->ip_summed = CHECKSUM_NONE;
@@ -1007,7 +1006,8 @@
                        goto err_out;
                }
 
-               memcpy(skb_put(bounce_skb, len), skb->data, skb->len);
+               skb_copy_from_linear_data(skb, skb_put(bounce_skb, len),
+                                         skb->len);
                dev_kfree_skb_any(skb);
                skb = bounce_skb;
        }

-- 
Greetings Michael.
-
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
Re: b44: regression in 2.6.22 (resend), Maximilian Engelhardt, (Sun May 27, 3:25 pm)
Re: b44: regression in 2.6.22 (resend), Michael Buesch, (Sun May 27, 5:13 pm)
Re: b44: regression in 2.6.22 (resend), Maximilian Engelhardt, (Sun May 27, 6:15 pm)
Re: b44: regression in 2.6.22 (resend), Michael Buesch, (Sun May 27, 8:24 pm)
Re: b44: regression in 2.6.22 (resend), Maximilian Engelhardt, (Sun May 27, 8:40 pm)
Re: b44: regression in 2.6.22 (resend), Michael Buesch, (Mon May 28, 6:49 am)
Re: b44: regression in 2.6.22 (resend), Maximilian Engelhardt, (Mon May 28, 10:12 am)
Re: b44: regression in 2.6.22 (resend), Michael Buesch, (Mon May 28, 10:55 am)
Re: b44: regression in 2.6.22 (resend), Gary Zambrano, (Tue May 29, 10:14 am)
Re: b44: regression in 2.6.22 (resend), Michael Buesch, (Tue May 29, 4:45 pm)
Re: b44: regression in 2.6.22 (resend), Gary Zambrano, (Tue May 29, 5:05 pm)
Re: b44: regression in 2.6.22 (resend), Jeff Garzik, (Tue May 29, 6:39 pm)
Re: b44: regression in 2.6.22 (resend), Gary Zambrano, (Tue May 29, 5:36 pm)
Re: b44: regression in 2.6.22 (resend), Michael Buesch, (Wed May 30, 6:45 am)
Re: b44: regression in 2.6.22 (resend), Stephen Hemminger, (Tue May 29, 5:01 pm)
Re: b44: regression in 2.6.22 (resend), Michael Buesch, (Mon May 28, 6:16 am)
Re: b44: regression in 2.6.22 (resend), Maximilian Engelhardt, (Mon May 28, 10:09 am)
Re: b44: regression in 2.6.22 (resend), Michael Buesch, (Mon May 28, 11:14 am)
Re: b44: regression in 2.6.22 (resend), Thomas Gleixner, (Mon May 28, 11:32 am)
Re: b44: regression in 2.6.22 (resend), Maximilian Engelhardt, (Mon May 28, 1:44 pm)
Re: b44: regression in 2.6.22 (resend), Thomas Gleixner, (Mon May 28, 3:23 pm)
Re: b44: regression in 2.6.22 (resend), Maximilian Engelhardt, (Sun Jun 3, 12:26 pm)
Re: b44: regression in 2.6.22 (resend), Thomas Gleixner, (Mon Jun 4, 2:39 am)
Re: b44: regression in 2.6.22 (resend), Stephen Hemminger, (Mon Jun 4, 12:09 pm)
Re: b44: regression in 2.6.22 (resend), Thomas Gleixner, (Mon Jun 4, 12:35 pm)
iperf: performance regression (was b44 driver problem?), Stephen Hemminger, (Mon Jun 4, 12:59 pm)
Re: iperf: performance regression (was b44 driver problem?), Stephen Hemminger, (Mon Jun 4, 1:51 pm)
Re: iperf: performance regression (was b44 driver problem?), Maximilian Engelhardt, (Mon Jun 4, 3:47 pm)
Re: iperf: performance regression (was b44 driver problem?), Stephen Hemminger, (Mon Jun 4, 4:02 pm)
Re: iperf: performance regression (was b44 driver problem?), Maximilian Engelhardt, (Mon Jun 4, 4:52 pm)
Re: b44: regression in 2.6.22 (resend), Maximilian Engelhardt, (Mon May 28, 4:55 pm)
Re: b44: regression in 2.6.22 (resend), Gary Zambrano, (Tue May 29, 9:58 am)
Re: b44: regression in 2.6.22 (resend), Maximilian Engelhardt, (Tue May 29, 1:23 pm)
Re: b44: regression in 2.6.22 (resend), Thomas Gleixner, (Mon May 28, 5:45 pm)
Re: b44: regression in 2.6.22 (resend), Maximilian Engelhardt, (Tue May 29, 2:28 pm)
Re: b44: regression in 2.6.22 (resend), Michael Buesch, (Mon May 28, 11:43 am)
Re: b44: regression in 2.6.22 (resend), Michael Buesch, (Sun May 27, 5:16 pm)
Re: b44: regression in 2.6.22 (resend), Maximilian Engelhardt, (Sun May 27, 5:50 pm)
Re: b44: regression in 2.6.22 (resend), Michael Buesch, (Sun May 27, 3:45 pm)
Re: b44: regression in 2.6.22 (resend), Maximilian Engelhardt, (Sun May 27, 4:36 pm)
Re: b44: regression in 2.6.22 (resend), Michael Buesch, (Sun May 27, 4:46 pm)
Re: b44: regression in 2.6.22 (resend), Maximilian Engelhardt, (Sun May 27, 5:46 pm)