Received-SPF: pass (google.com: domain of linux-bluetooth+bounces-3223-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
Precedence: bulk
MIME-Version: 1.0
References: <20240404114054.4987-1-mahesh.talewad@nxp.com> <20240404114054.4987-2-mahesh.talewad@nxp.com>
In-Reply-To: <20240404114054.4987-2-mahesh.talewad@nxp.com>
From: Luiz Augusto von Dentz <luiz.dentz@gmail.com>
Date: Thu, 4 Apr 2024 13:23:32 -0400
Message-ID: <CABBYNZKk_dQefyB33NrWpCVpPAnL_veRfhs0AcA9HOEmEifWXw@mail.gmail.com>
Subject: Re: [PATCH BlueZ v2 1/1] Increase autoconnect timeout max from
 16.384s to 20.480s
To: Mahesh Talewad <mahesh.talewad@nxp.com>
Cc: linux-bluetooth@vger.kernel.org, devyani.godbole@nxp.com, 
	sarveshwar.bajaj@nxp.com
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

Hi Mahesh,

On Thu, Apr 4, 2024 at 7:41=E2=80=AFAM Mahesh Talewad <mahesh.talewad@nxp.c=
om> wrote:
>
>  - In current implementation Autoconnecttimeout max value is
>    16384[0x4000] msecs.
>  - Since some controllers need some more time to respond to cmd
>    'LE Extended Create Connection' hence increased this
>    Autoconnecttimeout max value to 20480[0x5000] msecs.
>
> For ex:
> - In some controllers, If we include LE-Coded PHY in the initiating
>  PHY List, BLE INIT scheduler selects the 1M and Coded PHY scanning
>  as an initiator in round robin manner, and due to this available
>  bandwidth gets distributed between 1M and Coded PHY, and this
>  results in longer time taken for connection establishment by
>  the controller.
> - If ref device is advertising at 1.5 sec intervals, with create
>  connection timeout of 4 sec,  the controller gets only 2 opportunities
>  for the connection. Without the inclusion of LE-coded PHY,
>  DUT takes ~3.8 sec for the connection establishment.Hence as described
>  in above point  with the inclusion of LE-coded PHY it is difficult to
>  achieve 100% connection success with the device having
>  adv interval of 1.5 sec.

Have a look at:

https://patchwork.kernel.org/project/bluetooth/list/?series=3D841493

> Hence increased Autoconnecttimeout max value to 20480[0x5000] msecs.
>
> Signed-off-by: Mahesh Talewad <mahesh.talewad@nxp.com>
> ---
>  src/main.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/main.c b/src/main.c
> index f774670e4..081a388ad 100644
> --- a/src/main.c
> +++ b/src/main.c
> @@ -657,7 +657,7 @@ static void parse_le_config(GKeyFile *config)
>                   &btd_opts.defaults.le.autoconnect_timeout,
>                   sizeof(btd_opts.defaults.le.autoconnect_timeout),
>                   0x0001,
> -                 0x4000},
> +                 0x5000},  /*max val: 20480 msec*/
>                 { "AdvMonAllowlistScanDuration",
>                   &btd_opts.defaults.le.advmon_allowlist_scan_duration,
>                   sizeof(btd_opts.defaults.le.advmon_allowlist_scan_durat=
ion),
> --
> 2.34.1
>


--=20
Luiz Augusto von Dentz