public inbox for linux-btrfs@vger.kernel.org
 help / color / mirror / Atom feed
* Remaining BTRFS_PATH_AUTO_FREE work
@ 2025-08-18 17:10 David Sterba
  0 siblings, 0 replies; 4+ messages in thread
From: David Sterba @ 2025-08-18 17:10 UTC (permalink / raw)
  To: linux-btrfs

Hi,

there are still possible conversions of struct btrfs_path to
BTRFS_PATH_AUTO_FREE(). As this is fairly easy it's approachable for beginners
or drive-by contributions.

The basic pattern is explained in patch 4c74a32ad323 ("btrfs: DEFINE_FREE for
struct btrfs_path").

Examples of previous work:

- e23541811877 ("btrfs: do more trivial BTRFS_PATH_AUTO_FREE conversions")
- 516748f584fd ("btrfs: use BTRFS_PATH_AUTO_FREE in may_destroy_subvol()")

IIRC files beginning with [p-z] haven't been converted yet, search for
btrfs_alloc_path() and look if it matches the pattern.

Whoever is interested please reply to this mail so the work is not
unnecessarily duplicated.

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: Remaining BTRFS_PATH_AUTO_FREE work
@ 2025-08-19  2:03 Sun YangKai
  2025-08-19  3:13 ` Sun YangKai
  2025-08-21 12:36 ` David Sterba
  0 siblings, 2 replies; 4+ messages in thread
From: Sun YangKai @ 2025-08-19  2:03 UTC (permalink / raw)
  To: dsterba; +Cc: linux-btrfs

There's no such pattern in files beginning with p, and files beginning with z 
has been converted. So files beginning with [r-x] need this work. I'm doing 
this on files beginning with r.

BTW, should we also do this for files in the test dir?



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: Remaining BTRFS_PATH_AUTO_FREE work
  2025-08-19  2:03 Remaining BTRFS_PATH_AUTO_FREE work Sun YangKai
@ 2025-08-19  3:13 ` Sun YangKai
  2025-08-21 12:36 ` David Sterba
  1 sibling, 0 replies; 4+ messages in thread
From: Sun YangKai @ 2025-08-19  3:13 UTC (permalink / raw)
  To: dsterba; +Cc: linux-btrfs

> There's no such pattern in files beginning with p, and files beginning with
> z has been converted. So files beginning with [r-x] need this work. I'm
> doing this on files beginning with r.
Also for files beginning with s and t.
> 
> BTW, should we also do this for files in the test dir?





^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: Remaining BTRFS_PATH_AUTO_FREE work
  2025-08-19  2:03 Remaining BTRFS_PATH_AUTO_FREE work Sun YangKai
  2025-08-19  3:13 ` Sun YangKai
@ 2025-08-21 12:36 ` David Sterba
  1 sibling, 0 replies; 4+ messages in thread
From: David Sterba @ 2025-08-21 12:36 UTC (permalink / raw)
  To: Sun YangKai; +Cc: linux-btrfs

On Tue, Aug 19, 2025 at 10:03:45AM +0800, Sun YangKai wrote:
> There's no such pattern in files beginning with p, and files beginning with z 
> has been converted. So files beginning with [r-x] need this work. I'm doing 
> this on files beginning with r.
> 
> BTW, should we also do this for files in the test dir?

Yes, thanks.

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2025-08-21 12:36 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-08-19  2:03 Remaining BTRFS_PATH_AUTO_FREE work Sun YangKai
2025-08-19  3:13 ` Sun YangKai
2025-08-21 12:36 ` David Sterba
  -- strict thread matches above, loose matches on Subject: below --
2025-08-18 17:10 David Sterba

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox