diff options
| author | Jack Morgenstein <jackm@dev.mellanox.co.il> | 2012-03-07 09:56:35 +0400 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2012-03-08 12:28:01 +0400 |
| commit | dcf353b17007841f91e1735e33790508c84c9ad0 (patch) | |
| tree | c7733dca7ad0bd49b37660160be857b04bec0b84 /drivers | |
| parent | 783154573236f46b730b2eea17d2e6e04bdb3251 (diff) | |
| download | linux-dcf353b17007841f91e1735e33790508c84c9ad0.tar.xz | |
mlx4_core: fix bug in modify_cq wrapper for resize flow.
The actual FW command is called in procedure "handle_resize".
Code incorrectly invoked the FW command again (in good flow), in
the modify_cq wrapper function.
Fix by skipping second FW invocation unconditionally for resize.
Signed-off-by: Jack Morgenstein <jackm@dev.mellanox.co.il>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/net/ethernet/mellanox/mlx4/resource_tracker.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/net/ethernet/mellanox/mlx4/resource_tracker.c b/drivers/net/ethernet/mellanox/mlx4/resource_tracker.c index bfdb7af19e49..8752e6e08169 100644 --- a/drivers/net/ethernet/mellanox/mlx4/resource_tracker.c +++ b/drivers/net/ethernet/mellanox/mlx4/resource_tracker.c @@ -2255,8 +2255,7 @@ int mlx4_MODIFY_CQ_wrapper(struct mlx4_dev *dev, int slave, if (vhcr->op_modifier == 0) { err = handle_resize(dev, slave, vhcr, inbox, outbox, cmd, cq); - if (err) - goto ex_put; + goto ex_put; } err = mlx4_DMA_wrapper(dev, slave, vhcr, inbox, outbox, cmd); |