Browse Source

qcacmn: remove module parameter from dp code

Remove napi_budget module parameter from dp code.

Change-Id: Ib48d08044977da9091f1a1264d219420d7f53962
CRs-Fixed: 2096650
Venkateswara Swamy Bandaru 7 years ago
parent
commit
b3d6b4e446
4 changed files with 14 additions and 15 deletions
  1. 1 6
      dp/wifi3.0/dp_main.c
  2. 2 1
      hif/inc/hif.h
  3. 8 7
      hif/src/hif_exec.c
  4. 3 1
      hif/src/hif_exec.h

+ 1 - 6
dp/wifi3.0/dp_main.c

@@ -71,11 +71,6 @@ cdp_dump_flow_pool_info(struct cdp_soc_t *soc)
 #define RX_RING_MASK_VAL	0xF
 #endif
 
-unsigned int napi_budget = 128;
-module_param(napi_budget, uint, 0644);
-MODULE_PARM_DESC(napi_budget,
-		"tasklet mode: more than 0xffff , napi budget if <= 0xffff");
-
 bool rx_hash = 1;
 qdf_declare_param(rx_hash, bool);
 
@@ -1154,7 +1149,7 @@ static QDF_STATUS dp_soc_interrupt_attach(void *txrx_soc)
 		ret = hif_register_ext_group(soc->hif_handle,
 				num_irq, irq_id_map, dp_service_srngs,
 				&soc->intr_ctx[i], "dp_intr",
-				napi_budget);
+				HIF_EXEC_NAPI_TYPE, 2);
 
 		if (ret) {
 			QDF_TRACE(QDF_MODULE_ID_DP, QDF_TRACE_LEVEL_ERROR,

+ 2 - 1
hif/inc/hif.h

@@ -890,7 +890,8 @@ uint32_t hif_configure_ext_group_interrupts(struct hif_opaque_softc *hif_ctx);
 uint32_t  hif_register_ext_group(struct hif_opaque_softc *hif_ctx,
 		uint32_t numirq, uint32_t irq[], ext_intr_handler handler,
 		void *cb_ctx, const char *context_name,
-		uint32_t budget);
+		enum hif_exec_type type, uint32_t scale);
+
 void hif_deregister_exec_group(struct hif_opaque_softc *hif_ctx,
 				const char *context_name);
 

+ 8 - 7
hif/src/hif_exec.c

@@ -342,7 +342,8 @@ void hif_exec_kill(struct hif_opaque_softc *hif_ctx)
  */
 uint32_t hif_register_ext_group(struct hif_opaque_softc *hif_ctx,
 		uint32_t numirq, uint32_t irq[], ext_intr_handler handler,
-		void *cb_ctx, const char *context_name, uint32_t budget)
+		void *cb_ctx, const char *context_name,
+		enum hif_exec_type type, uint32_t scale)
 {
 	struct hif_softc *scn = HIF_GET_SOFTC(hif_ctx);
 	struct HIF_CE_state *hif_state = HIF_GET_CE_STATE(scn);
@@ -363,7 +364,7 @@ uint32_t hif_register_ext_group(struct hif_opaque_softc *hif_ctx,
 		return QDF_STATUS_E_FAILURE;
 	}
 
-	hif_ext_group = hif_exec_create(budget);
+	hif_ext_group = hif_exec_create(type, scale);
 	if (hif_ext_group == NULL)
 		return QDF_STATUS_E_FAILURE;
 
@@ -387,15 +388,15 @@ uint32_t hif_register_ext_group(struct hif_opaque_softc *hif_ctx,
  * hif_exec_create() - create an execution context
  * @type: the type of execution context to create
  */
-struct hif_exec_context *hif_exec_create(uint32_t budget)
+struct hif_exec_context *hif_exec_create(enum hif_exec_type type,
+						uint32_t scale)
 {
-	uint32_t type = budget <= 0xFFFF ?
-				HIF_EXEC_NAPI_TYPE : HIF_EXEC_TASKLET_TYPE;
+	HIF_INFO("%s: create exec_type %d budget %d\n",
+			__func__, type, QCA_NAPI_BUDGET * scale);
 
-	HIF_INFO("%s: create exec_type %d budget %d\n", __func__, type, budget);
 	switch (type) {
 	case HIF_EXEC_NAPI_TYPE:
-		return hif_exec_napi_create(budget);
+		return hif_exec_napi_create(QCA_NAPI_BUDGET * scale);
 
 	case HIF_EXEC_TASKLET_TYPE:
 		return hif_exec_tasklet_create();

+ 3 - 1
hif/src/hif_exec.h

@@ -107,7 +107,9 @@ static inline struct hif_tasklet_exec_context*
 	return (struct hif_tasklet_exec_context *) ctx;
 }
 
-struct hif_exec_context *hif_exec_create(enum hif_exec_type type);
+struct hif_exec_context *hif_exec_create(enum hif_exec_type type,
+						uint32_t scale);
+
 void hif_exec_destroy(struct hif_exec_context *ctx);
 
 int hif_grp_irq_configure(struct hif_softc *scn,