[linux-cifs-client] [PATCH 4/5] cifs: take reference to inode for oplock breaks

Steve French smfrench at gmail.com
Fri Aug 21 13:01:21 MDT 2009


On Fri, Aug 21, 2009 at 10:48 AM, Jeff Layton <jlayton at redhat.com> wrote:

>
> As long as you can do so in a non-racy way, then I'm not opposed to
> that long-term. The problem though is that I don't have a lot of
> confidence in the open file tracking code. It's extremely hard to
> follow and definitely has races. I don't think it's really possible to
> do what you suggest safely until the open file tracking code has been
> fixed.
>
> For now, I'm pretty sure this set should fix the problems that users
> are hitting in the oplock codepath today. I'd like to fix that first
> before we embark on a redesign of it.
>

You may be right that this should be two stages, but your 2nd and 3rd patch
are already large, so I doubt that it would grow (and might make it easier
to follow and more logical).

-- 
Thanks,

Steve
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.samba.org/pipermail/linux-cifs-client/attachments/20090821/0667654b/attachment.html>


More information about the linux-cifs-client mailing list