Re: [PATCH] Bluetooth: Prevent unintended pause by checking if advertising is active

[Yang Li <yang.li@amlogic.com>]

Hi Luiz,
> [ EXTERNAL EMAIL ]
>
> Hi Yang,
>
> On Mon, Jun 16, 2025 at 7:05 AM Yang Li via B4 Relay
> <devnull+yang.li.amlogic.com@kernel.org> wrote:
>> From: Yang Li <yang.li@amlogic.com>
>>
>> When PA Create Sync is enabled, advertising resumes unexpectedly.
>> Therefore, it's necessary to check whether advertising is currently
>> active before attempting to pause it.
>>
>>    < HCI Command: LE Add Device To... (0x08|0x0011) plen 7  #1345 [hci0] 48.306205
>>                  Address type: Random (0x01)
>>                  Address: 4F:84:84:5F:88:17 (Resolvable)
>>                  Identity type: Random (0x01)
>>                  Identity: FC:5B:8C:F7:5D:FB (Static)
>>    < HCI Command: LE Set Address Re.. (0x08|0x002d) plen 1  #1347 [hci0] 48.308023
>>                  Address resolution: Enabled (0x01)
>>    ...
>>    < HCI Command: LE Set Extended A.. (0x08|0x0039) plen 6  #1349 [hci0] 48.309650
>>                  Extended advertising: Enabled (0x01)
>>                  Number of sets: 1 (0x01)
>>                  Entry 0
>>                  Handle: 0x01
>>                  Duration: 0 ms (0x00)
>>                  Max ext adv events: 0
>>    ...
>>    < HCI Command: LE Periodic Adve.. (0x08|0x0044) plen 14  #1355 [hci0] 48.314575
>>                  Options: 0x0000
>>                  Use advertising SID, Advertiser Address Type and address
>>                  Reporting initially enabled
>>                  SID: 0x02
>>                  Adv address type: Random (0x01)
>>                  Adv address: 4F:84:84:5F:88:17 (Resolvable)
>>                  Identity type: Random (0x01)
>>                  Identity: FC:5B:8C:F7:5D:FB (Static)
>>                  Skip: 0x0000
>>                  Sync timeout: 20000 msec (0x07d0)
>>                  Sync CTE type: 0x0000
>>
>> Signed-off-by: Yang Li <yang.li@amlogic.com>
>> ---
>>   net/bluetooth/hci_sync.c | 4 ++++
>>   1 file changed, 4 insertions(+)
>>
>> diff --git a/net/bluetooth/hci_sync.c b/net/bluetooth/hci_sync.c
>> index 6687f2a4d1eb..4ea5a87f017d 100644
>> --- a/net/bluetooth/hci_sync.c
>> +++ b/net/bluetooth/hci_sync.c
>> @@ -2481,6 +2481,10 @@ static int hci_pause_advertising_sync(struct hci_dev *hdev)
>>          int err;
>>          int old_state;
>>
>> +       /* If controller is not advertising we are done. */
>> +       if (!hci_dev_test_flag(hdev, HCI_ADVERTISING))
>> +               return 0;
> HCI_ADVERTISING is for directed advertising, it shall probably be
> changed to HCI_LE_ADV like in hci_disable_advertising_sync.

Thanks for your valuable feedback.

I will do.

>>          /* If already been paused there is nothing to do. */
>>          if (hdev->advertising_paused)
>>                  return 0;
>>
>> ---
>> base-commit: 6c31dab4ff1e1f4e0f3412efb1cfc88033358d1f
>> change-id: 20250616-bluetooth_adv-6496e1ca6165
>>
>> Best regards,
>> --
>> Yang Li <yang.li@amlogic.com>
>>
>>
>
> --
> Luiz Augusto von Dentz