Commit 8b6573ff authored by Chandramohan Akula's avatar Chandramohan Akula Committed by Leon Romanovsky

bnxt_re: Update the debug counters for doorbell pacing

Add debug counters to track the Doorbell pacing events and report the
doorbell pacing debug stats.
Signed-off-by: default avatarChandramohan Akula <chandramohan.akula@broadcom.com>
Signed-off-by: default avatarSelvin Xavier <selvin.xavier@broadcom.com>
Link: https://lore.kernel.org/r/1690383081-15033-5-git-send-email-selvin.xavier@broadcom.comSigned-off-by: default avatarLeon Romanovsky <leon@kernel.org>
parent 4405baf8
...@@ -146,6 +146,10 @@ static const struct rdma_stat_desc bnxt_re_stat_descs[] = { ...@@ -146,6 +146,10 @@ static const struct rdma_stat_desc bnxt_re_stat_descs[] = {
[BNXT_RE_TX_CNP].name = "tx_cnp_pkts", [BNXT_RE_TX_CNP].name = "tx_cnp_pkts",
[BNXT_RE_RX_CNP].name = "rx_cnp_pkts", [BNXT_RE_RX_CNP].name = "rx_cnp_pkts",
[BNXT_RE_RX_ECN].name = "rx_ecn_marked_pkts", [BNXT_RE_RX_ECN].name = "rx_ecn_marked_pkts",
[BNXT_RE_PACING_RESCHED].name = "pacing_reschedule",
[BNXT_RE_PACING_CMPL].name = "pacing_complete",
[BNXT_RE_PACING_ALERT].name = "pacing_alerts",
[BNXT_RE_DB_FIFO_REG].name = "db_fifo_register",
}; };
static void bnxt_re_copy_ext_stats(struct bnxt_re_dev *rdev, static void bnxt_re_copy_ext_stats(struct bnxt_re_dev *rdev,
...@@ -278,6 +282,18 @@ static void bnxt_re_copy_err_stats(struct bnxt_re_dev *rdev, ...@@ -278,6 +282,18 @@ static void bnxt_re_copy_err_stats(struct bnxt_re_dev *rdev,
err_s->res_oos_drop_count; err_s->res_oos_drop_count;
} }
static void bnxt_re_copy_db_pacing_stats(struct bnxt_re_dev *rdev,
struct rdma_hw_stats *stats)
{
struct bnxt_re_db_pacing_stats *pacing_s = &rdev->stats.pacing;
stats->value[BNXT_RE_PACING_RESCHED] = pacing_s->resched;
stats->value[BNXT_RE_PACING_CMPL] = pacing_s->complete;
stats->value[BNXT_RE_PACING_ALERT] = pacing_s->alerts;
stats->value[BNXT_RE_DB_FIFO_REG] =
readl(rdev->en_dev->bar0 + rdev->pacing.dbr_db_fifo_reg_off);
}
int bnxt_re_ib_get_hw_stats(struct ib_device *ibdev, int bnxt_re_ib_get_hw_stats(struct ib_device *ibdev,
struct rdma_hw_stats *stats, struct rdma_hw_stats *stats,
u32 port, int index) u32 port, int index)
...@@ -350,6 +366,8 @@ int bnxt_re_ib_get_hw_stats(struct ib_device *ibdev, ...@@ -350,6 +366,8 @@ int bnxt_re_ib_get_hw_stats(struct ib_device *ibdev,
goto done; goto done;
} }
} }
if (rdev->pacing.dbr_pacing)
bnxt_re_copy_db_pacing_stats(rdev, stats);
} }
done: done:
......
...@@ -129,11 +129,21 @@ enum bnxt_re_hw_stats { ...@@ -129,11 +129,21 @@ enum bnxt_re_hw_stats {
BNXT_RE_TX_CNP, BNXT_RE_TX_CNP,
BNXT_RE_RX_CNP, BNXT_RE_RX_CNP,
BNXT_RE_RX_ECN, BNXT_RE_RX_ECN,
BNXT_RE_PACING_RESCHED,
BNXT_RE_PACING_CMPL,
BNXT_RE_PACING_ALERT,
BNXT_RE_DB_FIFO_REG,
BNXT_RE_NUM_EXT_COUNTERS BNXT_RE_NUM_EXT_COUNTERS
}; };
#define BNXT_RE_NUM_STD_COUNTERS (BNXT_RE_OUT_OF_SEQ_ERR + 1) #define BNXT_RE_NUM_STD_COUNTERS (BNXT_RE_OUT_OF_SEQ_ERR + 1)
struct bnxt_re_db_pacing_stats {
u64 resched;
u64 complete;
u64 alerts;
};
struct bnxt_re_res_cntrs { struct bnxt_re_res_cntrs {
atomic_t qp_count; atomic_t qp_count;
atomic_t rc_qp_count; atomic_t rc_qp_count;
...@@ -164,6 +174,7 @@ struct bnxt_re_rstat { ...@@ -164,6 +174,7 @@ struct bnxt_re_rstat {
struct bnxt_re_stats { struct bnxt_re_stats {
struct bnxt_re_rstat rstat; struct bnxt_re_rstat rstat;
struct bnxt_re_res_cntrs res; struct bnxt_re_res_cntrs res;
struct bnxt_re_db_pacing_stats pacing;
}; };
struct rdma_hw_stats *bnxt_re_ib_alloc_hw_port_stats(struct ib_device *ibdev, struct rdma_hw_stats *bnxt_re_ib_alloc_hw_port_stats(struct ib_device *ibdev,
......
...@@ -533,6 +533,7 @@ static void bnxt_re_db_fifo_check(struct work_struct *work) ...@@ -533,6 +533,7 @@ static void bnxt_re_db_fifo_check(struct work_struct *work)
pacing_data->pacing_th * BNXT_RE_PACING_ALARM_TH_MULTIPLE; pacing_data->pacing_th * BNXT_RE_PACING_ALARM_TH_MULTIPLE;
schedule_delayed_work(&rdev->dbq_pacing_work, schedule_delayed_work(&rdev->dbq_pacing_work,
msecs_to_jiffies(rdev->pacing.dbq_pacing_time)); msecs_to_jiffies(rdev->pacing.dbq_pacing_time));
rdev->stats.pacing.alerts++;
mutex_unlock(&rdev->pacing.dbq_lock); mutex_unlock(&rdev->pacing.dbq_lock);
} }
...@@ -563,12 +564,14 @@ static void bnxt_re_pacing_timer_exp(struct work_struct *work) ...@@ -563,12 +564,14 @@ static void bnxt_re_pacing_timer_exp(struct work_struct *work)
pacing_data->do_pacing = max_t(u32, rdev->pacing.dbr_def_do_pacing, pacing_data->do_pacing); pacing_data->do_pacing = max_t(u32, rdev->pacing.dbr_def_do_pacing, pacing_data->do_pacing);
if (pacing_data->do_pacing <= rdev->pacing.dbr_def_do_pacing) { if (pacing_data->do_pacing <= rdev->pacing.dbr_def_do_pacing) {
bnxt_re_set_default_pacing_data(rdev); bnxt_re_set_default_pacing_data(rdev);
rdev->stats.pacing.complete++;
goto dbq_unlock; goto dbq_unlock;
} }
restart_timer: restart_timer:
schedule_delayed_work(&rdev->dbq_pacing_work, schedule_delayed_work(&rdev->dbq_pacing_work,
msecs_to_jiffies(rdev->pacing.dbq_pacing_time)); msecs_to_jiffies(rdev->pacing.dbq_pacing_time));
rdev->stats.pacing.resched++;
dbq_unlock: dbq_unlock:
rdev->pacing.do_pacing_save = pacing_data->do_pacing; rdev->pacing.do_pacing_save = pacing_data->do_pacing;
mutex_unlock(&rdev->pacing.dbq_lock); mutex_unlock(&rdev->pacing.dbq_lock);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment