> -                       INIT_HLIST_NODE(&fmr->cache_node);

 > +                       if (params->cache) {
 > +                               INIT_HLIST_NODE(&fmr->cache_node);
 > +                       }

I guess we could do that but in practice it doesn't make any
difference, does it?  Adding the if statement makes the code bigger
(and it's not a fast path anyway).

(BTW the kernel style is to leave out the { } around a single line block)

 - R.
_______________________________________________
general mailing list
[email protected]
http://lists.openfabrics.org/cgi-bin/mailman/listinfo/general

To unsubscribe, please visit http://openib.org/mailman/listinfo/openib-general

Reply via email to