[ofa-general] Re: [PATCH V2] mlx4: check for FW version which properly supports resize_cq

Jack Morgenstein jackm at dev.mellanox.co.il
Wed Jul 15 22:38:44 PDT 2009


On Wednesday 15 July 2009 01:33, Roland Dreier wrote:
> It occurs to me that one change that makes sense and would help make
> this fix cleaner is the following -- since after all if a command # is
> out of range, that's really a different error than if a low-level driver
> just doesn't implement a certain method.
> 
This is exactly the patch I posted on Dec 2, 2008:
   http://lists.openfabrics.org/pipermail/general/2008-December/055734.html

However, see my comments accompanying the patch.  This is a global change for
all verbs -- and I'm not sure that it won't break some current applications.

MPI is OK with this change, however (in fact, MPI needs this change -- see the next
post in the openfabrics list thread above).

I suggest that you post to ewg, indicating the change (-ENOSYS for unimplemented
commands instead of -EINVAL) and see if anyone responds.  If it is OK, then you should
just do it.

-Jack



More information about the general mailing list