[xenial] SRU: Bluetooth: increase timeout for le auto connections

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
13 messages Options
Reply | Threaded
Open this post in threaded view
|

[xenial] SRU: Bluetooth: increase timeout for le auto connections

Konrad Zapalowicz
SRU Justification:

[Impact]

Not possible to pair BLE remote devices such as sensors and such with
Ubuntu when the host system uses BT+WiFi combo SoC. The connection
attempt is disturbed with a timeout.

[Fix]

It has been fixed by increasing the timeout value from 2 seconds to
4 seconds. It is enough for the events to reach the stack (measured
that it takes between 3 and 3.5 seconds).

[Testcase]

Tested with the device that failed to connect to Ubuntu Core gateway.
It fails w/o the patch, it connects just fine with the patch applied.

[Regression Potential]

Very small. The increased timeout is taken into consideration only for
new and scan report triggered connections. It will not make any already
working device to fail to pair.

[Other Info]

The bug has been discussed and fixed here:

https://marc.info/?l=linux-bluetooth&m=150824844606937&w=2

https://git.kernel.org/pub/scm/linux/kernel/git/bluetooth/bluetooth-next.git/commit/?id=1f01d8be0e6a04bd682a55f6d50c14c1679e7571

The patch has been accepted by the upstream and will be a part of the
next kernel release. Currently in the bluetooth-next tree.

[Patch]

From 2d5a130b6cf6cbf992f9a7a66dcab8ab4e8e15f9 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Konrad=20Zapa=C5=82owicz?= <[hidden email]>
Date: Tue, 17 Oct 2017 15:53:49 +0200
Subject: [PATCH] [xenial] (upstream) Bluetooth: increase timeout for le auto
 connections
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

BugLink: https://bugs.launchpad.net/bugs/1731467

This patch increases the connection timeout for LE connections that are
triggered by the advertising report to 4 seconds.

It has been observed that devices equipped with wifi+bt combo SoC fail
to create a connection with BLE devices due to their coexistence issues.
Increasing this timeout gives them enough time to complete the
connection with success.

Signed-off-by: Konrad Zapałowicz <[hidden email]>
Signed-off-by: Marcel Holtmann <[hidden email]>
(cherry picked from commit 1f01d8be0e6a04bd682a55f6d50c14c1679e7571)
---
 include/net/bluetooth/hci.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/net/bluetooth/hci.h b/include/net/bluetooth/hci.h
index 0205b80..630f6d7 100644
--- a/include/net/bluetooth/hci.h
+++ b/include/net/bluetooth/hci.h
@@ -266,7 +266,7 @@ enum {
 #define HCI_AUTO_OFF_TIMEOUT msecs_to_jiffies(2000) /* 2 seconds */
 #define HCI_POWER_OFF_TIMEOUT msecs_to_jiffies(5000) /* 5 seconds */
 #define HCI_LE_CONN_TIMEOUT msecs_to_jiffies(20000) /* 20 seconds */
-#define HCI_LE_AUTOCONN_TIMEOUT msecs_to_jiffies(2000) /* 2 seconds */
+#define HCI_LE_AUTOCONN_TIMEOUT msecs_to_jiffies(4000) /* 4 seconds */
 
 /* HCI data types */
 #define HCI_COMMAND_PKT 0x01
--
2.7.4


--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|

Re: [xenial] SRU: Bluetooth: increase timeout for le auto connections

Seth Forshee
On Fri, Nov 10, 2017 at 01:11:05PM +0100, Konrad Zapalowicz wrote:

> SRU Justification:
>
> [Impact]
>
> Not possible to pair BLE remote devices such as sensors and such with
> Ubuntu when the host system uses BT+WiFi combo SoC. The connection
> attempt is disturbed with a timeout.
>
> [Fix]
>
> It has been fixed by increasing the timeout value from 2 seconds to
> 4 seconds. It is enough for the events to reach the stack (measured
> that it takes between 3 and 3.5 seconds).
>
> [Testcase]
>
> Tested with the device that failed to connect to Ubuntu Core gateway.
> It fails w/o the patch, it connects just fine with the patch applied.
>
> [Regression Potential]
>
> Very small. The increased timeout is taken into consideration only for
> new and scan report triggered connections. It will not make any already
> working device to fail to pair.
>
> [Other Info]
>
> The bug has been discussed and fixed here:
>
> https://marc.info/?l=linux-bluetooth&m=150824844606937&w=2
>
> https://git.kernel.org/pub/scm/linux/kernel/git/bluetooth/bluetooth-next.git/commit/?id=1f01d8be0e6a04bd682a55f6d50c14c1679e7571
>
> The patch has been accepted by the upstream and will be a part of the
> next kernel release. Currently in the bluetooth-next tree.

Typically you should include this information in a cover letter (use
'git format-patch --cover-letter ...', then edit the cover letter and
send the patches with 'git send-email'). It's more difficult to apply
patches when sent like this, however it's not my job to apply this patch
so I'll let others tell you whether or not you should resend it.

> [Patch]
>
> From 2d5a130b6cf6cbf992f9a7a66dcab8ab4e8e15f9 Mon Sep 17 00:00:00 2001
> From: =?UTF-8?q?Konrad=20Zapa=C5=82owicz?= <[hidden email]>
> Date: Tue, 17 Oct 2017 15:53:49 +0200
> Subject: [PATCH] [xenial] (upstream) Bluetooth: increase timeout for le auto
>  connections
> MIME-Version: 1.0
> Content-Type: text/plain; charset=UTF-8
> Content-Transfer-Encoding: 8bit
>
> BugLink: https://bugs.launchpad.net/bugs/1731467
>
> This patch increases the connection timeout for LE connections that are
> triggered by the advertising report to 4 seconds.
>
> It has been observed that devices equipped with wifi+bt combo SoC fail
> to create a connection with BLE devices due to their coexistence issues.
> Increasing this timeout gives them enough time to complete the
> connection with success.
>
> Signed-off-by: Konrad Zapałowicz <[hidden email]>
> Signed-off-by: Marcel Holtmann <[hidden email]>
> (cherry picked from commit 1f01d8be0e6a04bd682a55f6d50c14c1679e7571)

Since this patch is only in linux-next right now, this should be changed
to read:

 (cherry picked from commit 1f01d8be0e6a04bd682a55f6d50c14c1679e7571 linux-next)

Otherwise we assume the sha1 is from Linus' tree, but it doesn't exist
there yet. With that change:

Acked-by: Seth Forshee <[hidden email]>

But again, I'll let others say whether or not you need to resend to fix
this.

Thanks,
Seth

--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|

[xenial] SRU: [PATCH v2 0/0] Bluetooth: increase timeout for le auto connections

Konrad Zapalowicz
From: Konrad Zapałowicz <[hidden email]>

SRU Justification:

BugLink: https://bugs.launchpad.net/bugs/1731467

[Impact]

Not possible to pair BLE remote devices such as sensors and such with
Ubuntu when the host system uses BT+WiFi combo SoC. The connection
attempt is disturbed with a timeout.

[Fix]

It has been fixed by increasing the timeout value from 2 seconds to
4 seconds. It is enough for the events to reach the stack (measured
that it takes between 3 and 3.5 seconds).

[Testcase]

Tested with the device that failed to connect to Ubuntu Core gateway.
It fails w/o the patch, it connects just fine with the patch applied.

[Regression Potential]

Very small. The increased timeout is taken into consideration only for
new and scan report triggered connections. It will not make any already
working device to fail to pair.

[Other Info]

The bug has been discussed and fixed here:

https://marc.info/?l=linux-bluetooth&m=150824844606937&w=2

https://git.kernel.org/pub/scm/linux/kernel/git/bluetooth/bluetooth-next.git/commit/?id=1f01d8be0e6a04bd682a55f6d50c14c1679e7571

The patch has been accepted by the upstream and will be a part of the
next kernel release. Currently in the bluetooth-next tree.

Konrad Zapałowicz (1):
  [xenial] (upstream) Bluetooth: increase timeout for le auto
    connections

 include/net/bluetooth/hci.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

--
2.7.4


--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|

[xenial] SRU: [PATCH v2 1/1] Bluetooth: increase timeout for le auto connections

Konrad Zapalowicz
From: Konrad Zapałowicz <[hidden email]>

BugLink: https://bugs.launchpad.net/bugs/1731467

This patch increases the connection timeout for LE connections that are
triggered by the advertising report to 4 seconds.

It has been observed that devices equipped with wifi+bt combo SoC fail
to create a connection with BLE devices due to their coexistence issues.
Increasing this timeout gives them enough time to complete the
connection with success.

Signed-off-by: Konrad Zapałowicz <[hidden email]>
Signed-off-by: Marcel Holtmann <[hidden email]>
(cherry picked from commit 1f01d8be0e6a04bd682a55f6d50c14c1679e7571 linux-next)
---
 include/net/bluetooth/hci.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/net/bluetooth/hci.h b/include/net/bluetooth/hci.h
index 0205b80..630f6d7 100644
--- a/include/net/bluetooth/hci.h
+++ b/include/net/bluetooth/hci.h
@@ -266,7 +266,7 @@ enum {
 #define HCI_AUTO_OFF_TIMEOUT msecs_to_jiffies(2000) /* 2 seconds */
 #define HCI_POWER_OFF_TIMEOUT msecs_to_jiffies(5000) /* 5 seconds */
 #define HCI_LE_CONN_TIMEOUT msecs_to_jiffies(20000) /* 20 seconds */
-#define HCI_LE_AUTOCONN_TIMEOUT msecs_to_jiffies(2000) /* 2 seconds */
+#define HCI_LE_AUTOCONN_TIMEOUT msecs_to_jiffies(4000) /* 4 seconds */
 
 /* HCI data types */
 #define HCI_COMMAND_PKT 0x01
--
2.7.4


--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|

ACKcmnt: [xenial] SRU: Bluetooth: increase timeout for le auto connections

Kleber Souza
In reply to this post by Konrad Zapalowicz
On 11/10/17 13:11, Konrad Zapalowicz wrote:

> SRU Justification:
>
> [Impact]
>
> Not possible to pair BLE remote devices such as sensors and such with
> Ubuntu when the host system uses BT+WiFi combo SoC. The connection
> attempt is disturbed with a timeout.
>
> [Fix]
>
> It has been fixed by increasing the timeout value from 2 seconds to
> 4 seconds. It is enough for the events to reach the stack (measured
> that it takes between 3 and 3.5 seconds).
>
> [Testcase]
>
> Tested with the device that failed to connect to Ubuntu Core gateway.
> It fails w/o the patch, it connects just fine with the patch applied.
>
> [Regression Potential]
>
> Very small. The increased timeout is taken into consideration only for
> new and scan report triggered connections. It will not make any already
> working device to fail to pair.
>
> [Other Info]
>
> The bug has been discussed and fixed here:
>
> https://marc.info/?l=linux-bluetooth&m=150824844606937&w=2
>
> https://git.kernel.org/pub/scm/linux/kernel/git/bluetooth/bluetooth-next.git/commit/?id=1f01d8be0e6a04bd682a55f6d50c14c1679e7571
>
> The patch has been accepted by the upstream and will be a part of the
> next kernel release. Currently in the bluetooth-next tree.
>
> [Patch]
>
> From 2d5a130b6cf6cbf992f9a7a66dcab8ab4e8e15f9 Mon Sep 17 00:00:00 2001
> From: =?UTF-8?q?Konrad=20Zapa=C5=82owicz?= <[hidden email]>
> Date: Tue, 17 Oct 2017 15:53:49 +0200
> Subject: [PATCH] [xenial] (upstream) Bluetooth: increase timeout for le auto
>  connections
> MIME-Version: 1.0
> Content-Type: text/plain; charset=UTF-8
> Content-Transfer-Encoding: 8bit
>
> BugLink: https://bugs.launchpad.net/bugs/1731467
>
> This patch increases the connection timeout for LE connections that are
> triggered by the advertising report to 4 seconds.
>
> It has been observed that devices equipped with wifi+bt combo SoC fail
> to create a connection with BLE devices due to their coexistence issues.
> Increasing this timeout gives them enough time to complete the
> connection with success.
>
> Signed-off-by: Konrad Zapałowicz <[hidden email]>
> Signed-off-by: Marcel Holtmann <[hidden email]>
> (cherry picked from commit 1f01d8be0e6a04bd682a55f6d50c14c1679e7571)
> ---
>  include/net/bluetooth/hci.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/include/net/bluetooth/hci.h b/include/net/bluetooth/hci.h
> index 0205b80..630f6d7 100644
> --- a/include/net/bluetooth/hci.h
> +++ b/include/net/bluetooth/hci.h
> @@ -266,7 +266,7 @@ enum {
>  #define HCI_AUTO_OFF_TIMEOUT msecs_to_jiffies(2000) /* 2 seconds */
>  #define HCI_POWER_OFF_TIMEOUT msecs_to_jiffies(5000) /* 5 seconds */
>  #define HCI_LE_CONN_TIMEOUT msecs_to_jiffies(20000) /* 20 seconds */
> -#define HCI_LE_AUTOCONN_TIMEOUT msecs_to_jiffies(2000) /* 2 seconds */
> +#define HCI_LE_AUTOCONN_TIMEOUT msecs_to_jiffies(4000) /* 4 seconds */
>  
>  /* HCI data types */
>  #define HCI_COMMAND_PKT 0x01
>

As pointed out by Seth, there are some issues with this patch
submission, but we can fix them while applying it. Please follow these
guidelines when sending future patches.

When submitting a patch to fix a bug in the kernel, the Launchpad bug
needs to be marked as affecting the "linux (Ubuntu)" package and
nominated for the relevant series (Xenial in this case). I have fixed it
on LP:#1731467.

Given that the issues pointed out by Seth is fixed while applying the patch:
Acked-by: Kleber Sacilotto de Souza <[hidden email]>

--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|

ACK/cmnt: [xenial] SRU: [PATCH v2 1/1] Bluetooth: increase timeout for le auto connections

Stefan Bader-2
In reply to this post by Konrad Zapalowicz
On 14.11.2017 11:51, Konrad Zapalowicz wrote:

> From: Konrad Zapałowicz <[hidden email]>
>
> BugLink: https://bugs.launchpad.net/bugs/1731467
>
> This patch increases the connection timeout for LE connections that are
> triggered by the advertising report to 4 seconds.
>
> It has been observed that devices equipped with wifi+bt combo SoC fail
> to create a connection with BLE devices due to their coexistence issues.
> Increasing this timeout gives them enough time to complete the
> connection with success.
>
> Signed-off-by: Konrad Zapałowicz <[hidden email]>
> Signed-off-by: Marcel Holtmann <[hidden email]>
> (cherry picked from commit 1f01d8be0e6a04bd682a55f6d50c14c1679e7571 linux-next)
Acked-by: Stefan Bader <[hidden email]>

> ---

How about at least also targeting this to Artful (and maybe Zesty, too) so that
this does not break for people switching to HWE kernels?

-Stefan

>  include/net/bluetooth/hci.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/include/net/bluetooth/hci.h b/include/net/bluetooth/hci.h
> index 0205b80..630f6d7 100644
> --- a/include/net/bluetooth/hci.h
> +++ b/include/net/bluetooth/hci.h
> @@ -266,7 +266,7 @@ enum {
>  #define HCI_AUTO_OFF_TIMEOUT msecs_to_jiffies(2000) /* 2 seconds */
>  #define HCI_POWER_OFF_TIMEOUT msecs_to_jiffies(5000) /* 5 seconds */
>  #define HCI_LE_CONN_TIMEOUT msecs_to_jiffies(20000) /* 20 seconds */
> -#define HCI_LE_AUTOCONN_TIMEOUT msecs_to_jiffies(2000) /* 2 seconds */
> +#define HCI_LE_AUTOCONN_TIMEOUT msecs_to_jiffies(4000) /* 4 seconds */
>  
>  /* HCI data types */
>  #define HCI_COMMAND_PKT 0x01
>


--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team

signature.asc (836 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: ACK/cmnt: [xenial] SRU: [PATCH v2 1/1] Bluetooth: increase timeout for le auto connections

Konrad Zapalowicz
On 11/15, Stefan Bader wrote:

> On 14.11.2017 11:51, Konrad Zapalowicz wrote:
> > From: Konrad Zapałowicz <[hidden email]>
> >
> > BugLink: https://bugs.launchpad.net/bugs/1731467
> >
> > This patch increases the connection timeout for LE connections that are
> > triggered by the advertising report to 4 seconds.
> >
> > It has been observed that devices equipped with wifi+bt combo SoC fail
> > to create a connection with BLE devices due to their coexistence issues.
> > Increasing this timeout gives them enough time to complete the
> > connection with success.
> >
> > Signed-off-by: Konrad Zapałowicz <[hidden email]>
> > Signed-off-by: Marcel Holtmann <[hidden email]>
> > (cherry picked from commit 1f01d8be0e6a04bd682a55f6d50c14c1679e7571 linux-next)
> Acked-by: Stefan Bader <[hidden email]>
>
> > ---
>
> How about at least also targeting this to Artful (and maybe Zesty, too) so that
> this does not break for people switching to HWE kernels?

From my point of view it is a good idea.

/K

>
> -Stefan
> >  include/net/bluetooth/hci.h | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/include/net/bluetooth/hci.h b/include/net/bluetooth/hci.h
> > index 0205b80..630f6d7 100644
> > --- a/include/net/bluetooth/hci.h
> > +++ b/include/net/bluetooth/hci.h
> > @@ -266,7 +266,7 @@ enum {
> >  #define HCI_AUTO_OFF_TIMEOUT msecs_to_jiffies(2000) /* 2 seconds */
> >  #define HCI_POWER_OFF_TIMEOUT msecs_to_jiffies(5000) /* 5 seconds */
> >  #define HCI_LE_CONN_TIMEOUT msecs_to_jiffies(20000) /* 20 seconds */
> > -#define HCI_LE_AUTOCONN_TIMEOUT msecs_to_jiffies(2000) /* 2 seconds */
> > +#define HCI_LE_AUTOCONN_TIMEOUT msecs_to_jiffies(4000) /* 4 seconds */
> >  
> >  /* HCI data types */
> >  #define HCI_COMMAND_PKT 0x01
> >
>
>




--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|

Re: ACKcmnt: [xenial] SRU: Bluetooth: increase timeout for le auto connections

Konrad Zapalowicz
In reply to this post by Kleber Souza
On 11/14, Kleber Souza wrote:

> On 11/10/17 13:11, Konrad Zapalowicz wrote:
> > SRU Justification:
> >
> > [Impact]
> >
> > Not possible to pair BLE remote devices such as sensors and such with
> > Ubuntu when the host system uses BT+WiFi combo SoC. The connection
> > attempt is disturbed with a timeout.
> >
> > [Fix]
> >
> > It has been fixed by increasing the timeout value from 2 seconds to
> > 4 seconds. It is enough for the events to reach the stack (measured
> > that it takes between 3 and 3.5 seconds).
> >
> > [Testcase]
> >
> > Tested with the device that failed to connect to Ubuntu Core gateway.
> > It fails w/o the patch, it connects just fine with the patch applied.
> >
> > [Regression Potential]
> >
> > Very small. The increased timeout is taken into consideration only for
> > new and scan report triggered connections. It will not make any already
> > working device to fail to pair.
> >
> > [Other Info]
> >
> > The bug has been discussed and fixed here:
> >
> > https://marc.info/?l=linux-bluetooth&m=150824844606937&w=2
> >
> > https://git.kernel.org/pub/scm/linux/kernel/git/bluetooth/bluetooth-next.git/commit/?id=1f01d8be0e6a04bd682a55f6d50c14c1679e7571
> >
> > The patch has been accepted by the upstream and will be a part of the
> > next kernel release. Currently in the bluetooth-next tree.
> >
> > [Patch]
> >
> > From 2d5a130b6cf6cbf992f9a7a66dcab8ab4e8e15f9 Mon Sep 17 00:00:00 2001
> > From: =?UTF-8?q?Konrad=20Zapa=C5=82owicz?= <[hidden email]>
> > Date: Tue, 17 Oct 2017 15:53:49 +0200
> > Subject: [PATCH] [xenial] (upstream) Bluetooth: increase timeout for le auto
> >  connections
> > MIME-Version: 1.0
> > Content-Type: text/plain; charset=UTF-8
> > Content-Transfer-Encoding: 8bit
> >
> > BugLink: https://bugs.launchpad.net/bugs/1731467
> >
> > This patch increases the connection timeout for LE connections that are
> > triggered by the advertising report to 4 seconds.
> >
> > It has been observed that devices equipped with wifi+bt combo SoC fail
> > to create a connection with BLE devices due to their coexistence issues.
> > Increasing this timeout gives them enough time to complete the
> > connection with success.
> >
> > Signed-off-by: Konrad Zapałowicz <[hidden email]>
> > Signed-off-by: Marcel Holtmann <[hidden email]>
> > (cherry picked from commit 1f01d8be0e6a04bd682a55f6d50c14c1679e7571)
> > ---
> >  include/net/bluetooth/hci.h | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/include/net/bluetooth/hci.h b/include/net/bluetooth/hci.h
> > index 0205b80..630f6d7 100644
> > --- a/include/net/bluetooth/hci.h
> > +++ b/include/net/bluetooth/hci.h
> > @@ -266,7 +266,7 @@ enum {
> >  #define HCI_AUTO_OFF_TIMEOUT msecs_to_jiffies(2000) /* 2 seconds */
> >  #define HCI_POWER_OFF_TIMEOUT msecs_to_jiffies(5000) /* 5 seconds */
> >  #define HCI_LE_CONN_TIMEOUT msecs_to_jiffies(20000) /* 20 seconds */
> > -#define HCI_LE_AUTOCONN_TIMEOUT msecs_to_jiffies(2000) /* 2 seconds */
> > +#define HCI_LE_AUTOCONN_TIMEOUT msecs_to_jiffies(4000) /* 4 seconds */
> >  
> >  /* HCI data types */
> >  #define HCI_COMMAND_PKT 0x01
> >
>
> As pointed out by Seth, there are some issues with this patch
> submission, but we can fix them while applying it. Please follow these
> guidelines when sending future patches.
>
> When submitting a patch to fix a bug in the kernel, the Launchpad bug
> needs to be marked as affecting the "linux (Ubuntu)" package and
> nominated for the relevant series (Xenial in this case). I have fixed it
> on LP:#1731467.
>
> Given that the issues pointed out by Seth is fixed while applying the patch:
> Acked-by: Kleber Sacilotto de Souza <[hidden email]>

Thanks. So it has been merged, right?

/Konrad

--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|

Re: ACKcmnt: [xenial] SRU: Bluetooth: increase timeout for le auto connections

Kleber Souza
On 11/17/17 08:47, Konrad Zapalowicz wrote:

> On 11/14, Kleber Souza wrote:
>> On 11/10/17 13:11, Konrad Zapalowicz wrote:
>>> SRU Justification:
>>>
>>> [Impact]
>>>
>>> Not possible to pair BLE remote devices such as sensors and such with
>>> Ubuntu when the host system uses BT+WiFi combo SoC. The connection
>>> attempt is disturbed with a timeout.
>>>
>>> [Fix]
>>>
>>> It has been fixed by increasing the timeout value from 2 seconds to
>>> 4 seconds. It is enough for the events to reach the stack (measured
>>> that it takes between 3 and 3.5 seconds).
>>>
>>> [Testcase]
>>>
>>> Tested with the device that failed to connect to Ubuntu Core gateway.
>>> It fails w/o the patch, it connects just fine with the patch applied.
>>>
>>> [Regression Potential]
>>>
>>> Very small. The increased timeout is taken into consideration only for
>>> new and scan report triggered connections. It will not make any already
>>> working device to fail to pair.
>>>
>>> [Other Info]
>>>
>>> The bug has been discussed and fixed here:
>>>
>>> https://marc.info/?l=linux-bluetooth&m=150824844606937&w=2
>>>
>>> https://git.kernel.org/pub/scm/linux/kernel/git/bluetooth/bluetooth-next.git/commit/?id=1f01d8be0e6a04bd682a55f6d50c14c1679e7571
>>>
>>> The patch has been accepted by the upstream and will be a part of the
>>> next kernel release. Currently in the bluetooth-next tree.
>>>
>>> [Patch]
>>>
>>> From 2d5a130b6cf6cbf992f9a7a66dcab8ab4e8e15f9 Mon Sep 17 00:00:00 2001
>>> From: =?UTF-8?q?Konrad=20Zapa=C5=82owicz?= <[hidden email]>
>>> Date: Tue, 17 Oct 2017 15:53:49 +0200
>>> Subject: [PATCH] [xenial] (upstream) Bluetooth: increase timeout for le auto
>>>  connections
>>> MIME-Version: 1.0
>>> Content-Type: text/plain; charset=UTF-8
>>> Content-Transfer-Encoding: 8bit
>>>
>>> BugLink: https://bugs.launchpad.net/bugs/1731467
>>>
>>> This patch increases the connection timeout for LE connections that are
>>> triggered by the advertising report to 4 seconds.
>>>
>>> It has been observed that devices equipped with wifi+bt combo SoC fail
>>> to create a connection with BLE devices due to their coexistence issues.
>>> Increasing this timeout gives them enough time to complete the
>>> connection with success.
>>>
>>> Signed-off-by: Konrad Zapałowicz <[hidden email]>
>>> Signed-off-by: Marcel Holtmann <[hidden email]>
>>> (cherry picked from commit 1f01d8be0e6a04bd682a55f6d50c14c1679e7571)
>>> ---
>>>  include/net/bluetooth/hci.h | 2 +-
>>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/include/net/bluetooth/hci.h b/include/net/bluetooth/hci.h
>>> index 0205b80..630f6d7 100644
>>> --- a/include/net/bluetooth/hci.h
>>> +++ b/include/net/bluetooth/hci.h
>>> @@ -266,7 +266,7 @@ enum {
>>>  #define HCI_AUTO_OFF_TIMEOUT msecs_to_jiffies(2000) /* 2 seconds */
>>>  #define HCI_POWER_OFF_TIMEOUT msecs_to_jiffies(5000) /* 5 seconds */
>>>  #define HCI_LE_CONN_TIMEOUT msecs_to_jiffies(20000) /* 20 seconds */
>>> -#define HCI_LE_AUTOCONN_TIMEOUT msecs_to_jiffies(2000) /* 2 seconds */
>>> +#define HCI_LE_AUTOCONN_TIMEOUT msecs_to_jiffies(4000) /* 4 seconds */
>>>  
>>>  /* HCI data types */
>>>  #define HCI_COMMAND_PKT 0x01
>>>
>>
>> As pointed out by Seth, there are some issues with this patch
>> submission, but we can fix them while applying it. Please follow these
>> guidelines when sending future patches.
>>
>> When submitting a patch to fix a bug in the kernel, the Launchpad bug
>> needs to be marked as affecting the "linux (Ubuntu)" package and
>> nominated for the relevant series (Xenial in this case). I have fixed it
>> on LP:#1731467.
>>
>> Given that the issues pointed out by Seth is fixed while applying the patch:
>> Acked-by: Kleber Sacilotto de Souza <[hidden email]>
>
> Thanks. So it has been merged, right?
>
> /Konrad
>

Hi,

The patch hasn't been applied to the Xenial tree yet, when that's done
we will send a reply with a "APPLIED" tag on the subject. However, it's
got the necessary ACK's on the mailing list and the next SRU cycle
begins next week, so it will be applied soon and the fix will become
available on the next kernel update.

Regards,
Kleber

--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|

Re: ACKcmnt: [xenial] SRU: Bluetooth: increase timeout for le auto connections

Konrad Zapalowicz
On Fri, Nov 17, 2017 at 10:57 AM, Kleber Souza
<[hidden email]> wrote:

> On 11/17/17 08:47, Konrad Zapalowicz wrote:
>> On 11/14, Kleber Souza wrote:
>>> On 11/10/17 13:11, Konrad Zapalowicz wrote:
>>>> SRU Justification:
>>>>
>>>> [Impact]
>>>>
>>>> Not possible to pair BLE remote devices such as sensors and such with
>>>> Ubuntu when the host system uses BT+WiFi combo SoC. The connection
>>>> attempt is disturbed with a timeout.
>>>>
>>>> [Fix]
>>>>
>>>> It has been fixed by increasing the timeout value from 2 seconds to
>>>> 4 seconds. It is enough for the events to reach the stack (measured
>>>> that it takes between 3 and 3.5 seconds).
>>>>
>>>> [Testcase]
>>>>
>>>> Tested with the device that failed to connect to Ubuntu Core gateway.
>>>> It fails w/o the patch, it connects just fine with the patch applied.
>>>>
>>>> [Regression Potential]
>>>>
>>>> Very small. The increased timeout is taken into consideration only for
>>>> new and scan report triggered connections. It will not make any already
>>>> working device to fail to pair.
>>>>
>>>> [Other Info]
>>>>
>>>> The bug has been discussed and fixed here:
>>>>
>>>> https://marc.info/?l=linux-bluetooth&m=150824844606937&w=2
>>>>
>>>> https://git.kernel.org/pub/scm/linux/kernel/git/bluetooth/bluetooth-next.git/commit/?id=1f01d8be0e6a04bd682a55f6d50c14c1679e7571
>>>>
>>>> The patch has been accepted by the upstream and will be a part of the
>>>> next kernel release. Currently in the bluetooth-next tree.
>>>>
>>>> [Patch]
>>>>
>>>> From 2d5a130b6cf6cbf992f9a7a66dcab8ab4e8e15f9 Mon Sep 17 00:00:00 2001
>>>> From: =?UTF-8?q?Konrad=20Zapa=C5=82owicz?= <[hidden email]>
>>>> Date: Tue, 17 Oct 2017 15:53:49 +0200
>>>> Subject: [PATCH] [xenial] (upstream) Bluetooth: increase timeout for le auto
>>>>  connections
>>>> MIME-Version: 1.0
>>>> Content-Type: text/plain; charset=UTF-8
>>>> Content-Transfer-Encoding: 8bit
>>>>
>>>> BugLink: https://bugs.launchpad.net/bugs/1731467
>>>>
>>>> This patch increases the connection timeout for LE connections that are
>>>> triggered by the advertising report to 4 seconds.
>>>>
>>>> It has been observed that devices equipped with wifi+bt combo SoC fail
>>>> to create a connection with BLE devices due to their coexistence issues.
>>>> Increasing this timeout gives them enough time to complete the
>>>> connection with success.
>>>>
>>>> Signed-off-by: Konrad Zapałowicz <[hidden email]>
>>>> Signed-off-by: Marcel Holtmann <[hidden email]>
>>>> (cherry picked from commit 1f01d8be0e6a04bd682a55f6d50c14c1679e7571)
>>>> ---
>>>>  include/net/bluetooth/hci.h | 2 +-
>>>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>>>
>>>> diff --git a/include/net/bluetooth/hci.h b/include/net/bluetooth/hci.h
>>>> index 0205b80..630f6d7 100644
>>>> --- a/include/net/bluetooth/hci.h
>>>> +++ b/include/net/bluetooth/hci.h
>>>> @@ -266,7 +266,7 @@ enum {
>>>>  #define HCI_AUTO_OFF_TIMEOUT       msecs_to_jiffies(2000)  /* 2 seconds */
>>>>  #define HCI_POWER_OFF_TIMEOUT      msecs_to_jiffies(5000)  /* 5 seconds */
>>>>  #define HCI_LE_CONN_TIMEOUT        msecs_to_jiffies(20000) /* 20 seconds */
>>>> -#define HCI_LE_AUTOCONN_TIMEOUT    msecs_to_jiffies(2000)  /* 2 seconds */
>>>> +#define HCI_LE_AUTOCONN_TIMEOUT    msecs_to_jiffies(4000)  /* 4 seconds */
>>>>
>>>>  /* HCI data types */
>>>>  #define HCI_COMMAND_PKT            0x01
>>>>
>>>
>>> As pointed out by Seth, there are some issues with this patch
>>> submission, but we can fix them while applying it. Please follow these
>>> guidelines when sending future patches.
>>>
>>> When submitting a patch to fix a bug in the kernel, the Launchpad bug
>>> needs to be marked as affecting the "linux (Ubuntu)" package and
>>> nominated for the relevant series (Xenial in this case). I have fixed it
>>> on LP:#1731467.
>>>
>>> Given that the issues pointed out by Seth is fixed while applying the patch:
>>> Acked-by: Kleber Sacilotto de Souza <[hidden email]>
>>
>> Thanks. So it has been merged, right?
>>
>> /Konrad
>>
>
> Hi,
>
> The patch hasn't been applied to the Xenial tree yet, when that's done
> we will send a reply with a "APPLIED" tag on the subject. However, it's
> got the necessary ACK's on the mailing list and the next SRU cycle
> begins next week, so it will be applied soon and the fix will become
> available on the next kernel update.

December 11th, right?

Sorry for pushing however I need to notify the customer about when the change
will be available. This one is highly expected.

Thanks,
Konrad

>
> Regards,
> Kleber

--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|

Re: ACKcmnt: [xenial] SRU: Bluetooth: increase timeout for le auto connections

Kleber Souza


On 11/17/17 11:23, Konrad Zapałowicz wrote:

> On Fri, Nov 17, 2017 at 10:57 AM, Kleber Souza
> <[hidden email]> wrote:
>> On 11/17/17 08:47, Konrad Zapalowicz wrote:
>>> On 11/14, Kleber Souza wrote:
>>>> On 11/10/17 13:11, Konrad Zapalowicz wrote:
>>>>> SRU Justification:
>>>>>
>>>>> [Impact]
>>>>>
>>>>> Not possible to pair BLE remote devices such as sensors and such with
>>>>> Ubuntu when the host system uses BT+WiFi combo SoC. The connection
>>>>> attempt is disturbed with a timeout.
>>>>>
>>>>> [Fix]
>>>>>
>>>>> It has been fixed by increasing the timeout value from 2 seconds to
>>>>> 4 seconds. It is enough for the events to reach the stack (measured
>>>>> that it takes between 3 and 3.5 seconds).
>>>>>
>>>>> [Testcase]
>>>>>
>>>>> Tested with the device that failed to connect to Ubuntu Core gateway.
>>>>> It fails w/o the patch, it connects just fine with the patch applied.
>>>>>
>>>>> [Regression Potential]
>>>>>
>>>>> Very small. The increased timeout is taken into consideration only for
>>>>> new and scan report triggered connections. It will not make any already
>>>>> working device to fail to pair.
>>>>>
>>>>> [Other Info]
>>>>>
>>>>> The bug has been discussed and fixed here:
>>>>>
>>>>> https://marc.info/?l=linux-bluetooth&m=150824844606937&w=2
>>>>>
>>>>> https://git.kernel.org/pub/scm/linux/kernel/git/bluetooth/bluetooth-next.git/commit/?id=1f01d8be0e6a04bd682a55f6d50c14c1679e7571
>>>>>
>>>>> The patch has been accepted by the upstream and will be a part of the
>>>>> next kernel release. Currently in the bluetooth-next tree.
>>>>>
>>>>> [Patch]
>>>>>
>>>>> From 2d5a130b6cf6cbf992f9a7a66dcab8ab4e8e15f9 Mon Sep 17 00:00:00 2001
>>>>> From: =?UTF-8?q?Konrad=20Zapa=C5=82owicz?= <[hidden email]>
>>>>> Date: Tue, 17 Oct 2017 15:53:49 +0200
>>>>> Subject: [PATCH] [xenial] (upstream) Bluetooth: increase timeout for le auto
>>>>>  connections
>>>>> MIME-Version: 1.0
>>>>> Content-Type: text/plain; charset=UTF-8
>>>>> Content-Transfer-Encoding: 8bit
>>>>>
>>>>> BugLink: https://bugs.launchpad.net/bugs/1731467
>>>>>
>>>>> This patch increases the connection timeout for LE connections that are
>>>>> triggered by the advertising report to 4 seconds.
>>>>>
>>>>> It has been observed that devices equipped with wifi+bt combo SoC fail
>>>>> to create a connection with BLE devices due to their coexistence issues.
>>>>> Increasing this timeout gives them enough time to complete the
>>>>> connection with success.
>>>>>
>>>>> Signed-off-by: Konrad Zapałowicz <[hidden email]>
>>>>> Signed-off-by: Marcel Holtmann <[hidden email]>
>>>>> (cherry picked from commit 1f01d8be0e6a04bd682a55f6d50c14c1679e7571)
>>>>> ---
>>>>>  include/net/bluetooth/hci.h | 2 +-
>>>>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>>>>
>>>>> diff --git a/include/net/bluetooth/hci.h b/include/net/bluetooth/hci.h
>>>>> index 0205b80..630f6d7 100644
>>>>> --- a/include/net/bluetooth/hci.h
>>>>> +++ b/include/net/bluetooth/hci.h
>>>>> @@ -266,7 +266,7 @@ enum {
>>>>>  #define HCI_AUTO_OFF_TIMEOUT       msecs_to_jiffies(2000)  /* 2 seconds */
>>>>>  #define HCI_POWER_OFF_TIMEOUT      msecs_to_jiffies(5000)  /* 5 seconds */
>>>>>  #define HCI_LE_CONN_TIMEOUT        msecs_to_jiffies(20000) /* 20 seconds */
>>>>> -#define HCI_LE_AUTOCONN_TIMEOUT    msecs_to_jiffies(2000)  /* 2 seconds */
>>>>> +#define HCI_LE_AUTOCONN_TIMEOUT    msecs_to_jiffies(4000)  /* 4 seconds */
>>>>>
>>>>>  /* HCI data types */
>>>>>  #define HCI_COMMAND_PKT            0x01
>>>>>
>>>>
>>>> As pointed out by Seth, there are some issues with this patch
>>>> submission, but we can fix them while applying it. Please follow these
>>>> guidelines when sending future patches.
>>>>
>>>> When submitting a patch to fix a bug in the kernel, the Launchpad bug
>>>> needs to be marked as affecting the "linux (Ubuntu)" package and
>>>> nominated for the relevant series (Xenial in this case). I have fixed it
>>>> on LP:#1731467.
>>>>
>>>> Given that the issues pointed out by Seth is fixed while applying the patch:
>>>> Acked-by: Kleber Sacilotto de Souza <[hidden email]>
>>>
>>> Thanks. So it has been merged, right?
>>>
>>> /Konrad
>>>
>>
>> Hi,
>>
>> The patch hasn't been applied to the Xenial tree yet, when that's done
>> we will send a reply with a "APPLIED" tag on the subject. However, it's
>> got the necessary ACK's on the mailing list and the next SRU cycle
>> begins next week, so it will be applied soon and the fix will become
>> available on the next kernel update.
>
> December 11th, right?

Right, that's the scheduled date for the next SRU cycle going to -updates.

>
> Sorry for pushing however I need to notify the customer about when the change
> will be available. This one is highly expected.
>
> Thanks,
> Konrad
>
>>
>> Regards,
>> Kleber

--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|

APPLIED[X/Z/A]: [xenial] SRU: Bluetooth: increase timeout for le auto connections

Stefan Bader-2
In reply to this post by Konrad Zapalowicz
On 10.11.2017 13:11, Konrad Zapalowicz wrote:

> SRU Justification:
>
> [Impact]
>
> Not possible to pair BLE remote devices such as sensors and such with
> Ubuntu when the host system uses BT+WiFi combo SoC. The connection
> attempt is disturbed with a timeout.
>
> [Fix]
>
> It has been fixed by increasing the timeout value from 2 seconds to
> 4 seconds. It is enough for the events to reach the stack (measured
> that it takes between 3 and 3.5 seconds).
>
> [Testcase]
>
> Tested with the device that failed to connect to Ubuntu Core gateway.
> It fails w/o the patch, it connects just fine with the patch applied.
>
> [Regression Potential]
>
> Very small. The increased timeout is taken into consideration only for
> new and scan report triggered connections. It will not make any already
> working device to fail to pair.
>
> [Other Info]
>
> The bug has been discussed and fixed here:
>
> https://marc.info/?l=linux-bluetooth&m=150824844606937&w=2
>
> https://git.kernel.org/pub/scm/linux/kernel/git/bluetooth/bluetooth-next.git/commit/?id=1f01d8be0e6a04bd682a55f6d50c14c1679e7571
>
> The patch has been accepted by the upstream and will be a part of the
> next kernel release. Currently in the bluetooth-next tree.
>
> [Patch]
>
> From 2d5a130b6cf6cbf992f9a7a66dcab8ab4e8e15f9 Mon Sep 17 00:00:00 2001
> From: =?UTF-8?q?Konrad=20Zapa=C5=82owicz?= <[hidden email]>
> Date: Tue, 17 Oct 2017 15:53:49 +0200
> Subject: [PATCH] [xenial] (upstream) Bluetooth: increase timeout for le auto
>  connections
> MIME-Version: 1.0
> Content-Type: text/plain; charset=UTF-8
> Content-Transfer-Encoding: 8bit
>
> BugLink: https://bugs.launchpad.net/bugs/1731467
>
> This patch increases the connection timeout for LE connections that are
> triggered by the advertising report to 4 seconds.
>
> It has been observed that devices equipped with wifi+bt combo SoC fail
> to create a connection with BLE devices due to their coexistence issues.
> Increasing this timeout gives them enough time to complete the
> connection with success.
>
> Signed-off-by: Konrad Zapałowicz <[hidden email]>
> Signed-off-by: Marcel Holtmann <[hidden email]>
> (cherry picked from commit 1f01d8be0e6a04bd682a55f6d50c14c1679e7571)
> ---
>  include/net/bluetooth/hci.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/include/net/bluetooth/hci.h b/include/net/bluetooth/hci.h
> index 0205b80..630f6d7 100644
> --- a/include/net/bluetooth/hci.h
> +++ b/include/net/bluetooth/hci.h
> @@ -266,7 +266,7 @@ enum {
>  #define HCI_AUTO_OFF_TIMEOUT msecs_to_jiffies(2000) /* 2 seconds */
>  #define HCI_POWER_OFF_TIMEOUT msecs_to_jiffies(5000) /* 5 seconds */
>  #define HCI_LE_CONN_TIMEOUT msecs_to_jiffies(20000) /* 20 seconds */
> -#define HCI_LE_AUTOCONN_TIMEOUT msecs_to_jiffies(2000) /* 2 seconds */
> +#define HCI_LE_AUTOCONN_TIMEOUT msecs_to_jiffies(4000) /* 4 seconds */
>  
>  /* HCI data types */
>  #define HCI_COMMAND_PKT 0x01
>
The fix is upstream by now. So I took the liberty to do a real cherry-pick and
also to apply the same patch across Xenial to Artful as otherwise there would be
a risk of getting regressions when using HWE kernels.
I added the additional tasks to the bug report as well.

-Stefan


--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team

signature.asc (836 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: ACKcmnt: [xenial] SRU: Bluetooth: increase timeout for le auto connections

Stefan Bader-2
In reply to this post by Konrad Zapalowicz
On 17.11.2017 11:23, Konrad Zapałowicz wrote:

> On Fri, Nov 17, 2017 at 10:57 AM, Kleber Souza
> <[hidden email]> wrote:
>> On 11/17/17 08:47, Konrad Zapalowicz wrote:
>>> On 11/14, Kleber Souza wrote:
>>>> On 11/10/17 13:11, Konrad Zapalowicz wrote:
>>>>> SRU Justification:
>>>>>
>>>>> [Impact]
>>>>>
>>>>> Not possible to pair BLE remote devices such as sensors and such with
>>>>> Ubuntu when the host system uses BT+WiFi combo SoC. The connection
>>>>> attempt is disturbed with a timeout.
>>>>>
>>>>> [Fix]
>>>>>
>>>>> It has been fixed by increasing the timeout value from 2 seconds to
>>>>> 4 seconds. It is enough for the events to reach the stack (measured
>>>>> that it takes between 3 and 3.5 seconds).
>>>>>
>>>>> [Testcase]
>>>>>
>>>>> Tested with the device that failed to connect to Ubuntu Core gateway.
>>>>> It fails w/o the patch, it connects just fine with the patch applied.
>>>>>
>>>>> [Regression Potential]
>>>>>
>>>>> Very small. The increased timeout is taken into consideration only for
>>>>> new and scan report triggered connections. It will not make any already
>>>>> working device to fail to pair.
>>>>>
>>>>> [Other Info]
>>>>>
>>>>> The bug has been discussed and fixed here:
>>>>>
>>>>> https://marc.info/?l=linux-bluetooth&m=150824844606937&w=2
>>>>>
>>>>> https://git.kernel.org/pub/scm/linux/kernel/git/bluetooth/bluetooth-next.git/commit/?id=1f01d8be0e6a04bd682a55f6d50c14c1679e7571
>>>>>
>>>>> The patch has been accepted by the upstream and will be a part of the
>>>>> next kernel release. Currently in the bluetooth-next tree.
>>>>>
>>>>> [Patch]
>>>>>
>>>>> From 2d5a130b6cf6cbf992f9a7a66dcab8ab4e8e15f9 Mon Sep 17 00:00:00 2001
>>>>> From: =?UTF-8?q?Konrad=20Zapa=C5=82owicz?= <[hidden email]>
>>>>> Date: Tue, 17 Oct 2017 15:53:49 +0200
>>>>> Subject: [PATCH] [xenial] (upstream) Bluetooth: increase timeout for le auto
>>>>>  connections
>>>>> MIME-Version: 1.0
>>>>> Content-Type: text/plain; charset=UTF-8
>>>>> Content-Transfer-Encoding: 8bit
>>>>>
>>>>> BugLink: https://bugs.launchpad.net/bugs/1731467
>>>>>
>>>>> This patch increases the connection timeout for LE connections that are
>>>>> triggered by the advertising report to 4 seconds.
>>>>>
>>>>> It has been observed that devices equipped with wifi+bt combo SoC fail
>>>>> to create a connection with BLE devices due to their coexistence issues.
>>>>> Increasing this timeout gives them enough time to complete the
>>>>> connection with success.
>>>>>
>>>>> Signed-off-by: Konrad Zapałowicz <[hidden email]>
>>>>> Signed-off-by: Marcel Holtmann <[hidden email]>
>>>>> (cherry picked from commit 1f01d8be0e6a04bd682a55f6d50c14c1679e7571)
>>>>> ---
>>>>>  include/net/bluetooth/hci.h | 2 +-
>>>>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>>>>
>>>>> diff --git a/include/net/bluetooth/hci.h b/include/net/bluetooth/hci.h
>>>>> index 0205b80..630f6d7 100644
>>>>> --- a/include/net/bluetooth/hci.h
>>>>> +++ b/include/net/bluetooth/hci.h
>>>>> @@ -266,7 +266,7 @@ enum {
>>>>>  #define HCI_AUTO_OFF_TIMEOUT       msecs_to_jiffies(2000)  /* 2 seconds */
>>>>>  #define HCI_POWER_OFF_TIMEOUT      msecs_to_jiffies(5000)  /* 5 seconds */
>>>>>  #define HCI_LE_CONN_TIMEOUT        msecs_to_jiffies(20000) /* 20 seconds */
>>>>> -#define HCI_LE_AUTOCONN_TIMEOUT    msecs_to_jiffies(2000)  /* 2 seconds */
>>>>> +#define HCI_LE_AUTOCONN_TIMEOUT    msecs_to_jiffies(4000)  /* 4 seconds */
>>>>>
>>>>>  /* HCI data types */
>>>>>  #define HCI_COMMAND_PKT            0x01
>>>>>
>>>>
>>>> As pointed out by Seth, there are some issues with this patch
>>>> submission, but we can fix them while applying it. Please follow these
>>>> guidelines when sending future patches.
>>>>
>>>> When submitting a patch to fix a bug in the kernel, the Launchpad bug
>>>> needs to be marked as affecting the "linux (Ubuntu)" package and
>>>> nominated for the relevant series (Xenial in this case). I have fixed it
>>>> on LP:#1731467.
>>>>
>>>> Given that the issues pointed out by Seth is fixed while applying the patch:
>>>> Acked-by: Kleber Sacilotto de Souza <[hidden email]>
>>>
>>> Thanks. So it has been merged, right?
>>>
>>> /Konrad
>>>
>>
>> Hi,
>>
>> The patch hasn't been applied to the Xenial tree yet, when that's done
>> we will send a reply with a "APPLIED" tag on the subject. However, it's
>> got the necessary ACK's on the mailing list and the next SRU cycle
>> begins next week, so it will be applied soon and the fix will become
>> available on the next kernel update.
>
> December 11th, right?
>
> Sorry for pushing however I need to notify the customer about when the change
> will be available. This one is highly expected.
I just applied it to git trees in preparation for the next cycle starting this
week. Release of this to updates will not be before the end of that cycle (Dec-11)

-Stefan

>
> Thanks,
> Konrad
>
>>
>> Regards,
>> Kleber
>



--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team

signature.asc (836 bytes) Download Attachment