On Thu, 30 Jul 2015, Marciniszyn, Mike wrote:

> > That is obvious and useless. Patches should have a meaningful description
> > and justify the changes.
> >
>
> The driver uses the CNP opcode for congestion control.

And that requires a new transport protocol???

> > Why do you add the CNP opcode and what in the world does it do? CNP is
> > what? And why do the other enum values not work for you?
>
> The driver supports congestion control in software vs. outboard
> firmware, so the opcode should be available in the appropriate kernel
> include file.

So is CNP an operation or a protocol?

--
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