qed: Handle possible race in SB config
authorYuval Mintz <Yuval.Mintz@qlogic.com>
Sun, 21 Feb 2016 09:40:08 +0000 (11:40 +0200)
committerDavid S. Miller <davem@davemloft.net>
Mon, 22 Feb 2016 03:49:15 +0000 (22:49 -0500)
Due to HW design, some of the memories are wide-bus and access to those
needs to be sequentialized on a per-HW-block level; Read/write to a
given HW-block might break other read/write to wide-bus memory done at
~same time.

Status blocks initialization in CAU is done into such a wide-bus memory.
This moves the initialization into using DMAE which is guaranteed to be
safe to use on such memories.

Signed-off-by: Yuval Mintz <Yuval.Mintz@qlogic.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/qlogic/qed/qed_int.c

index 90ba1d71e67a7e17f7b7543471c8c21e7b096c2f..fa73daa9465540a2c941c24fd2e2bd5c1f20b26d 100644 (file)
@@ -473,20 +473,20 @@ void qed_int_cau_conf_sb(struct qed_hwfn *p_hwfn,
                         u8 vf_valid)
 {
        struct cau_sb_entry sb_entry;
-       u32 val;
 
        qed_init_cau_sb_entry(p_hwfn, &sb_entry, p_hwfn->rel_pf_id,
                              vf_number, vf_valid);
 
        if (p_hwfn->hw_init_done) {
-               val = CAU_REG_SB_ADDR_MEMORY + igu_sb_id * sizeof(u64);
-               qed_wr(p_hwfn, p_ptt, val, lower_32_bits(sb_phys));
-               qed_wr(p_hwfn, p_ptt, val + sizeof(u32),
-                      upper_32_bits(sb_phys));
-
-               val = CAU_REG_SB_VAR_MEMORY + igu_sb_id * sizeof(u64);
-               qed_wr(p_hwfn, p_ptt, val, sb_entry.data);
-               qed_wr(p_hwfn, p_ptt, val + sizeof(u32), sb_entry.params);
+               /* Wide-bus, initialize via DMAE */
+               u64 phys_addr = (u64)sb_phys;
+
+               qed_dmae_host2grc(p_hwfn, p_ptt, (u64)(uintptr_t)&phys_addr,
+                                 CAU_REG_SB_ADDR_MEMORY +
+                                 igu_sb_id * sizeof(u64), 2, 0);
+               qed_dmae_host2grc(p_hwfn, p_ptt, (u64)(uintptr_t)&sb_entry,
+                                 CAU_REG_SB_VAR_MEMORY +
+                                 igu_sb_id * sizeof(u64), 2, 0);
        } else {
                /* Initialize Status Block Address */
                STORE_RT_REG_AGG(p_hwfn,