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