Note: This is a beta release of Red Hat Bugzilla 5.0. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Also email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback here.
Bug 236102 - Merge transactions during ->evict_inode() for gfs2
Summary: Merge transactions during ->evict_inode() for gfs2
Alias: None
Product: Fedora
Classification: Fedora
Component: GFS-kernel
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Steve Whitehouse
QA Contact:
Depends On: 242066
Blocks: 698394 434407
TreeView+ depends on / blocked
Reported: 2007-04-11 22:22 UTC by Steve Whitehouse
Modified: 2019-02-13 22:29 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: Enhancement
Doc Text:
Clone Of:
Last Closed:

Attachments (Terms of Use)

Description Steve Whitehouse 2007-04-11 22:22:06 UTC
Deallocation of a file is broken into several transactions. It should be
possible to merge some of these. Also there is considerable scope for
optimisation by reading things ahead in various places.

Also, if it is possible to deallocate directories without having to journal all
the hash table blocks and write them back to disk, then that should also result
in a faster deallocation.

The main problem here is the indeterminate number of rgrps in which the blocks
of the inode might reside. This means that deallocation may have to be broken
into separate transactions in order not to overflow the max transaction size.

We also need to spend some time looking for other oprimisations in this code.

Comment 12 Steve Whitehouse 2011-05-16 10:45:17 UTC
->delete_inode() has become ->evict_inode() in the upstream code

Note You need to log in before you can comment on or make changes to this bug.