<br><br><div class="gmail_quote">On Fri, Aug 21, 2009 at 10:48 AM, Jeff Layton <span dir="ltr"><<a href="mailto:jlayton@redhat.com">jlayton@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="im">
<br>
</div>As long as you can do so in a non-racy way, then I'm not opposed to<br>
that long-term. The problem though is that I don't have a lot of<br>
confidence in the open file tracking code. It's extremely hard to<br>
follow and definitely has races. I don't think it's really possible to<br>
do what you suggest safely until the open file tracking code has been<br>
fixed.<br>
<br>
For now, I'm pretty sure this set should fix the problems that users<br>
are hitting in the oplock codepath today. I'd like to fix that first<br>
before we embark on a redesign of it.<br>
</blockquote></div><br>You may be right that this should be two stages, but your 2nd and 3rd patch<br>are already large, so I doubt that it would grow (and might make it easier<br>to follow and more logical).<br clear="all">
<br>-- <br>Thanks,<br><br>Steve<br>