Gitweb:     
http://git.kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=82c3aca6ad9004169df8f2f8c0747686fe4003b3
Commit:     82c3aca6ad9004169df8f2f8c0747686fe4003b3
Parent:     3ec7393a6858a1716e74aa81be6af76fd180021d
Author:     Michael S. Tsirkin <[EMAIL PROTECTED]>
AuthorDate: Wed Jun 20 19:22:15 2007 +0300
Committer:  Roland Dreier <[EMAIL PROTECTED]>
CommitDate: Thu Jun 21 13:38:08 2007 -0700

    IPoIB/cm: Fix interoperability when MTU doesn't match
    
    IPoIB connected mode currently rejects a connection request unless the
    supported MTU is >= the local netdevice MTU. This breaks
    interoperability with implementations that might have tweaked
    IPOIB_CM_MTU, and there's real no longer a reason to do so: this test
    is just a leftover from when we did not tweak MTU per-connection.  Fix
    this by making the test as permissive as possible.
    
    Signed-off-by: Michael S. Tsirkin <[EMAIL PROTECTED]>
    Signed-off-by: Roland Dreier <[EMAIL PROTECTED]>
---
 drivers/infiniband/ulp/ipoib/ipoib_cm.c |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/infiniband/ulp/ipoib/ipoib_cm.c 
b/drivers/infiniband/ulp/ipoib/ipoib_cm.c
index c64249f..472c632 100644
--- a/drivers/infiniband/ulp/ipoib/ipoib_cm.c
+++ b/drivers/infiniband/ulp/ipoib/ipoib_cm.c
@@ -759,9 +759,9 @@ static int ipoib_cm_rep_handler(struct ib_cm_id *cm_id, 
struct ib_cm_event *even
 
        p->mtu = be32_to_cpu(data->mtu);
 
-       if (p->mtu < priv->dev->mtu + IPOIB_ENCAP_LEN) {
-               ipoib_warn(priv, "Rejecting connection: mtu %d < device mtu %d 
+ 4\n",
-                          p->mtu, priv->dev->mtu);
+       if (p->mtu <= IPOIB_ENCAP_LEN) {
+               ipoib_warn(priv, "Rejecting connection: mtu %d <= %d\n",
+                          p->mtu, IPOIB_ENCAP_LEN);
                return -EINVAL;
        }
 
-
To unsubscribe from this list: send the line "unsubscribe git-commits-head" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to