Re: [PATCH 12/15] btrfs: remove the free space inode ordered extent special cases

[Tal Zussman]

On 9/8/26 6:42 PM, David Sterba wrote:
> On Mon, Sep 07, 2026 at 09:19:27PM -0400, Tal Zussman wrote:
>> --- a/fs/btrfs/transaction.c
>> +++ b/fs/btrfs/transaction.c
>> @@ -125,17 +125,14 @@ static const unsigned int btrfs_blocked_trans_types[TRANS_STATE_MAX] = {
>>  	[TRANS_STATE_UNBLOCKED]		= (__TRANS_START |
>>  					   __TRANS_ATTACH |
>>  					   __TRANS_JOIN |
>> -					   __TRANS_JOIN_NOLOCK |
>>  					   __TRANS_JOIN_NOSTART),
>>  	[TRANS_STATE_SUPER_COMMITTED]	= (__TRANS_START |
>>  					   __TRANS_ATTACH |
>>  					   __TRANS_JOIN |
>> -					   __TRANS_JOIN_NOLOCK |
>>  					   __TRANS_JOIN_NOSTART),
>>  	[TRANS_STATE_COMPLETED]		= (__TRANS_START |
>>  					   __TRANS_ATTACH |
>>  					   __TRANS_JOIN |
>> -					   __TRANS_JOIN_NOLOCK |
>>  					   __TRANS_JOIN_NOSTART),
>>  };
> 
> This touches the transaction engine, there was exception for the space
> cache, I'm not sure if we want to remove it completely. At least it
> would be good to separate the space cache usage and the the API in case
> we'd want to restore it some day.
> 

Sure, I can split out the transaction changes from the rest of this patch
for v2. I'll move the transaction changes to the end of the series so we
can easily drop them if necessary.