Skip to content

Improve File Cache, other small performance fixes#842

Merged
jfantinhardesty merged 21 commits intomainfrom
improve-statfs-and-getattr
Feb 23, 2026
Merged

Improve File Cache, other small performance fixes#842
jfantinhardesty merged 21 commits intomainfrom
improve-statfs-and-getattr

Conversation

@jfantinhardesty
Copy link
Contributor

@jfantinhardesty jfantinhardesty commented Feb 23, 2026

What type of Pull Request is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Optimization
  • Documentation Update

Describe your changes in brief

Added dirty file tracking in file cache to allow returning attributes when file is open and dirty and prevent stale data from being returned. Improved TruncateFile to handle missing file descriptors by reopening the cached file if needed, and to avoid marking the file dirty if the truncate is a no-op (these happen frequently in the VMS use case). Adjusted the LRU eviction logic to avoid deleting files that are still in use. Read now returns the correct FUSE error code on access denied (happens some what frequently with VMS). Improved Readdir to correctly handle end-of-directory conditions and prevent unnecessary warnings or errors. Fix issue with limit on download in file cache by changing GetUsage to just return bytes.

Checklist

  • Tested locally
  • Added new dependencies
  • Updated documentation
  • Added tests

Related Issues

  • Related Issue #
  • Closes #

Copy link
Contributor

@Ferelith-maker Ferelith-maker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well done!

@jfantinhardesty jfantinhardesty merged commit 07e607f into main Feb 23, 2026
36 of 39 checks passed
@jfantinhardesty jfantinhardesty deleted the improve-statfs-and-getattr branch February 23, 2026 23:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants