lkml.org 
[lkml]   [2017]   [Nov]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 0/5] orangefs: misc attribute fixes
On Tue, Nov 28, 2017 at 03:21:56PM -0500, Martin Brandenburg wrote:
> Mike,
>
> Here's a few fixes for OrangeFS.
>
> Most important is to run getattr for size during a mmap page fault.
>
> I have also found a bug in d_revalidate. The test is backwards.
> However I haven't found a test that demonstrates that it is wrong, which
> makes me doubt that all the code here is completely optimal.

The reason I can't find a test is because this code only gets run
if the inode being tested has the same handle as the inode in the
dcache. Of course this happens all the time (it wasn't changed). We
would return that it was not valid even though it was, which didn't
affect correctness but caused OrangeFS to perform extra round-trips with
the server.

We would return that it was valid even though it wasn't when an inode
is deleted and re-created on the server as a different type or different
symlink target AND has the same handle as the one in the cache. This
practically never happens.

Martin

\
 
 \ /
  Last update: 2017-11-28 22:03    [W:0.074 / U:0.560 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site