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 #9338|review?([EMAIL PROTECTED])|review+
               Flag|                            |


(From update of attachment 9338)
Looks good.
You need a special test developed to excercise error in ldlm_cb_interpret to
make sure clients are actually being evicted as expected now.

Now about limiting amount of ASTs, I think the change is simple enough not to
require anmother round of DLD/CODE.
Basically what you need is to look at number of ASTs added to set in
ldlm_run_*_ast_work and once it grows to certain value, you just do
set_wait/set_destroy and then create the set again (you need to inherit
"restart" value over this.).
May be some code reusing with those two functions won't' hurt as well.

_______________________________________________
Lustre-devel mailing list
[email protected]
https://mail.clusterfs.com/mailman/listinfo/lustre-devel

Reply via email to