2024-07-12 02:47:32 |
Leo Lin |
bug |
|
|
added bug |
2024-07-12 02:47:45 |
Leo Lin |
linux (Ubuntu): assignee |
|
Leo Lin (0xff07) |
|
2024-07-19 05:49:14 |
Rex Tsai |
bug task added |
|
oem-priority |
|
2024-07-19 07:36:13 |
Leo Lin |
tags |
|
jammy noble |
|
2024-07-23 03:45:02 |
Leo Lin |
description |
SRU Jusitification for Kernel
[Impact]
Noble failed the L2CAP/LE/CPU/BI-02-C test in the Porfile Tuning Suite (PTS), which Jammy could pass.
This is due to new behavior introduced in e4b019515f950b4e6e5b74b2e1bb03a90cb33039 (Bluetooth: Enforce validation on max value of connection interval). The kernel only accept Connection Parameter Update Requests whose incoming conn_max_interval are lower than the current conn_max_interval, and adjust the newer conn_max_interval to that received max value.
However, this behavior means that conn_max_interval can only decrease, but never increase. This could potentially make the conditions for connection parameters narrower over time, causing subsequent connections failed on some devices. See the issue 847 in bluez upstream[1]. The patch 806a5198c05987b748b50f3d0c0cfb3d417381a4 (Bluetooth: L2CAP: Fix rejecting L2CAP_CONN_PARAM_UPDATE_REQ) in the linux-next fixed this by accepting connection parameter unconditionally. The relavent test procedure is also on the mailing list[2].
[1] https://github.com/bluez/bluez/issues/847
[2] https://linuxlists.cc/l/15/linux-bluetooth/t/5350289/(patch_v3)_bluetooth:_l2cap:_fix_rejecting_l2cap_conn_param_update_req#post5352326
[Fix]
Backport the from commit 806a5198c05987b748b50f3d0c0cfb3d417381a4 (Bluetooth: L2CAP: Fix rejecting L2CAP_CONN_PARAM_UPDATE_REQ), which currently is in the linux-next.
[1] https://patchwork.kernel.org/project/bluetooth/patch/20240521143521.1568672-1-luiz.dentz@gmail.com/
[Test Case]
1. Install the kernel with the backported patch
2. Run the following test case in the PTS:
L2CAP/LE/CPU/BI-02-C
GAP/CONN/CPUP/BV-05-C
[Where problems could occur]
This essentially revert the behavior of accepting L2CAP connection parameters back to its original state before e4b019515f950b4e6e5b74b2e1bb03a90cb33039 (Bluetooth: Enforce validation on max value of connection interval).
Note that implementing restriction to the conenction parameters may take greater effort than just adding a few checks in the kernel. The user space, notably the bluetoothd may also need adjustments[1]. So in this case, removing the half-done boundary checks in kernel may still do greater good if there's no plan to make those additional changes.
[1] https://github.com/bluez/bluez/issues/717#issuecomment-1885719058 |
SRU Jusitification for Kernel
[Impact]
Noble failed the L2CAP/LE/CPU/BI-02-C test in the Porfile Tuning Suite (PTS), which Jammy previously could pass.
This is due to new behavior introduced in e4b019515f950b4e6e5b74b2e1bb03a90cb33039 (Bluetooth: Enforce validation on max value of connection interval). The kernel only accept Connection Parameter Update Requests whose incoming conn_max_interval are lower than the current conn_max_interval, and adjust the newer conn_max_interval to that received max value.
However, this behavior means that conn_max_interval can only decrease, but never increase. This could potentially make the conditions for connection parameters narrower over time, causing subsequent connections failed on some devices. See the issue 847 in bluez upstream[1]. The patch 806a5198c05987b748b50f3d0c0cfb3d417381a4 (Bluetooth: L2CAP: Fix rejecting L2CAP_CONN_PARAM_UPDATE_REQ) in the linux-next fixed this by accepting connection parameter unconditionally. The relavent test procedure is also on the mailing list[2].
[1] https://github.com/bluez/bluez/issues/847
[2] https://linuxlists.cc/l/15/linux-bluetooth/t/5350289/(patch_v3)_bluetooth:_l2cap:_fix_rejecting_l2cap_conn_param_update_req#post5352326
[Fix]
Backport the from commit 806a5198c05987b748b50f3d0c0cfb3d417381a4 (Bluetooth: L2CAP: Fix rejecting L2CAP_CONN_PARAM_UPDATE_REQ), which currently is in the linux-next.
[1] https://patchwork.kernel.org/project/bluetooth/patch/20240521143521.1568672-1-luiz.dentz@gmail.com/
[Test Case]
1. Install the kernel with the backported patch
2. Run the following test case in the PTS:
L2CAP/LE/CPU/BI-02-C
GAP/CONN/CPUP/BV-05-C
[Where problems could occur]
This essentially revert the behavior of accepting L2CAP connection parameters back to its original state before e4b019515f950b4e6e5b74b2e1bb03a90cb33039 (Bluetooth: Enforce validation on max value of connection interval).
Note that implementing restriction to the conenction parameters may take greater effort than just adding a few checks in the kernel. The user space, notably the bluetoothd may also need adjustments[1]. So in this case, removing the half-done boundary checks in kernel may still do greater good if there's no plan to make those additional changes.
[1] https://github.com/bluez/bluez/issues/717#issuecomment-1885719058 |
|
2024-07-23 05:51:01 |
En-Wei Wu |
nominated for series |
|
Ubuntu Noble |
|
2024-07-23 05:51:01 |
En-Wei Wu |
bug task added |
|
linux (Ubuntu Noble) |
|
2024-07-23 05:51:12 |
En-Wei Wu |
nominated for series |
|
Ubuntu Jammy |
|
2024-07-23 05:51:12 |
En-Wei Wu |
bug task added |
|
linux (Ubuntu Jammy) |
|
2024-07-23 05:51:48 |
En-Wei Wu |
linux (Ubuntu Jammy): assignee |
|
Leo Lin (0xff07) |
|
2024-07-23 05:52:03 |
En-Wei Wu |
linux (Ubuntu Noble): assignee |
|
Leo Lin (0xff07) |
|
2024-07-23 05:52:25 |
En-Wei Wu |
oem-priority: assignee |
|
Leo Lin (0xff07) |
|
2024-07-26 09:57:56 |
Roxana Nicolescu |
linux (Ubuntu Jammy): status |
New |
Fix Committed |
|
2024-07-26 09:57:58 |
Roxana Nicolescu |
linux (Ubuntu Noble): status |
New |
Fix Committed |
|
2024-07-26 09:58:03 |
Roxana Nicolescu |
linux (Ubuntu): status |
New |
Invalid |
|
2024-08-08 15:13:33 |
Ubuntu Kernel Bot |
tags |
jammy noble |
jammy kernel-spammed-noble-linux-v2 noble verification-needed-noble-linux |
|
2024-08-09 03:27:20 |
Ubuntu Kernel Bot |
tags |
jammy kernel-spammed-noble-linux-v2 noble verification-needed-noble-linux |
jammy kernel-spammed-jammy-linux-v2 kernel-spammed-noble-linux-v2 noble verification-needed-jammy-linux verification-needed-noble-linux |
|
2024-08-11 20:48:09 |
Rex Tsai |
tags |
jammy kernel-spammed-jammy-linux-v2 kernel-spammed-noble-linux-v2 noble verification-needed-jammy-linux verification-needed-noble-linux |
jammy kernel-spammed-jammy-linux-v2 kernel-spammed-noble-linux-v2 noble oem-priority verification-needed-jammy-linux verification-needed-noble-linux |
|
2024-08-12 03:54:49 |
Leo Lin |
summary |
Fix L2CAP/LE/CPU/BI-02-C bluetooth certification failure |
Fix L2CAP/LE/CPU/BV-02-C bluetooth certification failure |
|
2024-08-12 03:55:21 |
Leo Lin |
description |
SRU Jusitification for Kernel
[Impact]
Noble failed the L2CAP/LE/CPU/BI-02-C test in the Porfile Tuning Suite (PTS), which Jammy previously could pass.
This is due to new behavior introduced in e4b019515f950b4e6e5b74b2e1bb03a90cb33039 (Bluetooth: Enforce validation on max value of connection interval). The kernel only accept Connection Parameter Update Requests whose incoming conn_max_interval are lower than the current conn_max_interval, and adjust the newer conn_max_interval to that received max value.
However, this behavior means that conn_max_interval can only decrease, but never increase. This could potentially make the conditions for connection parameters narrower over time, causing subsequent connections failed on some devices. See the issue 847 in bluez upstream[1]. The patch 806a5198c05987b748b50f3d0c0cfb3d417381a4 (Bluetooth: L2CAP: Fix rejecting L2CAP_CONN_PARAM_UPDATE_REQ) in the linux-next fixed this by accepting connection parameter unconditionally. The relavent test procedure is also on the mailing list[2].
[1] https://github.com/bluez/bluez/issues/847
[2] https://linuxlists.cc/l/15/linux-bluetooth/t/5350289/(patch_v3)_bluetooth:_l2cap:_fix_rejecting_l2cap_conn_param_update_req#post5352326
[Fix]
Backport the from commit 806a5198c05987b748b50f3d0c0cfb3d417381a4 (Bluetooth: L2CAP: Fix rejecting L2CAP_CONN_PARAM_UPDATE_REQ), which currently is in the linux-next.
[1] https://patchwork.kernel.org/project/bluetooth/patch/20240521143521.1568672-1-luiz.dentz@gmail.com/
[Test Case]
1. Install the kernel with the backported patch
2. Run the following test case in the PTS:
L2CAP/LE/CPU/BI-02-C
GAP/CONN/CPUP/BV-05-C
[Where problems could occur]
This essentially revert the behavior of accepting L2CAP connection parameters back to its original state before e4b019515f950b4e6e5b74b2e1bb03a90cb33039 (Bluetooth: Enforce validation on max value of connection interval).
Note that implementing restriction to the conenction parameters may take greater effort than just adding a few checks in the kernel. The user space, notably the bluetoothd may also need adjustments[1]. So in this case, removing the half-done boundary checks in kernel may still do greater good if there's no plan to make those additional changes.
[1] https://github.com/bluez/bluez/issues/717#issuecomment-1885719058 |
SRU Jusitification for Kernel
[Impact]
Noble failed the L2CAP/LE/CPU/BV-02-C test in the Porfile Tuning Suite (PTS), which Jammy previously could pass.
This is due to new behavior introduced in e4b019515f950b4e6e5b74b2e1bb03a90cb33039 (Bluetooth: Enforce validation on max value of connection interval). The kernel only accept Connection Parameter Update Requests whose incoming conn_max_interval are lower than the current conn_max_interval, and adjust the newer conn_max_interval to that received max value.
However, this behavior means that conn_max_interval can only decrease, but never increase. This could potentially make the conditions for connection parameters narrower over time, causing subsequent connections failed on some devices. See the issue 847 in bluez upstream[1]. The patch 806a5198c05987b748b50f3d0c0cfb3d417381a4 (Bluetooth: L2CAP: Fix rejecting L2CAP_CONN_PARAM_UPDATE_REQ) in the linux-next fixed this by accepting connection parameter unconditionally. The relavent test procedure is also on the mailing list[2].
[1] https://github.com/bluez/bluez/issues/847
[2] https://linuxlists.cc/l/15/linux-bluetooth/t/5350289/(patch_v3)_bluetooth:_l2cap:_fix_rejecting_l2cap_conn_param_update_req#post5352326
[Fix]
Backport the from commit 806a5198c05987b748b50f3d0c0cfb3d417381a4 (Bluetooth: L2CAP: Fix rejecting L2CAP_CONN_PARAM_UPDATE_REQ), which currently is in the linux-next.
[1] https://patchwork.kernel.org/project/bluetooth/patch/20240521143521.1568672-1-luiz.dentz@gmail.com/
[Test Case]
1. Install the kernel with the backported patch
2. Run the following test case in the PTS:
L2CAP/LE/CPU/BV-02-C
GAP/CONN/CPUP/BV-05-C
[Where problems could occur]
This essentially revert the behavior of accepting L2CAP connection parameters back to its original state before e4b019515f950b4e6e5b74b2e1bb03a90cb33039 (Bluetooth: Enforce validation on max value of connection interval).
Note that implementing restriction to the conenction parameters may take greater effort than just adding a few checks in the kernel. The user space, notably the bluetoothd may also need adjustments[1]. So in this case, removing the half-done boundary checks in kernel may still do greater good if there's no plan to make those additional changes.
[1] https://github.com/bluez/bluez/issues/717#issuecomment-1885719058 |
|
2024-08-12 04:03:39 |
Leo Lin |
tags |
jammy kernel-spammed-jammy-linux-v2 kernel-spammed-noble-linux-v2 noble oem-priority verification-needed-jammy-linux verification-needed-noble-linux |
jammy kernel-spammed-jammy-linux-v2 kernel-spammed-noble-linux-v2 noble oem-priority verification-done-noble-linux verification-needed-jammy-linux |
|
2024-08-12 06:15:35 |
Leo Lin |
tags |
jammy kernel-spammed-jammy-linux-v2 kernel-spammed-noble-linux-v2 noble oem-priority verification-done-noble-linux verification-needed-jammy-linux |
jammy kernel-spammed-jammy-linux-v2 kernel-spammed-noble-linux-v2 noble oem-priority verification-done-jammy-linux verification-done-noble-linux |
|
2024-08-26 08:51:46 |
Ubuntu Kernel Bot |
tags |
jammy kernel-spammed-jammy-linux-v2 kernel-spammed-noble-linux-v2 noble oem-priority verification-done-jammy-linux verification-done-noble-linux |
jammy kernel-spammed-jammy-linux-nvidia-tegra-v2 kernel-spammed-jammy-linux-v2 kernel-spammed-noble-linux-v2 noble oem-priority verification-done-jammy-linux verification-done-noble-linux verification-needed-jammy-linux-nvidia-tegra |
|
2024-08-26 13:37:27 |
Ubuntu Kernel Bot |
tags |
jammy kernel-spammed-jammy-linux-nvidia-tegra-v2 kernel-spammed-jammy-linux-v2 kernel-spammed-noble-linux-v2 noble oem-priority verification-done-jammy-linux verification-done-noble-linux verification-needed-jammy-linux-nvidia-tegra |
jammy kernel-spammed-jammy-linux-gcp-6.8-v2 kernel-spammed-jammy-linux-nvidia-tegra-v2 kernel-spammed-jammy-linux-v2 kernel-spammed-noble-linux-v2 noble oem-priority verification-done-jammy-linux verification-done-noble-linux verification-needed-jammy-linux-gcp-6.8 verification-needed-jammy-linux-nvidia-tegra |
|
2024-08-27 18:46:35 |
Ubuntu Kernel Bot |
tags |
jammy kernel-spammed-jammy-linux-gcp-6.8-v2 kernel-spammed-jammy-linux-nvidia-tegra-v2 kernel-spammed-jammy-linux-v2 kernel-spammed-noble-linux-v2 noble oem-priority verification-done-jammy-linux verification-done-noble-linux verification-needed-jammy-linux-gcp-6.8 verification-needed-jammy-linux-nvidia-tegra |
jammy kernel-spammed-jammy-linux-gcp-6.8-v2 kernel-spammed-jammy-linux-nvidia-tegra-igx-v2 kernel-spammed-jammy-linux-nvidia-tegra-v2 kernel-spammed-jammy-linux-v2 kernel-spammed-noble-linux-v2 noble oem-priority verification-done-jammy-linux verification-done-noble-linux verification-needed-jammy-linux-gcp-6.8 verification-needed-jammy-linux-nvidia-tegra verification-needed-jammy-linux-nvidia-tegra-igx |
|