On 05/14/2012 02:29 PM, levin li wrote: > +static inline int bsearch_in_entries(struct sd_vnode *entries, > + int nr_entries, uint64_t id) Maybe get_vnode_pos() is a better name to hide the low level implementation and more descriptive of what we gets from the function. Thanks, Yuan