diff options
author | Anton Blanchard <anton@samba.org> | 2010-09-06 21:21:41 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2010-09-06 21:21:41 -0400 |
commit | f89e49e79f839dbe79364ec6cd7e2274ad11b120 (patch) | |
tree | 45d75386c47d38f940770326ade59bc57a27c93c /drivers/net/ibmveth.c | |
parent | 8d86c61ae41d9068fd5e5cc01a4abd53c4fe3ab5 (diff) |
ibmveth: Use lighter weight read memory barrier in ibmveth_poll
We want to order the read in ibmveth_rxq_pending_buffer and the read of
ibmveth_rxq_buffer_valid which are both cacheable memory. smp_rmb() is good
enough for this.
Signed-off-by: Anton Blanchard <anton@samba.org>
Signed-off-by: Santiago Leon <santil@linux.vnet.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/ibmveth.c')
-rw-r--r-- | drivers/net/ibmveth.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/ibmveth.c b/drivers/net/ibmveth.c index c236c8af3367..aedcb1802182 100644 --- a/drivers/net/ibmveth.c +++ b/drivers/net/ibmveth.c | |||
@@ -1010,7 +1010,7 @@ static int ibmveth_poll(struct napi_struct *napi, int budget) | |||
1010 | if (!ibmveth_rxq_pending_buffer(adapter)) | 1010 | if (!ibmveth_rxq_pending_buffer(adapter)) |
1011 | break; | 1011 | break; |
1012 | 1012 | ||
1013 | rmb(); | 1013 | smp_rmb(); |
1014 | if (!ibmveth_rxq_buffer_valid(adapter)) { | 1014 | if (!ibmveth_rxq_buffer_valid(adapter)) { |
1015 | wmb(); /* suggested by larson1 */ | 1015 | wmb(); /* suggested by larson1 */ |
1016 | adapter->rx_invalid_buffer++; | 1016 | adapter->rx_invalid_buffer++; |