Bluetooth: hci_intel: Give priority to LPM packets
authorLoic Poulain <loic.poulain@intel.com>
Wed, 9 Sep 2015 17:08:02 +0000 (19:08 +0200)
committerMarcel Holtmann <marcel@holtmann.org>
Thu, 17 Sep 2015 11:20:06 +0000 (13:20 +0200)
Change the way to insert LPM packets into the txq.
Use skb_queue_head instead of skb_queue_tail to always prioritise LPM
packets over potential tx queue content.

Signed-off-by: Loic Poulain <loic.poulain@intel.com>
Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
drivers/bluetooth/hci_intel.c

index b17386b..249e096 100644 (file)
@@ -190,7 +190,8 @@ static int intel_lpm_suspend(struct hci_uart *hu)
 
        set_bit(STATE_LPM_TRANSACTION, &intel->flags);
 
-       skb_queue_tail(&intel->txq, skb);
+       /* LPM flow is a priority, enqueue packet at list head */
+       skb_queue_head(&intel->txq, skb);
        hci_uart_tx_wakeup(hu);
 
        intel_wait_lpm_transaction(hu);
@@ -233,7 +234,8 @@ static int intel_lpm_resume(struct hci_uart *hu)
 
        set_bit(STATE_LPM_TRANSACTION, &intel->flags);
 
-       skb_queue_tail(&intel->txq, skb);
+       /* LPM flow is a priority, enqueue packet at list head */
+       skb_queue_head(&intel->txq, skb);
        hci_uart_tx_wakeup(hu);
 
        intel_wait_lpm_transaction(hu);
@@ -272,7 +274,8 @@ static int intel_lpm_host_wake(struct hci_uart *hu)
               sizeof(lpm_resume_ack));
        bt_cb(skb)->pkt_type = HCI_LPM_PKT;
 
-       skb_queue_tail(&intel->txq, skb);
+       /* LPM flow is a priority, enqueue packet at list head */
+       skb_queue_head(&intel->txq, skb);
        hci_uart_tx_wakeup(hu);
 
        bt_dev_dbg(hu->hdev, "Resumed by controller");