Received-SPF: pass (google.com: domain of linux-bluetooth+bounces-3756-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) client-ip=2604:1380:45e3:2400::1;
From: Vlad Pruteanu <vlad.pruteanu@nxp.com>
To: linux-bluetooth@vger.kernel.org
Cc: claudia.rosu@nxp.com,
	mihai-octavian.urzica@nxp.com,
	silviu.barbulescu@nxp.com,
	iulia.tanasescu@nxp.com,
	andrei.istodorescu@nxp.com,
	luiz.dentz@gmail.com,
	Vlad Pruteanu <vlad.pruteanu@nxp.com>
Subject: [PATCH BlueZ v2 0/1] client/player: Fix transport.send command's transfer of packets
Date: Fri, 19 Apr 2024 17:58:04 +0300
Message-Id: <20240419145805.46263-1-vlad.pruteanu@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?oCkWfTwVkkWDz0koJsGdAqOWVi5t4DzirhdRIfBVIK4yT2N4U30CRMKxHsK8?=
 =?us-ascii?Q?gT9YaAZbcbPb0uqPKRVBZy+trR7rcY57B1cTS7V8LjqIQ+rMQBfTIs3KIgSb?=
 =?us-ascii?Q?SgJPLL/Mb0pKzdiuCtC181ojQucavewSREQLHJo8/OTEBnh7iqihrMTlkSNH?=
 =?us-ascii?Q?csn+rAF/1690l05QSouBOZ45p6u0bGzqdGjgT2n+l1pXAKsIeDqlW28x8kKD?=
 =?us-ascii?Q?oiruk+AyrcYBfIOisf8PB1l1NAvhBfrpVAkUo+BKFcfdOjw8O2uWDMBLPBBT?=
 =?us-ascii?Q?OciB+qhbcY9y5qXhvcX53MXePn7NL2QONTaggp+enep4OljXFqvUXdE89C1f?=
 =?us-ascii?Q?QC8ofeE3WLhkkMW8xO7OfRDGJk0U9e7FmC9nyAARvAJehmSD4V8qGCm2/1Nn?=
 =?us-ascii?Q?0/wUl0NRYz19POPB/4yQ6XeKE/E1pPGPIqIDeTAhSW1+oOoNmHf8BV/ulglh?=
 =?us-ascii?Q?ojH5H/n8M2EHOmN5jcsB3fOlgrnu3xY+mWkpTf30QgjCtVFwRpKpLGCTxRd/?=
 =?us-ascii?Q?ch0TPWa5Y9DERDxvVuYq7+8NjFSfQj3U7bQV1cqLqXXKYmTPuIa1sQyxpsQi?=
 =?us-ascii?Q?6m34rjJaoyRWYe+xyJjxGeRu4g3qC/hed4Ix8hQUl1TIDPOIpIRCiYWaHvvQ?=
 =?us-ascii?Q?WwceUvEnZ+/9/1zMskZ6M/IWjL/VY0omYLh3J5UPvLvS+YoL3mQGpZC82zoI?=
 =?us-ascii?Q?0irWd0k0iIWX0fX2zz33HriY67rucCTR5q2/FIvbXFb/upOiEfxDe1FjrJld?=
 =?us-ascii?Q?/dJA3e4R+7CMXjUxPUZ53eyJzQO0rVZcIiPa+z8skt0fNo6eSyZl5h6cOF8U?=
 =?us-ascii?Q?+82trwLWv5kQvEOFqydueuyXrS6SV91yeBmD6uDzBfwup0LMIZmuKBnJD4pK?=
 =?us-ascii?Q?CTGXYuSWbgYpk5SHhaBcGQToqAwSdzE+mzom7Noh9DMirJd9sEJkzNLuIh9I?=
 =?us-ascii?Q?B3ob3MAYxlT9FDMZ1C4BgC7ddd8+YZBlvDk4NHNyChMiEFWF8tfQ6TB70vYq?=
 =?us-ascii?Q?pmBnH464h8wqRT5JSG7pPtZqP5ja/GI12FUT3liUbJ8pRpAmagLnaa7f1FNH?=
 =?us-ascii?Q?dw0y3BdAO21C0u84jDXiM1P2Lq0HhCuOONNiEN1Rf1kgy/TV5ofT6J/o5BAk?=
 =?us-ascii?Q?xNQglShWohQ3gNMs+WjbDJMmyf856ZG2JmK64bM8qcdnfoz3gUxfAOvQOqyX?=
 =?us-ascii?Q?FVy2VXLJxJFJGX7xIabDDQU704LHLfwv9Q5LEuQBcAYDU9yAKVODF0LAz/ni?=
 =?us-ascii?Q?+q4ZZl7Z4fF0QUfD6yKUrgJXk/27YKoCH3tQVhysyQmuWNReIAq8WyiiSLMK?=
 =?us-ascii?Q?Zcx2TVCm/motbYEhU74wAvW+xoKAgRpCLkKyavp+yWRi6fcz+52Ofcg6ie8x?=
 =?us-ascii?Q?IqeixXGUjfq7Kd74wSOAXXWlZ70l70DvODE4khZxLMUudLqm3OaNMYj9KS8G?=
 =?us-ascii?Q?0vQ4B4GHq8FOJs+ihz/2qb/6K/XYC9KbdVSlZbIJGB4fUzPU9rB4WXswvvne?=
 =?us-ascii?Q?Pm3rlNTaz8OGW6ur1M0Fke0lylCSNlY5LGv+eP4VA15k1htXidEiQB4X44K+?=
 =?us-ascii?Q?8FK73isRhwFi545HeHMNzDAph94afbPk7pyF3qsB?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 930142a4-91d5-4af7-e7a8-08dc60812a1a
X-MS-Exchange-CrossTenant-AuthSource: VI1PR04MB5389.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 19 Apr 2024 14:58:25.6810
 (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: DcX7pGq/lvllBNJG3w4fGSqRwDMGojpYT6N8xpi55VvLkyD61xdGR/sFZ7kWQ3oPAbbGBT2ukOGIdHg03NT88Q==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DUZPR04MB9947

The transport.send command sends a number num of packets at intervals of
transport latency.

Num is defined as qos.ucast.out.latency * 1000 / qos.ucast.out.interval.

Since this latency could be smaller than the SDU interval, the resulting
num could be 0, causing the file transfer to stop after the first packet.
In this case num will be set to 1 so that at least 1 packet is always sent.

It the transport send timer is set to a value smaller than that of SDU
interval, the available buffers for ISO Data will eventually become full.
Thus, if a packet can't be sent due to resource temporarily being
unavailable decrease the fd offset so that next time the same packet will
be sent.

This patch is a temporary fix until the appropriate solution that uses
number of completed packets to control the flow of ISO Data packets is
implemented.

Since both Unicast and Broadcast scenarios use the same transport functions
differentiate between the 2 cases when accessing the qos structures to get
the transport latency.

Vlad Pruteanu (1):
  client/player: Fix transport.send command's transfer of packets

 client/player.c | 55 +++++++++++++++++++++++++++++++++++++++++++------
 1 file changed, 49 insertions(+), 6 deletions(-)

-- 
2.40.1