Bluetooth: Compress the size of struct hci_ctrl
authorJohan Hedberg <johan.hedberg@intel.com>
Thu, 5 Nov 2015 07:31:40 +0000 (09:31 +0200)
committerMarcel Holtmann <marcel@holtmann.org>
Thu, 19 Nov 2015 16:50:27 +0000 (17:50 +0100)
We can reduce the size of the hci_ctrl struct by converting
'bool req_start' to 'u8 req_flags' and making the two function
pointers a union (since only one is ever set at a time).

Signed-off-by: Johan Hedberg <johan.hedberg@intel.com>
Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
include/net/bluetooth/bluetooth.h
net/bluetooth/hci_core.c
net/bluetooth/hci_request.c
net/bluetooth/hci_sock.c

index a85e6d3d75ef287462ce76392c9b990282ef297d..8d38f411009c43ae03f23177de9780fa5bde0ec0 100644 (file)
@@ -296,12 +296,17 @@ typedef void (*hci_req_complete_t)(struct hci_dev *hdev, u8 status, u16 opcode);
 typedef void (*hci_req_complete_skb_t)(struct hci_dev *hdev, u8 status,
                                       u16 opcode, struct sk_buff *skb);
 
+#define HCI_REQ_START  BIT(0)
+#define HCI_REQ_SKB    BIT(1)
+
 struct hci_ctrl {
        __u16 opcode;
-       bool req_start;
+       u8 req_flags;
        u8 req_event;
-       hci_req_complete_t req_complete;
-       hci_req_complete_skb_t req_complete_skb;
+       union {
+               hci_req_complete_t req_complete;
+               hci_req_complete_skb_t req_complete_skb;
+       };
 };
 
 struct bt_skb_cb {
index db26cbd1cd9d3f2c55514ae8143a881f2b90c00a..bc97fc6de8765689e42e98c21e9c10ca2ca6587c 100644 (file)
@@ -3695,7 +3695,7 @@ int hci_send_cmd(struct hci_dev *hdev, __u16 opcode, __u32 plen,
        /* Stand-alone HCI commands must be flagged as
         * single-command requests.
         */
-       bt_cb(skb)->hci.req_start = true;
+       bt_cb(skb)->hci.req_flags |= HCI_REQ_START;
 
        skb_queue_tail(&hdev->cmd_q, skb);
        queue_work(hdev->workqueue, &hdev->cmd_work);
@@ -4392,7 +4392,7 @@ static bool hci_req_is_complete(struct hci_dev *hdev)
        if (!skb)
                return true;
 
-       return bt_cb(skb)->hci.req_start;
+       return (bt_cb(skb)->hci.req_flags & HCI_REQ_START);
 }
 
 static void hci_resend_last(struct hci_dev *hdev)
@@ -4452,20 +4452,20 @@ void hci_req_cmd_complete(struct hci_dev *hdev, u16 opcode, u8 status,
         * callback would be found in hdev->sent_cmd instead of the
         * command queue (hdev->cmd_q).
         */
-       if (bt_cb(hdev->sent_cmd)->hci.req_complete) {
-               *req_complete = bt_cb(hdev->sent_cmd)->hci.req_complete;
+       if (bt_cb(hdev->sent_cmd)->hci.req_flags & HCI_REQ_SKB) {
+               *req_complete_skb = bt_cb(hdev->sent_cmd)->hci.req_complete_skb;
                return;
        }
 
-       if (bt_cb(hdev->sent_cmd)->hci.req_complete_skb) {
-               *req_complete_skb = bt_cb(hdev->sent_cmd)->hci.req_complete_skb;
+       if (bt_cb(hdev->sent_cmd)->hci.req_complete) {
+               *req_complete = bt_cb(hdev->sent_cmd)->hci.req_complete;
                return;
        }
 
        /* Remove all pending commands belonging to this request */
        spin_lock_irqsave(&hdev->cmd_q.lock, flags);
        while ((skb = __skb_dequeue(&hdev->cmd_q))) {
-               if (bt_cb(skb)->hci.req_start) {
+               if (bt_cb(skb)->hci.req_flags & HCI_REQ_START) {
                        __skb_queue_head(&hdev->cmd_q, skb);
                        break;
                }
index bdb170995966b913c2be07c78a2cfe2728152910..5ba27c30e8f254df00f446d97d4f105c70279ac7 100644 (file)
@@ -56,8 +56,12 @@ static int req_run(struct hci_request *req, hci_req_complete_t complete,
                return -ENODATA;
 
        skb = skb_peek_tail(&req->cmd_q);
-       bt_cb(skb)->hci.req_complete = complete;
-       bt_cb(skb)->hci.req_complete_skb = complete_skb;
+       if (complete) {
+               bt_cb(skb)->hci.req_complete = complete;
+       } else if (complete_skb) {
+               bt_cb(skb)->hci.req_complete_skb = complete_skb;
+               bt_cb(skb)->hci.req_flags |= HCI_REQ_SKB;
+       }
 
        spin_lock_irqsave(&hdev->cmd_q.lock, flags);
        skb_queue_splice_tail(&req->cmd_q, &hdev->cmd_q);
@@ -128,7 +132,7 @@ void hci_req_add_ev(struct hci_request *req, u16 opcode, u32 plen,
        }
 
        if (skb_queue_empty(&req->cmd_q))
-               bt_cb(skb)->hci.req_start = true;
+               bt_cb(skb)->hci.req_flags |= HCI_REQ_START;
 
        bt_cb(skb)->hci.req_event = event;
 
index 235ad0fa357192552721c6ca68aca860bd0b8fae..19b23013c4f668341b1f80435d696d98da156db1 100644 (file)
@@ -1249,7 +1249,7 @@ static int hci_sock_sendmsg(struct socket *sock, struct msghdr *msg,
                        /* Stand-alone HCI commands must be flagged as
                         * single-command requests.
                         */
-                       bt_cb(skb)->hci.req_start = true;
+                       bt_cb(skb)->hci.req_flags |= HCI_REQ_START;
 
                        skb_queue_tail(&hdev->cmd_q, skb);
                        queue_work(hdev->workqueue, &hdev->cmd_work);