On Sun, Sep 8, 2013 at 2:44 PM, Doug Ledford <[email protected]> wrote:
>
> -                                       ret = find_gid_port(cma_dev->device, 
> &iboe_gid, port);
> +                                       ret = 
> ib_find_cached_gid(cma_dev->device, &iboe_gid, &found_port, NULL);

This type of change is kind of unfortunate -- I've been on a
multi-year quest to get rid of the whole "ib...cached..." set of APIs,
since the part of the code that handles maintaining that cache is racy
and hard to get the locking right with.  It would be better if we
could declare that the device driver's query GID methods had to be
fast (non-sleeping) and use them directly (and device drivers could
maintain a cache if they need to, maybe with a library to help them).

But I guess that's a bigger project than fixing this immediate issue.
--
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