From: Michal Kalderon <michal.kalde...@cavium.com>
Date: Mon, 2 Oct 2017 11:23:48 +0300

> When more than one ll2 queue is opened ( that is not an OOO queue )
> ll2 code does not have enough information to determine whether
> the queue is the main one or not, so a new field is added to the
> acquire input data to expose the control of determining whether
> the queue is the main queue or a secondary queue.
> 
> Signed-off-by: Michal Kalderon <michal.kalde...@cavium.com>
> Signed-off-by: Ariel Elior <ariel.el...@cavium.com>
> ---
>  drivers/net/ethernet/qlogic/qed/qed_ll2.c | 7 ++++++-
>  drivers/net/ethernet/qlogic/qed/qed_ll2.h | 1 +
>  include/linux/qed/qed_ll2_if.h            | 1 +
>  3 files changed, 8 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/net/ethernet/qlogic/qed/qed_ll2.c 
> b/drivers/net/ethernet/qlogic/qed/qed_ll2.c
> index 10e3a43..1dd0cca 100644
> --- a/drivers/net/ethernet/qlogic/qed/qed_ll2.c
> +++ b/drivers/net/ethernet/qlogic/qed/qed_ll2.c
> @@ -894,7 +894,7 @@ static int qed_sp_ll2_rx_queue_start(struct qed_hwfn 
> *p_hwfn,
>       p_ramrod->drop_ttl0_flg = p_ll2_conn->input.rx_drop_ttl0_flg;
>       p_ramrod->inner_vlan_removal_en = p_ll2_conn->input.rx_vlan_removal_en;
>       p_ramrod->queue_id = p_ll2_conn->queue_id;
> -     p_ramrod->main_func_queue = (conn_type == QED_LL2_TYPE_OOO) ? 0 : 1;
> +     p_ramrod->main_func_queue = p_ll2_conn->main_func_queue;
>  
>       if ((IS_MF_DEFAULT(p_hwfn) || IS_MF_SI(p_hwfn)) &&
>           p_ramrod->main_func_queue && (conn_type != QED_LL2_TYPE_ROCE) &&
> @@ -1265,6 +1265,11 @@ int qed_ll2_acquire_connection(void *cxt, struct 
> qed_ll2_acquire_data *data)
>  
>       p_ll2_info->tx_dest = (data->input.tx_dest == QED_LL2_TX_DEST_NW) ?
>                             CORE_TX_DEST_NW : CORE_TX_DEST_LB;
> +     if (data->input.conn_type == QED_LL2_TYPE_OOO ||
> +         data->input.secondary_queue)
> +             p_ll2_info->main_func_queue = false;
> +     else
> +             p_ll2_info->main_func_queue = true;
 ...
> +     u8 main_func_queue;

If these things are bools please use the 'bool' type.

Thank you.

Reply via email to