Please don't reply to lustre-devel. Instead, comment in Bugzilla by using the
following link:
https://bugzilla.lustre.org/show_bug.cgi?id=11301
What |Removed |Added
----------------------------------------------------------------------------
Attachment #9391|review?([EMAIL PROTECTED])|review+
Flag| |
(From update of attachment 9391)
>+ if (ast_count > 0)
>+ ldlm_send_and_maybe_create_set(&arg, 0);
>+ else
>+ /* In case when number of ASTs is multiply of
>+ * PARALLEL_AST_LIMIT or @rpc_list was initially empty,
>+ * @arg.set must be destroyed here, otherwise we get
>+ * write memory leaking. */
>+ ptlrpc_set_destroy(arg.set);
I was thinking along the line that you can just drop the if in front of
ldlm_send_and_maybe_create_set() call,
but this will also do, of course.
About sanityN.sh, test 29, if it executes that ast error path - then yes, we
can use it.
_______________________________________________
Lustre-devel mailing list
[email protected]
https://mail.clusterfs.com/mailman/listinfo/lustre-devel
