Hi Howard,
On Thu, Sep 17, 2020 at 1:47 AM Howard Chung <howardchung@xxxxxxxxxx> wrote:
>
> From: Miao-chen Chou <mcchou@xxxxxxxxxxxx>
>
> This fixes the count of Adv monitor upon monitor removal.
>
> The following test was performed.
> - Start two btmgmt consoles, issue a btmgmt advmon-remove command on one
> console and observe a MGMT_EV_ADV_MONITOR_REMOVED event on the other.
>
> Signed-off-by: Miao-chen Chou <mcchou@xxxxxxxxxxxx>
> Signed-off-by: Howard Chung <howardchung@xxxxxxxxxx>
> Reviewed-by: Alain Michaud <alainm@xxxxxxxxxxxx>
> ---
>
> Changes in v2:
> - delete 'case 0x001c' in mgmt_config.c
>
> net/bluetooth/hci_core.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/net/bluetooth/hci_core.c b/net/bluetooth/hci_core.c
> index 8a2645a833013..f30a1f5950e15 100644
> --- a/net/bluetooth/hci_core.c
> +++ b/net/bluetooth/hci_core.c
> @@ -3061,6 +3061,7 @@ static int free_adv_monitor(int id, void *ptr, void *data)
>
> idr_remove(&hdev->adv_monitors_idr, monitor->handle);
> hci_free_adv_monitor(monitor);
> + hdev->adv_monitors_cnt--;
>
> return 0;
> }
> @@ -3077,6 +3078,7 @@ int hci_remove_adv_monitor(struct hci_dev *hdev, u16 handle)
>
> idr_remove(&hdev->adv_monitors_idr, monitor->handle);
> hci_free_adv_monitor(monitor);
> + hdev->adv_monitors_cnt--;
> } else {
> /* Remove all monitors if handle is 0. */
> idr_for_each(&hdev->adv_monitors_idr, &free_adv_monitor, hdev);
> --
> 2.28.0.618.gf4bc123cb7-goog
This looks like a kernel patch so you shouldn't be prefixing it with
BlueZ as it might confuse CI.
--
Luiz Augusto von Dentz
