-
Notifications
You must be signed in to change notification settings - Fork 6.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Bluetooth: BAP: Shell: Improve recv statistics #59386
Conversation
We should keep track of all the places that needs to be changed for multi stream support ;) |
subsys/bluetooth/audio/shell/bap.c
Outdated
lost_pkts++; | ||
} | ||
|
||
/* TODO: Make it possible to only print every X packets, and make X settable by the shell */ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sounds like a good idea to make it configurable. Also it seems like a fairly small change. Could this be added in the PR? otherwise LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it makes sense to add to this PR :)
Yeah, it's getting increasingly difficult :) |
Add more information when we print the recv every 100th packet, and remove all per-recv printing. This also resets all information on stream start. This does, however, no properly support multiple streams. Signed-off-by: Emil Gydesen <[email protected]>
The internval of how often we report receive stats can now be configured via the cmd_recv_stats command. Signed-off-by: Emil Gydesen <[email protected]>
Add more information when we print the recv every 100th packet, and remove all per-recv printing.
This also resets all information on stream start. This does, however, no properly support multiple streams.