diff options
| author | Kent Overstreet <kent.overstreet@linux.dev> | 2022-11-25 02:03:55 +0300 |
|---|---|---|
| committer | Kent Overstreet <kent.overstreet@linux.dev> | 2023-10-23 00:09:49 +0300 |
| commit | 393a1f6863790fddf8b53bfb81f2c984cdbc1990 (patch) | |
| tree | 49d028ee32ff607fcc69bf16aba948463d4a91cd /fs/bcachefs/extents.c | |
| parent | 19a614d2e4beed7faf52ab95cb48ce38a3c38c04 (diff) | |
| download | linux-393a1f6863790fddf8b53bfb81f2c984cdbc1990.tar.xz | |
bcachefs: Better inlining in core write path
Provide inline versions of some allocation functions
- bch2_alloc_sectors_done_inlined()
- bch2_alloc_sectors_append_ptrs_inlined()
and use them in the core IO path.
Also, inline bch2_extent_update_i_size_sectors() and
bch2_bkey_append_ptr().
In the core write path, function call overhead matters - every function
call is a jump to a new location and a potential cache miss.
Signed-off-by: Kent Overstreet <kent.overstreet@linux.dev>
Diffstat (limited to 'fs/bcachefs/extents.c')
| -rw-r--r-- | fs/bcachefs/extents.c | 23 |
1 files changed, 0 insertions, 23 deletions
diff --git a/fs/bcachefs/extents.c b/fs/bcachefs/extents.c index 422adca7230b..3d124dc5bbef 100644 --- a/fs/bcachefs/extents.c +++ b/fs/bcachefs/extents.c @@ -698,29 +698,6 @@ void bch2_bkey_extent_entry_drop(struct bkey_i *k, union bch_extent_entry *entry k->k.u64s -= extent_entry_u64s(entry); } -void bch2_bkey_append_ptr(struct bkey_i *k, - struct bch_extent_ptr ptr) -{ - EBUG_ON(bch2_bkey_has_device(bkey_i_to_s_c(k), ptr.dev)); - - switch (k->k.type) { - case KEY_TYPE_btree_ptr: - case KEY_TYPE_btree_ptr_v2: - case KEY_TYPE_extent: - EBUG_ON(bkey_val_u64s(&k->k) >= BKEY_EXTENT_VAL_U64s_MAX); - - ptr.type = 1 << BCH_EXTENT_ENTRY_ptr; - - memcpy((void *) &k->v + bkey_val_bytes(&k->k), - &ptr, - sizeof(ptr)); - k->u64s++; - break; - default: - BUG(); - } -} - static inline void __extent_entry_insert(struct bkey_i *k, union bch_extent_entry *dst, union bch_extent_entry *new) |