net/smc: use smc_curs_copy() for SMC-D
authorUrsula Braun <ubraun@linux.ibm.com>
Thu, 7 Feb 2019 14:56:16 +0000 (15:56 +0100)
committerDavid S. Miller <davem@davemloft.net>
Fri, 8 Feb 2019 02:06:18 +0000 (18:06 -0800)
SMC already provides a wrapper for atomic64 calls to be
architecture independent. Use this wrapper for SMC-D as well.

Reported-by: Jens Remus <jremus@linux.ibm.com>
Signed-off-by: Ursula Braun <ubraun@linux.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/smc/smc_cdc.h

index b5bfe38c7f9b6a87258adc0aecce58c31c2a164a..e8c214b992b69e5158e93c6eeb7b1c720c1a3036 100644 (file)
@@ -245,17 +245,18 @@ static inline void smcr_cdc_msg_to_host(struct smc_host_cdc_msg *local,
 }
 
 static inline void smcd_cdc_msg_to_host(struct smc_host_cdc_msg *local,
-                                       struct smcd_cdc_msg *peer)
+                                       struct smcd_cdc_msg *peer,
+                                       struct smc_connection *conn)
 {
        union smc_host_cursor temp;
 
        temp.wrap = peer->prod.wrap;
        temp.count = peer->prod.count;
-       atomic64_set(&local->prod.acurs, atomic64_read(&temp.acurs));
+       smc_curs_copy(&local->prod, &temp, conn);
 
        temp.wrap = peer->cons.wrap;
        temp.count = peer->cons.count;
-       atomic64_set(&local->cons.acurs, atomic64_read(&temp.acurs));
+       smc_curs_copy(&local->cons, &temp, conn);
        local->prod_flags = peer->cons.prod_flags;
        local->conn_state_flags = peer->cons.conn_state_flags;
 }
@@ -265,7 +266,7 @@ static inline void smc_cdc_msg_to_host(struct smc_host_cdc_msg *local,
                                       struct smc_connection *conn)
 {
        if (conn->lgr->is_smcd)
-               smcd_cdc_msg_to_host(local, (struct smcd_cdc_msg *)peer);
+               smcd_cdc_msg_to_host(local, (struct smcd_cdc_msg *)peer, conn);
        else
                smcr_cdc_msg_to_host(local, peer, conn);
 }