lkml.org 
[lkml]   [2008]   [Nov]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Date
    From
    Subject[patch 10/22] hostap: pad the skb->cb usage in lieu of a proper fix
    2.6.27-stable review patch.  If anyone has any objections, please let us know.

    ------------------

    From: Johannes Berg <johannes@sipsolutions.net>

    commit f7cd168645dda3e9067f24fabbfa787f9a237488 upstream.

    Like mac80211 did, this driver makes 'clever' use of skb->cb to pass
    information along with an skb as it is requeued from the virtual device
    to the physical wireless device. Unfortunately, that trick no longer
    works...

    Unlike mac80211, code complexity and driver apathy makes this hack
    the best option we have in the short run. Hopefully someone will
    eventually be motivated to code a proper fix before all the effected
    hardware dies.

    (Above text by me. Johannes officially disavows all knowledge of this
    hack. -- JWL)

    Signed-off-by: Johannes Berg <johannes@sipsolutions.net>
    Signed-off-by: John W. Linville <linville@tuxdriver.com>
    Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

    ---
    drivers/net/wireless/hostap/hostap_wlan.h | 5 ++++-
    1 file changed, 4 insertions(+), 1 deletion(-)

    --- a/drivers/net/wireless/hostap/hostap_wlan.h
    +++ b/drivers/net/wireless/hostap/hostap_wlan.h
    @@ -918,9 +918,12 @@ struct hostap_interface {

    /*
    * TX meta data - stored in skb->cb buffer, so this must not be increased over
    - * the 40-byte limit
    + * the 48-byte limit.
    + * THE PADDING THIS STARTS WITH IS A HORRIBLE HACK THAT SHOULD NOT LIVE
    + * TO SEE THE DAY.
    */
    struct hostap_skb_tx_data {
    + unsigned int __padding_for_default_qdiscs;
    u32 magic; /* HOSTAP_SKB_TX_DATA_MAGIC */
    u8 rate; /* transmit rate */
    #define HOSTAP_TX_FLAGS_WDS BIT(0)
    --


    \
     
     \ /
      Last update: 2008-11-15 06:31    [W:0.022 / U:1.056 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site