Commit d6bfd59c authored by Johan Hedberg's avatar Johan Hedberg Committed by Marcel Holtmann

Bluetooth: Add timer for regenerating local RPA

This patch adds a timer for updating the local RPA periodically. The
default timeout is set to 15 minutes.
Signed-off-by: default avatarJohan Hedberg <johan.hedberg@intel.com>
Signed-off-by: default avatarMarcel Holtmann <marcel@holtmann.org>
parent b1e2b3ae
...@@ -127,6 +127,7 @@ enum { ...@@ -127,6 +127,7 @@ enum {
HCI_SC_ENABLED, HCI_SC_ENABLED,
HCI_SC_ONLY, HCI_SC_ONLY,
HCI_PRIVACY, HCI_PRIVACY,
HCI_RPA_EXPIRED,
HCI_RPA_RESOLVING, HCI_RPA_RESOLVING,
HCI_HS_ENABLED, HCI_HS_ENABLED,
HCI_LE_ENABLED, HCI_LE_ENABLED,
......
...@@ -130,6 +130,9 @@ struct oob_data { ...@@ -130,6 +130,9 @@ struct oob_data {
#define HCI_MAX_SHORT_NAME_LENGTH 10 #define HCI_MAX_SHORT_NAME_LENGTH 10
/* Default LE RPA expiry time, 15 minutes */
#define HCI_DEFAULT_RPA_TIMEOUT (15 * 60)
struct amp_assoc { struct amp_assoc {
__u16 len; __u16 len;
__u16 offset; __u16 offset;
...@@ -304,6 +307,8 @@ struct hci_dev { ...@@ -304,6 +307,8 @@ struct hci_dev {
__u8 scan_rsp_data_len; __u8 scan_rsp_data_len;
__u8 irk[16]; __u8 irk[16];
__u32 rpa_timeout;
struct delayed_work rpa_expired;
int (*open)(struct hci_dev *hdev); int (*open)(struct hci_dev *hdev);
int (*close)(struct hci_dev *hdev); int (*close)(struct hci_dev *hdev);
......
...@@ -2102,6 +2102,7 @@ static int hci_dev_do_open(struct hci_dev *hdev) ...@@ -2102,6 +2102,7 @@ static int hci_dev_do_open(struct hci_dev *hdev)
if (!ret) { if (!ret) {
hci_dev_hold(hdev); hci_dev_hold(hdev);
set_bit(HCI_RPA_EXPIRED, &hdev->dev_flags);
set_bit(HCI_UP, &hdev->flags); set_bit(HCI_UP, &hdev->flags);
hci_notify(hdev, HCI_DEV_UP); hci_notify(hdev, HCI_DEV_UP);
if (!test_bit(HCI_SETUP, &hdev->dev_flags) && if (!test_bit(HCI_SETUP, &hdev->dev_flags) &&
...@@ -2199,6 +2200,7 @@ static int hci_dev_do_close(struct hci_dev *hdev) ...@@ -2199,6 +2200,7 @@ static int hci_dev_do_close(struct hci_dev *hdev)
cancel_delayed_work(&hdev->service_cache); cancel_delayed_work(&hdev->service_cache);
cancel_delayed_work_sync(&hdev->le_scan_disable); cancel_delayed_work_sync(&hdev->le_scan_disable);
cancel_delayed_work_sync(&hdev->rpa_expired);
hci_dev_lock(hdev); hci_dev_lock(hdev);
hci_inquiry_cache_flush(hdev); hci_inquiry_cache_flush(hdev);
...@@ -3300,6 +3302,8 @@ struct hci_dev *hci_alloc_dev(void) ...@@ -3300,6 +3302,8 @@ struct hci_dev *hci_alloc_dev(void)
hdev->le_conn_min_interval = 0x0028; hdev->le_conn_min_interval = 0x0028;
hdev->le_conn_max_interval = 0x0038; hdev->le_conn_max_interval = 0x0038;
hdev->rpa_timeout = HCI_DEFAULT_RPA_TIMEOUT;
mutex_init(&hdev->lock); mutex_init(&hdev->lock);
mutex_init(&hdev->req_lock); mutex_init(&hdev->req_lock);
......
...@@ -881,12 +881,39 @@ static void service_cache_off(struct work_struct *work) ...@@ -881,12 +881,39 @@ static void service_cache_off(struct work_struct *work)
hci_req_run(&req, NULL); hci_req_run(&req, NULL);
} }
static void rpa_expired(struct work_struct *work)
{
struct hci_dev *hdev = container_of(work, struct hci_dev,
rpa_expired.work);
struct hci_request req;
BT_DBG("");
set_bit(HCI_RPA_EXPIRED, &hdev->dev_flags);
if (!test_bit(HCI_ADVERTISING, &hdev->dev_flags) ||
hci_conn_num(hdev, LE_LINK) > 0)
return;
/* The generation of a new RPA and programming it into the
* controller happens in the enable_advertising() function.
*/
hci_req_init(&req, hdev);
disable_advertising(&req);
enable_advertising(&req);
hci_req_run(&req, NULL);
}
static void mgmt_init_hdev(struct sock *sk, struct hci_dev *hdev) static void mgmt_init_hdev(struct sock *sk, struct hci_dev *hdev)
{ {
if (test_and_set_bit(HCI_MGMT, &hdev->dev_flags)) if (test_and_set_bit(HCI_MGMT, &hdev->dev_flags))
return; return;
INIT_DELAYED_WORK(&hdev->service_cache, service_cache_off); INIT_DELAYED_WORK(&hdev->service_cache, service_cache_off);
INIT_DELAYED_WORK(&hdev->rpa_expired, rpa_expired);
/* Non-mgmt controlled devices get this bit set /* Non-mgmt controlled devices get this bit set
* implicitly so that pairing works for them, however * implicitly so that pairing works for them, however
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment