linux.git/drivers/infiniband/ulp, branch v2.6.15 Linux kernel source tree IPoIB: fix error handling in ipoib_open 2005-11-29T18:55:58+00:00 Roland Dreier rolandd@cisco.com 2005-11-29T18:55:58+00:00 267ee88ed34c76dc527eeb3d95f9f9558ac99973 If ipoib_ib_dev_up() fails after ipoib_ib_dev_open() is called, then ipoib_ib_dev_stop() needs to be called to clean up. Signed-off-by: Roland Dreier <rolandd@cisco.com> 
If ipoib_ib_dev_up() fails after ipoib_ib_dev_open() is called, then
ipoib_ib_dev_stop() needs to be called to clean up.

Signed-off-by: Roland Dreier <rolandd@cisco.com>
IPoIB: protect child list in ipoib_ib_dev_flush 2005-11-29T18:53:30+00:00 Michael S. Tsirkin mst@mellanox.co.il 2005-11-29T18:53:30+00:00 4f71055a45a503273c039d80db8ba9b13cb17549 race condition: ipoib_ib_dev_flush is accessing child list without locks. Signed-off-by: Michael S. Tsirkin <mst@mellanox.co.il> Signed-off-by: Roland Dreier <rolandd@cisco.com> 
race condition: ipoib_ib_dev_flush is accessing child list without locks.

Signed-off-by: Michael S. Tsirkin <mst@mellanox.co.il>
Signed-off-by: Roland Dreier <rolandd@cisco.com>
IPoIB: don't zero members after we allocate with kzalloc 2005-11-29T18:25:23+00:00 Roland Dreier rolandd@cisco.com 2005-11-29T18:25:23+00:00 2e86541ec878de9ec5771600a77f451a80bebfc4 ipoib_mcast_alloc() uses kzalloc(), so there's no need to zero out members of the mcast struct after it's allocated. Signed-off-by: Roland Dreier <rolandd@cisco.com> 
ipoib_mcast_alloc() uses kzalloc(), so there's no need to zero out
members of the mcast struct after it's allocated.

Signed-off-by: Roland Dreier <rolandd@cisco.com>
IPoIB: reinitialize mcast structs' completions for every query 2005-11-29T18:18:45+00:00 Michael S. Tsirkin mst@mellanox.co.il 2005-11-29T18:18:45+00:00 de922487890936470660e89f9095aee980637989 Make sure mcast->done is initialized to uncompleted value before we submit a new query, so that it's safe to wait on. Signed-off-by: Michael S. Tsirkin <mst@mellanox.co.il> Signed-off-by: Roland Dreier <rolandd@cisco.com> 
Make sure mcast->done is initialized to uncompleted value before we
submit a new query, so that it's safe to wait on.

Signed-off-by: Michael S. Tsirkin <mst@mellanox.co.il>
Signed-off-by: Roland Dreier <rolandd@cisco.com>
IPoIB: always set path->query to NULL when query finishes 2005-11-29T18:13:54+00:00 Roland Dreier rolandd@cisco.com 2005-11-29T18:13:54+00:00 5872a9fc28e6cd3a4e51479a50970d19a01573b3 Always set path->query to NULL when the SA path record query completes, rather than only when we don't have an address handle. Signed-off-by: Roland Dreier <rolandd@cisco.com> 
Always set path->query to NULL when the SA path record query
completes, rather than only when we don't have an address handle.

Signed-off-by: Roland Dreier <rolandd@cisco.com>
IPoIB: reinitialize path struct's completion for every query 2005-11-29T05:20:34+00:00 Roland Dreier rolandd@cisco.com 2005-11-29T05:20:34+00:00 65c7eddaba33995e013ef3c04718f6dc8fdf2335 It's possible that IPoIB will issue multiple SA queries for the same path struct. Therefore the struct's completion needs to be initialized for each query rather than only once when the struct is allocated, or else we might not wait long enough for later queries to finish and free the path struct too soon. Signed-off-by: Roland Dreier <rolandd@cisco.com> 
It's possible that IPoIB will issue multiple SA queries for the same
path struct.  Therefore the struct's completion needs to be
initialized for each query rather than only once when the struct is
allocated, or else we might not wait long enough for later queries to
finish and free the path struct too soon.

Signed-off-by: Roland Dreier <rolandd@cisco.com>
[IB] srp: don't post receive if no send buf available 2005-11-15T08:19:21+00:00 Roland Dreier rolandd@cisco.com 2005-11-15T08:19:21+00:00 47f2bce9021b4974ed33b072ebb8348c8145c946 Have __srp_get_tx_iu() fail if the target port's request limit will not allow the initiator to post a send. This avoids continuing on and posting a receive, and then failing to post a corresponding send. If that happens, then the initiator will end up with an extra receive posted, and if this happens to much, the receive queue will overflow. Signed-off-by: Roland Dreier <rolandd@cisco.com> 
Have __srp_get_tx_iu() fail if the target port's request limit will
not allow the initiator to post a send.  This avoids continuing on and
posting a receive, and then failing to post a corresponding send.  If
that happens, then the initiator will end up with an extra receive
posted, and if this happens to much, the receive queue will overflow.

Signed-off-by: Roland Dreier <rolandd@cisco.com>
[IB] srp: increase max_luns 2005-11-11T22:06:01+00:00 Roland Dreier rolandd@cisco.com 2005-11-11T22:06:01+00:00 5f068992a1bccda5574b4f6d33458ef806686d7f Increase SRP max_luns to 512 to match the kernel's default, since SRP storage targets can have lots of LUNs and the SRP initiator itself doesn't have any particular limit. Signed-off-by: Roland Dreier <rolandd@cisco.com> 
Increase SRP max_luns to 512 to match the kernel's default, since SRP
storage targets can have lots of LUNs and the SRP initiator itself
doesn't have any particular limit.

Signed-off-by: Roland Dreier <rolandd@cisco.com>
Merge branch 'for-linus' of master.kernel.org:/pub/scm/linux/kernel/git/roland/infiniband 2005-11-10T21:27:06+00:00 Linus Torvalds torvalds@g5.osdl.org 2005-11-10T21:27:06+00:00 78b9c0f91cf908616b8f9f356e1d1220e727ea88 






[IPoIB] no need to set skb->dev right before freeing skb
2005-11-10T18:22:50+00:00

Roland Dreier
rolandd@cisco.com

2005-11-07T18:49:38+00:00

8c608a32e3cd7ff14498ad996ca32d1452245a97

For cut-and-paste reasons, the IPoIB driver was setting skb->dev right
before calling dev_kfree_skb_any().  Get rid of this.

Signed-off-by: Roland Dreier <rolandd@cisco.com>





For cut-and-paste reasons, the IPoIB driver was setting skb->dev right
before calling dev_kfree_skb_any().  Get rid of this.

Signed-off-by: Roland Dreier <rolandd@cisco.com>