Bluetooth: hci_intel: Give priority to LPM packets

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>

diff --git a/drivers/bluetooth/hci_intel.c b/drivers/bluetooth/hci_intel.c
index b17386b5792c45ffa592e826bbd684216516162f..249e0963170d5834ed634d092d32065f96da97fb 100644 (file)
--- a/drivers/bluetooth/hci_intel.c
+++ b/drivers/bluetooth/hci_intel.c
@@ -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");