diff options
| author | Roland Dreier | 2005-04-17 00:26:24 +0200 |
|---|---|---|
| committer | Linus Torvalds | 2005-04-17 00:26:24 +0200 |
| commit | 44ea66879d5638cfed5b5ecf628badfd8ec26f36 (patch) | |
| tree | 987bb5368a8e4a058cffc2eef4752d7cca4335e5 /drivers/infiniband/hw/mthca/mthca_dev.h | |
| parent | [PATCH] IB/mthca: implement RDMA/atomic operations for mem-free mode (diff) | |
| download | kernel-qcow2-linux-44ea66879d5638cfed5b5ecf628badfd8ec26f36.tar.gz kernel-qcow2-linux-44ea66879d5638cfed5b5ecf628badfd8ec26f36.tar.xz kernel-qcow2-linux-44ea66879d5638cfed5b5ecf628badfd8ec26f36.zip | |
[PATCH] IB/mthca: fix MTT allocation in mem-free mode
Fix bug in MTT allocation in mem-free mode.
I misunderstood the MTT size value returned by the firmware -- it is really
the size of a single MTT entry, since mem-free mode does not segment the MTT
as the original firmware did. This meant that our MTT addresses ended up
being off by a factor of 8. This meant that our MTT allocations might
overlap, and so we could overwrite and corrupt earlier memory regions when
writing new MTT entries.
We fix this by always using our 64-byte MTT segment size. This allows some
simplification of the code as well, since there's no reason to put the MTT
segment size in a variable -- we can always use our enum value directly.
Signed-off-by: Roland Dreier <roland@topspin.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'drivers/infiniband/hw/mthca/mthca_dev.h')
| -rw-r--r-- | drivers/infiniband/hw/mthca/mthca_dev.h | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/infiniband/hw/mthca/mthca_dev.h b/drivers/infiniband/hw/mthca/mthca_dev.h index 56b2bfb5adb1..f437979e8967 100644 --- a/drivers/infiniband/hw/mthca/mthca_dev.h +++ b/drivers/infiniband/hw/mthca/mthca_dev.h @@ -121,7 +121,6 @@ struct mthca_limits { int reserved_eqs; int num_mpts; int num_mtt_segs; - int mtt_seg_size; int reserved_mtts; int reserved_mrws; int reserved_uars; |