2024-02-20 09:28:24

by Kunwu Chan

Subject: [PATCH 0/6] btrfs: Use KMEM_CACHE instead of kmem_cache_create

As David Sterba said in
https://lore.kernel.org/all/[email protected]/
I'm using a patchset to cleanup the same issues in the 'brtfs' module.

For where the cache name and the structure name match.
Use the new KMEM_CACHE() macro instead of direct kmem_cache_create
to simplify the creation of SLAB caches.

Kunwu Chan (6):
btrfs: Simplify the allocation of slab caches in
btrfs_delayed_inode_init
btrfs: Simplify the allocation of slab caches in ordered_data_init
btrfs: Simplify the allocation of slab caches in
btrfs_transaction_init
btrfs: Simplify the allocation of slab caches in btrfs_ctree_init
btrfs: Simplify the allocation of slab caches in
btrfs_delayed_ref_init
btrfs: Simplify the allocation of slab caches in btrfs_free_space_init

fs/btrfs/ctree.c | 4 +---
fs/btrfs/delayed-inode.c | 6 +-----
fs/btrfs/delayed-ref.c | 24 ++++++++----------------
fs/btrfs/free-space-cache.c | 4 +---
fs/btrfs/ordered-data.c | 5 +----
fs/btrfs/transaction.c | 5 ++---
6 files changed, 14 insertions(+), 34 deletions(-)

--
2.39.2

2024-02-21 12:01:30

by David Sterba

Subject: Re: [PATCH 0/6] btrfs: Use KMEM_CACHE instead of kmem_cache_create

On Tue, Feb 20, 2024 at 05:06:39PM +0800, Kunwu Chan wrote:
> As David Sterba said in
> https://lore.kernel.org/all/[email protected]/
> I'm using a patchset to cleanup the same issues in the 'brtfs' module.
>
> For where the cache name and the structure name match.
> Use the new KMEM_CACHE() macro instead of direct kmem_cache_create
> to simplify the creation of SLAB caches.
>
> Kunwu Chan (6):
> btrfs: Simplify the allocation of slab caches in
> btrfs_delayed_inode_init
> btrfs: Simplify the allocation of slab caches in ordered_data_init
> btrfs: Simplify the allocation of slab caches in
> btrfs_transaction_init
> btrfs: Simplify the allocation of slab caches in btrfs_ctree_init
> btrfs: Simplify the allocation of slab caches in
> btrfs_delayed_ref_init
> btrfs: Simplify the allocation of slab caches in btrfs_free_space_init

Added to for-next, thanks. I've edited the changels so the name of the
structure is mentioned rather than the function where it happens, and
did some minor formatting adjustments.

2024-02-22 03:10:40

by Kunwu Chan

Subject: Re: [PATCH 0/6] btrfs: Use KMEM_CACHE instead of kmem_cache_create

On 2024/2/21 20:00, David Sterba wrote:
> On Tue, Feb 20, 2024 at 05:06:39PM +0800, Kunwu Chan wrote:
>> As David Sterba said in
>> https://lore.kernel.org/all/[email protected]/
>> I'm using a patchset to cleanup the same issues in the 'brtfs' module.
>>
>> For where the cache name and the structure name match.
>> Use the new KMEM_CACHE() macro instead of direct kmem_cache_create
>> to simplify the creation of SLAB caches.
>>
>> Kunwu Chan (6):
>> btrfs: Simplify the allocation of slab caches in
>> btrfs_delayed_inode_init
>> btrfs: Simplify the allocation of slab caches in ordered_data_init
>> btrfs: Simplify the allocation of slab caches in
>> btrfs_transaction_init
>> btrfs: Simplify the allocation of slab caches in btrfs_ctree_init
>> btrfs: Simplify the allocation of slab caches in
>> btrfs_delayed_ref_init
>> btrfs: Simplify the allocation of slab caches in btrfs_free_space_init
>
> Added to for-next, thanks. I've edited the changels so the name of the
> structure is mentioned rather than the function where it happens, and
> did some minor formatting adjustments.
It's ok, thanks for your adjustments.
--
Thanks,
Kunwu