Re: [rds-devel] net-next pull request: RDS

2010-09-15 Thread Or Gerlitz
Andrew Grover wrote: > Once net-next gets pushed to mainline and Roland pulls from that, > then we'll be in a good position to put these helpers where they should go, > and change other ULPs to use them. Andy, as Roland commented, you can push such helpers through Dave once Roland made a review

Re: [rds-devel] net-next pull request: RDS

2010-09-14 Thread Roland Dreier
> Since rds goes upstream via netdev not linux-rdma, it's a little > tricky.. any core ib helpers we added would not be in net-next so rds > wouldn't build. :) Once net-next gets pushed to mainline and Roland > pulls from that, then we'll be in a good position to put these helpers > where they

Re: [rds-devel] net-next pull request: RDS

2010-09-14 Thread Andrew Grover
On Tue, Sep 14, 2010 at 11:35 AM, Zach Brown wrote: >> Zach Brown's "RDS/IB: print IB event strings as well as their number" - >> commit >> 1bde04a63d532c2540d6fdee0a661530a62b1686 in net-next-2.6 looks perfect to >> reside as a helper function in the core IB stack which can be in use by other >>

Re: [rds-devel] net-next pull request: RDS

2010-09-14 Thread Zach Brown
> Zach Brown's "RDS/IB: print IB event strings as well as their number" - commit > 1bde04a63d532c2540d6fdee0a661530a62b1686 in net-next-2.6 looks perfect to > reside as a helper function in the core IB stack which can be in use by other > rdma drivers (e.g ipoib, iser, srp, etc). Yeah, if those o

Re: [rds-devel] net-next pull request: RDS

2010-09-13 Thread Chris Mason
On Sun, Sep 12, 2010 at 10:49:38AM +0200, Or Gerlitz wrote: > Hi Andy, > > Some clarifications/questions from whatever quick look one can have over 107 > patches... > > Zach Brown's "RDS/IB: print IB event strings as well as their number" - commit > 1bde04a63d532c2540d6fdee0a661530a62b1686 in n