Merge branch 'mlx5-fixes'

Saeed Mahameed says:

====================
Mellanox 100G mlx5 fixes 2016-12-04

Some bug fixes for mlx5 core and mlx5e driver.

v1->v2:
 - replace "uint" with "unsigned int"
====================

Signed-off-by: David S. Miller <davem@davemloft.net>

8 files changed, 53 insertions, 42 deletions

diff --git a/drivers/net/ethernet/mellanox/mlx5/core/cmd.c b/drivers/net/ethernet/mellanox/mlx5/core/cmd.c
index 1e639f886021..bfe410e8a469 100644
--- a/drivers/net/ethernet/mellanox/mlx5/core/cmd.c
+++ b/drivers/net/ethernet/mellanox/mlx5/core/cmd.c
@@ -268,11 +268,6 @@ static void dump_buf(void *buf, int size, int data_only, int offset)
                 pr_debug("\n");
 }
 
-enum {
-        MLX5_DRIVER_STATUS_ABORTED = 0xfe,
-        MLX5_DRIVER_SYND = 0xbadd00de,
-};
-
 static int mlx5_internal_err_ret_value(struct mlx5_core_dev *dev, u16 op,
                                        u32 *synd, u8 *status)
 {
diff --git a/drivers/net/ethernet/mellanox/mlx5/core/en.h b/drivers/net/ethernet/mellanox/mlx5/core/en.h
index 7a43502a89cc..71382df59fc0 100644
--- a/drivers/net/ethernet/mellanox/mlx5/core/en.h
+++ b/drivers/net/ethernet/mellanox/mlx5/core/en.h
@@ -241,7 +241,7 @@ struct mlx5e_tstamp {
 };
 
 enum {
-        MLX5E_RQ_STATE_FLUSH,
+        MLX5E_RQ_STATE_ENABLED,
         MLX5E_RQ_STATE_UMR_WQE_IN_PROGRESS,
         MLX5E_RQ_STATE_AM,
 };
@@ -394,7 +394,7 @@ struct mlx5e_sq_dma {
 };
 
 enum {
-        MLX5E_SQ_STATE_FLUSH,
+        MLX5E_SQ_STATE_ENABLED,
         MLX5E_SQ_STATE_BF_ENABLE,
 };
 
diff --git a/drivers/net/ethernet/mellanox/mlx5/core/en_main.c b/drivers/net/ethernet/mellanox/mlx5/core/en_main.c
index 84e8b250e2af..246d98ebb588 100644
--- a/drivers/net/ethernet/mellanox/mlx5/core/en_main.c
+++ b/drivers/net/ethernet/mellanox/mlx5/core/en_main.c
@@ -759,6 +759,7 @@ static int mlx5e_open_rq(struct mlx5e_channel *c,
         if (err)
                 goto err_destroy_rq;
 
+        set_bit(MLX5E_RQ_STATE_ENABLED, &rq->state);
         err = mlx5e_modify_rq_state(rq, MLX5_RQC_STATE_RST, MLX5_RQC_STATE_RDY);
         if (err)
                 goto err_disable_rq;
@@ -773,6 +774,7 @@ static int mlx5e_open_rq(struct mlx5e_channel *c,
         return 0;
 
 err_disable_rq:
+        clear_bit(MLX5E_RQ_STATE_ENABLED, &rq->state);
         mlx5e_disable_rq(rq);
 err_destroy_rq:
         mlx5e_destroy_rq(rq);
@@ -782,7 +784,7 @@ err_destroy_rq:
 
 static void mlx5e_close_rq(struct mlx5e_rq *rq)
 {
-        set_bit(MLX5E_RQ_STATE_FLUSH, &rq->state);
+        clear_bit(MLX5E_RQ_STATE_ENABLED, &rq->state);
         napi_synchronize(&rq->channel->napi); /* prevent mlx5e_post_rx_wqes */
         cancel_work_sync(&rq->am.work);
 
@@ -1006,7 +1008,6 @@ static int mlx5e_enable_sq(struct mlx5e_sq *sq, struct mlx5e_sq_param *param)
         MLX5_SET(sqc,  sqc, min_wqe_inline_mode, sq->min_inline_mode);
         MLX5_SET(sqc,  sqc, state,              MLX5_SQC_STATE_RST);
         MLX5_SET(sqc,  sqc, tis_lst_sz, param->type == MLX5E_SQ_ICO ? 0 : 1);
-        MLX5_SET(sqc,  sqc, flush_in_error_en,  1);
 
         MLX5_SET(wq,   wq, wq_type,       MLX5_WQ_TYPE_CYCLIC);
         MLX5_SET(wq,   wq, uar_page,      sq->uar.index);
@@ -1083,6 +1084,7 @@ static int mlx5e_open_sq(struct mlx5e_channel *c,
         if (err)
                 goto err_destroy_sq;
 
+        set_bit(MLX5E_SQ_STATE_ENABLED, &sq->state);
         err = mlx5e_modify_sq(sq, MLX5_SQC_STATE_RST, MLX5_SQC_STATE_RDY,
                               false, 0);
         if (err)
@@ -1096,6 +1098,7 @@ static int mlx5e_open_sq(struct mlx5e_channel *c,
         return 0;
 
 err_disable_sq:
+        clear_bit(MLX5E_SQ_STATE_ENABLED, &sq->state);
         mlx5e_disable_sq(sq);
 err_destroy_sq:
         mlx5e_destroy_sq(sq);
@@ -1112,7 +1115,7 @@ static inline void netif_tx_disable_queue(struct netdev_queue *txq)
 
 static void mlx5e_close_sq(struct mlx5e_sq *sq)
 {
-        set_bit(MLX5E_SQ_STATE_FLUSH, &sq->state);
+        clear_bit(MLX5E_SQ_STATE_ENABLED, &sq->state);
         /* prevent netif_tx_wake_queue */
         napi_synchronize(&sq->channel->napi);
 
@@ -3092,7 +3095,7 @@ static void mlx5e_tx_timeout(struct net_device *dev)
                 if (!netif_xmit_stopped(netdev_get_tx_queue(dev, i)))
                         continue;
                 sched_work = true;
-                set_bit(MLX5E_SQ_STATE_FLUSH, &sq->state);
+                clear_bit(MLX5E_SQ_STATE_ENABLED, &sq->state);
                 netdev_err(dev, "TX timeout on queue: %d, SQ: 0x%x, CQ: 0x%x, SQ Cons: 0x%x SQ Prod: 0x%x\n",
                            i, sq->sqn, sq->cq.mcq.cqn, sq->cc, sq->pc);
         }
@@ -3147,13 +3150,13 @@ static int mlx5e_xdp_set(struct net_device *netdev, struct bpf_prog *prog)
         for (i = 0; i < priv->params.num_channels; i++) {
                 struct mlx5e_channel *c = priv->channel[i];
 
-                set_bit(MLX5E_RQ_STATE_FLUSH, &c->rq.state);
+                clear_bit(MLX5E_RQ_STATE_ENABLED, &c->rq.state);
                 napi_synchronize(&c->napi);
                 /* prevent mlx5e_poll_rx_cq from accessing rq->xdp_prog */
 
                 old_prog = xchg(&c->rq.xdp_prog, prog);
 
-                clear_bit(MLX5E_RQ_STATE_FLUSH, &c->rq.state);
+                set_bit(MLX5E_RQ_STATE_ENABLED, &c->rq.state);
                 /* napi_schedule in case we have missed anything */
                 set_bit(MLX5E_CHANNEL_NAPI_SCHED, &c->flags);
                 napi_schedule(&c->napi);
diff --git a/drivers/net/ethernet/mellanox/mlx5/core/en_rx.c b/drivers/net/ethernet/mellanox/mlx5/core/en_rx.c
index c6de6fba5843..33495d88aeb2 100644
--- a/drivers/net/ethernet/mellanox/mlx5/core/en_rx.c
+++ b/drivers/net/ethernet/mellanox/mlx5/core/en_rx.c
@@ -340,7 +340,7 @@ static inline void mlx5e_post_umr_wqe(struct mlx5e_rq *rq, u16 ix)
         while ((pi = (sq->pc & wq->sz_m1)) > sq->edge) {
                 sq->db.ico_wqe[pi].opcode = MLX5_OPCODE_NOP;
                 sq->db.ico_wqe[pi].num_wqebbs = 1;
-                mlx5e_send_nop(sq, true);
+                mlx5e_send_nop(sq, false);
         }
 
         wqe = mlx5_wq_cyc_get_wqe(wq, pi);
@@ -412,7 +412,7 @@ void mlx5e_post_rx_mpwqe(struct mlx5e_rq *rq)
 
         clear_bit(MLX5E_RQ_STATE_UMR_WQE_IN_PROGRESS, &rq->state);
 
-        if (unlikely(test_bit(MLX5E_RQ_STATE_FLUSH, &rq->state))) {
+        if (unlikely(!test_bit(MLX5E_RQ_STATE_ENABLED, &rq->state))) {
                 mlx5e_free_rx_mpwqe(rq, &rq->mpwqe.info[wq->head]);
                 return;
         }
@@ -445,7 +445,7 @@ void mlx5e_dealloc_rx_mpwqe(struct mlx5e_rq *rq, u16 ix)
 }
 
 #define RQ_CANNOT_POST(rq) \
-        (test_bit(MLX5E_RQ_STATE_FLUSH, &rq->state) || \
+        (!test_bit(MLX5E_RQ_STATE_ENABLED, &rq->state) || \
          test_bit(MLX5E_RQ_STATE_UMR_WQE_IN_PROGRESS, &rq->state))
 
 bool mlx5e_post_rx_wqes(struct mlx5e_rq *rq)
@@ -924,7 +924,7 @@ int mlx5e_poll_rx_cq(struct mlx5e_cq *cq, int budget)
         struct mlx5e_sq *xdp_sq = &rq->channel->xdp_sq;
         int work_done = 0;
 
-        if (unlikely(test_bit(MLX5E_RQ_STATE_FLUSH, &rq->state)))
+        if (unlikely(!test_bit(MLX5E_RQ_STATE_ENABLED, &rq->state)))
                 return 0;
 
         if (cq->decmprs_left)
diff --git a/drivers/net/ethernet/mellanox/mlx5/core/en_tx.c b/drivers/net/ethernet/mellanox/mlx5/core/en_tx.c
index 70a717382357..cfb68371c397 100644
--- a/drivers/net/ethernet/mellanox/mlx5/core/en_tx.c
+++ b/drivers/net/ethernet/mellanox/mlx5/core/en_tx.c
@@ -409,7 +409,7 @@ bool mlx5e_poll_tx_cq(struct mlx5e_cq *cq, int napi_budget)
 
         sq = container_of(cq, struct mlx5e_sq, cq);
 
-        if (unlikely(test_bit(MLX5E_SQ_STATE_FLUSH, &sq->state)))
+        if (unlikely(!test_bit(MLX5E_SQ_STATE_ENABLED, &sq->state)))
                 return false;
 
         npkts = 0;
diff --git a/drivers/net/ethernet/mellanox/mlx5/core/en_txrx.c b/drivers/net/ethernet/mellanox/mlx5/core/en_txrx.c
index 5703f19a6a24..e5c12a732aa1 100644
--- a/drivers/net/ethernet/mellanox/mlx5/core/en_txrx.c
+++ b/drivers/net/ethernet/mellanox/mlx5/core/en_txrx.c
@@ -56,7 +56,7 @@ static void mlx5e_poll_ico_cq(struct mlx5e_cq *cq)
         struct mlx5_cqe64 *cqe;
         u16 sqcc;
 
-        if (unlikely(test_bit(MLX5E_SQ_STATE_FLUSH, &sq->state)))
+        if (unlikely(!test_bit(MLX5E_SQ_STATE_ENABLED, &sq->state)))
                 return;
 
         cqe = mlx5e_get_cqe(cq);
@@ -113,7 +113,7 @@ static inline bool mlx5e_poll_xdp_tx_cq(struct mlx5e_cq *cq)
 
         sq = container_of(cq, struct mlx5e_sq, cq);
 
-        if (unlikely(test_bit(MLX5E_SQ_STATE_FLUSH, &sq->state)))
+        if (unlikely(!test_bit(MLX5E_SQ_STATE_ENABLED, &sq->state)))
                 return false;
 
         /* sq->cc must be updated only after mlx5_cqwq_update_db_record(),
diff --git a/drivers/net/ethernet/mellanox/mlx5/core/main.c b/drivers/net/ethernet/mellanox/mlx5/core/main.c
index 3b7c6a9f2b5f..ada24e103b02 100644
--- a/drivers/net/ethernet/mellanox/mlx5/core/main.c
+++ b/drivers/net/ethernet/mellanox/mlx5/core/main.c
@@ -62,13 +62,13 @@ MODULE_DESCRIPTION("Mellanox Connect-IB, ConnectX-4 core driver");
 MODULE_LICENSE("Dual BSD/GPL");
 MODULE_VERSION(DRIVER_VERSION);
 
-int mlx5_core_debug_mask;
-module_param_named(debug_mask, mlx5_core_debug_mask, int, 0644);
+unsigned int mlx5_core_debug_mask;
+module_param_named(debug_mask, mlx5_core_debug_mask, uint, 0644);
 MODULE_PARM_DESC(debug_mask, "debug mask: 1 = dump cmd data, 2 = dump cmd exec time, 3 = both. Default=0");
 
 #define MLX5_DEFAULT_PROF       2
-static int prof_sel = MLX5_DEFAULT_PROF;
-module_param_named(prof_sel, prof_sel, int, 0444);
+static unsigned int prof_sel = MLX5_DEFAULT_PROF;
+module_param_named(prof_sel, prof_sel, uint, 0444);
 MODULE_PARM_DESC(prof_sel, "profile selector. Valid range 0 - 2");
 
 enum {
@@ -732,13 +732,15 @@ static int mlx5_core_set_issi(struct mlx5_core_dev *dev)
                 u8 status;
 
                 mlx5_cmd_mbox_status(query_out, &status, &syndrome);
-                if (status == MLX5_CMD_STAT_BAD_OP_ERR) {
-                        pr_debug("Only ISSI 0 is supported\n");
-                        return 0;
+                if (!status || syndrome == MLX5_DRIVER_SYND) {
+                        mlx5_core_err(dev, "Failed to query ISSI err(%d) status(%d) synd(%d)\n",
+                                      err, status, syndrome);
+                        return err;
                 }
 
-                pr_err("failed to query ISSI err(%d)\n", err);
-                return err;
+                mlx5_core_warn(dev, "Query ISSI is not supported by FW, ISSI is 0\n");
+                dev->issi = 0;
+                return 0;
         }
 
         sup_issi = MLX5_GET(query_issi_out, query_out, supported_issi_dw0);
@@ -752,7 +754,8 @@ static int mlx5_core_set_issi(struct mlx5_core_dev *dev)
                 err = mlx5_cmd_exec(dev, set_in, sizeof(set_in),
                                     set_out, sizeof(set_out));
                 if (err) {
-                        pr_err("failed to set ISSI=1 err(%d)\n", err);
+                        mlx5_core_err(dev, "Failed to set ISSI to 1 err(%d)\n",
+                                      err);
                         return err;
                 }
 
@@ -1227,13 +1230,6 @@ static int init_one(struct pci_dev *pdev,
 
         dev->pdev = pdev;
         dev->event = mlx5_core_event;
-
-        if (prof_sel < 0 || prof_sel >= ARRAY_SIZE(profile)) {
-                mlx5_core_warn(dev,
-                               "selected profile out of range, selecting default (%d)\n",
-                               MLX5_DEFAULT_PROF);
-                prof_sel = MLX5_DEFAULT_PROF;
-        }
         dev->profile = &profile[prof_sel];
 
         INIT_LIST_HEAD(&priv->ctx_list);
@@ -1450,10 +1446,22 @@ static struct pci_driver mlx5_core_driver = {
         .sriov_configure   = mlx5_core_sriov_configure,
 };
 
+static void mlx5_core_verify_params(void)
+{
+        if (prof_sel >= ARRAY_SIZE(profile)) {
+                pr_warn("mlx5_core: WARNING: Invalid module parameter prof_sel %d, valid range 0-%zu, changing back to default(%d)\n",
+                        prof_sel,
+                        ARRAY_SIZE(profile) - 1,
+                        MLX5_DEFAULT_PROF);
+                prof_sel = MLX5_DEFAULT_PROF;
+        }
+}
+
 static int __init init(void)
 {
         int err;
 
+        mlx5_core_verify_params();
         mlx5_register_debugfs();
 
         err = pci_register_driver(&mlx5_core_driver);
diff --git a/drivers/net/ethernet/mellanox/mlx5/core/mlx5_core.h b/drivers/net/ethernet/mellanox/mlx5/core/mlx5_core.h
index 187662c8ea96..63b9a0dba885 100644
--- a/drivers/net/ethernet/mellanox/mlx5/core/mlx5_core.h
+++ b/drivers/net/ethernet/mellanox/mlx5/core/mlx5_core.h
@@ -44,11 +44,11 @@
 
 #define MLX5_TOTAL_VPORTS(mdev) (1 + pci_sriov_get_totalvfs(mdev->pdev))
 
-extern int mlx5_core_debug_mask;
+extern uint mlx5_core_debug_mask;
 
 #define mlx5_core_dbg(__dev, format, ...)                               \
-        dev_dbg(&(__dev)->pdev->dev, "%s:%s:%d:(pid %d): " format,      \
-                 (__dev)->priv.name, __func__, __LINE__, current->pid,  \
+        dev_dbg(&(__dev)->pdev->dev, "%s:%d:(pid %d): " format,         \
+                 __func__, __LINE__, current->pid,                      \
                  ##__VA_ARGS__)
 
 #define mlx5_core_dbg_mask(__dev, mask, format, ...)                    \
@@ -63,8 +63,8 @@ do {									\
                ##__VA_ARGS__)
 
 #define mlx5_core_warn(__dev, format, ...)                              \
-        dev_warn(&(__dev)->pdev->dev, "%s:%s:%d:(pid %d): " format,     \
-                (__dev)->priv.name, __func__, __LINE__, current->pid,   \
+        dev_warn(&(__dev)->pdev->dev, "%s:%d:(pid %d): " format,        \
+                 __func__, __LINE__, current->pid,                      \
                 ##__VA_ARGS__)
 
 #define mlx5_core_info(__dev, format, ...)                              \
@@ -75,6 +75,11 @@ enum {
         MLX5_CMD_TIME, /* print command execution time */
 };
 
+enum {
+        MLX5_DRIVER_STATUS_ABORTED = 0xfe,
+        MLX5_DRIVER_SYND = 0xbadd00de,
+};
+
 int mlx5_query_hca_caps(struct mlx5_core_dev *dev);
 int mlx5_query_board_id(struct mlx5_core_dev *dev);
 int mlx5_cmd_init_hca(struct mlx5_core_dev *dev);