On 7/11/2018 12:31 PM, Jason Gunthorpe wrote:
It badly violates all accepted limits for patch size and patch series
length. I don't see what you think anyone else is going to do with
this.
Oh I completely agree. Unfortunately I didn't see a better way to do this.
Kaike did a lot of work to try and break things up into logical chunks. Any
suggestions on how to make it more digestable, we are open to it?
You must split it, I'm not sure why this is a problem?
Oh we can certainly do that. It's still the same amount of code to
review in the end, but if that makes it more consumable, let me see what
we can do.
I would have liked the patches to come out over time and be made more
incrementally, but because of the complexity that wasn't really possible.
The obvious split is 5 series in something like
responder RDMA WRITE support
initiator RDMA WRITE support
responder RDMA READ support
initiator RDMA READ support
enablement and negotiation
Seems reasonable to me.
-Denny
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
