Re: [BlueZ PATCH v2 1/6] Bluetooth: Update Adv monitor count upon removal

[Luiz Augusto von Dentz]

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