>From 8dd219ed2966f5ecc555e15f688a5340cb8f9fe6 Mon Sep 17 00:00:00 2001
From: Eliad Peller <[email protected]>
Date: Sun, 19 Sep 2010 18:55:08 +0200
Subject: [PATCH 08/26] wl1271: avoid redundant memcpy of rx_status

copy the rx_status directly to skb->cb (control buffer) instead of copying
it to a local struct and then copying it again (for each rx packet)

Signed-off-by: Eliad Peller <[email protected]>
Acked-by: Luciano Coelho <[email protected]>
Signed-off-by: John W. Linville <[email protected]>
Signed-off-by: Claude Brouat <[email protected]>
---
 drivers/net/wireless/wl12xx/wl1271_rx.c |    4 +---
 1 files changed, 1 insertions(+), 3 deletions(-)

diff --git a/drivers/net/wireless/wl12xx/wl1271_rx.c 
b/drivers/net/wireless/wl12xx/wl1271_rx.c
index b98fb64..bbfd7f0 100644
--- a/drivers/net/wireless/wl12xx/wl1271_rx.c
+++ b/drivers/net/wireless/wl12xx/wl1271_rx.c
@@ -75,7 +75,6 @@ static void wl1271_rx_status(struct wl1271 *wl,

 static void wl1271_rx_handle_data(struct wl1271 *wl, u32 length)
 {
-    struct ieee80211_rx_status rx_status;
     struct wl1271_rx_descriptor *desc;
     struct sk_buff *skb;
     u16 *fc;
@@ -108,14 +107,13 @@ static void wl1271_rx_handle_data(struct wl1271 *wl, u32 
length)
     if ((*fc & IEEE80211_FCTL_STYPE) == IEEE80211_STYPE_BEACON)
           beacon = 1;

-    wl1271_rx_status(wl, desc, &rx_status, beacon);
+    wl1271_rx_status(wl, desc, IEEE80211_SKB_RXCB(skb), beacon);

     wl1271_debug(DEBUG_RX, "rx skb 0x%p: %d B %s", skb, skb->len,
                beacon ? "beacon" : "");

     skb_trim(skb, skb->len - desc->pad_len);

-    memcpy(IEEE80211_SKB_RXCB(skb), &rx_status, sizeof(rx_status));
     ieee80211_rx_ni(wl->hw, skb);
 }

--
1.6.3.3




Claude BROUAT
UMG/MIPE/WSIV  System Integrator
Office:    +33 (0)1 72 21 04 54
mailto: mailto:[email protected]

Intel Corp. SAS
134, av du Général Eisenhower
BP 73586
31100 TOULOUSE
France



---------------------------------------------------------------------
Intel Corporation SAS (French simplified joint stock company)
Registered headquarters: "Les Montalets"- 2, rue de Paris, 
92196 Meudon Cedex, France
Registration Number:  302 456 199 R.C.S. NANTERRE
Capital: 4,572,000 Euros

This e-mail and any attachments may contain confidential material for
the sole use of the intended recipient(s). Any review or distribution
by others is strictly prohibited. If you are not the intended
recipient, please contact the sender and delete all copies.

Attachment: 0008-wl1271-avoid-redundant-memcpy-of-rx_status.patch
Description: 0008-wl1271-avoid-redundant-memcpy-of-rx_status.patch

_______________________________________________
Meego-kernel mailing list
[email protected]
http://lists.meego.com/listinfo/meego-kernel

Reply via email to