net/fm10k: fix Rx descriptor read timing
authorXiao Wang <xiao.w.wang@intel.com>
Mon, 4 Jul 2016 07:51:08 +0000 (15:51 +0800)
committerBruce Richardson <bruce.richardson@intel.com>
Tue, 5 Jul 2016 09:43:49 +0000 (11:43 +0200)
commitaf15ee640dfe2b45f1358638d41037e82604be4a
tree7f58f25c9297a2adcc8625e33bccc8df1aa72e9b
parentd1d861efef8ba6ddeec4560bff88b842288b7810
net/fm10k: fix Rx descriptor read timing

We find that when traffic is light, a small number of packets will have
wrong metadata (e.g. packet type), however this issue will not happen
when traffic is heavy.

The root cause is some fields in fm10k_rx_desc are read at the wrong time,
since the descriptor (being 16-bytes big) is not read as a single atomic
operation. When the input speed is slower than software's capability,
fm10k scalar Rx function accesses descriptors at about the same time
as HW writes them, so the scenario can occur: some fields like pkt_info
in fm10k_rx_desc are read before HW writeback but some fields like DD bit
are read after HW writeback, this will lead to the later packet parsing
function using incorrect value.

This patch fixes this issue by reading and parsing Rx descriptor only after
first checking that the DD bit is set.

Fixes: 4b61d3bfa941 ("fm10k: add receive and tranmit")
Fixes: c82dd0a7bfa5 ("fm10k: add scatter receive")

Signed-off-by: Wang Xiao W <xiao.w.wang@intel.com>
Acked-by: Bruce Richardson <bruce.richardson@intel.com>
drivers/net/fm10k/fm10k_rxtx.c