Minor cleanups in GFP_NOFS flags use
Get series with b4
b4 am [email protected]Versions newest first
- v1 Latest16 patches 2 repliesIn review
Patches in v1 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/16 | btrfs: sink gfp parameter to set_extent_bitsfs/ fs/ fs/ +3 more | +10 | 0 replies | |
| 2/16 | btrfs: sink gfp parameter to clear_extent_bitsfs/ fs/ fs/ +6 more | +15 | 0 replies | |
| 3/16 | btrfs: sink gfp parameter to clear_record_extent_bitsfs/ fs/ fs/ | +5 | 0 replies | |
| 4/16 | btrfs: sink gfp parameter to clear_extent_dirtyfs/ fs/ fs/ | +4 | 0 replies | |
| 5/16 | btrfs: sink gfp parameter to set_extent_delallocfs/ fs/ fs/ | +6 | 0 replies | |
| 6/16 | btrfs: sink gfp parameter to set_extent_defragfs/ fs/ | +3 | 0 replies | |
| 7/16 | btrfs: sink gfp parameter to set_extent_newfs/ fs/ | +4 | 0 replies | |
| 8/16 | btrfs: sink gfp parameter to set_record_extent_bitsfs/ fs/ fs/ | +4 | 0 replies | |
| 9/16 | btrfs: untangle gotos a bit in __set_extent_bitfs/ | +7 | 0 replies | |
| 10/16 | btrfs: untangle gotos a bit in __clear_extent_bitfs/ | +8 | 0 replies | |
| 11/16 | btrfs: untangle gotos a bit in convert_extent_bitfs/ | +7 | 0 replies | |
| 12/16 | btrfs: make state preallocation more speculative in __set_extent_bitfs/ | +7 | 0 replies | |
| 13/16 | btrfs: __set_extent_bit, try preallocation out of locked section with lighter gfp flagsfs/ | +7 | 2 replies | |
| 14/16 | btrfs: __clear_extent_bit, try preallocation out of locked section with lighter gfp flagsfs/ | +7 | 0 replies | |
| 15/16 | btrfs: convert_extent_bit, try preallocation out of locked section with lighter gfp flagsfs/ | +7 | 0 replies | |
| 16/16 | btrfs: sink gfp parameter to convert_extent_bitfs/ fs/ fs/ | +9 | 0 replies |
Reviews trailers per patch
No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.
Cover letter [PATCH 00/16] Minor cleanups in GFP_NOFS flags use
Hi,
the first part of patchset mostly moves the GFP_NOFS flags from several callsites into the callers. The point is to move all GFP_NOFS to the actual point of use. If all the callers pass the same flag, there's no point in doing that, which as a small benefit saves a few bytes on the stack.
The second part tries to improve extent_state preallocation logic in the set/clear/convert functions. Depending on the internal state, we might not need the allocation at all, the actual failure is left to the place where we really need the memory, that's done in a locked section with GFP_ATOMIC. If we leave this section, we speculatively try to allocate new extent state unless we already have one spare.
I'd like to get this to 4.7, the patchset has low functionality impact but helps me to continue in the NOFS crusade.
David Sterba (16):
btrfs: sink gfp parameter to set_extent_bits
btrfs: sink gfp parameter to clear_extent_bits
btrfs: sink gfp parameter to clear_record_extent_bits
btrfs: sink gfp parameter to clear_extent_dirty
btrfs: sink gfp parameter to set_extent_delalloc
btrfs: sink gfp parameter to set_extent_defrag
btrfs: sink gfp parameter to set_extent_new
btrfs: sink gfp parameter to set_record_extent_bits
btrfs: untangle gotos a bit in __set_extent_bit
btrfs: untangle gotos a bit in __clear_extent_bit
btrfs: untangle gotos a bit in convert_extent_bit
btrfs: make state preallocation more speculative in __set_extent_bit
btrfs: __set_extent_bit, try preallocation out of locked section with
lighter gfp flags
btrfs: __clear_extent_bit, try preallocation out of locked section
with lighter gfp flags
btrfs: convert_extent_bit, try preallocation out of locked section
with lighter gfp flags
btrfs: sink gfp parameter to convert_extent_bit
fs/btrfs/disk-io.c | 4 +- fs/btrfs/extent-tree.c | 16 +++--- fs/btrfs/extent_io.c | 105 +++++++++++++++++++++++---------------- fs/btrfs/extent_io.h | 34 ++++++------- fs/btrfs/file-item.c | 2 +- fs/btrfs/inode.c | 5 +- fs/btrfs/ioctl.c | 2 +- fs/btrfs/qgroup.c | 8 ++- fs/btrfs/relocation.c | 7 ++- fs/btrfs/scrub.c | 4 +- fs/btrfs/tests/extent-io-tests.c | 8 +-- fs/btrfs/transaction.c | 2 +- fs/btrfs/tree-log.c | 2 +- 13 files changed, 108 insertions(+), 91 deletions(-)
Signature · 1 line
2.7.1