From 1affbbad46125739273eee1b4514e340dd64de13 Mon Sep 17 00:00:00 2001 From: Venkata Sharath Chandra Manchala Date: Thu, 12 Dec 2019 14:29:54 -0800 Subject: [PATCH] qcacmn: Move pktlog API to platform specific file Move process_sw_event API to pktlog_wifi2.c file as it is only applicable to legacy product line. Change-Id: I8eee102ea2eb70c0ffcec4ac20dfdf6ef48bca5b CRs-Fixed: 2585613 --- utils/pktlog/include/pktlog_ac_i.h | 78 ++++++++-- utils/pktlog/include/pktlog_wifi2.h | 57 ++++++++ utils/pktlog/include/pktlog_wifi3.h | 78 ++++++++++ utils/pktlog/pktlog_ac.c | 5 + utils/pktlog/pktlog_internal.c | 211 +--------------------------- utils/pktlog/pktlog_wifi2.c | 179 +++++++++++++++++++++++ 6 files changed, 391 insertions(+), 217 deletions(-) diff --git a/utils/pktlog/include/pktlog_ac_i.h b/utils/pktlog/include/pktlog_ac_i.h index d222affb73..5f7d9beba2 100644 --- a/utils/pktlog/include/pktlog_ac_i.h +++ b/utils/pktlog/include/pktlog_ac_i.h @@ -1,5 +1,5 @@ /* - * Copyright (c) 2012-2019 The Linux Foundation. All rights reserved. + * Copyright (c) 2012-2020 The Linux Foundation. All rights reserved. * * Permission to use, copy, modify, and/or distribute this software for * any purpose with or without fee is hereby granted, provided that the @@ -60,14 +60,72 @@ char *pktlog_getbuf(struct pktlog_dev_t *pl_dev, struct ath_pktlog_info *pl_info, size_t log_size, struct ath_pktlog_hdr *pl_hdr); -A_STATUS process_tx_info(struct cdp_pdev *pdev, void *data); -A_STATUS process_rx_info(void *pdev, void *data); -A_STATUS process_rx_info_remote(void *pdev, void *data); -A_STATUS process_rate_find(void *pdev, void *data); -A_STATUS process_rate_update(void *pdev, void *data); -A_STATUS process_sw_event(void *pdev, void *data); -int process_pktlog_lite(void *context, void *log_data, uint16_t log_type); -int process_rx_desc_remote(void *pdev, void *data); -A_STATUS process_offload_pktlog(struct cdp_pdev *pdev, void *data); +#ifdef PKTLOG_HAS_SPECIFIC_DATA +/** + * pktlog_hdr_set_specific_data() - set type specific data + * @log_hdr: pktlog header + * @type_specific_data: type specific data + * + * Return: None + */ +void +pktlog_hdr_set_specific_data(struct ath_pktlog_hdr *log_hdr, + uint32_t type_specific_data); + +/** + * pktlog_hdr_get_specific_data() - get type specific data + * @log_hdr: pktlog header + * @type_specific_data: type specific data + * + * Return: pktlog subtype + */ +uint32_t +pktlog_hdr_get_specific_data(struct ath_pktlog_hdr *log_hdr); + +/** + * pktlog_arg_set_specific_data() - set type specific data + * @log_hdr: pktlog arg + * @type_specific_data: type specific data + * + * Return: None + */ +void +pktlog_arg_set_specific_data(struct ath_pktlog_arg *plarg, + uint32_t type_specific_data); + +/** + * pktlog_arg_get_specific_data() - set type specific data + * @log_hdr: pktlog arg + * @type_specific_data: type specific data + * + * Return: pktlog subtype + */ +uint32_t +pktlog_arg_get_specific_data(struct ath_pktlog_arg *plarg); +#else +static inline void +pktlog_hdr_set_specific_data(struct ath_pktlog_hdr *log_hdr, + uint32_t type_specific_data) +{ +} + +static inline uint32_t +pktlog_hdr_get_specific_data(struct ath_pktlog_hdr *log_hdr) +{ + return 0; +} + +static inline void +pktlog_arg_set_specific_data(struct ath_pktlog_arg *plarg, + uint32_t type_specific_data) +{ +} + +static inline uint32_t +pktlog_arg_get_specific_data(struct ath_pktlog_arg *plarg) +{ + return 0; +} +#endif /* PKTLOG_HAS_SPECIFIC_DATA */ #endif /* REMOVE_PKT_LOG */ #endif diff --git a/utils/pktlog/include/pktlog_wifi2.h b/utils/pktlog/include/pktlog_wifi2.h index a86140bfde..d24d527095 100644 --- a/utils/pktlog/include/pktlog_wifi2.h +++ b/utils/pktlog/include/pktlog_wifi2.h @@ -74,6 +74,15 @@ A_STATUS process_rate_find(void *pdev, void *data); * Return: 0 - success/non-zero - failure */ A_STATUS process_rate_update(void *pdev, void *data); + +/** + * process_sw_event() - process sw event pktlog buffers + * @txrx_pdev: ol pdev handle + * @data: pktlog buffer + * + * Return: 0 - success/non-zero - failure + */ +A_STATUS process_sw_event(void *pdev, void *data); #else static inline A_STATUS process_tx_info(struct cdp_pdev *txrx_pdev, void *data) @@ -104,4 +113,52 @@ A_STATUS process_rate_update(void *pdev, void *data) { return 0; } + +static inline +A_STATUS process_sw_event(void *pdev, void *data) +{ + return 0; +} #endif /* REMOVE_PKT_LOG */ + +/** + * process_offload_pktlog_wifi3() - Process full pktlog events + * pdev: abstract pdev handle + * data: pktlog buffer + * + * Return: zero on success, non-zero on failure + */ +static inline A_STATUS +process_offload_pktlog_wifi3(struct cdp_pdev *pdev, void *data) +{ + return 0; +} + +/** + * process_rx_desc_remote_wifi3() - Process pktlog buffers received + * from monitor status ring + * @pdev: pdev handle + * @data: pktlog buffer pointer + * + * Return: 0 - success/non-zero - failure + */ +static inline int +process_rx_desc_remote_wifi3(void *pdev, void *data) +{ + return 0; +} + +/** + * process_pktlog_lite_wifi3() - Process pktlog buffers received + * from monitor status ring + * @pdev: pdev handle + * @data: pktlog buffer pointer + * + * Return: 0 - success/non-zero - failure + */ +static inline int +process_pktlog_lite_wifi3(void *context, void *log_data, + uint16_t log_type) +{ + return 0; +} diff --git a/utils/pktlog/include/pktlog_wifi3.h b/utils/pktlog/include/pktlog_wifi3.h index 77927227fd..5ffe4f85f9 100644 --- a/utils/pktlog/include/pktlog_wifi3.h +++ b/utils/pktlog/include/pktlog_wifi3.h @@ -68,3 +68,81 @@ process_pktlog_lite_wifi3(void *context, void *log_data, return 0; } #endif /* REMOVE_PKT_LOG */ + +/** + * process_tx_info() - process tx pktlog buffers + * @txrx_pdev: ol pdev handle + * @data: pktlog buffer + * + * Return: 0 - success/non-zero - failure + */ +static inline +A_STATUS process_tx_info(struct cdp_pdev *txrx_pdev, void *data) +{ + return 0; +} + +/** + * process_rx_info_remote() - process rx pktlog buffers + * @txrx_pdev: ol pdev handle + * @data: pktlog buffer + * + * Return: 0 - success/non-zero - failure + */ +static inline +A_STATUS process_rx_info_remote(void *pdev, void *data) +{ + return 0; +} + +/** + * process_rx_remote() - process rx pktlog buffers + * @txrx_pdev: ol pdev handle + * @data: pktlog buffer + * + * Return: 0 - success/non-zero - failure + */ +static inline +A_STATUS process_rx_info(void *pdev, void *data) +{ + return 0; +} + +/** + * process_rate_find() - process rate event pktlog buffers + * @txrx_pdev: ol pdev handle + * @data: pktlog buffer + * + * Return: 0 - success/non-zero - failure + */ +static inline +A_STATUS process_rate_find(void *pdev, void *data) +{ + return 0; +} + +/** + * process_rate_update() - process rate event pktlog buffers + * @txrx_pdev: ol pdev handle + * @data: pktlog buffer + * + * Return: 0 - success/non-zero - failure + */ +static inline +A_STATUS process_rate_update(void *pdev, void *data) +{ + return 0; +} + +/** + * process_sw_event() - process sw event pktlog buffers + * @txrx_pdev: ol pdev handle + * @data: pktlog buffer + * + * Return: 0 - success/non-zero - failure + */ +static inline +A_STATUS process_sw_event(void *pdev, void *data) +{ + return 0; +} diff --git a/utils/pktlog/pktlog_ac.c b/utils/pktlog/pktlog_ac.c index cd083b204a..3dcd09a9bc 100644 --- a/utils/pktlog/pktlog_ac.c +++ b/utils/pktlog/pktlog_ac.c @@ -41,6 +41,11 @@ #include "htc.h" #include #include +#ifdef PKTLOG_LEGACY +#include "pktlog_wifi2.h" +#else +#include "pktlog_wifi3.h" +#endif /* PKTLOG_LEGACY */ wdi_event_subscribe PKTLOG_TX_SUBSCRIBER; wdi_event_subscribe PKTLOG_RX_SUBSCRIBER; diff --git a/utils/pktlog/pktlog_internal.c b/utils/pktlog/pktlog_internal.c index a4d1604318..d028931f01 100644 --- a/utils/pktlog/pktlog_internal.c +++ b/utils/pktlog/pktlog_internal.c @@ -43,56 +43,31 @@ #include "wlan_logging_sock_svc.h" #ifdef PKTLOG_HAS_SPECIFIC_DATA -static inline void +void pktlog_hdr_set_specific_data(struct ath_pktlog_hdr *log_hdr, uint32_t type_specific_data) { log_hdr->type_specific_data = type_specific_data; } -static inline uint32_t +uint32_t pktlog_hdr_get_specific_data(struct ath_pktlog_hdr *log_hdr) { return log_hdr->type_specific_data; } -static inline void +void pktlog_arg_set_specific_data(struct ath_pktlog_arg *plarg, uint32_t type_specific_data) { plarg->type_specific_data = type_specific_data; } -static inline uint32_t +uint32_t pktlog_arg_get_specific_data(struct ath_pktlog_arg *plarg) { return plarg->type_specific_data; } - -#else -static inline void -pktlog_hdr_set_specific_data(struct ath_pktlog_hdr *log_hdr, - uint32_t type_specific_data) -{ -} - -static inline uint32_t -pktlog_hdr_get_specific_data(struct ath_pktlog_hdr *log_hdr) -{ - return 0; -} - -static inline void -pktlog_arg_set_specific_data(struct ath_pktlog_arg *plarg, - uint32_t type_specific_data) -{ -} - -static inline uint32_t -pktlog_arg_get_specific_data(struct ath_pktlog_arg *plarg) -{ - return 0; -} #endif /* PKTLOG_HAS_SPECIFIC_DATA */ void pktlog_getbuf_intsafe(struct ath_pktlog_arg *plarg) @@ -221,182 +196,4 @@ char *pktlog_getbuf(struct pktlog_dev_t *pl_dev, return plarg.buf; } - -#ifdef HELIUMPLUS -A_STATUS process_sw_event(void *pdev, void *data) -{ - struct pktlog_dev_t *pl_dev = get_pktlog_handle(); - struct ath_pktlog_hdr pl_hdr; - struct ath_pktlog_info *pl_info; - size_t log_size; - uint32_t len; - struct ol_fw_data *fw_data; - - /* - * Will be uncommented when the rate control find - * for pktlog is implemented in the firmware. - * Currently derived from the TX PPDU status - */ - struct ath_pktlog_sw_event sw_event; - uint32_t *pl_tgt_hdr; - - if (!pdev) { - qdf_print("Invalid pdev in %s", __func__); - return A_ERROR; - } - if (!data) { - qdf_print("Invalid data in %s", __func__); - return A_ERROR; - } - if (!pl_dev) { - qdf_print("Invalid pl_dev in %s", __func__); - return A_ERROR; - } - - fw_data = (struct ol_fw_data *)data; - len = fw_data->len; - if (len < (sizeof(uint32_t) * - (ATH_PKTLOG_HDR_FLAGS_OFFSET + 1)) || - len < (sizeof(uint32_t) * - (ATH_PKTLOG_HDR_MISSED_CNT_OFFSET + 1)) || - len < (sizeof(uint32_t) * - (ATH_PKTLOG_HDR_LOG_TYPE_OFFSET + 1)) || - len < (sizeof(uint32_t) * - (ATH_PKTLOG_HDR_MAC_ID_OFFSET + 1)) || - len < (sizeof(uint32_t) * - (ATH_PKTLOG_HDR_SIZE_OFFSET + 1)) || - len < (sizeof(uint32_t) * - (ATH_PKTLOG_HDR_TYPE_SPECIFIC_DATA_OFFSET + 1))) { - qdf_print("Invalid msdu len in %s", __func__); - qdf_assert(0); - return A_ERROR; - } - - pl_tgt_hdr = (uint32_t *)fw_data->data; - /* - * Makes the short words (16 bits) portable b/w little endian - * and big endian - */ - pl_hdr.flags = (*(pl_tgt_hdr + ATH_PKTLOG_HDR_FLAGS_OFFSET) & - ATH_PKTLOG_HDR_FLAGS_MASK) >> - ATH_PKTLOG_HDR_FLAGS_SHIFT; - pl_hdr.missed_cnt = (*(pl_tgt_hdr + ATH_PKTLOG_HDR_MISSED_CNT_OFFSET) & - ATH_PKTLOG_HDR_MISSED_CNT_MASK) >> - ATH_PKTLOG_HDR_MISSED_CNT_SHIFT; - pl_hdr.log_type = (*(pl_tgt_hdr + ATH_PKTLOG_HDR_LOG_TYPE_OFFSET) & - ATH_PKTLOG_HDR_LOG_TYPE_MASK) >> - ATH_PKTLOG_HDR_LOG_TYPE_SHIFT; - pl_hdr.macId = (*(pl_tgt_hdr + ATH_PKTLOG_HDR_MAC_ID_OFFSET) & - ATH_PKTLOG_HDR_MAC_ID_MASK) >> - ATH_PKTLOG_HDR_MAC_ID_SHIFT; - pl_hdr.size = (*(pl_tgt_hdr + ATH_PKTLOG_HDR_SIZE_OFFSET) & - ATH_PKTLOG_HDR_SIZE_MASK) >> ATH_PKTLOG_HDR_SIZE_SHIFT; - pl_hdr.timestamp = *(pl_tgt_hdr + ATH_PKTLOG_HDR_TIMESTAMP_OFFSET); - - pl_hdr.type_specific_data = - *(pl_tgt_hdr + ATH_PKTLOG_HDR_TYPE_SPECIFIC_DATA_OFFSET); - pl_info = pl_dev->pl_info; - log_size = pl_hdr.size; - sw_event.sw_event = (void *)pktlog_getbuf(pl_dev, pl_info, - log_size, &pl_hdr); - if (sizeof(struct ath_pktlog_hdr) + pl_hdr.size > len) { - qdf_assert(0); - return A_ERROR; - } - qdf_mem_copy(sw_event.sw_event, - ((char *)fw_data->data + sizeof(struct ath_pktlog_hdr)), - pl_hdr.size); - - cds_pkt_stats_to_logger_thread(&pl_hdr, NULL, sw_event.sw_event); - - return A_OK; -} - -#else -A_STATUS process_sw_event(void *pdev, void *data) -{ - struct pktlog_dev_t *pl_dev = get_pktlog_handle(); - struct ath_pktlog_hdr pl_hdr; - struct ath_pktlog_info *pl_info; - size_t log_size; - uint32_t len; - struct ol_fw_data *fw_data; - - /* - * Will be uncommented when the rate control find - * for pktlog is implemented in the firmware. - * Currently derived from the TX PPDU status - */ - struct ath_pktlog_sw_event sw_event; - uint32_t *pl_tgt_hdr; - - if (!pdev) { - qdf_print("Invalid pdev in %s", __func__); - return A_ERROR; - } - if (!data) { - qdf_print("Invalid data in %s", __func__); - return A_ERROR; - } - if (!pl_dev) { - qdf_print("Invalid pl_dev in %s", __func__); - return A_ERROR; - } - - fw_data = (struct ol_fw_data *)data; - len = fw_data->len; - if (len < (sizeof(uint32_t) * - (ATH_PKTLOG_HDR_FLAGS_OFFSET + 1)) || - len < (sizeof(uint32_t) * - (ATH_PKTLOG_HDR_MISSED_CNT_OFFSET + 1)) || - len < (sizeof(uint32_t) * - (ATH_PKTLOG_HDR_LOG_TYPE_OFFSET + 1)) || - len < (sizeof(uint32_t) * - (ATH_PKTLOG_HDR_SIZE_OFFSET + 1)) || - len < (sizeof(uint32_t) * - (ATH_PKTLOG_HDR_TYPE_SPECIFIC_DATA_OFFSET + 1))) { - qdf_print("Invalid msdu len in %s", __func__); - qdf_assert(0); - return A_ERROR; - } - - pl_tgt_hdr = (uint32_t *)fw_data->data; - /* - * Makes the short words (16 bits) portable b/w little endian - * and big endian - */ - pl_hdr.flags = (*(pl_tgt_hdr + ATH_PKTLOG_HDR_FLAGS_OFFSET) & - ATH_PKTLOG_HDR_FLAGS_MASK) >> - ATH_PKTLOG_HDR_FLAGS_SHIFT; - pl_hdr.missed_cnt = (*(pl_tgt_hdr + ATH_PKTLOG_HDR_MISSED_CNT_OFFSET) & - ATH_PKTLOG_HDR_MISSED_CNT_MASK) >> - ATH_PKTLOG_HDR_MISSED_CNT_SHIFT; - pl_hdr.log_type = (*(pl_tgt_hdr + ATH_PKTLOG_HDR_LOG_TYPE_OFFSET) & - ATH_PKTLOG_HDR_LOG_TYPE_MASK) >> - ATH_PKTLOG_HDR_LOG_TYPE_SHIFT; - pl_hdr.size = (*(pl_tgt_hdr + ATH_PKTLOG_HDR_SIZE_OFFSET) & - ATH_PKTLOG_HDR_SIZE_MASK) >> ATH_PKTLOG_HDR_SIZE_SHIFT; - pl_hdr.timestamp = *(pl_tgt_hdr + ATH_PKTLOG_HDR_TIMESTAMP_OFFSET); - - pktlog_hdr_set_specific_data(&pl_hdr, - *(pl_tgt_hdr + - ATH_PKTLOG_HDR_TYPE_SPECIFIC_DATA_OFFSET)); - - pl_info = pl_dev->pl_info; - log_size = pl_hdr.size; - sw_event.sw_event = (void *)pktlog_getbuf(pl_dev, pl_info, - log_size, &pl_hdr); - if (sizeof(struct ath_pktlog_hdr) + pl_hdr.size > len) { - qdf_assert(0); - return A_ERROR; - } - qdf_mem_copy(sw_event.sw_event, - ((char *)fw_data->data + sizeof(struct ath_pktlog_hdr)), - pl_hdr.size); - - cds_pkt_stats_to_logger_thread(&pl_hdr, NULL, sw_event.sw_event); - - return A_OK; -} -#endif #endif /*REMOVE_PKT_LOG */ diff --git a/utils/pktlog/pktlog_wifi2.c b/utils/pktlog/pktlog_wifi2.c index 06ddcc794d..2cc8ae7866 100644 --- a/utils/pktlog/pktlog_wifi2.c +++ b/utils/pktlog/pktlog_wifi2.c @@ -17,6 +17,7 @@ /* WIFI2 - Refers to legacy platforms */ #include "pktlog_wifi2.h" +#ifndef REMOVE_PKT_LOG static struct txctl_frm_hdr frm_hdr; #ifndef HELIUMPLUS @@ -1015,3 +1016,181 @@ A_STATUS process_rate_update(void *pdev, void *data) return A_OK; } #endif /* HELIUMPLUS */ + +#ifdef HELIUMPLUS +A_STATUS process_sw_event(void *pdev, void *data) +{ + struct pktlog_dev_t *pl_dev = get_pktlog_handle(); + struct ath_pktlog_hdr pl_hdr; + struct ath_pktlog_info *pl_info; + size_t log_size; + uint32_t len; + struct ol_fw_data *fw_data; + + /* + * Will be uncommented when the rate control find + * for pktlog is implemented in the firmware. + * Currently derived from the TX PPDU status + */ + struct ath_pktlog_sw_event sw_event; + uint32_t *pl_tgt_hdr; + + if (!pdev) { + qdf_print("Invalid pdev in %s", __func__); + return A_ERROR; + } + if (!data) { + qdf_print("Invalid data in %s", __func__); + return A_ERROR; + } + if (!pl_dev) { + qdf_print("Invalid pl_dev in %s", __func__); + return A_ERROR; + } + + fw_data = (struct ol_fw_data *)data; + len = fw_data->len; + if (len < (sizeof(uint32_t) * + (ATH_PKTLOG_HDR_FLAGS_OFFSET + 1)) || + len < (sizeof(uint32_t) * + (ATH_PKTLOG_HDR_MISSED_CNT_OFFSET + 1)) || + len < (sizeof(uint32_t) * + (ATH_PKTLOG_HDR_LOG_TYPE_OFFSET + 1)) || + len < (sizeof(uint32_t) * + (ATH_PKTLOG_HDR_MAC_ID_OFFSET + 1)) || + len < (sizeof(uint32_t) * + (ATH_PKTLOG_HDR_SIZE_OFFSET + 1)) || + len < (sizeof(uint32_t) * + (ATH_PKTLOG_HDR_TYPE_SPECIFIC_DATA_OFFSET + 1))) { + qdf_print("Invalid msdu len in %s", __func__); + qdf_assert(0); + return A_ERROR; + } + + pl_tgt_hdr = (uint32_t *)fw_data->data; + /* + * Makes the short words (16 bits) portable b/w little endian + * and big endian + */ + pl_hdr.flags = (*(pl_tgt_hdr + ATH_PKTLOG_HDR_FLAGS_OFFSET) & + ATH_PKTLOG_HDR_FLAGS_MASK) >> + ATH_PKTLOG_HDR_FLAGS_SHIFT; + pl_hdr.missed_cnt = (*(pl_tgt_hdr + ATH_PKTLOG_HDR_MISSED_CNT_OFFSET) & + ATH_PKTLOG_HDR_MISSED_CNT_MASK) >> + ATH_PKTLOG_HDR_MISSED_CNT_SHIFT; + pl_hdr.log_type = (*(pl_tgt_hdr + ATH_PKTLOG_HDR_LOG_TYPE_OFFSET) & + ATH_PKTLOG_HDR_LOG_TYPE_MASK) >> + ATH_PKTLOG_HDR_LOG_TYPE_SHIFT; + pl_hdr.macId = (*(pl_tgt_hdr + ATH_PKTLOG_HDR_MAC_ID_OFFSET) & + ATH_PKTLOG_HDR_MAC_ID_MASK) >> + ATH_PKTLOG_HDR_MAC_ID_SHIFT; + pl_hdr.size = (*(pl_tgt_hdr + ATH_PKTLOG_HDR_SIZE_OFFSET) & + ATH_PKTLOG_HDR_SIZE_MASK) >> ATH_PKTLOG_HDR_SIZE_SHIFT; + pl_hdr.timestamp = *(pl_tgt_hdr + ATH_PKTLOG_HDR_TIMESTAMP_OFFSET); + + pl_hdr.type_specific_data = + *(pl_tgt_hdr + ATH_PKTLOG_HDR_TYPE_SPECIFIC_DATA_OFFSET); + pl_info = pl_dev->pl_info; + log_size = pl_hdr.size; + sw_event.sw_event = (void *)pktlog_getbuf(pl_dev, pl_info, + log_size, &pl_hdr); + if (sizeof(struct ath_pktlog_hdr) + pl_hdr.size > len) { + qdf_assert(0); + return A_ERROR; + } + qdf_mem_copy(sw_event.sw_event, + ((char *)fw_data->data + sizeof(struct ath_pktlog_hdr)), + pl_hdr.size); + + cds_pkt_stats_to_logger_thread(&pl_hdr, NULL, sw_event.sw_event); + + return A_OK; +} +#else +A_STATUS process_sw_event(void *pdev, void *data) +{ + struct pktlog_dev_t *pl_dev = get_pktlog_handle(); + struct ath_pktlog_hdr pl_hdr; + struct ath_pktlog_info *pl_info; + size_t log_size; + uint32_t len; + struct ol_fw_data *fw_data; + + /* + * Will be uncommented when the rate control find + * for pktlog is implemented in the firmware. + * Currently derived from the TX PPDU status + */ + struct ath_pktlog_sw_event sw_event; + uint32_t *pl_tgt_hdr; + + if (!pdev) { + qdf_print("Invalid pdev in %s", __func__); + return A_ERROR; + } + if (!data) { + qdf_print("Invalid data in %s", __func__); + return A_ERROR; + } + if (!pl_dev) { + qdf_print("Invalid pl_dev in %s", __func__); + return A_ERROR; + } + + fw_data = (struct ol_fw_data *)data; + len = fw_data->len; + if (len < (sizeof(uint32_t) * + (ATH_PKTLOG_HDR_FLAGS_OFFSET + 1)) || + len < (sizeof(uint32_t) * + (ATH_PKTLOG_HDR_MISSED_CNT_OFFSET + 1)) || + len < (sizeof(uint32_t) * + (ATH_PKTLOG_HDR_LOG_TYPE_OFFSET + 1)) || + len < (sizeof(uint32_t) * + (ATH_PKTLOG_HDR_SIZE_OFFSET + 1)) || + len < (sizeof(uint32_t) * + (ATH_PKTLOG_HDR_TYPE_SPECIFIC_DATA_OFFSET + 1))) { + qdf_print("Invalid msdu len in %s", __func__); + qdf_assert(0); + return A_ERROR; + } + + pl_tgt_hdr = (uint32_t *)fw_data->data; + /* + * Makes the short words (16 bits) portable b/w little endian + * and big endian + */ + pl_hdr.flags = (*(pl_tgt_hdr + ATH_PKTLOG_HDR_FLAGS_OFFSET) & + ATH_PKTLOG_HDR_FLAGS_MASK) >> + ATH_PKTLOG_HDR_FLAGS_SHIFT; + pl_hdr.missed_cnt = (*(pl_tgt_hdr + ATH_PKTLOG_HDR_MISSED_CNT_OFFSET) & + ATH_PKTLOG_HDR_MISSED_CNT_MASK) >> + ATH_PKTLOG_HDR_MISSED_CNT_SHIFT; + pl_hdr.log_type = (*(pl_tgt_hdr + ATH_PKTLOG_HDR_LOG_TYPE_OFFSET) & + ATH_PKTLOG_HDR_LOG_TYPE_MASK) >> + ATH_PKTLOG_HDR_LOG_TYPE_SHIFT; + pl_hdr.size = (*(pl_tgt_hdr + ATH_PKTLOG_HDR_SIZE_OFFSET) & + ATH_PKTLOG_HDR_SIZE_MASK) >> ATH_PKTLOG_HDR_SIZE_SHIFT; + pl_hdr.timestamp = *(pl_tgt_hdr + ATH_PKTLOG_HDR_TIMESTAMP_OFFSET); + + pktlog_hdr_set_specific_data(&pl_hdr, + *(pl_tgt_hdr + + ATH_PKTLOG_HDR_TYPE_SPECIFIC_DATA_OFFSET)); + + pl_info = pl_dev->pl_info; + log_size = pl_hdr.size; + sw_event.sw_event = (void *)pktlog_getbuf(pl_dev, pl_info, + log_size, &pl_hdr); + if (sizeof(struct ath_pktlog_hdr) + pl_hdr.size > len) { + qdf_assert(0); + return A_ERROR; + } + qdf_mem_copy(sw_event.sw_event, + ((char *)fw_data->data + sizeof(struct ath_pktlog_hdr)), + pl_hdr.size); + + cds_pkt_stats_to_logger_thread(&pl_hdr, NULL, sw_event.sw_event); + + return A_OK; +} +#endif /* HELIUMPLUS */ +#endif /* REMOVE_PKT_LOG */