From 12c7f7a08b0166c916a662ef07be11a162998aaa Mon Sep 17 00:00:00 2001 From: Harsh Kumar Bijlani Date: Tue, 8 Sep 2020 19:49:33 +0530 Subject: [PATCH] qcacmn: Change max headroom for monitor destination buffer Macro MAX_MONITOR_HEADER is used to specify the max. headroom size for different buffer rings. With the monitor frag feature, macro DP_RX_MON_MAX_MONITOR_HEADER is required to specify the headroom size for the monitor destination buffers as per the the approch being used. Change-Id: Ic4c0132292008942d7d443235e5f54d7eaa77e74 CRs-Fixed: 2772048 --- dp/wifi3.0/dp_rx_mon.h | 16 +++++++++++----- 1 file changed, 11 insertions(+), 5 deletions(-) diff --git a/dp/wifi3.0/dp_rx_mon.h b/dp/wifi3.0/dp_rx_mon.h index ff19b224ac43..b7347a3b2966 100644 --- a/dp/wifi3.0/dp_rx_mon.h +++ b/dp/wifi3.0/dp_rx_mon.h @@ -26,7 +26,10 @@ */ #define MON_BUF_MIN_ENTRIES 64 -/* The maxinum buffer length allocated for radio tap */ +/* + * The maximum headroom reserved for monitor destination buffer to + * accomodate radiotap header and protocol flow tag + */ #ifdef DP_RX_MON_MEM_FRAG /* *---------------------------------- @@ -37,11 +40,14 @@ * actual offset, data gets written to actual offset after updating * radiotap HDR. */ -#define MAX_MONITOR_HEADER (256) +#define DP_RX_MON_MAX_MONITOR_HEADER (256) #else -#define MAX_MONITOR_HEADER (512) +#define DP_RX_MON_MAX_MONITOR_HEADER (512) #endif +/* The maximum buffer length allocated for radiotap for monitor status buffer */ +#define MAX_MONITOR_HEADER (512) + /* l2 header pad byte in case of Raw frame is Zero and 2 in non raw */ #define DP_RX_MON_RAW_L2_HDR_PAD_BYTE (0) #define DP_RX_MON_NONRAW_L2_HDR_PAD_BYTE (2) @@ -412,8 +418,8 @@ QDF_STATUS dp_rx_mon_alloc_parent_buffer(qdf_nbuf_t *head_msdu) * | 64 B | Length(128 B) | * -------------------------------------- */ - *head_msdu = qdf_nbuf_alloc_no_recycler(MAX_MONITOR_HEADER, - MAX_MONITOR_HEADER, 4); + *head_msdu = qdf_nbuf_alloc_no_recycler(DP_RX_MON_MAX_MONITOR_HEADER, + DP_RX_MON_MAX_MONITOR_HEADER, 4); if (!(*head_msdu)) return QDF_STATUS_E_FAILURE;