Re: [PATCH 3/3] infiniband: rxe: fix 32-bit build warnings

2016-06-13 Thread Moni Shoua
thanks. will be applied to next series. On Mon, Jun 13, 2016 at 3:54 PM, Arnd Bergmann wrote: > The new rxe infinband driver passes around pointers that have been > converted to 64-bit integers. This is valid, but causes compile-time > warnings on all 32-bit architectures: > > infiniband/hw/rxe/r

Re: [PATCH 1/3] infiniband: rxe: avoid 64-bit division

2016-06-13 Thread Moni Shoua
thanks. will be applied to next series On Mon, Jun 13, 2016 at 4:18 PM, Leon Romanovsky wrote: > On Mon, Jun 13, 2016 at 02:54:53PM +0200, Arnd Bergmann wrote: >> The rxe driver fails to build on 32-bit because of a 64-bit division: >> >> In function `rxe_qp_from_attr': >> :(.text+0x53158): undef

Re: [PATCH 2/3] infiniband: rxe: add UDP_TUNNEL dependency

2016-06-13 Thread Moni Shoua
thanks. will be applied to next series On Mon, Jun 13, 2016 at 3:54 PM, Arnd Bergmann wrote: > The newly added rxe driver links against the UDP tunneling code, > which causes build errors when CONFIG_UDP_TUNNEL is disabled: > > ERROR: "setup_udp_tunnel_sock" [drivers/infiniband/hw/rxe/ib_rxe.ko]

Possible bug in bonding

2007-11-12 Thread Moni Shoua
Hi, As reported last week by Or Gerlitz and confirmed by me, there is a kernel crash when trying to unenslave all ib slaves (which leads to bonding master destruction). I also found that it happens with Ethernet slaves if following the steps: 1. unenslaving all slaves via sysfs 2. deleting th

Re: bonding / 2.6.24-rc1 issues

2007-11-07 Thread Moni Shoua
Or Gerlitz wrote: > Jay, Moni > > I did some tests with 2.6.24-rc1 and the first patch to bonding that Jay > sent last night to netdev. Basic operation and fail over work fine. > However, I see some crashes which are somehow related to destroying the > bond when the slaves are ipoib ones, I don't

Re: [PATCH 1/2] bonding: fix rtnl locking merge error

2007-11-07 Thread Moni Shoua
d so the first function doesn't mess with rtnl, and the > second correctly acquires and releases rtnl. > > Bug reported by Moni Shoua <[EMAIL PROTECTED]> I ran some shallow tests and it seems that the patch fixes the problem. thanks - To unsubscribe from this list: send the

Re: problems with ib-bonding of 2.6.24-rc1

2007-11-06 Thread Moni Shoua
Same thing happens with Ethernet slave - Intel 82546GB Gigabit Ethernet Controller (rev 03) - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: problems with ib-bonding of 2.6.24-rc1

2007-11-05 Thread Moni Shoua
Jay Vosburgh wrote: > Moni Shoua <[EMAIL PROTECTED]> wrote: > >> Basically, what I see is that after a while commands like ifconfig or ip >> stucks. >> I only use sysfs to configure bonding (which also stucks after a while). > > I've fooled with

Re: problems with ib-bonding of 2.6.24-rc1

2007-11-05 Thread Moni Shoua
Moni Shoua wrote: > Basically, what I see is that after a while commands like ifconfig or ip > stucks. > I only use sysfs to configure bonding (which also stucks after a while). > Maybe some extra information might help This is what happens in the kernel while the command '

problems with ib-bonding of 2.6.24-rc1

2007-11-04 Thread Moni Shoua
Hi, I've been doing some tests for bonding of 2.6.24-rc1 and noticed some problems. My first goal was to see how bonding works with IPoIB slaves but I also tried it with Ethernet. Basically, what I see is that after a while commands like ifconfig or ip stucks. I only use sysfs to configure bondin

Re: [PATCH linux-2.6] bonding: two small fixes for IPoIB support

2007-10-16 Thread Moni Shoua
ding to version 3.2.1. > > Signed-off-by: Moni Shoua > Signed-off-by: Jay Vosburgh <[EMAIL PROTECTED]> > > --- > drivers/net/bonding/bond_main.c | 11 +-- > drivers/net/bonding/bonding.h |4 ++-- > 2 files changed, 7 insertions(+), 8 deletions(-)

[PATCH V7 8/8] net/bonding: Destroy bonding master when last slave is gone

2007-10-15 Thread Moni Shoua
when slave_cnt is zero ensures that these functions be used anymore. Signed-off-by: Moni Shoua --- drivers/net/bonding/bond_main.c | 37 - drivers/net/bonding/bond_sysfs.c |9 + drivers/net/bonding/bonding.h|3 +++ 3 files changed, 44 inser

PATCH V6 7/8] net/bonding: Delay sending of gratuitous ARP to avoid failure

2007-10-15 Thread Moni Shoua
Delay sending a gratuitous_arp when LINK_STATE_LINKWATCH_PENDING bit in dev->state field is on. This improves the chances for the arp packet to be transmitted. Signed-off-by: Moni Shoua Acked-by: Jay Vosburgh <[EMAIL PROTECTED]> --- drivers/net/bonding/bond_main

[PATCH V7 5/8] net/bonding: Enable IP multicast for bonding IPoIB devices

2007-10-15 Thread Moni Shoua
where for multicast joins taking place after the enslavement another ip_xxx_mc_map() is called (eg ip_ib_mc_map() when the bond type is ARPHRD_INFINIBAND) Signed-off-by: Moni Shoua Signed-off-by: Or Gerlitz Acked-by: Jay Vosburgh <[EMAIL PROTECTED]> --- drivers/net/bonding/bond_main.c

[PATCH V7 6/8] net/bonding: Handlle wrong assumptions that slave is always an Ethernet device

2007-10-15 Thread Moni Shoua
bonding sometimes uses Ethernet constants (such as MTU and address length) which are not good when it enslaves non Ethernet devices (such as InfiniBand). Signed-off-by: Moni Shoua Acked-by: Jay Vosburgh <[EMAIL PROTECTED]> --- drivers/net/bonding/bond_main.c |3 ++- drivers/net/b

[PATCH V7 4/8] net/bonding: Enable bonding to enslave netdevices not supporting set_mac_address()

2007-10-15 Thread Moni Shoua
already done by the bonding code). Signed-off-by: Moni Shoua Signed-off-by: Or Gerlitz Acked-by: Jay Vosburgh <[EMAIL PROTECTED]> --- drivers/net/bonding/bond_main.c | 87 +++- drivers/net/bonding/bonding.h |1 2 files changed, 60 insertions(

[PATCH V7 2/8] IB/ipoib: Verify address handle validity on send

2007-10-15 Thread Moni Shoua
ipoib_neigh. This patch detects and prevents this from happenning. Signed-off-by: Moni Shoua Signed-off-by: Or Gerlitz Acked-by: Roland Dreier <[EMAIL PROTECTED]> --- drivers/infiniband/ulp/ipoib/ipoib_main.c |5 +++-- 1 files changed, 3 insertions(+), 2 deletions(-) diff --git a/d

[PATCH V7 3/8] net/bonding: Enable bonding to enslave non ARPHRD_ETHER

2007-10-15 Thread Moni Shoua
) of the port this IPoIB device is bounded to. The QP is a resource created by the IB HW and the GID is an identifier burned into the HCA (i have omitted here some details which are not important for the bonding RFC). Signed-off-by: Moni Shoua Signed-off-by: Or Gerlitz --- drivers/net/bonding

Re: [ofa-general] [PATCH V7 0/8] net/bonding: ADD IPoIB support for the bonding driver

2007-10-15 Thread Moni Shoua
Moni Shoua wrote: > This is the 7th version of this patch series. See link to V6 below. > I forgot to mention that the patches are relative to jgarzik/netdev-2.6.git#master. I couldn't compile the 2.6.24 or the upstream branches so I used master branch to test the fixes. - To

[PATCH V7 1/8] IB/ipoib: Bound the net device to the ipoib_neigh structue

2007-10-15 Thread Moni Shoua
as the IFF_MASTER bit set. Signed-off-by: Moni Shoua Signed-off-by: Or Gerlitz Acked-by: Roland Dreier <[EMAIL PROTECTED]> --- drivers/infiniband/ulp/ipoib/ipoib.h |4 +++- drivers/infiniband/ulp/ipoib/ipoib_main.c | 24 +++- drivers/infiniband/

[PATCH V7 0/8] net/bonding: ADD IPoIB support for the bonding driver

2007-10-15 Thread Moni Shoua
This is the 7th version of this patch series. See link to V6 below. Changes from the previous version - * Some patches required modifications to remove offsets so they can be applied with git-apply * Patch #3 was first modified by Jay and later by me to make it wo

Re: [PATCH] IB/ipoib: Bound the net device to the ipoib_neigh structue

2007-10-14 Thread Moni Shoua
Roland Dreier wrote: > > It happens only when ib interfaces are slaves of a bonding device. > > I thought before that the stuck is in napi_disable() but it's almost right. > > I put prints before and after call to napi_disable and see that it is > called twice. > > I'll try to investigate in t

Re: [PATCH] IB/ipoib: Bound the net device to the ipoib_neigh structue

2007-10-13 Thread Moni Shoua
I will be near my lab only tomorrow... I will check this and let you know. On 10/11/07, Roland Dreier <[EMAIL PROTECTED]> wrote: > > It happens only when ib interfaces are slaves of a bonding device. > > I thought before that the stuck is in napi_disable() but it's almost right. > > I put print

Re: [PATCH] IB/ipoib: Bound the net device to the ipoib_neigh structue

2007-10-11 Thread Moni Shoua
Roland Dreier wrote: > > I also ran a test for the code in the branch of 2.6.24 and found a problem. > > I see that ifconfig down doesn't return (for IPoIB interfaces) and it's > stuck in napi_disable() in the kernel (any idea why?) > > For what it's worth, I took the upstream 2.6.23 git tree a

Re: [PATCH] IB/ipoib: Bound the net device to the ipoib_neigh structue

2007-10-10 Thread Moni Shoua
Jay Vosburgh wrote: > David Miller <[EMAIL PROTECTED]> wrote: > >> From: Jeff Garzik <[EMAIL PROTECTED]> >> Date: Tue, 09 Oct 2007 20:56:35 -0400 >> >>> Jeff Garzik wrote: applied patches 1-9 the only thing that was a hiccup during submission is that your email subject lines d

Re: [ofa-general] Re: [PATCH V6 0/9] net/bonding: ADD IPoIB support for the bonding driver

2007-10-09 Thread Moni Shoua
Jay Vosburgh wrote: > Jeff Garzik <[EMAIL PROTECTED]> wrote: > >> Moni Shoua wrote: >>> Jay Vosburgh wrote: >>>> ACK patches 3 - 9. >>>> >>>>Roland, are you comfortable with the IB changes in patches 1 and 2? >>

Re: [ofa-general] Re: [PATCH V6 0/9] net/bonding: ADD IPoIB support for the bonding driver

2007-10-02 Thread Moni Shoua
Jay Vosburgh wrote: > ACK patches 3 - 9. > > Roland, are you comfortable with the IB changes in patches 1 and 2? > > Jeff, when Roland acks patches 1 and 2, please apply all 9. > > -J Hi Jeff, Roland acked the IPoIB patches. If you haven't done so already can you please apply

Re: [ofa-general] [PATCH V6 0/9] net/bonding: ADD IPoIB support for the bonding driver

2007-09-25 Thread Moni Shoua
Jay, I think that all comments to the patches were discussed and handled. If you agree, can you please push then to the networking tree so they will be merged into 2.6.24? This includes the IPoIB patches (agreed with Roland). Note that there are *no* patches to net/core (like in V5). thanks Moni

Re: [PATCH V6 5/9] net/bonding: Enable IP multicast for bonding IPoIB devices

2007-09-25 Thread Moni Shoua
> > Please get rid of the warning. Make bonding work correctly and allow > enslave/remove > of device when bonding is down. > - > To unsubscribe from this list: send the line "unsubscribe netdev" in > the body of a message to [EMAIL PROTECTED] > More majordomo info at http://vger.kernel.org/maj

[PATCH V6 8/9] net/bonding: Destroy bonding master when last slave is gone

2007-09-24 Thread Moni Shoua
when slave_cnt is zero ensures that these functions be used anymore. Signed-off-by: Moni Shoua --- drivers/net/bonding/bond_main.c | 37 + drivers/net/bonding/bond_sysfs.c |9 + drivers/net/bonding/bonding.h|3 +++ 3 files changed, 45 inser

[PATCH 9/9] bonding: Optionally allow ethernet slaves to keep own MAC

2007-09-24 Thread Moni Shoua
Update the "don't change MAC of slaves" functionality added in previous changes to be a generic option, rather than something tied to IB devices, as it's occasionally useful for regular ethernet devices as well. Adds "fail_over_mac" option (which is automatically enabled for IB slaves), ap

PATCH V6 7/9] net/bonding: Delay sending of gratuitous ARP to avoid failure

2007-09-24 Thread Moni Shoua
Delay sending a gratuitous_arp when LINK_STATE_LINKWATCH_PENDING bit in dev->state field is on. This improves the chances for the arp packet to be transmitted. Signed-off-by: Moni Shoua --- drivers/net/bonding/bond_main.c | 24 +--- drivers/net/bonding/bonding.h |

[PATCH V6 6/9] net/bonding: Handlle wrong assumptions that slave is always an Ethernet device

2007-09-24 Thread Moni Shoua
bonding sometimes uses Ethernet constants (such as MTU and address length) which are not good when it enslaves non Ethernet devices (such as InfiniBand). Signed-off-by: Moni Shoua --- drivers/net/bonding/bond_main.c |3 ++- drivers/net/bonding/bond_sysfs.c | 10 -- drivers/net

[PATCH V6 5/9] net/bonding: Enable IP multicast for bonding IPoIB devices

2007-09-24 Thread Moni Shoua
where for multicast joins taking place after the enslavement another ip_xxx_mc_map() is called (eg ip_ib_mc_map() when the bond type is ARPHRD_INFINIBAND) Signed-off-by: Moni Shoua Signed-off-by: Or Gerlitz --- drivers/net/bonding/bond_main.c |5 +++-- drivers/net/bonding/bond_sysfs.c |6

[PATCH V6 4/9] net/bonding: Enable bonding to enslave netdevices not supporting set_mac_address()

2007-09-24 Thread Moni Shoua
already done by the bonding code). Signed-off-by: Moni Shoua Signed-off-by: Or Gerlitz --- drivers/net/bonding/bond_main.c | 87 +++- drivers/net/bonding/bonding.h |1 2 files changed, 60 insertions(+), 28 deletions(-) Index: net-2.6/drivers/net

[PATCH V6 3/9] net/bonding: Enable bonding to enslave non ARPHRD_ETHER

2007-09-24 Thread Moni Shoua
) of the port this IPoIB device is bounded to. The QP is a resource created by the IB HW and the GID is an identifier burned into the HCA (i have omitted here some details which are not important for the bonding RFC). Signed-off-by: Moni Shoua Signed-off-by: Or Gerlitz --- drivers/net/bonding

[PATCH V6 2/9] IB/ipoib: Verify address handle validity on send

2007-09-24 Thread Moni Shoua
ipoib_neigh. This patch detects and prevents this from happenning. Signed-off-by: Moni Shoua Signed-off-by: Or Gerlitz --- drivers/infiniband/ulp/ipoib/ipoib_main.c |5 +++-- 1 files changed, 3 insertions(+), 2 deletions(-) Index: net-2.6/drivers/infiniband/ulp/ipoib/ipoib_main.c

[PATCH V6 1/9] IB/ipoib: Bound the net device to the ipoib_neigh structue

2007-09-24 Thread Moni Shoua
as the IFF_MASTER bit set. Signed-off-by: Moni Shoua Signed-off-by: Or Gerlitz --- drivers/infiniband/ulp/ipoib/ipoib.h |4 +++- drivers/infiniband/ulp/ipoib/ipoib_main.c | 24 +++- drivers/infiniband/ulp/ipoib/ipoib_multicast.c |3 ++- 3 files changed,

[PATCH V6 0/9] net/bonding: ADD IPoIB support for the bonding driver

2007-09-24 Thread Moni Shoua
This patch series is the sixth version (see below link to V5) of the suggested changes to the bonding driver so it would be able to support non ARPHRD_ETHER netdevices for its High-Availability (active-backup) mode. Patches 1-8 were originally submitted in V5 and patch 9 is an addition by Jay.

Re: [ofa-general] Re: [PATCH V5 2/11] IB/ipoib: Notify the world before doing unregister

2007-09-24 Thread Moni Shoua
Roland Dreier wrote: > > The action in bonding to a detach of slave is to unregister the master > (see patch 10). > > This can't be done from the context of unregister_netdevice itself (it is > protected by rtnl_lock). > > I'm confused. Your patch has: > > > + ipoib_slave_detach(cp

Re: [ofa-general] Re: [PATCH V5 2/11] IB/ipoib: Notify the world before doing unregister

2007-09-23 Thread Moni Shoua
Roland Dreier wrote: > > + ipoib_slave_detach(cpriv->dev); > >unregister_netdev(cpriv->dev); > > Maybe you already answered this before, but I'm still not clear why > this notifier call can't just be added to the start of > unregister_netdevice(), so we can avoid having driv

[PATCH V5 5/11] net/bonding: Enable bonding to enslave non ARPHRD_ETHER

2007-09-20 Thread Moni Shoua
) of the port this IPoIB device is bounded to. The QP is a resource created by the IB HW and the GID is an identifier burned into the HCA (i have omitted here some details which are not important for the bonding RFC). Signed-off-by: Moni Shoua Signed-off-by: Or Gerlitz --- drivers/net/bonding

[PATCH 11/11] bonding: Optionally allow ethernet slaves to keep own MAC

2007-09-20 Thread Moni Shoua
Update the "don't change MAC of slaves" functionality added in previous changes to be a generic option, rather than something tied to IB devices, as it's occasionally useful for regular ethernet devices as well. Adds "fail_over_mac" option (which is automatically enabled for IB slaves), ap

PATCH V5 9/11] net/bonding: Delay sending of gratuitous ARP to avoid failure

2007-09-20 Thread Moni Shoua
Delay sending a gratuitous_arp when LINK_STATE_LINKWATCH_PENDING bit in dev->state field is on. This improves the chances for the arp packet to be transmitted. Signed-off-by: Moni Shoua --- drivers/net/bonding/bond_main.c | 24 +--- drivers/net/bonding/bonding.h |

[PATCH V5 10/11] net/bonding: Destroy bonding master when last slave is gone

2007-09-20 Thread Moni Shoua
when slave_cnt is zero ensures that these functions be used anymore. Signed-off-by: Moni Shoua --- drivers/net/bonding/bond_main.c | 45 +++- drivers/net/bonding/bonding.h |3 ++ 2 files changed, 47 insertions(+), 1 deletion(-) Index: net-2.6/driver

[PATCH V5 8/11] net/bonding: Handlle wrong assumptions that slave is always an Ethernet device

2007-09-20 Thread Moni Shoua
bonding sometimes uses Ethernet constants (such as MTU and address length) which are not good when it enslaves non Ethernet devices (such as InfiniBand). Signed-off-by: Moni Shoua --- drivers/net/bonding/bond_main.c |3 ++- drivers/net/bonding/bond_sysfs.c | 19

[PATCH V5 7/11] net/bonding: Enable IP multicast for bonding IPoIB devices

2007-09-20 Thread Moni Shoua
where for multicast joins taking place after the enslavement another ip_xxx_mc_map() is called (eg ip_ib_mc_map() when the bond type is ARPHRD_INFINIBAND) Signed-off-by: Moni Shoua Signed-off-by: Or Gerlitz --- drivers/net/bonding/bond_main.c |5 +++-- drivers/net/bonding/bond_sysfs.c |6

[PATCH V5 6/11] net/bonding: Enable bonding to enslave netdevices not supporting set_mac_address()

2007-09-20 Thread Moni Shoua
already done by the bonding code). Signed-off-by: Moni Shoua Signed-off-by: Or Gerlitz --- drivers/net/bonding/bond_main.c | 87 +++- drivers/net/bonding/bonding.h |1 2 files changed, 60 insertions(+), 28 deletions(-) Index: net-2.6/drivers/net

[PATCH V5 5/11] net/bonding: Enable bonding to enslave non ARPHRD_ETHER

2007-09-20 Thread Moni Shoua
) of the port this IPoIB device is bounded to. The QP is a resource created by the IB HW and the GID is an identifier burned into the HCA (i have omitted here some details which are not important for the bonding RFC). Signed-off-by: Moni Shoua Signed-off-by: Or Gerlitz --- drivers/net/bonding

[PATCH V5 4/11] IB/ipoib: Verify address handle validity on send

2007-09-20 Thread Moni Shoua
ipoib_neigh. This patch detects and prevents this from happenning. Signed-off-by: Moni Shoua Signed-off-by: Or Gerlitz --- drivers/infiniband/ulp/ipoib/ipoib_main.c |5 +++-- 1 files changed, 3 insertions(+), 2 deletions(-) Index: net-2.6/drivers/infiniband/ulp/ipoib/ipoib_main.c

[PATCH V5 3/11] IB/ipoib: Bound the net device to the ipoib_neigh structue

2007-09-20 Thread Moni Shoua
as the IFF_MASTER bit set. Signed-off-by: Moni Shoua Signed-off-by: Or Gerlitz --- drivers/infiniband/ulp/ipoib/ipoib.h |4 +++- drivers/infiniband/ulp/ipoib/ipoib_main.c | 24 +++- drivers/infiniband/ulp/ipoib/ipoib_multicast.c |3 ++- 3 files changed,

[PATCH V5 1/11] net/core: add a netdev notification for slave detach

2007-09-20 Thread Moni Shoua
A slave of a bonding master that wants to send a notification before going down should call netdev_slave_detach(). The handling of this notification will be done outside the context of unregister_netdevice() which is sometimes necessary, as with IPoIB slave for example. Signed-off-by: Moni Shoua

[PATCH V5 2/11] IB/ipoib: Notify the world before doing unregister

2007-09-20 Thread Moni Shoua
master, let the master know that the IPoIB device is about to unregister (but before calling unregister). Signed-off-by: Moni Shoua --- drivers/infiniband/ulp/ipoib/ipoib.h |7 +++ drivers/infiniband/ulp/ipoib/ipoib_main.c |3 +++ drivers/infiniband/ulp/ipoib/ipoib_vlan.c |1

[PATCH V5 0/11] net/bonding: ADD IPoIB support for the bonding driver

2007-09-20 Thread Moni Shoua
This patch series is the fifth version (see below link to V4) of the suggested changes to the bonding driver so it would be able to support non ARPHRD_ETHER netdevices for its High-Availability (active-backup) mode. Patches 1-10 were originally submitted in V4 and patch 11 is an addition by Jay

Re: [ofa-general] Re: [PATCH 02/11] IB/ipoib: Notify the world before doing unregister

2007-09-19 Thread Moni Shoua
Roland, Jay, Thanks a lot for the comments. I'd like to summarize the points raised so far 1. Reduce the indentation in patch #4 (Roland) I will resend 2. Remove the "if (n->dev->flags & IFF_MASTER)" from patch #3 (Roland) I will resend 3. Consider making ipoib_slave_detach() net

Re: [ofa-general] [PATCH V4 0/10] net/bonding: ADD IPoIB support for the bonding driver

2007-09-10 Thread Moni Shoua
Hi all, This patch series is a bit neglected. Since our goal is to have bonding support for IPoIB in kernel 2.6.24 it is very important for us to get comments soon. We would appreciate if you take some time to look at this and help us push this code upstream. thanks MoniS - To unsubscribe

Re: [ofa-general] Re: [PATCH V4 10/10] net/bonding: Destroy bonding master when last slave is gone

2007-09-02 Thread Moni Shoua
Jay Vosburgh wrote: > Moni Shoua <[EMAIL PROTECTED]> wrote: > >> Jay Vosburgh wrote: >>> Moni Shoua <[EMAIL PROTECTED]> wrote: >>> >>>> When bonding enslaves non Ethernet devices it takes pointers to functions >>>> in the module

Re: [ofa-general] Re: [PATCH V4 10/10] net/bonding: Destroy bonding master when last slave is gone

2007-08-29 Thread Moni Shoua
Jay Vosburgh wrote: > Moni Shoua <[EMAIL PROTECTED]> wrote: > >> When bonding enslaves non Ethernet devices it takes pointers to functions >> in the module that owns the slaves. In this case it becomes unsafe >> to keep the bonding master registered after last slave

Re: [ofa-general] Re: [PATCH V4 8/10] net/bonding: Handlle wrong assumptions that slave is always an Ethernet device

2007-08-29 Thread Moni Shoua
Jay Vosburgh wrote: > Moni Shoua <[EMAIL PROTECTED]> wrote: > >> bonding sometimes uses Ethernet constants (such as MTU and address length) >> which >> are not good when it enslaves non Ethernet devices (such as InfiniBand). >> >> Signed-off-by: Moni S

[PATCH V4 10/10] net/bonding: Destroy bonding master when last slave is gone

2007-08-20 Thread Moni Shoua
when slave_cnt is zero ensures that these functions be used anymore. Signed-off-by: Moni Shoua <[EMAIL PROTECTED]> --- drivers/net/bonding/bond_main.c | 45 +++- drivers/net/bonding/bonding.h |3 ++ 2 files changed, 47 insertions(+), 1 deletion(-)

PATCH V4 9/10] net/bonding: Delay sending of gratuitous ARP to avoid failure

2007-08-20 Thread Moni Shoua
Delay sending a gratuitous_arp when LINK_STATE_LINKWATCH_PENDING bit in dev->state field is on. This improves the chances for the arp packet to be transmitted. Signed-off-by: Moni Shoua <[EMAIL PROTECTED]> --- drivers/net/bonding/bond_main.c | 24 +--- drivers/ne

[PATCH V4 7/10] net/bonding: Enable IP multicast for bonding IPoIB devices

2007-08-20 Thread Moni Shoua
where for multicast joins taking place after the enslavement another ip_xxx_mc_map() is called (eg ip_ib_mc_map() when the bond type is ARPHRD_INFINIBAND) Signed-off-by: Moni Shoua <[EMAIL PROTECTED]> Signed-off-by: Or Gerlitz <[EMAIL PROTECTED]> --- drivers/net/bonding/bond_ma

[PATCH V4 8/10] net/bonding: Handlle wrong assumptions that slave is always an Ethernet device

2007-08-20 Thread Moni Shoua
bonding sometimes uses Ethernet constants (such as MTU and address length) which are not good when it enslaves non Ethernet devices (such as InfiniBand). Signed-off-by: Moni Shoua <[EMAIL PROTECTED]> --- drivers/net/bonding/bond_main.c |3 ++- drivers/net/bonding/bond_sysfs.c

[PATCH V4 6/10] net/bonding: Enable bonding to enslave netdevices not supporting set_mac_address()

2007-08-20 Thread Moni Shoua
already done by the bonding code). Signed-off-by: Moni Shoua <[EMAIL PROTECTED]> Signed-off-by: Or Gerlitz <[EMAIL PROTECTED]> --- drivers/net/bonding/bond_main.c | 87 +++- drivers/net/bonding/bonding.h |1 2 files changed, 60 inser

[PATCH V4 5/10] net/bonding: Enable bonding to enslave non ARPHRD_ETHER

2007-08-20 Thread Moni Shoua
) of the port this IPoIB device is bounded to. The QP is a resource created by the IB HW and the GID is an identifier burned into the HCA (i have omitted here some details which are not important for the bonding RFC). Signed-off-by: Moni Shoua <[EMAIL PROTECTED]> Signed-off-by: Or Gerlitz &

[PATCH V4 3/10] IB/ipoib: Bound the net device to the ipoib_neigh structue

2007-08-20 Thread Moni Shoua
as the IFF_MASTER bit set. Signed-off-by: Moni Shoua <[EMAIL PROTECTED]> Signed-off-by: Or Gerlitz <[EMAIL PROTECTED]> --- drivers/infiniband/ulp/ipoib/ipoib.h |4 +++- drivers/infiniband/ulp/ipoib/ipoib_main.c | 17 +++-- drivers/infiniband/ulp/ipoi

[PATCH V4 4/10] IB/ipoib: Verify address handle validity on send

2007-08-20 Thread Moni Shoua
ipoib_neigh. This patch detects and prevents this from happenning. Signed-off-by: Moni Shoua <[EMAIL PROTECTED]> Signed-off-by: Or Gerlitz <[EMAIL PROTECTED]> --- drivers/infiniband/ulp/ipoib/ipoib_main.c |5 +++-- 1 files changed, 3 insertions(+), 2 deletions(-) Index: net-2.6/driver

[PATCH V4 2/10] IB/ipoib: Notify the world before doing unregister

2007-08-20 Thread Moni Shoua
master, let the master know that the IPoIB device is about to unregister (but before calling unregister). Signed-off-by: Moni Shoua <[EMAIL PROTECTED]> --- drivers/infiniband/ulp/ipoib/ipoib_main.c | 15 +++ 1 files changed, 15 insertions(+) Index: net-2.6/drivers/infiniba

[PATCH V4 1/10] IB/ipoib: Export call to call_netdevice_notifiers and add new private flag

2007-08-20 Thread Moni Shoua
wish to inform the bonding master about coming detachment. Signed-off-by: Moni Shoua <[EMAIL PROTECTED]> --- include/linux/if.h |1 + net/core/dev.c |1 + 2 files changed, 2 insertions(+) Index: net-2.6/net/core

[PATCH V4 0/10] net/bonding: ADD IPoIB support for the bonding driver

2007-08-20 Thread Moni Shoua
This patch series is the fourth version (see below link to V3) of the suggested changes to the bonding driver so it would be able to support non ARPHRD_ETHER netdevices for its High-Availability (active-backup) mode. The motivation is to enable the bonding driver on its HA mode to work with th

Re: [ofa-general] Re: Re: Re: [PATCH V3 0/7] net/bonding: ADD IPoIB support for the bonding driver

2007-08-01 Thread Moni Shoua
> It's always wrong to copy symbols from another module without > referencing it. Michael, It seems like the preferred approach is to prevent ib_ipoib from being unloaded while bonding is on top it, right? It seems like it would handle all safety issues (not just neigh cleanup). - To unsubscri

Re: [ofa-general] Re: [PATCH V3 7/7] net/bonding: Delay sending of gratuitous ARP to avoid failure

2007-07-31 Thread Moni Shoua
Jay Vosburgh wrote: > Moni Shoua <[EMAIL PROTECTED]> wrote: > >> Delay sending a gratuitous_arp when LINK_STATE_LINKWATCH_PENDING bit >> in dev->state field is on. This improves the chances for the arp packet to >> be transmitted. > > Under what circ

Re: [ofa-general] Re: [PATCH V3 0/7] net/bonding: ADD IPoIB support for the bonding driver

2007-07-31 Thread Moni Shoua
Roland Dreier wrote: > > 1. When bonding enslaves an IPoIB device the bonding neighbor holds a > > reference to a cleanup function in the IPoIB drives. This makes it unsafe > to > > unload the IPoIB module if there are bonding neighbors in the air. So, to > > avoid this race one must unload

[PATCH V3 7/7] net/bonding: Delay sending of gratuitous ARP to avoid failure

2007-07-30 Thread Moni Shoua
Delay sending a gratuitous_arp when LINK_STATE_LINKWATCH_PENDING bit in dev->state field is on. This improves the chances for the arp packet to be transmitted. Signed-off-by: Moni Shoua <[EMAIL PROTECTED]> --- drivers/net/bonding/bond_main.c | 25 + drivers/ne

[PATCH V3 6/7] net/bonding: Handlle wrong assumptions that slave is always an Ethernet device

2007-07-30 Thread Moni Shoua
bonding sometimes uses Ethernet constants (such as MTU and address length) which are not good when it enslaves non Ethernet devices (such as InfiniBand). Signed-off-by: Moni Shoua <[EMAIL PROTECTED]> --- drivers/net/bonding/bond_main.c |2 +- drivers/net/bonding/bond_sysfs.c

[PATCH V3 0/7] net/bonding: ADD IPoIB support for the bonding driver

2007-07-30 Thread Moni Shoua
This patch series is the third version (see below link to V2) of the suggested changes to the bonding driver so it would be able to support non ARPHRD_ETHER netdevices for its High-Availability (active-backup) mode. The motivation is to enable the bonding driver on its HA mode to work with the

[PATCH V3 3/7] net/bonding: Enable bonding to enslave non ARPHRD_ETHER

2007-07-30 Thread Moni Shoua
) of the port this IPoIB device is bounded to. The QP is a resource created by the IB HW and the GID is an identifier burned into the HCA (i have omitted here some details which are not important for the bonding RFC). Signed-off-by: Moni Shoua <[EMAIL PROTECTED]> Signed-off-by: Or Gerlitz &

[PATCH V3 4/7] net/bonding: Enable bonding to enslave netdevices not supporting set_mac_address()

2007-07-30 Thread Moni Shoua
already done by the bonding code). Signed-off-by: Moni Shoua <[EMAIL PROTECTED]> Signed-off-by: Or Gerlitz <[EMAIL PROTECTED]> --- drivers/net/bonding/bond_main.c | 88 +++- drivers/net/bonding/bonding.h |1 2 files changed, 61 inser

[PATCH V3 5/7] net/bonding: Enable IP multicast for bonding IPoIB devices

2007-07-30 Thread Moni Shoua
where for multicast joins taking place after the enslavement another ip_xxx_mc_map() is called (eg ip_ib_mc_map() when the bond type is ARPHRD_INFINIBAND) Signed-off-by: Moni Shoua <[EMAIL PROTECTED]> Signed-off-by: Or Gerlitz <[EMAIL PROTECTED]> --- drivers/net/bonding/bond_ma

[PATCH V3 2/7] IB/ipoib: Verify address handle validity on send

2007-07-30 Thread Moni Shoua
ipoib_neigh. This patch detects and prevents this from happenning. Signed-off-by: Moni Shoua <[EMAIL PROTECTED]> Signed-off-by: Or Gerlitz <[EMAIL PROTECTED]> --- drivers/infiniband/ulp/ipoib/ipoib_main.c |5 +++-- 1 files changed, 3 insertions(+), 2 deletions(-) Index: net-2.6/driver

[PATCH V3 1/7] IB/ipoib: Bound the net device to the ipoib_neigh structue

2007-07-30 Thread Moni Shoua
as the IFF_MASTER bit set. Signed-off-by: Moni Shoua <[EMAIL PROTECTED]> Signed-off-by: Or Gerlitz <[EMAIL PROTECTED]> --- drivers/infiniband/ulp/ipoib/ipoib.h |4 +++- drivers/infiniband/ulp/ipoib/ipoib_main.c | 17 +++-- drivers/infiniband/ulp/ipoib/ipo