* [PATCH] Revert "btrfs: zoned: fail mount if the device does not support zone append"
@ 2021-07-01 8:13 Johannes Thumshirn
2021-07-01 9:34 ` David Sterba
0 siblings, 1 reply; 3+ messages in thread
From: Johannes Thumshirn @ 2021-07-01 8:13 UTC (permalink / raw)
To: David Sterba
Cc: Johannes Thumshirn, linux-btrfs, Naohiro Aota, Damien Le Moal
Now that the device-mapper code can fully emulate zone append there's no
need for this check anymore.
This reverts commit 1d68128c107a0b8c0c9125cb05d4771ddc438369.
Cc: Naohiro Aota <naohiro.aota@wdc.com>
Cc: Damien Le Moal <damien.lemoal@wdc.com>
Singed-off-by: Johannes Thumshirn <johannes.thumshirn@wdc.com>
---
fs/btrfs/zoned.c | 7 -------
1 file changed, 7 deletions(-)
diff --git a/fs/btrfs/zoned.c b/fs/btrfs/zoned.c
index 297c0b1c0634..e4087a2364a2 100644
--- a/fs/btrfs/zoned.c
+++ b/fs/btrfs/zoned.c
@@ -354,13 +354,6 @@ int btrfs_get_dev_zone_info(struct btrfs_device *device)
if (!IS_ALIGNED(nr_sectors, zone_sectors))
zone_info->nr_zones++;
- if (bdev_is_zoned(bdev) && zone_info->max_zone_append_size == 0) {
- btrfs_err(fs_info, "zoned: device %pg does not support zone append",
- bdev);
- ret = -EINVAL;
- goto out;
- }
-
zone_info->seq_zones = bitmap_zalloc(zone_info->nr_zones, GFP_KERNEL);
if (!zone_info->seq_zones) {
ret = -ENOMEM;
--
2.31.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] Revert "btrfs: zoned: fail mount if the device does not support zone append"
2021-07-01 8:13 [PATCH] Revert "btrfs: zoned: fail mount if the device does not support zone append" Johannes Thumshirn
@ 2021-07-01 9:34 ` David Sterba
2021-07-01 9:49 ` Johannes Thumshirn
0 siblings, 1 reply; 3+ messages in thread
From: David Sterba @ 2021-07-01 9:34 UTC (permalink / raw)
To: Johannes Thumshirn
Cc: David Sterba, linux-btrfs, Naohiro Aota, Damien Le Moal
On Thu, Jul 01, 2021 at 05:13:37PM +0900, Johannes Thumshirn wrote:
> Now that the device-mapper code can fully emulate zone append there's no
> need for this check anymore.
Which commit implements that? I did a quick search for 'append' in the
recent git history there and found nothing.
>
> This reverts commit 1d68128c107a0b8c0c9125cb05d4771ddc438369.
Please do commit references as 1d68128c107a ("btrfs: zoned: fail mount if the
device does not support zone append") .
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] Revert "btrfs: zoned: fail mount if the device does not support zone append"
2021-07-01 9:34 ` David Sterba
@ 2021-07-01 9:49 ` Johannes Thumshirn
0 siblings, 0 replies; 3+ messages in thread
From: Johannes Thumshirn @ 2021-07-01 9:49 UTC (permalink / raw)
To: dsterba; +Cc: David Sterba, linux-btrfs, Naohiro Aota, Damien Le Moal
On 01/07/2021 11:37, David Sterba wrote:
> On Thu, Jul 01, 2021 at 05:13:37PM +0900, Johannes Thumshirn wrote:
>> Now that the device-mapper code can fully emulate zone append there's no
>> need for this check anymore.
>
> Which commit implements that? I did a quick search for 'append' in the
> recent git history there and found nothing.
The series is on it's way to Linus:
https://lore.kernel.org/linux-block/YNyparaGoPleiSxX@redhat.com
I'll resend the patch once it hits Linus' tree and has a stable commit id.
>>
>> This reverts commit 1d68128c107a0b8c0c9125cb05d4771ddc438369.
>
> Please do commit references as 1d68128c107a ("btrfs: zoned: fail mount if the
> device does not support zone append") .
>
Will do.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2021-07-01 9:49 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-07-01 8:13 [PATCH] Revert "btrfs: zoned: fail mount if the device does not support zone append" Johannes Thumshirn
2021-07-01 9:34 ` David Sterba
2021-07-01 9:49 ` Johannes Thumshirn
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.