mac802154: tx: remove xmit channel context switch
authorAlexander Aring <alex.aring@gmail.com>
Sun, 26 Oct 2014 08:37:04 +0000 (09:37 +0100)
committerMarcel Holtmann <marcel@holtmann.org>
Sun, 26 Oct 2014 16:24:03 +0000 (17:24 +0100)
This patch removes the channel hopping feature before xmit. There are
several issues to provide a real channel hopping (timing requirements,
etc...).

We don't have any known kernelspace protocol which really use this
feature. And I don't know an real user of this feature.
We simply drop this feature now.

This patch removes also the hold of pib lock which isn't needed by any
real driver xmit callback implementation.

Signed-off-by: Alexander Aring <alex.aring@gmail.com>
Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
include/net/mac802154.h
net/mac802154/tx.c

index b07d431c0b1927cd3946dbe523a246a28908785d..ba8ddff70bb66a9ce831e592ce67cb5cf75b6f63 100644 (file)
@@ -113,8 +113,7 @@ struct ieee802154_hw {
  *       skb cntains the buffer starting from the IEEE 802.15.4 header.
  *       The low-level driver should send the frame based on available
  *       configuration.
- *       This function should return zero or negative errno. Called with
- *       pib_lock held.
+ *       This function should return zero or negative errno.
  *
  * ed:    Handler that 802.15.4 module calls for Energy Detection.
  *       This function should place the value for detected energy
index d0ceb46134d95f35ec313af3e02112e6f9a7c500..be8deae193867d3996087b0c9c97e1d4765f98cc 100644 (file)
@@ -34,8 +34,6 @@ struct wpan_xmit_cb {
        struct sk_buff *skb;
        struct work_struct work;
        struct ieee802154_local *local;
-       u8 chan;
-       u8 page;
 };
 
 static inline struct wpan_xmit_cb *wpan_xmit_cb(const struct sk_buff *skb)
@@ -53,26 +51,10 @@ static void mac802154_xmit_worker(struct work_struct *work)
        struct sk_buff *skb = cb->skb;
        int res;
 
-       mutex_lock(&local->phy->pib_lock);
-       if (local->phy->current_channel != cb->chan ||
-           local->phy->current_page != cb->page) {
-               res = local->ops->set_channel(&local->hw, cb->page, cb->chan);
-               if (res) {
-                       pr_debug("set_channel failed\n");
-                       goto out;
-               }
-
-               local->phy->current_channel = cb->chan;
-               local->phy->current_page = cb->page;
-       }
-
        res = local->ops->xmit(&local->hw, skb);
        if (res)
                pr_debug("transmission failed\n");
 
-out:
-       mutex_unlock(&local->phy->pib_lock);
-
        /* Restart the netif queue on each sub_if_data object. */
        rcu_read_lock();
        list_for_each_entry_rcu(sdata, &local->interfaces, list)
@@ -82,17 +64,12 @@ out:
        dev_kfree_skb(skb);
 }
 
-static netdev_tx_t mac802154_tx(struct ieee802154_local *local,
-                               struct sk_buff *skb, u8 page, u8 chan)
+static netdev_tx_t
+mac802154_tx(struct ieee802154_local *local, struct sk_buff *skb)
 {
        struct ieee802154_sub_if_data *sdata;
        struct wpan_xmit_cb *cb = wpan_xmit_cb(skb);
 
-       if (!(local->phy->channels_supported[page] & (1 << chan))) {
-               WARN_ON(1);
-               goto err_tx;
-       }
-
        mac802154_monitors_rx(local, skb);
 
        if (!(local->hw.flags & IEEE802154_HW_OMIT_CKSUM)) {
@@ -115,8 +92,6 @@ static netdev_tx_t mac802154_tx(struct ieee802154_local *local,
        INIT_WORK(&cb->work, mac802154_xmit_worker);
        cb->skb = skb;
        cb->local = local;
-       cb->page = page;
-       cb->chan = chan;
 
        queue_work(local->workqueue, &cb->work);
 
@@ -130,44 +105,19 @@ err_tx:
 netdev_tx_t mac802154_monitor_xmit(struct sk_buff *skb, struct net_device *dev)
 {
        struct ieee802154_sub_if_data *sdata = IEEE802154_DEV_TO_SUB_IF(dev);
-       u8 chan, page;
-
-       /* FIXME: locking */
-       chan = sdata->local->phy->current_channel;
-       page = sdata->local->phy->current_page;
-
-       if (chan == MAC802154_CHAN_NONE) /* not initialized */
-               return NETDEV_TX_OK;
-
-       if (WARN_ON(page >= WPAN_NUM_PAGES) ||
-           WARN_ON(chan >= WPAN_NUM_CHANNELS))
-               return NETDEV_TX_OK;
 
        skb->skb_iif = dev->ifindex;
        dev->stats.tx_packets++;
        dev->stats.tx_bytes += skb->len;
 
-       return mac802154_tx(sdata->local, skb, page, chan);
+       return mac802154_tx(sdata->local, skb);
 }
 
 netdev_tx_t mac802154_wpan_xmit(struct sk_buff *skb, struct net_device *dev)
 {
        struct ieee802154_sub_if_data *sdata = IEEE802154_DEV_TO_SUB_IF(dev);
-       u8 chan, page;
        int rc;
 
-       spin_lock_bh(&sdata->mib_lock);
-       chan = sdata->chan;
-       page = sdata->page;
-       spin_unlock_bh(&sdata->mib_lock);
-
-       if (chan == MAC802154_CHAN_NONE ||
-           page >= WPAN_NUM_PAGES ||
-           chan >= WPAN_NUM_CHANNELS) {
-               kfree_skb(skb);
-               return NETDEV_TX_OK;
-       }
-
        rc = mac802154_llsec_encrypt(&sdata->sec, skb);
        if (rc) {
                pr_warn("encryption failed: %i\n", rc);
@@ -179,5 +129,5 @@ netdev_tx_t mac802154_wpan_xmit(struct sk_buff *skb, struct net_device *dev)
        dev->stats.tx_packets++;
        dev->stats.tx_bytes += skb->len;
 
-       return mac802154_tx(sdata->local, skb, page, chan);
+       return mac802154_tx(sdata->local, skb);
 }