Please don't reply to lustre-devel. Instead, comment in Bugzilla by using the 
following link:
https://bugzilla.lustre.org/show_bug.cgi?id=10902



(In reply to comment #106)
> > 1. Oleg, your optimization above does not look very helpful.
> > Once a lock with a compatible mode occurs, we skip all the
> > locks of this mode by mode skip list -- locks are grouped by
> > mode. As we do not walk through locks with compatible modes,
> > your optimization works for the very first lock in every compatible
> > mode group only. I do not think we need an optimization that
> > works for 1 lock per mode group of locks.
> well, i didn't change it in this patch, need Oleg's feedback of your opinion.

This is in fact quite right suggestion. I somehow thought that we iterate
through every group, but since we skips all compatible mode locks, this makes no
sense to adopt of course.

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

Reply via email to