• Roland Dreier's avatar
    IB/mthca: Fix mthca_write_mtt() on HCAs with hidden memory · 532c3b58
    Roland Dreier authored
    Commit b2875d4c ("IB/mthca: Always fill MTTs from CPU") causes a crash
    in mthca_write_mtt() with non-memfree HCAs that have their memory
    hidden (that is, have only two PCI BARs instead of having a third BAR
    that allows access to the RAM attached to the HCA) on 64-bit
    architectures.  This is because the commit just before, c20e20ab
    ("IB/mthca: Merge MR and FMR space on 64-bit systems") makes
    dev->mr_table.fmr_mtt_buddy equal to &dev->mr_table.mtt_buddy and
    hence mthca_write_mtt() tries to write directly into the HCA's MTT
    table.  However, since that table is in the HCA's memory, this is
    impossible without the PCI BAR that gives access to that memory.
    
    This causes a crash because mthca_tavor_write_mtt_seg() basically
    tries to dereference some offset of a NULL pointer.  Fix this by
    adding a test of MTHCA_FLAG_FMR in mthca_write_mtt() so that we always
    use the WRITE_MTT firmware command rather than writing directly if
    FMRs are not enabled.
    Signed-off-by: default avatarRoland Dreier <rolandd@cisco.com>
    532c3b58
mthca_mr.c 23.7 KB