qcacmn: Fill skb->priority only for macro ATH_RX_PRI_SAVE
skb->priority is needed only for ATH_RX_PRI_SAVE. This access can
be avoided in regular datapath by using skb->cb for faster access.
Change-Id: I5ef970fa45fa314db7e14284d898c98a59d8591e
diff --git a/dp/wifi3.0/dp_rx.c b/dp/wifi3.0/dp_rx.c
index c1cd694..979a9c6 100644
--- a/dp/wifi3.0/dp_rx.c
+++ b/dp/wifi3.0/dp_rx.c
@@ -30,6 +30,13 @@
#include "dp_rx_mon.h"
#include "dp_ipa.h"
+#ifdef ATH_RX_PRI_SAVE
+#define DP_RX_TID_SAVE(_nbuf, _tid) \
+ (qdf_nbuf_set_priority(_nbuf, _tid))
+#else
+#define DP_RX_TID_SAVE(_nbuf, _tid)
+#endif
+
#ifdef CONFIG_MCL
static inline bool dp_rx_check_ap_bridge(struct dp_vdev *vdev)
{
@@ -45,23 +52,6 @@
}
#endif
-#ifdef ATH_RX_PRI_SAVE
-static inline void dp_rx_save_tid_ts(qdf_nbuf_t nbuf, uint8_t tid, bool flag)
-{
- qdf_nbuf_set_priority(nbuf, tid);
- if (qdf_unlikely(flag))
- qdf_nbuf_set_timestamp(nbuf);
-}
-#else
-static inline void dp_rx_save_tid_ts(qdf_nbuf_t nbuf, uint8_t tid, bool flag)
-{
- if (qdf_unlikely(flag)) {
- qdf_nbuf_set_priority(nbuf, tid);
- qdf_nbuf_set_timestamp(nbuf);
- }
-}
-#endif
-
/*
* dp_rx_dump_info_and_assert() - dump RX Ring info and Rx Desc info
*
@@ -1942,7 +1932,10 @@
* to be written
*/
rx_pdev = soc->pdev_list[rx_desc->pool_id];
- dp_rx_save_tid_ts(nbuf, tid, rx_pdev->delay_stats_flag);
+ DP_RX_TID_SAVE(nbuf, tid);
+ if (qdf_unlikely(rx_pdev->delay_stats_flag))
+ qdf_nbuf_set_timestamp(nbuf);
+
tid_stats = &rx_pdev->stats.tid_stats.tid_rx_stats[tid];
if (qdf_unlikely(!hal_rx_attn_msdu_done_get(rx_tlv_hdr))) {
dp_err("MSDU DONE failure");