Re: [PATCH v3 3/5] btrfs: zoned: call blkdev_zone_mgmt in nofs scope

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Johannes Thumshirn <johannes.thumshirn@xxxxxxx>, Naohiro Aota <naohiro.aota@xxxxxxx>, Johannes Thumshirn <jth@xxxxxxxxxx>, Alasdair Kergon <agk@xxxxxxxxxx>, Mike Snitzer <snitzer@xxxxxxxxxx>, Mikulas Patocka <mpatocka@xxxxxxxxxx>, dm-devel@xxxxxxxxxxxxxxx, Chris Mason <clm@xxxxxx>, Josef Bacik <josef@xxxxxxxxxxxxxx>, David Sterba <dsterba@xxxxxxxx>, Jaegeuk Kim <jaegeuk@xxxxxxxxxx>, Chao Yu <chao@xxxxxxxxxx>, Jens Axboe <axboe@xxxxxxxxx>, Christoph Hellwig <hch@xxxxxx>, Sagi Grimberg <sagi@xxxxxxxxxxx>, Chaitanya Kulkarni <kch@xxxxxxxxxx>
Subject: Re: [PATCH v3 3/5] btrfs: zoned: call blkdev_zone_mgmt in nofs scope
From: Damien Le Moal <dlemoal@xxxxxxxxxx>
Date: Tue, 30 Jan 2024 08:27:18 +0900
Cc: linux-fsdevel@xxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, linux-btrfs@xxxxxxxxxxxxxxx, linux-f2fs-devel@xxxxxxxxxxxxxxxxxxxxx, linux-block@xxxxxxxxxxxxxxx, linux-nvme@xxxxxxxxxxxxxxxxxxx
In-reply-to: <20240128-zonefs_nofs-v3-3-ae3b7c8def61@wdc.com>
Organization: Western Digital Research
User-agent: Mozilla Thunderbird

On 1/29/24 16:52, Johannes Thumshirn wrote:
> Add a memalloc_nofs scope around all calls to blkdev_zone_mgmt(). This
> allows us to further get rid of the GFP_NOFS argument for
> blkdev_zone_mgmt().
> 
> Signed-off-by: Johannes Thumshirn <johannes.thumshirn@xxxxxxx>

Looks OK to me.

Reviewed-by: Damien Le Moal <dlemoal@xxxxxxxxxx>


-- 
Damien Le Moal
Western Digital Research

References:
- [PATCH v3 0/5] block: remove gfp_mask for blkdev_zone_mgmt()
  - From: Johannes Thumshirn
- [PATCH v3 3/5] btrfs: zoned: call blkdev_zone_mgmt in nofs scope
  - From: Johannes Thumshirn

Prev by Date: Re: [PATCH v3 2/5] dm: dm-zoned: guard blkdev_zone_mgmt with noio scope
Next by Date: Re: [PATCH v3 4/5] f2fs: guard blkdev_zone_mgmt with nofs scope
Previous by thread: [PATCH v3 3/5] btrfs: zoned: call blkdev_zone_mgmt in nofs scope
Next by thread: Re: [PATCH v3 3/5] btrfs: zoned: call blkdev_zone_mgmt in nofs scope
Index(es):
- Date
- Thread

[Index of Archives] [Linux Ext4 Filesystem] [Union Filesystem] [Filesystem Testing] [Ceph Users] [Ecryptfs] [NTFS 3] [AutoFS] [Kernel Newbies] [Share Photos] [Security] [Netfilter] [Bugtraq] [Yosemite News] [MIPS Linux] [ARM Linux] [Linux Security] [Linux Cachefs] [Reiser Filesystem] [Linux RAID] [NTFS 3] [Samba] [Device Mapper] [CEPH Development]