On Sat,  7 Nov 2020 13:59:46 +0100 Karsten Graul wrote:
> +static inline void smc_switch_link_and_count(struct smc_connection *conn,
> +                                          struct smc_link *to_lnk)

Please avoid the use of static inline functions.

The compiler should inline this 3 instruction function anyway.

Please fix other instances in this submission as well.

> +{
> +     atomic_dec(&conn->lnk->conn_cnt);
> +     conn->lnk = to_lnk;
> +     atomic_inc(&conn->lnk->conn_cnt);
> +}

Reply via email to