Re: [openib-general] please pull for 2.6.21: fix + add IB multicast support

2007-02-16 Thread Roland Dreier
OK, I pulled this in to my for-2.6.21 branch and I will ask Linus to pull later today. Thanks. - R. ___ openib-general mailing list openib-general@openib.org http://openib.org/mailman/listinfo/openib-general To unsubscribe, please visit

Re: [openib-general] please pull for 2.6.21: fix + add IB multicast support

2007-02-16 Thread Sean Hefty
Roland Dreier wrote: OK, I pulled this in to my for-2.6.21 branch and I will ask Linus to pull later today. Thanks for the review. ___ openib-general mailing list openib-general@openib.org http://openib.org/mailman/listinfo/openib-general To

Re: [openib-general] please pull for 2.6.21: fix + add IB multicast support

2007-02-14 Thread Or Gerlitz
Or Gerlitz wrote: Roland Dreier wrote: I merged the increment port number and remove redundant '_wq' patches from git.openfabrics.org/~shefty/scm/rdma-dev.git for-roland I plan to review to multicast stuff next week and I hope to merge it for 2.6.21. Or, have you or anyone else at Voltaire

Re: [openib-general] please pull for 2.6.21: fix + add IB multicast support

2007-02-11 Thread Or Gerlitz
Roland Dreier wrote: I merged the increment port number and remove redundant '_wq' patches from git.openfabrics.org/~shefty/scm/rdma-dev.git for-roland I plan to review to multicast stuff next week and I hope to merge it for 2.6.21. Or, have you or anyone else at Voltaire read over the

Re: [openib-general] please pull for 2.6.21: fix + add IB multicast support

2007-02-10 Thread Michael S. Tsirkin
+ It seems same goes for + mc = kzalloc(sizeof(*mc), GFP_KERNEL); + if (!mc) + return NULL; We would need to set events_reported. IMO, probably worth it to init just this one field rather than use up initialized memory - and I think it's clearer. --

Re: [openib-general] please pull for 2.6.21: fix + add IB multicast support

2007-02-10 Thread Roland Dreier
IMO, probably worth it to init just this one field rather than use up initialized memory - and I think it's clearer. What do you mean by using up initialized memory? kzalloc() just does a memset(0), and it's not like there's a limit on the number of times we're allowed to call memset(). -

Re: [openib-general] please pull for 2.6.21: fix + add IB multicast support

2007-02-09 Thread Michael S. Tsirkin
Quoting Roland Dreier [EMAIL PROTECTED]: Subject: Re: please pull for 2.6.21: fix + add IB multicast support I merged the increment port number and remove redundant '_wq' patches from git.openfabrics.org/~shefty/scm/rdma-dev.git for-roland I plan to review to multicast stuff next week and

Re: [openib-general] please pull for 2.6.21: fix + add IB multicast support

2007-02-09 Thread Or Gerlitz
On 2/9/07, Roland Dreier [EMAIL PROTECTED] wrote: I plan to review to multicast stuff next week and I hope to merge it for 2.6.21 thanks, good news! Or, have you or anyone else at Voltaire read over the code in addition to using it? Do you see anything that should be cleaned up? OK, I

Re: [openib-general] please pull for 2.6.21: fix + add IB multicast support

2007-02-09 Thread Michael S. Tsirkin
Or, have you or anyone else at Voltaire read over the code in addition to using it? Do you see anything that should be cleaned up? OK, I most the the review i did (and interaction with Sean to add changes) was on the rdma_cm: add multicast communication support patch, and i was less

Re: [openib-general] please pull for 2.6.21: fix + add IB multicast support

2007-02-09 Thread Sean Hefty
+ member = kzalloc(sizeof *member, gfp_mask); + if (!member) + return ERR_PTR(-ENOMEM); This appears okay to replace with kmalloc. + group = kzalloc(sizeof *group, gfp_mask); + if (!group) + return NULL; + We would need additional

Re: [openib-general] please pull for 2.6.21: fix + add IB multicast support

2007-02-08 Thread Or Gerlitz
Or Gerlitz wrote: Sean Hefty wrote: Sean Hefty (3): rdma_cm: Increment port number after close to avoid re-use. ib_sa: track multicast join/leave requests rdma_cm: add multicast communication support Assuming that you haven't look at this yet, I updated the ib_sa patch

Re: [openib-general] please pull for 2.6.21: fix + add IB multicast support

2007-02-08 Thread Roland Dreier
I merged the increment port number and remove redundant '_wq' patches from git.openfabrics.org/~shefty/scm/rdma-dev.git for-roland I plan to review to multicast stuff next week and I hope to merge it for 2.6.21. Or, have you or anyone else at Voltaire read over the code in addition to using it?

Re: [openib-general] please pull for 2.6.21: fix + add IB multicast support

2007-02-06 Thread Or Gerlitz
Sean Hefty wrote: Sean Hefty (3): rdma_cm: Increment port number after close to avoid re-use. ib_sa: track multicast join/leave requests rdma_cm: add multicast communication support Assuming that you haven't look at this yet, I updated the ib_sa patch above to shorten

Re: [openib-general] please pull for 2.6.21: fix + add IB multicast support

2007-02-06 Thread Roland Dreier
Can you comment on the multicast changes merge for 2.6.21 status? Where are the final patches that you want to merge? - R. ___ openib-general mailing list openib-general@openib.org http://openib.org/mailman/listinfo/openib-general To unsubscribe,

Re: [openib-general] please pull for 2.6.21: fix + add IB multicast support

2007-02-06 Thread Sean Hefty
Can you comment on the multicast changes merge for 2.6.21 status? Where are the final patches that you want to merge? Try the for-roland branch at git.openfabrics.org/~shefty/scm/rdma-dev.git. If this doesn't work, or you hit any snags, let me know, and I'll try to correct any issues so that

Re: [openib-general] please pull for 2.6.21: fix + add IB multicast support

2007-02-02 Thread Sean Hefty
Sean Hefty (3): rdma_cm: Increment port number after close to avoid re-use. ib_sa: track multicast join/leave requests rdma_cm: add multicast communication support Assuming that you haven't look at this yet, I updated the ib_sa patch above to shorten the workqueue name,

[openib-general] please pull for 2.6.21: fix + add IB multicast support

2007-01-30 Thread Sean Hefty
Roland, I've created a 'for-roland' branch off of my git tree: git://git.openfabrics.org/~shefty/rdma-dev.git with the following changes: Sean Hefty (3): rdma_cm: Increment port number after close to avoid re-use. ib_sa: track multicast join/leave requests rdma_cm: add