From: Goldwyn Rodrigues <rgoldwyn@suse.de>
To: linux-btrfs@vger.kernel.org
Subject: [PATCH 0/7] Allocate structures on stack instead of kmalloc()
Date: Tue, 27 Jul 2021 16:17:24 -0500 [thread overview]
Message-ID: <20210727211731.23394-1-rgoldwyn@suse.de> (raw)
Here are some structs which can be converted to allocation on stack in order
to save on post-checks on kmalloc() and kfree() each of them.
Sizes of the structures are also in the commit message in case you feel they
are too bit to be allocated on stack.
There are two more structs in ioctl.c which can be converted, but
I was not sure of them:
1. In create_snapshot(), pending_snapshot can be converted. pending_snapshot
is used in the transaction.
2. In btrfs_ioctl_set_received_subvol_32, args64 can be converted, but args32
cannot. All Pointers associated with memdup_user() can also be converted
by using copy_from_user() instead. This would include many more structs.
Goldwyn Rodrigues (7):
btrfs: Allocate walk_control on stack
btrfs: Allocate file_ra_state on stack
btrfs: Allocate btrfs_ioctl_get_subvol_info_args on stack
btrfs: Allocate btrfs_ioctl_balance_args on stack
btrfs: Allocate btrfs_ioctl_quota_rescan_args on stack
btrfs: Allocate btrfs_ioctl_defrag_range_args on stack
btrfs: Alloc backref_ctx on stack
fs/btrfs/extent-tree.c | 89 +++++++++++++-----------------
fs/btrfs/free-space-cache.c | 12 ++---
fs/btrfs/ioctl.c | 105 ++++++++++++++----------------------
fs/btrfs/send.c | 29 ++++------
4 files changed, 89 insertions(+), 146 deletions(-)
next reply other threads:[~2021-07-27 21:19 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-07-27 21:17 Goldwyn Rodrigues [this message]
2021-07-29 16:51 ` [PATCH 0/7] Allocate structures on stack instead of kmalloc() David Sterba
-- strict thread matches above, loose matches on Subject: below --
2021-07-28 12:51 [PATCH 0/7] Change allocation from kmalloc() to stack Goldwyn Rodrigues
2021-07-27 21:17 ` [PATCH 1/7] btrfs: Allocate walk_control on stack Goldwyn Rodrigues
2021-07-28 5:11 ` Anand Jain
2021-07-28 5:25 ` Anand Jain
2021-07-28 11:08 ` David Sterba
2021-07-27 21:17 ` [PATCH 2/7] btrfs: Allocate file_ra_state " Goldwyn Rodrigues
2021-07-28 5:29 ` Anand Jain
2021-07-27 21:17 ` [PATCH 3/7] btrfs: Allocate btrfs_ioctl_get_subvol_info_args " Goldwyn Rodrigues
2021-07-28 5:59 ` Anand Jain
2021-07-29 17:08 ` David Sterba
2021-07-29 17:22 ` David Sterba
2021-07-27 21:17 ` [PATCH 4/7] btrfs: Allocate btrfs_ioctl_balance_args " Goldwyn Rodrigues
2021-07-28 0:02 ` Darrick J. Wong
2021-07-28 2:04 ` Goldwyn Rodrigues
2021-07-28 1:19 ` kernel test robot
2021-07-27 21:17 ` [PATCH 5/7] btrfs: Allocate btrfs_ioctl_quota_rescan_args " Goldwyn Rodrigues
2021-07-28 6:01 ` Anand Jain
2021-07-27 21:17 ` [PATCH 6/7] btrfs: Allocate btrfs_ioctl_defrag_range_args " Goldwyn Rodrigues
2021-07-28 6:27 ` Anand Jain
2021-07-27 21:17 ` [PATCH 7/7] btrfs: Alloc backref_ctx " Goldwyn Rodrigues
2021-07-28 6:30 ` Anand Jain
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20210727211731.23394-1-rgoldwyn@suse.de \
--to=rgoldwyn@suse.de \
--cc=linux-btrfs@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).