On Tue, 2011-11-08 at 08:21 +0100, Bart Van Assche wrote:
> On Mon, Nov 7, 2011 at 9:46 PM, Nicholas A. Bellinger
> <[email protected]> wrote:
> > On Mon, 2011-11-07 at 20:00 +0100, Bart Van Assche wrote:
> >> On Mon, Nov 7, 2011 at 7:10 PM, Roland Dreier <[email protected]> 
> >> wrote:
> >> > On Sun, Nov 6, 2011 at 9:31 AM, Bart Van Assche <[email protected]> 
> >> > wrote:
> >> >> Make sure that the last WQE event is handled properly if it arrives
> >> >> before the QP has been transitioned into the error state.
> >
> > This patch fails to compile with the following:
> >
> > drivers/infiniband/ulp/srpt/ib_srpt.c: In function ‘srpt_perform_rdmas’:
> > drivers/infiniband/ulp/srpt/ib_srpt.c:2936: error: ‘CH_RELEASING’ 
> > undeclared (first use in this function)
> > drivers/infiniband/ulp/srpt/ib_srpt.c:2936: error: (Each undeclared 
> > identifier is reported only once
> > drivers/infiniband/ulp/srpt/ib_srpt.c:2936: error: for each function it 
> > appears in.)
> >
> > I'm assuming this needs to be changed to CH_DRAINING instead, yes..?
> 
> Yes. The tests I ran were with that change included - I'm not sure how
> it's possible that that change was not included in the patch I posted.
> 

Thanks for verifying this one.  Pushed into lio-core and will squash
into a single patch for target-pending.

Thanks Bart!

--nab


--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to