Krishna> 1. In ib_umem_get(), I see you set ret = 0, which is
Krishna> unnecessary because chunk-> nents is set based on ret
Krishna> value. Plus you already have a "while (ret)" to break
Krishna> out. "ret = 0" can be safely removed.
Actually, I know why "ret = 0" is there: although it's not strictly
needed, gcc isn't smart enough to see that, and without the
initialization, it warns that "'ret' might be used uninitialized in
this function."
- R.
_______________________________________________
openib-general mailing list
[email protected]
http://openib.org/mailman/listinfo/openib-general
To unsubscribe, please visit http://openib.org/mailman/listinfo/openib-general
