On 6/4/2013 3:01 PM, Steve Wise wrote:
On 6/4/2013 2:46 PM, Hefty, Sean wrote:
+
#ifdef HAVE_IBV_REGISTER_DRIVER
static __attribute__((constructor)) void mlx4_register_driver(void)
{
- ibv_register_driver("mlx4", mlx4_driver_init);
+ verbs_register_driver("mlx4", mlx4_driver_init);
+
}
#else
Shouldn't ibv_register_driver() need to be called in the lib
constructor
function if HAVE_IBV_REGISTER_DRIVER is not defined?
? If HAVE_IBV_REGISTER_DRIVER is not defined, then we can't call
ibv_register_driver...
I thought HAVE_IBV_REGISTER_DRIVER was something new for deciding if
the lib should call verbs_register_driver().
We should just remove the HAVE_IBV_... check completely, since with
this change, libmlx4 requires an updated version of libibverbs.
Ah. I was thinking it would use the old interface if it was compiled
against a libibverbs that didn't support the extensions.
So old provider libs will work with the new libibverbs but new provider
libs will not work with the old libibverbs? Is there no way around
this? That dependency can be painful.
--
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
