summary |
shortlog |
log |
commit | commitdiff |
tree
raw |
patch |
inline | side by side (from parent 1:
4f35d4c)
then returning with flags=0, instead of flags=LDLM_FL_BLOCK_CONV. Thus, the client
believed that the conversion had succeeded immediately, marked the lock accordingly,
and continued on its merry way, even if it didn't actually have the lock.
if (lock->l_completion_ast)
lock->l_completion_ast(lock, 0);
}
if (lock->l_completion_ast)
lock->l_completion_ast(lock, 0);
}
+ } else {
+ /* FIXME: We should try the conversion right away and possibly
+ * return success without the need for an extra AST */
list_add_tail(&lock->l_res_link, &res->lr_converting);
list_add_tail(&lock->l_res_link, &res->lr_converting);
+ *flags |= LDLM_FL_BLOCK_CONV;
+ }