Re: [PATCH 04/11] btrfs: reada: use sprout device_list_mutex

Anand Jain
2020-09-01T09:05:12Z

On 1/9/20 12:08 am, Josef Bacik wrote:
> On 8/30/20 10:40 AM, Anand Jain wrote:
>> On an fs mounted using a sprout-device, the seed fs_devices are
>> maintained
>> in a linked list under fs_info->fs_devices. Each seed's fs_devices also
>> have device_list_mutex initialized to protect against the potential race
>> with delete threads. But the delete thread (at btrfs_rm_device()) is
>> holding
>> the fs_info::fs_devices::device_list_mutex mutex which is sprout's
>> device_list_mutex instead of seed's device_list_mutex. Moreover, there
>> aren't any significient benefits in using the seed::device_list_mutex
>> instead of sprout::device_list_mutex.
>>
>> So this patch converts them of using the seed::device_list_mutex to
>> sprout::device_list_mutex.
>>
>> Signed-off-by: Anand Jain <anand.jain@oracle.com>
>
> This doesn't apply cleanly to misc-next as of this morning.  Thanks,

Kdave renamed the function to reada_start_for_fsdevs() dropping
the __ prefix. Rebased my misc-next. I am sending V2.

Thanks, Anand

>
> Josef