Bug 7826
Summary: | BUG: at mm/truncate.c:425 invalidate_inode_pages2_range() | ||
---|---|---|---|
Product: | File System | Reporter: | Andrew Clayton (andrew) |
Component: | NFS | Assignee: | Andrew Morton (akpm) |
Status: | RESOLVED CODE_FIX | ||
Severity: | normal | CC: | bunk |
Priority: | P2 | ||
Hardware: | i386 | ||
OS: | Linux | ||
Kernel Version: | 2.6.20-rc5 | Subsystem: | |
Regression: | --- | Bisected commit-id: |
Description
Andrew Clayton
2007-01-15 04:30:13 UTC
Not a bug! It is perfectly legal and acceptable for the user to ^C out of a hanging process. In that case, the NFS client will also abandon the attempt to invalidate the page cache, and just defer doing so until the next process needs to access that data. That WARN_ON crap that Andrew merged into invalidate_inode_pages2_range() needs to be thrown out. Agreed, this is not a bug if the callers handle it correctly. Can we send a patch to remove the WARN_ON_ONCE, and close this? we already did, I think. |