| Message ID | 20230517215852.1395512-1-luiz.dentz@gmail.com |
|---|---|
| State | New |
| Headers | show |
| Series | [BlueZ] monitor: Fix using PACKET_FILTER_SHOW_SCO_DATA for ISO packets | expand |
This is automated email and please do not reply to this email! Dear submitter, Thank you for submitting the patches to the linux bluetooth mailing list. This is a CI test results with your patch series: PW Link:https://patchwork.kernel.org/project/bluetooth/list/?series=748669 ---Test result--- Test Summary: CheckPatch PASS 0.45 seconds GitLint PASS 0.30 seconds BuildEll PASS 26.47 seconds BluezMake PASS 791.91 seconds MakeCheck PASS 11.49 seconds MakeDistcheck PASS 155.08 seconds CheckValgrind PASS 247.47 seconds CheckSmatch WARNING 336.29 seconds bluezmakeextell PASS 101.00 seconds IncrementalBuild PASS 660.64 seconds ScanBuild PASS 1013.03 seconds Details ############################## Test: CheckSmatch - WARNING Desc: Run smatch tool with source Output: monitor/packet.c: note: in included file:monitor/display.h:82:26: warning: Variable length array is used.monitor/packet.c:1801:26: warning: Variable length array is used.monitor/packet.c: note: in included file:monitor/bt.h:3552:52: warning: array of flexible structuresmonitor/bt.h:3540:40: warning: array of flexible structures --- Regards, Linux Bluetooth
Hello: This patch was applied to bluetooth/bluez.git (master) by Luiz Augusto von Dentz <luiz.von.dentz@intel.com>: On Wed, 17 May 2023 14:58:52 -0700 you wrote: > From: Luiz Augusto von Dentz <luiz.von.dentz@intel.com> > > This fixes the uses of PACKET_FILTER_SHOW_SCO_DATA for ISO packets and > introduce a dedicated flag for it in the form of > PACKET_FILTER_SHOW_ISO_DATA which can be enabled with use of -I/--iso. > --- > monitor/main.c | 9 +++++++-- > monitor/packet.c | 5 +++-- > monitor/packet.h | 1 + > 3 files changed, 11 insertions(+), 4 deletions(-) Here is the summary with links: - [BlueZ] monitor: Fix using PACKET_FILTER_SHOW_SCO_DATA for ISO packets https://git.kernel.org/pub/scm/bluetooth/bluez.git/?id=9526ef46e465 You are awesome, thank you!
diff --git a/monitor/main.c b/monitor/main.c index 3ec3a5f08a01..decf7cc467f2 100644 --- a/monitor/main.c +++ b/monitor/main.c @@ -62,6 +62,7 @@ static void usage(void) "\t-T, --date Show time and date information\n" "\t-S, --sco Dump SCO traffic\n" "\t-A, --a2dp Dump A2DP stream traffic\n" + "\t-I, --iso Dump ISO traffic\n" "\t-E, --ellisys [ip] Send Ellisys HCI Injection\n" "\t-P, --no-pager Disable pager usage\n" "\t-J --jlink <device>,[<serialno>],[<interface>],[<speed>]\n" @@ -89,6 +90,7 @@ static const struct option main_options[] = { { "date", no_argument, NULL, 'T' }, { "sco", no_argument, NULL, 'S' }, { "a2dp", no_argument, NULL, 'A' }, + { "iso", no_argument, NULL, 'I' }, { "ellisys", required_argument, NULL, 'E' }, { "no-pager", no_argument, NULL, 'P' }, { "jlink", required_argument, NULL, 'J' }, @@ -126,8 +128,8 @@ int main(int argc, char *argv[]) struct sockaddr_un addr; opt = getopt_long(argc, argv, - "r:w:a:s:p:i:d:B:V:MNtTSAE:PJ:R:C:c:vh", - main_options, NULL); + "r:w:a:s:p:i:d:B:V:MNtTSAIE:PJ:R:C:c:vh", + main_options, NULL); if (opt < 0) break; @@ -197,6 +199,9 @@ int main(int argc, char *argv[]) case 'A': filter_mask |= PACKET_FILTER_SHOW_A2DP_STREAM; break; + case 'I': + filter_mask |= PACKET_FILTER_SHOW_ISO_DATA; + break; case 'E': ellisys_server = optarg; ellisys_port = 24352; diff --git a/monitor/packet.c b/monitor/packet.c index 94561b65ea75..aafb81eefb13 100644 --- a/monitor/packet.c +++ b/monitor/packet.c @@ -66,6 +66,7 @@ #define COLOR_HCI_EVENT_UNKNOWN COLOR_WHITE_BG #define COLOR_HCI_ACLDATA COLOR_CYAN #define COLOR_HCI_SCODATA COLOR_YELLOW +#define COLOR_HCI_ISODATA COLOR_YELLOW #define COLOR_UNKNOWN_ERROR COLOR_WHITE_BG #define COLOR_UNKNOWN_FEATURE_BIT COLOR_WHITE_BG @@ -12357,7 +12358,7 @@ void packet_hci_isodata(struct timeval *tv, struct ucred *cred, uint16_t index, sprintf(handle_str, "Handle %d", acl_handle(handle)); sprintf(extra_str, "flags 0x%2.2x dlen %d", flags, hdr->dlen); - print_packet(tv, cred, in ? '>' : '<', index, NULL, COLOR_HCI_SCODATA, + print_packet(tv, cred, in ? '>' : '<', index, NULL, COLOR_HCI_ISODATA, in ? "ISO Data RX" : "ISO Data TX", handle_str, extra_str); @@ -12368,7 +12369,7 @@ void packet_hci_isodata(struct timeval *tv, struct ucred *cred, uint16_t index, return; } - if (filter_mask & PACKET_FILTER_SHOW_SCO_DATA) + if (filter_mask & PACKET_FILTER_SHOW_ISO_DATA) packet_hexdump(data, size); } diff --git a/monitor/packet.h b/monitor/packet.h index b07d5d18ce21..55ba854ec1f7 100644 --- a/monitor/packet.h +++ b/monitor/packet.h @@ -22,6 +22,7 @@ #define PACKET_FILTER_SHOW_SCO_DATA (1 << 5) #define PACKET_FILTER_SHOW_A2DP_STREAM (1 << 6) #define PACKET_FILTER_SHOW_MGMT_SOCKET (1 << 7) +#define PACKET_FILTER_SHOW_ISO_DATA (1 << 8) struct packet_conn_data { uint16_t index;
From: Luiz Augusto von Dentz <luiz.von.dentz@intel.com> This fixes the uses of PACKET_FILTER_SHOW_SCO_DATA for ISO packets and introduce a dedicated flag for it in the form of PACKET_FILTER_SHOW_ISO_DATA which can be enabled with use of -I/--iso. --- monitor/main.c | 9 +++++++-- monitor/packet.c | 5 +++-- monitor/packet.h | 1 + 3 files changed, 11 insertions(+), 4 deletions(-)