[OpenAFS-devel] Linux cache manager throwing away callbacks

Chas Williams (CONTRACTOR) chas@cmf.nrl.navy.mil
Mon, 19 Jan 2009 11:15:08 -0500


In message <db6e3f110901190757t4f4936cdo5cf4a2a9ea34b28b@mail.gmail.com>,"Derri
ck Brashear" writes:
>> afs_InactiveVCache() already has a conditional check for CUnlinked so
>> i didnt think it was helpful to check twice.
>
>But the call to clear e.g. CDirty and InvalidateAllSegments() happens
>outside that check, which I'm pretty sure at best can cause us to need
>to revalidate that our data is current and at worst will discard
>unwritten data.

checking CUnlinked doesnt prevent that.  this code in .dentry_iput has
been around quite a while i think.