IB/mlx5: Add modify_flow_action_esp verb

Adding implementation in mlx5 driver to modify action_xfrm object. This
merely call the accel layer. Currently a user can modify only the
ESN parameters.

Reviewed-by: Yishai Hadas <yishaih@mellanox.com>
Signed-off-by: Matan Barak <matanb@mellanox.com>
Signed-off-by: Aviad Yehezkel <aviadye@mellanox.com>
Signed-off-by: Leon Romanovsky <leonro@mellanox.com>
Signed-off-by: Jason Gunthorpe <jgg@mellanox.com>

diff --git a/drivers/infiniband/hw/mlx5/main.c b/drivers/infiniband/hw/mlx5/main.c
index 8b16a42..bc46589 100644 (file)
--- a/drivers/infiniband/hw/mlx5/main.c
+++ b/drivers/infiniband/hw/mlx5/main.c
@@ -3201,6 +3201,54 @@ err_parse:
        return ERR_PTR(err);
 }
 
+static int
+mlx5_ib_modify_flow_action_esp(struct ib_flow_action *action,
+                              const struct ib_flow_action_attrs_esp *attr,
+                              struct uverbs_attr_bundle *attrs)
+{
+       struct mlx5_ib_flow_action *maction = to_mflow_act(action);
+       struct mlx5_accel_esp_xfrm_attrs accel_attrs;
+       int err = 0;
+
+       if (attr->keymat || attr->replay || attr->encap ||
+           attr->spi || attr->seq || attr->tfc_pad ||
+           attr->hard_limit_pkts ||
+           (attr->flags & ~(IB_FLOW_ACTION_ESP_FLAGS_ESN_TRIGGERED |
+                            IB_FLOW_ACTION_ESP_FLAGS_MOD_ESP_ATTRS |
+                            IB_UVERBS_FLOW_ACTION_ESP_FLAGS_ESN_NEW_WINDOW)))
+               return -EOPNOTSUPP;
+
+       /* Only the ESN value or the MLX5_ACCEL_ESP_FLAGS_ESN_STATE_OVERLAP can
+        * be modified.
+        */
+       if (!(maction->esp_aes_gcm.ib_flags &
+             IB_FLOW_ACTION_ESP_FLAGS_ESN_TRIGGERED) &&
+           attr->flags & (IB_FLOW_ACTION_ESP_FLAGS_ESN_TRIGGERED |
+                          IB_UVERBS_FLOW_ACTION_ESP_FLAGS_ESN_NEW_WINDOW))
+               return -EINVAL;
+
+       memcpy(&accel_attrs, &maction->esp_aes_gcm.ctx->attrs,
+              sizeof(accel_attrs));
+
+       accel_attrs.esn = attr->esn;
+       if (attr->flags & IB_UVERBS_FLOW_ACTION_ESP_FLAGS_ESN_NEW_WINDOW)
+               accel_attrs.flags |= MLX5_ACCEL_ESP_FLAGS_ESN_STATE_OVERLAP;
+       else
+               accel_attrs.flags &= ~MLX5_ACCEL_ESP_FLAGS_ESN_STATE_OVERLAP;
+
+       err = mlx5_accel_esp_modify_xfrm(maction->esp_aes_gcm.ctx,
+                                        &accel_attrs);
+       if (err)
+               return err;
+
+       maction->esp_aes_gcm.ib_flags &=
+               ~IB_UVERBS_FLOW_ACTION_ESP_FLAGS_ESN_NEW_WINDOW;
+       maction->esp_aes_gcm.ib_flags |=
+               attr->flags & IB_UVERBS_FLOW_ACTION_ESP_FLAGS_ESN_NEW_WINDOW;
+
+       return 0;
+}
+
 static int mlx5_ib_destroy_flow_action(struct ib_flow_action *action)
 {
        struct mlx5_ib_flow_action *maction = to_mflow_act(action);
@@ -4927,6 +4975,7 @@ int mlx5_ib_stage_caps_init(struct mlx5_ib_dev *dev)
                        (1ull << IB_USER_VERBS_EX_CMD_DESTROY_FLOW);
        dev->ib_dev.create_flow_action_esp = mlx5_ib_create_flow_action_esp;
        dev->ib_dev.destroy_flow_action = mlx5_ib_destroy_flow_action;
+       dev->ib_dev.modify_flow_action_esp = mlx5_ib_modify_flow_action_esp;
        dev->ib_dev.driver_id = RDMA_DRIVER_MLX5;
 
        err = init_node_data(dev);