Received-SPF: pass (google.com: domain of linux-bluetooth+bounces-2685-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
From: Silviu Florian Barbulescu <silviu.barbulescu@nxp.com>
To: linux-bluetooth@vger.kernel.org
Cc: mihai-octavian.urzica@nxp.com,
	silviu.barbulescu@nxp.com,
	vlad.pruteanu@nxp.com,
	andrei.istodorescu@nxp.com,
	luiz.dentz@gmail.com,
	iulia.tanasescu@nxp.com
Subject: [PATCH BlueZ 0/1] Adjust SDU size based on the number of locations
Date: Thu, 21 Mar 2024 18:12:54 +0200
Message-Id: <20240321161255.24892-1-silviu.barbulescu@nxp.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?us-ascii?Q?2b4tOHdP6q7ec+gPet/TIBl6m2sWJ62oU49fNuHTLkyicWpMGTTFFCulYtUS?=
 =?us-ascii?Q?c/A2J6/qHD75QuEo8r7k9iP4RQ88wnMnqNcqxPOM6cY3SLCgnNfY+yj52KOf?=
 =?us-ascii?Q?Sc98h51ZyqMU78us+WI/u0C9LyzrkXwzIoAKujFuauO8TTdntj3mq2TBxMeQ?=
 =?us-ascii?Q?wVzpKhawxOPHQLByVyas5kHh+o7iE5Nbmp9rpagXPbkgKtj0PRNCaiHvTHYy?=
 =?us-ascii?Q?pgoFNl2We4gTZ1zcSpiHf3gY8IXrfO0XVXxFiBlIQ191mPdgEWlO11KSLgN1?=
 =?us-ascii?Q?mDFTnD6Eg33E0SnpuRTkIlJLj7fTirRKw2/wGokazKkPLnz7kazyBtuM83F7?=
 =?us-ascii?Q?sK8sKJNIdF1XMiTOpAVEKziTDa0XQiYc4KXmprbDtzHbJe3Rf9abCtpGOYJH?=
 =?us-ascii?Q?lC9F1RVTR3djc1ofQoqWeds1rFpvYR6Dzf2qrhsza66+uGNd6WXyvFAmUtV4?=
 =?us-ascii?Q?LHVmLaqOd2+rNooWLNi8vYHae81PnO0jXIPJaHPvdzAI1Htz10wzO+SA33/a?=
 =?us-ascii?Q?TxL80+rzxJBW8KAzNRC6xS2Gg+LEdQE4Y53kPVdvlGlDodRvKRMJifogNCfr?=
 =?us-ascii?Q?OJ1i6GJgtwKCbRBUAQyab2gre46hCQ666wyD7Apq5kfFBYI/MzfEmN+rco1v?=
 =?us-ascii?Q?M+IBTHP4X07XblZSF89cjaQDEEoP2tOtYf3TcSmXV95JfXZDoxr2NXi0uiuD?=
 =?us-ascii?Q?kX572O7uILNe7R7/Qu5sqHiXabJb7LJKX7qYtbvqz0Uxg/8tIs4kumd0FVO1?=
 =?us-ascii?Q?boiFTUvVZsXDlCpNHTTpn/bHlxecYFljMkSAdArQPI4GPRXm9Wn1YWH6YhNP?=
 =?us-ascii?Q?c6vNJQ/KCj3SgGk65GXmUfo4vYltOARLzdsosVlKBRemYd8dT70yKWAiv4ub?=
 =?us-ascii?Q?p/p8QKYYcPsJe/7gRRF1ESPRnXPdbRhlvdMuc77jjFhY7MsFT3usjavIEk4s?=
 =?us-ascii?Q?9APV7N6pJ6Nt4fJST4Azl/55qtx48PGlJqfz5QCIRCIcjGorKVUc3POBYj+X?=
 =?us-ascii?Q?AlaLa+GDBAyCD3zNbefo+1OEOrRgJ6LlQQceOtsCKhKiNL5YYXYaA26CFCD5?=
 =?us-ascii?Q?uvvCixY6MAoN95dPUdPQ3Y+FL/uMZmEr+ds7lmjQFcspNpM7Ne3lqpspCLD2?=
 =?us-ascii?Q?c4w9/7ktCA2OBwXuHc7kKwHsyLNBwICENjZZeaKBRx1RzxHIfGMI8GSLxYKO?=
 =?us-ascii?Q?Q0QEB9z/j0Jmmc5o9qcvF3Y028ApKbsT4Ntsux/L5QzT7aO6L8l2k3L67vxi?=
 =?us-ascii?Q?vcq96Q4lXPWZN2nhXqXr+aVvptxJyphhPgcFWE4+sRaamHfm83CnMxOxGlne?=
 =?us-ascii?Q?qFA/abvgnTXPb19Fb1RcDaqBlnrtVjAxYx+E4UmdxQGaag1DEPtyTfpXmpRT?=
 =?us-ascii?Q?CoRgC96dWehDA8aD3P1hYOD57+rGRgwOYX7BR4eRbuzlfjpwYX/WkBWZASxY?=
 =?us-ascii?Q?p0awPMuwu/t+Y6dYkZZZgmbz+SKjb3zgCzepvbOIL0UMBEECT/Xd8qBWTsSE?=
 =?us-ascii?Q?3WwWzzpl5wX9sXEj+gggWpN0kAfCOns1RsoXnLQQN0tDCGCDgnexnG9dPvzr?=
 =?us-ascii?Q?c0c9FbI1w9qvVbjvcegx5thCwdUKDbeiJDJTx7p+3d+RzuW9olpITBST0z5K?=
 =?us-ascii?Q?yg=3D=3D?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 7e17fb67-e9f7-4cc5-c22d-08dc49c1c6ae
X-MS-Exchange-CrossTenant-AuthSource: VI1PR04MB10073.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 21 Mar 2024 16:12:59.1765
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: zHpHGU6JAieIlYqNWZ4WRU8sfjPMv7ghsazETRig+brzdcrH48koMmJNh93eiEg35KuqgbX4Pyf5oyC8Z+Buqb6PEEp/XIEVxXxIHMJAT8s=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR04MB9956

Adjust the SDU size based on the number of locations/channels
that is being requested.

Tested with patch [BlueZ,1/1] Fix crash in iov_append_ltv function
applied

Waiting to connect to bluetoothd...
[bluetooth]# Agent registered
endpoint.register 00001852-0000-1000-8000-00805f9b34fb 0x065f9b34fb 0x06
[/local/endpoint/ep0] Auto Accept (yes/no): y
[/local/endpoint/ep0] Max Transports (auto/value): a
[/local/endpoint/ep0] Locations: 3
[/local/endpoint/ep0] Supported Context (value): 0
Capabilities:
03 01 ff 00 02 02 03 02 03 03 05 04 1a 00 f0 00 ................
Metadata:
[bluetooth]# [NEW] Endpoint /org/bluez/hci0/pac_bcast0
[bluetooth]# Endpoint /local/endpoint/ep0 registered
endpoint.config /org/bluez/hci0/pac_bcast0 /local/endpoint/ep0 48_4_1
[/local/endpoint/ep0] BIG (auto/value): 0
[/local/endpoint/ep0] Enter channel location (value/no): 3
[/local/endpoint/ep0] Enter Metadata (value/no): n
Configuration.#0: len 0x02 type 0x01
Configuration.Sampling Frequency: 48 Khz (0x08)
Configuration.#1: len 0x02 type 0x02
Configuration.Frame Duration: 10 ms (0x01)
Configuration.#2: len 0x03 type 0x04
Configuration.Frame Length: 120 (0x0078)
Configuration.#3: len 0x05 type 0x03
Configuration.Location: 0x00000003
Configuration.Location: Front Left (0x00000001)
Configuration.Location: Front Right (0x00000002)
BIG 0x00
BIS 0x00
BCode:
01 02 68 05 53 f1 41 5a a2 65 bb af c6 ea 03 b8 ..h.S.AZ.e......
Framing 0x00
PresentationDelay 40000
Interval 10000
PHY 0x02
SDU 240
Retransmissions 4
Latency 20
[bluetooth]# [NEW] Transport /org/bluez/hci0/pac_bcast0/fd0
[bluetooth]# Endpoint: SetConfiguration
[bluetooth]# Transport /org/bluez/hci0/pac_bcast0/fd0
[bluetooth]# Properties.Device: /org/bluez/hci0
[bluetooth]# Auto Accepting...
[bluetooth]# Endpoint /local/endpoint/ep0 configured
transport.acquire /org/bluez/hci0/pac_bcast0/fd0 c_bcast0/fd0
auto acquiring...
Transport /org/bluez/hci0/pac_bcast0/fd0 acquiring
[bluetooth]# Endpoint: SetConfiguration
[bluetooth]# Transport /org/bluez/hci0/pac_bcast0/fd0
[bluetooth]# Properties.Device: /org/bluez/hci0
[bluetooth]# Auto Accepting...
[CHG] Transport /org/bluez/hci0/pac_bcast0/fd0 QoS.BIG: 0x00 (0)
[CHG] Transport /org/bluez/hci0/pac_bcast0/fd0 QoS.BIS: 0x01 (1)
[CHG] Transport /org/bluez/hci0/pac_bcast0/fd0 QoS.SyncFactor: 0x01 (1)
[CHG] Transport /org/bluez/hci0/pac_bcast0/fd0 QoS.Packing: 0x00 (0)
[CHG] Transport /org/bluez/hci0/pac_bcast0/fd0 QoS.Framing: 0x00 (0)
[CHG] Transport /org/bluez/hci0/pac_bcast0/fd0 QoS.BCode:
01 02 68 05 53 f1 41 5a a2 65 bb af c6 ea 03 b8 ..h.S.AZ.e......
[CHG] Transport /org/bluez/hci0/pac_bcast0/fd0 QoS.Options: 0x00 (0)
[CHG] Transport /org/bluez/hci0/pac_bcast0/fd0 QoS.Skip: 0x0000 (0)
[CHG] Transport /org/bluez/hci0/pac_bcast0/fd0 QoS.SyncTimeout:
0x07d0 (2000)
[CHG] Transport /org/bluez/hci0/pac_bcast0/fd0 QoS.SyncType: 0x00 (0)
[CHG] Transport /org/bluez/hci0/pac_bcast0/fd0 QoS.MSE: 0x00 (0)
[CHG] Transport /org/bluez/hci0/pac_bcast0/fd0 QoS.Timeout: 0x07d0 (2000)
[CHG] Transport /org/bluez/hci0/pac_bcast0/fd0 QoS.Interval: 0x00002710
(10000)
[CHG] Transport /org/bluez/hci0/pac_bcast0/fd0 QoS.Latency: 0x0014 (20)
[CHG] Transport /org/bluez/hci0/pac_bcast0/fd0 QoS.SDU: 0x00f0 (240)
[CHG] Transport /org/bluez/hci0/pac_bcast0/fd0 QoS.PHY: 0x02 (2)
[CHG] Transport /org/bluez/hci0/pac_bcast0/fd0 QoS.Retransmissions:
0x04 (4)
[CHG] Transport /org/bluez/hci0/pac_bcast0/fd0 Codec: 0x06 (6)
[CHG] Transport /org/bluez/hci0/pac_bcast0/fd0 Configuration:
02 01 08 02 02 01 03 04 78 00 05 03 03 00 00 00 ........x.......
Transport /org/bluez/hci0/pac_bcast0/fd0 acquiring complete
Acquire successful: fd 8 MTU 0:240
[CHG] Transport /org/bluez/hci0/pac_bcast0/fd0 State: active

Silviu Florian Barbulescu (1):
  player: Adjust SDU size based on the number of locations

 client/player.c | 8 ++++++++
 1 file changed, 8 insertions(+)


base-commit: b8ad3490a3507476844d6c6a87b2cb336f7d4eb9
-- 
2.39.2