All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Christoph Böhmwalder" <christoph.boehmwalder@linbit.com>
To: Christoph Hellwig <hch@lst.de>
Cc: drbd-dev@lists.linbit.com, linux-block@vger.kernel.org,
	Jens Axboe <axboe@kernel.dk>
Subject: Re: [Drbd-dev] [PATCH 25/27] block: remove QUEUE_FLAG_DISCARD
Date: Wed, 6 Apr 2022 11:43:28 +0200	[thread overview]
Message-ID: <287eee33-720d-79f2-c1c6-862001164673@linbit.com> (raw)
In-Reply-To: <20220406060516.409838-26-hch@lst.de>

Am 06.04.22 um 08:05 schrieb Christoph Hellwig:
> Just use a non-zero max_discard_sectors as an indicator for discard
> support, similar to what is done for write zeroes.
> 
> The only places where needs special attention is the RAID5 driver,
> which must clear discard support for security reasons by default,
> even if the default stacking rules would allow for it.
> 
> Signed-off-by: Christoph Hellwig <hch@lst.de>
> ---
>  arch/um/drivers/ubd_kern.c    |  2 --
>  block/blk-mq-debugfs.c        |  1 -
>  drivers/block/drbd/drbd_nl.c  | 15 ---------------

For the DRBD part:

Acked-by: Christoph Böhmwalder <christoph.boehmwalder@linbit.com>

>  drivers/block/loop.c          |  2 --
>  drivers/block/nbd.c           |  3 ---
>  drivers/block/null_blk/main.c |  1 -
>  drivers/block/rbd.c           |  1 -
>  drivers/block/rnbd/rnbd-clt.c |  2 --
>  drivers/block/virtio_blk.c    |  2 --
>  drivers/block/xen-blkfront.c  |  2 --
>  drivers/block/zram/zram_drv.c |  1 -
>  drivers/md/bcache/super.c     |  1 -
>  drivers/md/dm-table.c         |  5 +----
>  drivers/md/dm-thin.c          |  2 --
>  drivers/md/dm.c               |  1 -
>  drivers/md/md-linear.c        |  9 ---------
>  drivers/md/raid0.c            |  7 -------
>  drivers/md/raid1.c            | 14 --------------
>  drivers/md/raid10.c           | 14 --------------
>  drivers/md/raid5.c            | 12 ++++--------
>  drivers/mmc/core/queue.c      |  1 -
>  drivers/mtd/mtd_blkdevs.c     |  1 -
>  drivers/nvme/host/core.c      |  6 ++----
>  drivers/s390/block/dasd_fba.c |  1 -
>  drivers/scsi/sd.c             |  2 --
>  include/linux/blkdev.h        |  2 --
>  26 files changed, 7 insertions(+), 103 deletions(-)
> 
> diff --git a/arch/um/drivers/ubd_kern.c b/arch/um/drivers/ubd_kern.c
> index b03269faef714..085ffdf98e57e 100644
> --- a/arch/um/drivers/ubd_kern.c
> +++ b/arch/um/drivers/ubd_kern.c
> @@ -483,7 +483,6 @@ static void ubd_handler(void)
>  			if ((io_req->error == BLK_STS_NOTSUPP) && (req_op(io_req->req) == REQ_OP_DISCARD)) {
>  				blk_queue_max_discard_sectors(io_req->req->q, 0);
>  				blk_queue_max_write_zeroes_sectors(io_req->req->q, 0);
> -				blk_queue_flag_clear(QUEUE_FLAG_DISCARD, io_req->req->q);
>  			}
>  			blk_mq_end_request(io_req->req, io_req->error);
>  			kfree(io_req);
> @@ -803,7 +802,6 @@ static int ubd_open_dev(struct ubd *ubd_dev)
>  		ubd_dev->queue->limits.discard_alignment = SECTOR_SIZE;
>  		blk_queue_max_discard_sectors(ubd_dev->queue, UBD_MAX_REQUEST);
>  		blk_queue_max_write_zeroes_sectors(ubd_dev->queue, UBD_MAX_REQUEST);
> -		blk_queue_flag_set(QUEUE_FLAG_DISCARD, ubd_dev->queue);
>  	}
>  	blk_queue_flag_set(QUEUE_FLAG_NONROT, ubd_dev->queue);
>  	return 0;
> diff --git a/block/blk-mq-debugfs.c b/block/blk-mq-debugfs.c
> index aa0349e9f083b..fd111c5001256 100644
> --- a/block/blk-mq-debugfs.c
> +++ b/block/blk-mq-debugfs.c
> @@ -113,7 +113,6 @@ static const char *const blk_queue_flag_name[] = {
>  	QUEUE_FLAG_NAME(FAIL_IO),
>  	QUEUE_FLAG_NAME(NONROT),
>  	QUEUE_FLAG_NAME(IO_STAT),
> -	QUEUE_FLAG_NAME(DISCARD),
>  	QUEUE_FLAG_NAME(NOXMERGES),
>  	QUEUE_FLAG_NAME(ADD_RANDOM),
>  	QUEUE_FLAG_NAME(SECERASE),
> diff --git a/drivers/block/drbd/drbd_nl.c b/drivers/block/drbd/drbd_nl.c
> index 94ac3737723a8..0b3e43be6414d 100644
> --- a/drivers/block/drbd/drbd_nl.c
> +++ b/drivers/block/drbd/drbd_nl.c
> @@ -1230,30 +1230,16 @@ static void decide_on_discard_support(struct drbd_device *device,
>  	 */
>  	blk_queue_discard_granularity(q, 512);
>  	q->limits.max_discard_sectors = drbd_max_discard_sectors(connection);
> -	blk_queue_flag_set(QUEUE_FLAG_DISCARD, q);
>  	q->limits.max_write_zeroes_sectors =
>  		drbd_max_discard_sectors(connection);
>  	return;
>  
>  not_supported:
> -	blk_queue_flag_clear(QUEUE_FLAG_DISCARD, q);
>  	blk_queue_discard_granularity(q, 0);
>  	q->limits.max_discard_sectors = 0;
>  	q->limits.max_write_zeroes_sectors = 0;
>  }
>  
> -static void fixup_discard_if_not_supported(struct request_queue *q)
> -{
> -	/* To avoid confusion, if this queue does not support discard, clear
> -	 * max_discard_sectors, which is what lsblk -D reports to the user.
> -	 * Older kernels got this wrong in "stack limits".
> -	 * */
> -	if (!blk_queue_discard(q)) {
> -		blk_queue_max_discard_sectors(q, 0);
> -		blk_queue_discard_granularity(q, 0);
> -	}
> -}
> -
>  static void fixup_write_zeroes(struct drbd_device *device, struct request_queue *q)
>  {
>  	/* Fixup max_write_zeroes_sectors after blk_stack_limits():
> @@ -1300,7 +1286,6 @@ static void drbd_setup_queue_param(struct drbd_device *device, struct drbd_backi
>  		blk_stack_limits(&q->limits, &b->limits, 0);
>  		disk_update_readahead(device->vdisk);
>  	}
> -	fixup_discard_if_not_supported(q);
>  	fixup_write_zeroes(device, q);
>  }
>  
> diff --git a/drivers/block/loop.c b/drivers/block/loop.c
> index d5499795a1fec..976cf987b3920 100644
> --- a/drivers/block/loop.c
> +++ b/drivers/block/loop.c
> @@ -784,12 +784,10 @@ static void loop_config_discard(struct loop_device *lo)
>  		q->limits.discard_granularity = granularity;
>  		blk_queue_max_discard_sectors(q, max_discard_sectors);
>  		blk_queue_max_write_zeroes_sectors(q, max_discard_sectors);
> -		blk_queue_flag_set(QUEUE_FLAG_DISCARD, q);
>  	} else {
>  		q->limits.discard_granularity = 0;
>  		blk_queue_max_discard_sectors(q, 0);
>  		blk_queue_max_write_zeroes_sectors(q, 0);
> -		blk_queue_flag_clear(QUEUE_FLAG_DISCARD, q);
>  	}
>  	q->limits.discard_alignment = 0;
>  }
> diff --git a/drivers/block/nbd.c b/drivers/block/nbd.c
> index 5a1f98494dddf..c7e03eabd205f 100644
> --- a/drivers/block/nbd.c
> +++ b/drivers/block/nbd.c
> @@ -1231,8 +1231,6 @@ static void nbd_parse_flags(struct nbd_device *nbd)
>  		set_disk_ro(nbd->disk, true);
>  	else
>  		set_disk_ro(nbd->disk, false);
> -	if (config->flags & NBD_FLAG_SEND_TRIM)
> -		blk_queue_flag_set(QUEUE_FLAG_DISCARD, nbd->disk->queue);
>  	if (config->flags & NBD_FLAG_SEND_FLUSH) {
>  		if (config->flags & NBD_FLAG_SEND_FUA)
>  			blk_queue_write_cache(nbd->disk->queue, true, true);
> @@ -1320,7 +1318,6 @@ static void nbd_config_put(struct nbd_device *nbd)
>  		nbd->disk->queue->limits.discard_granularity = 0;
>  		nbd->disk->queue->limits.discard_alignment = 0;
>  		blk_queue_max_discard_sectors(nbd->disk->queue, UINT_MAX);
> -		blk_queue_flag_clear(QUEUE_FLAG_DISCARD, nbd->disk->queue);
>  
>  		mutex_unlock(&nbd->config_lock);
>  		nbd_put(nbd);
> diff --git a/drivers/block/null_blk/main.c b/drivers/block/null_blk/main.c
> index 05b1120e66234..f6493a9e85ed3 100644
> --- a/drivers/block/null_blk/main.c
> +++ b/drivers/block/null_blk/main.c
> @@ -1767,7 +1767,6 @@ static void null_config_discard(struct nullb *nullb)
>  	nullb->q->limits.discard_granularity = nullb->dev->blocksize;
>  	nullb->q->limits.discard_alignment = nullb->dev->blocksize;
>  	blk_queue_max_discard_sectors(nullb->q, UINT_MAX >> 9);
> -	blk_queue_flag_set(QUEUE_FLAG_DISCARD, nullb->q);
>  }
>  
>  static const struct block_device_operations null_bio_ops = {
> diff --git a/drivers/block/rbd.c b/drivers/block/rbd.c
> index b844432bad20b..2b21f717cce1a 100644
> --- a/drivers/block/rbd.c
> +++ b/drivers/block/rbd.c
> @@ -4942,7 +4942,6 @@ static int rbd_init_disk(struct rbd_device *rbd_dev)
>  	blk_queue_io_opt(q, rbd_dev->opts->alloc_size);
>  
>  	if (rbd_dev->opts->trim) {
> -		blk_queue_flag_set(QUEUE_FLAG_DISCARD, q);
>  		q->limits.discard_granularity = rbd_dev->opts->alloc_size;
>  		blk_queue_max_discard_sectors(q, objset_bytes >> SECTOR_SHIFT);
>  		blk_queue_max_write_zeroes_sectors(q, objset_bytes >> SECTOR_SHIFT);
> diff --git a/drivers/block/rnbd/rnbd-clt.c b/drivers/block/rnbd/rnbd-clt.c
> index b66e8840b94b8..efa99a3884507 100644
> --- a/drivers/block/rnbd/rnbd-clt.c
> +++ b/drivers/block/rnbd/rnbd-clt.c
> @@ -1364,8 +1364,6 @@ static void setup_request_queue(struct rnbd_clt_dev *dev)
>  	blk_queue_max_discard_sectors(dev->queue, dev->max_discard_sectors);
>  	dev->queue->limits.discard_granularity	= dev->discard_granularity;
>  	dev->queue->limits.discard_alignment	= dev->discard_alignment;
> -	if (dev->max_discard_sectors)
> -		blk_queue_flag_set(QUEUE_FLAG_DISCARD, dev->queue);
>  	if (dev->secure_discard)
>  		blk_queue_flag_set(QUEUE_FLAG_SECERASE, dev->queue);
>  
> diff --git a/drivers/block/virtio_blk.c b/drivers/block/virtio_blk.c
> index a8bcf3f664af1..6ccf15253dee1 100644
> --- a/drivers/block/virtio_blk.c
> +++ b/drivers/block/virtio_blk.c
> @@ -888,8 +888,6 @@ static int virtblk_probe(struct virtio_device *vdev)
>  			v = sg_elems;
>  		blk_queue_max_discard_segments(q,
>  					       min(v, MAX_DISCARD_SEGMENTS));
> -
> -		blk_queue_flag_set(QUEUE_FLAG_DISCARD, q);
>  	}
>  
>  	if (virtio_has_feature(vdev, VIRTIO_BLK_F_WRITE_ZEROES)) {
> diff --git a/drivers/block/xen-blkfront.c b/drivers/block/xen-blkfront.c
> index 003056d4f7f5f..253bf835aca1f 100644
> --- a/drivers/block/xen-blkfront.c
> +++ b/drivers/block/xen-blkfront.c
> @@ -944,7 +944,6 @@ static void blkif_set_queue_limits(struct blkfront_info *info)
>  	blk_queue_flag_set(QUEUE_FLAG_VIRT, rq);
>  
>  	if (info->feature_discard) {
> -		blk_queue_flag_set(QUEUE_FLAG_DISCARD, rq);
>  		blk_queue_max_discard_sectors(rq, get_capacity(gd));
>  		rq->limits.discard_granularity = info->discard_granularity ?:
>  						 info->physical_sector_size;
> @@ -1606,7 +1605,6 @@ static irqreturn_t blkif_interrupt(int irq, void *dev_id)
>  				blkif_req(req)->error = BLK_STS_NOTSUPP;
>  				info->feature_discard = 0;
>  				info->feature_secdiscard = 0;
> -				blk_queue_flag_clear(QUEUE_FLAG_DISCARD, rq);
>  				blk_queue_flag_clear(QUEUE_FLAG_SECERASE, rq);
>  			}
>  			break;
> diff --git a/drivers/block/zram/zram_drv.c b/drivers/block/zram/zram_drv.c
> index e9474b02012de..59ff444bf6c76 100644
> --- a/drivers/block/zram/zram_drv.c
> +++ b/drivers/block/zram/zram_drv.c
> @@ -1952,7 +1952,6 @@ static int zram_add(void)
>  	blk_queue_io_opt(zram->disk->queue, PAGE_SIZE);
>  	zram->disk->queue->limits.discard_granularity = PAGE_SIZE;
>  	blk_queue_max_discard_sectors(zram->disk->queue, UINT_MAX);
> -	blk_queue_flag_set(QUEUE_FLAG_DISCARD, zram->disk->queue);
>  
>  	/*
>  	 * zram_bio_discard() will clear all logical blocks if logical block
> diff --git a/drivers/md/bcache/super.c b/drivers/md/bcache/super.c
> index 296f200b2e208..2f49e31142f62 100644
> --- a/drivers/md/bcache/super.c
> +++ b/drivers/md/bcache/super.c
> @@ -973,7 +973,6 @@ static int bcache_device_init(struct bcache_device *d, unsigned int block_size,
>  
>  	blk_queue_flag_set(QUEUE_FLAG_NONROT, d->disk->queue);
>  	blk_queue_flag_clear(QUEUE_FLAG_ADD_RANDOM, d->disk->queue);
> -	blk_queue_flag_set(QUEUE_FLAG_DISCARD, d->disk->queue);
>  
>  	blk_queue_write_cache(q, true, true);
>  
> diff --git a/drivers/md/dm-table.c b/drivers/md/dm-table.c
> index 4297c38328a9b..0dff6907fd00d 100644
> --- a/drivers/md/dm-table.c
> +++ b/drivers/md/dm-table.c
> @@ -1968,15 +1968,12 @@ int dm_table_set_restrictions(struct dm_table *t, struct request_queue *q,
>  		blk_queue_flag_clear(QUEUE_FLAG_NOWAIT, q);
>  
>  	if (!dm_table_supports_discards(t)) {
> -		blk_queue_flag_clear(QUEUE_FLAG_DISCARD, q);
> -		/* Must also clear discard limits... */
>  		q->limits.max_discard_sectors = 0;
>  		q->limits.max_hw_discard_sectors = 0;
>  		q->limits.discard_granularity = 0;
>  		q->limits.discard_alignment = 0;
>  		q->limits.discard_misaligned = 0;
> -	} else
> -		blk_queue_flag_set(QUEUE_FLAG_DISCARD, q);
> +	}
>  
>  	if (dm_table_supports_secure_erase(t))
>  		blk_queue_flag_set(QUEUE_FLAG_SECERASE, q);
> diff --git a/drivers/md/dm-thin.c b/drivers/md/dm-thin.c
> index cd333a3e4c33b..eded4bcc4545f 100644
> --- a/drivers/md/dm-thin.c
> +++ b/drivers/md/dm-thin.c
> @@ -4050,8 +4050,6 @@ static void pool_io_hints(struct dm_target *ti, struct queue_limits *limits)
>  		/*
>  		 * Must explicitly disallow stacking discard limits otherwise the
>  		 * block layer will stack them if pool's data device has support.
> -		 * QUEUE_FLAG_DISCARD wouldn't be set but there is no way for the
> -		 * user to see that, so make sure to set all discard limits to 0.
>  		 */
>  		limits->discard_granularity = 0;
>  		return;
> diff --git a/drivers/md/dm.c b/drivers/md/dm.c
> index ba75933cc22ca..dbbf64ce7e927 100644
> --- a/drivers/md/dm.c
> +++ b/drivers/md/dm.c
> @@ -955,7 +955,6 @@ void disable_discard(struct mapped_device *md)
>  
>  	/* device doesn't really support DISCARD, disable it */
>  	limits->max_discard_sectors = 0;
> -	blk_queue_flag_clear(QUEUE_FLAG_DISCARD, md->queue);
>  }
>  
>  void disable_write_zeroes(struct mapped_device *md)
> diff --git a/drivers/md/md-linear.c b/drivers/md/md-linear.c
> index 4dd5afff72844..138a3b25c5c82 100644
> --- a/drivers/md/md-linear.c
> +++ b/drivers/md/md-linear.c
> @@ -64,7 +64,6 @@ static struct linear_conf *linear_conf(struct mddev *mddev, int raid_disks)
>  	struct linear_conf *conf;
>  	struct md_rdev *rdev;
>  	int i, cnt;
> -	bool discard_supported = false;
>  
>  	conf = kzalloc(struct_size(conf, disks, raid_disks), GFP_KERNEL);
>  	if (!conf)
> @@ -96,9 +95,6 @@ static struct linear_conf *linear_conf(struct mddev *mddev, int raid_disks)
>  
>  		conf->array_sectors += rdev->sectors;
>  		cnt++;
> -
> -		if (bdev_max_discard_sectors(rdev->bdev))
> -			discard_supported = true;
>  	}
>  	if (cnt != raid_disks) {
>  		pr_warn("md/linear:%s: not enough drives present. Aborting!\n",
> @@ -106,11 +102,6 @@ static struct linear_conf *linear_conf(struct mddev *mddev, int raid_disks)
>  		goto out;
>  	}
>  
> -	if (!discard_supported)
> -		blk_queue_flag_clear(QUEUE_FLAG_DISCARD, mddev->queue);
> -	else
> -		blk_queue_flag_set(QUEUE_FLAG_DISCARD, mddev->queue);
> -
>  	/*
>  	 * Here we calculate the device offsets.
>  	 */
> diff --git a/drivers/md/raid0.c b/drivers/md/raid0.c
> index 02ac3ab213c72..7231f5e1eaa73 100644
> --- a/drivers/md/raid0.c
> +++ b/drivers/md/raid0.c
> @@ -399,7 +399,6 @@ static int raid0_run(struct mddev *mddev)
>  	conf = mddev->private;
>  	if (mddev->queue) {
>  		struct md_rdev *rdev;
> -		bool discard_supported = false;
>  
>  		blk_queue_max_hw_sectors(mddev->queue, mddev->chunk_sectors);
>  		blk_queue_max_write_zeroes_sectors(mddev->queue, mddev->chunk_sectors);
> @@ -412,13 +411,7 @@ static int raid0_run(struct mddev *mddev)
>  		rdev_for_each(rdev, mddev) {
>  			disk_stack_limits(mddev->gendisk, rdev->bdev,
>  					  rdev->data_offset << 9);
> -			if (bdev_max_discard_sectors(rdev->bdev))
> -				discard_supported = true;
>  		}
> -		if (!discard_supported)
> -			blk_queue_flag_clear(QUEUE_FLAG_DISCARD, mddev->queue);
> -		else
> -			blk_queue_flag_set(QUEUE_FLAG_DISCARD, mddev->queue);
>  	}
>  
>  	/* calculate array device size */
> diff --git a/drivers/md/raid1.c b/drivers/md/raid1.c
> index 39b9cb4d54ee0..3da749d150a17 100644
> --- a/drivers/md/raid1.c
> +++ b/drivers/md/raid1.c
> @@ -1826,8 +1826,6 @@ static int raid1_add_disk(struct mddev *mddev, struct md_rdev *rdev)
>  			break;
>  		}
>  	}
> -	if (mddev->queue && bdev_max_discard_sectors(rdev->bdev))
> -		blk_queue_flag_set(QUEUE_FLAG_DISCARD, mddev->queue);
>  	print_conf(conf);
>  	return err;
>  }
> @@ -3106,7 +3104,6 @@ static int raid1_run(struct mddev *mddev)
>  	int i;
>  	struct md_rdev *rdev;
>  	int ret;
> -	bool discard_supported = false;
>  
>  	if (mddev->level != 1) {
>  		pr_warn("md/raid1:%s: raid level not set to mirroring (%d)\n",
> @@ -3141,8 +3138,6 @@ static int raid1_run(struct mddev *mddev)
>  			continue;
>  		disk_stack_limits(mddev->gendisk, rdev->bdev,
>  				  rdev->data_offset << 9);
> -		if (bdev_max_discard_sectors(rdev->bdev))
> -			discard_supported = true;
>  	}
>  
>  	mddev->degraded = 0;
> @@ -3179,15 +3174,6 @@ static int raid1_run(struct mddev *mddev)
>  
>  	md_set_array_sectors(mddev, raid1_size(mddev, 0, 0));
>  
> -	if (mddev->queue) {
> -		if (discard_supported)
> -			blk_queue_flag_set(QUEUE_FLAG_DISCARD,
> -						mddev->queue);
> -		else
> -			blk_queue_flag_clear(QUEUE_FLAG_DISCARD,
> -						  mddev->queue);
> -	}
> -
>  	ret = md_integrity_register(mddev);
>  	if (ret) {
>  		md_unregister_thread(&mddev->thread);
> diff --git a/drivers/md/raid10.c b/drivers/md/raid10.c
> index eaa86c6a35a55..36a460015cf58 100644
> --- a/drivers/md/raid10.c
> +++ b/drivers/md/raid10.c
> @@ -2144,8 +2144,6 @@ static int raid10_add_disk(struct mddev *mddev, struct md_rdev *rdev)
>  		rcu_assign_pointer(p->rdev, rdev);
>  		break;
>  	}
> -	if (mddev->queue && bdev_max_discard_sectors(rdev->bdev))
> -		blk_queue_flag_set(QUEUE_FLAG_DISCARD, mddev->queue);
>  
>  	print_conf(conf);
>  	return err;
> @@ -4069,7 +4067,6 @@ static int raid10_run(struct mddev *mddev)
>  	sector_t size;
>  	sector_t min_offset_diff = 0;
>  	int first = 1;
> -	bool discard_supported = false;
>  
>  	if (mddev_init_writes_pending(mddev) < 0)
>  		return -ENOMEM;
> @@ -4140,20 +4137,9 @@ static int raid10_run(struct mddev *mddev)
>  					  rdev->data_offset << 9);
>  
>  		disk->head_position = 0;
> -
> -		if (bdev_max_discard_sectors(rdev->bdev))
> -			discard_supported = true;
>  		first = 0;
>  	}
>  
> -	if (mddev->queue) {
> -		if (discard_supported)
> -			blk_queue_flag_set(QUEUE_FLAG_DISCARD,
> -						mddev->queue);
> -		else
> -			blk_queue_flag_clear(QUEUE_FLAG_DISCARD,
> -						  mddev->queue);
> -	}
>  	/* need to check that every block has at least one working mirror */
>  	if (!enough(conf, -1)) {
>  		pr_err("md/raid10:%s: not enough operational mirrors.\n",
> diff --git a/drivers/md/raid5.c b/drivers/md/raid5.c
> index 0bbae0e638666..59f91e392a2ae 100644
> --- a/drivers/md/raid5.c
> +++ b/drivers/md/raid5.c
> @@ -7776,14 +7776,10 @@ static int raid5_run(struct mddev *mddev)
>  		 * A better idea might be to turn DISCARD into WRITE_ZEROES
>  		 * requests, as that is required to be safe.
>  		 */
> -		if (devices_handle_discard_safely &&
> -		    mddev->queue->limits.max_discard_sectors >= (stripe >> 9) &&
> -		    mddev->queue->limits.discard_granularity >= stripe)
> -			blk_queue_flag_set(QUEUE_FLAG_DISCARD,
> -						mddev->queue);
> -		else
> -			blk_queue_flag_clear(QUEUE_FLAG_DISCARD,
> -						mddev->queue);
> +		if (!devices_handle_discard_safely ||
> +		    mddev->queue->limits.max_discard_sectors < (stripe >> 9) ||
> +		    mddev->queue->limits.discard_granularity < stripe)
> +			blk_queue_max_discard_sectors(mddev->queue, 0);
>  
>  		blk_queue_max_hw_sectors(mddev->queue, UINT_MAX);
>  	}
> diff --git a/drivers/mmc/core/queue.c b/drivers/mmc/core/queue.c
> index c69b2d9df6f16..cac6315010a3d 100644
> --- a/drivers/mmc/core/queue.c
> +++ b/drivers/mmc/core/queue.c
> @@ -183,7 +183,6 @@ static void mmc_queue_setup_discard(struct request_queue *q,
>  	if (!max_discard)
>  		return;
>  
> -	blk_queue_flag_set(QUEUE_FLAG_DISCARD, q);
>  	blk_queue_max_discard_sectors(q, max_discard);
>  	q->limits.discard_granularity = card->pref_erase << 9;
>  	/* granularity must not be greater than max. discard */
> diff --git a/drivers/mtd/mtd_blkdevs.c b/drivers/mtd/mtd_blkdevs.c
> index 64d2b093f114b..f731721114655 100644
> --- a/drivers/mtd/mtd_blkdevs.c
> +++ b/drivers/mtd/mtd_blkdevs.c
> @@ -377,7 +377,6 @@ int add_mtd_blktrans_dev(struct mtd_blktrans_dev *new)
>  	blk_queue_flag_clear(QUEUE_FLAG_ADD_RANDOM, new->rq);
>  
>  	if (tr->discard) {
> -		blk_queue_flag_set(QUEUE_FLAG_DISCARD, new->rq);
>  		blk_queue_max_discard_sectors(new->rq, UINT_MAX);
>  		new->rq->limits.discard_granularity = tr->blksize;
>  	}
> diff --git a/drivers/nvme/host/core.c b/drivers/nvme/host/core.c
> index efb85c6d8e2d5..7e07dd69262a7 100644
> --- a/drivers/nvme/host/core.c
> +++ b/drivers/nvme/host/core.c
> @@ -1607,10 +1607,8 @@ static void nvme_config_discard(struct gendisk *disk, struct nvme_ns *ns)
>  	struct request_queue *queue = disk->queue;
>  	u32 size = queue_logical_block_size(queue);
>  
> -	if (ctrl->max_discard_sectors == 0) {
> -		blk_queue_flag_clear(QUEUE_FLAG_DISCARD, queue);
> +	if (ctrl->max_discard_sectors == 0)
>  		return;
> -	}
>  
>  	BUILD_BUG_ON(PAGE_SIZE / sizeof(struct nvme_dsm_range) <
>  			NVME_DSM_MAX_RANGES);
> @@ -1619,7 +1617,7 @@ static void nvme_config_discard(struct gendisk *disk, struct nvme_ns *ns)
>  	queue->limits.discard_granularity = size;
>  
>  	/* If discard is already enabled, don't reset queue limits */
> -	if (blk_queue_flag_test_and_set(QUEUE_FLAG_DISCARD, queue))
> +	if (queue->limits.max_discard_sectors)
>  		return;
>  
>  	blk_queue_max_discard_sectors(queue, ctrl->max_discard_sectors);
> diff --git a/drivers/s390/block/dasd_fba.c b/drivers/s390/block/dasd_fba.c
> index e084f4deddddd..8bd5665db9198 100644
> --- a/drivers/s390/block/dasd_fba.c
> +++ b/drivers/s390/block/dasd_fba.c
> @@ -791,7 +791,6 @@ static void dasd_fba_setup_blk_queue(struct dasd_block *block)
>  
>  	blk_queue_max_discard_sectors(q, max_discard_sectors);
>  	blk_queue_max_write_zeroes_sectors(q, max_discard_sectors);
> -	blk_queue_flag_set(QUEUE_FLAG_DISCARD, q);
>  }
>  
>  static int dasd_fba_pe_handler(struct dasd_device *device,
> diff --git a/drivers/scsi/sd.c b/drivers/scsi/sd.c
> index a390679cf4584..444479657b7fd 100644
> --- a/drivers/scsi/sd.c
> +++ b/drivers/scsi/sd.c
> @@ -797,7 +797,6 @@ static void sd_config_discard(struct scsi_disk *sdkp, unsigned int mode)
>  	case SD_LBP_FULL:
>  	case SD_LBP_DISABLE:
>  		blk_queue_max_discard_sectors(q, 0);
> -		blk_queue_flag_clear(QUEUE_FLAG_DISCARD, q);
>  		return;
>  
>  	case SD_LBP_UNMAP:
> @@ -830,7 +829,6 @@ static void sd_config_discard(struct scsi_disk *sdkp, unsigned int mode)
>  	}
>  
>  	blk_queue_max_discard_sectors(q, max_blocks * (logical_block_size >> 9));
> -	blk_queue_flag_set(QUEUE_FLAG_DISCARD, q);
>  }
>  
>  static blk_status_t sd_setup_unmap_cmnd(struct scsi_cmnd *cmd)
> diff --git a/include/linux/blkdev.h b/include/linux/blkdev.h
> index 7b9c0cf95d2d5..f1cf557ea20ef 100644
> --- a/include/linux/blkdev.h
> +++ b/include/linux/blkdev.h
> @@ -540,7 +540,6 @@ struct request_queue {
>  #define QUEUE_FLAG_NONROT	6	/* non-rotational device (SSD) */
>  #define QUEUE_FLAG_VIRT		QUEUE_FLAG_NONROT /* paravirt device */
>  #define QUEUE_FLAG_IO_STAT	7	/* do disk/partitions IO accounting */
> -#define QUEUE_FLAG_DISCARD	8	/* supports DISCARD */
>  #define QUEUE_FLAG_NOXMERGES	9	/* No extended merges */
>  #define QUEUE_FLAG_ADD_RANDOM	10	/* Contributes to random pool */
>  #define QUEUE_FLAG_SECERASE	11	/* supports secure erase */
> @@ -582,7 +581,6 @@ bool blk_queue_flag_test_and_set(unsigned int flag, struct request_queue *q);
>  	test_bit(QUEUE_FLAG_STABLE_WRITES, &(q)->queue_flags)
>  #define blk_queue_io_stat(q)	test_bit(QUEUE_FLAG_IO_STAT, &(q)->queue_flags)
>  #define blk_queue_add_random(q)	test_bit(QUEUE_FLAG_ADD_RANDOM, &(q)->queue_flags)
> -#define blk_queue_discard(q)	test_bit(QUEUE_FLAG_DISCARD, &(q)->queue_flags)
>  #define blk_queue_zone_resetall(q)	\
>  	test_bit(QUEUE_FLAG_ZONE_RESETALL, &(q)->queue_flags)
>  #define blk_queue_secure_erase(q) \

  reply	other threads:[~2022-04-06 13:08 UTC|newest]

Thread overview: 395+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-04-06  6:04 use block_device based APIs in block layer consumers Christoph Hellwig
2022-04-06  6:04 ` [Ocfs2-devel] " Christoph Hellwig via Ocfs2-devel
2022-04-06  6:04 ` [dm-devel] " Christoph Hellwig
2022-04-06  6:04 ` Christoph Hellwig
2022-04-06  6:04 ` Christoph Hellwig
2022-04-06  6:04 ` [f2fs-dev] " Christoph Hellwig
2022-04-06  6:04 ` [PATCH 01/27] target: remove an incorrect unmap zeroes data deduction Christoph Hellwig
2022-04-06  6:04   ` [Ocfs2-devel] " Christoph Hellwig via Ocfs2-devel
2022-04-06  6:04   ` [dm-devel] " Christoph Hellwig
2022-04-06  6:04   ` Christoph Hellwig
2022-04-06  6:04   ` Christoph Hellwig
2022-04-06  6:04   ` [f2fs-dev] " Christoph Hellwig
2022-04-07  3:05   ` [dm-devel] " Martin K. Petersen
2022-04-07  3:05     ` Martin K. Petersen
2022-04-07  3:05     ` [f2fs-dev] " Martin K. Petersen
2022-04-07  3:05     ` Martin K. Petersen
2022-04-07  3:05     ` Martin K. Petersen
2022-04-07  3:05     ` [Ocfs2-devel] " Martin K. Petersen via Ocfs2-devel
2022-04-07  3:06   ` Martin K. Petersen
2022-04-07  3:06     ` Martin K. Petersen
2022-04-07  3:06     ` Martin K. Petersen
2022-04-07  3:06     ` Martin K. Petersen
2022-04-07  3:06     ` [f2fs-dev] " Martin K. Petersen
2022-04-07  3:06     ` [Ocfs2-devel] " Martin K. Petersen via Ocfs2-devel
2022-04-06  6:04 ` [f2fs-dev] [PATCH 02/27] target: pass a block_device to target_configure_unmap_from_queue Christoph Hellwig
2022-04-06  6:04   ` Christoph Hellwig
2022-04-06  6:04   ` [Ocfs2-devel] " Christoph Hellwig via Ocfs2-devel
2022-04-06  6:04   ` Christoph Hellwig
2022-04-06  6:04   ` [dm-devel] " Christoph Hellwig
2022-04-06  6:04   ` Christoph Hellwig
2022-04-07  3:07   ` Martin K. Petersen
2022-04-07  3:07     ` [dm-devel] " Martin K. Petersen
2022-04-07  3:07     ` Martin K. Petersen
2022-04-07  3:07     ` Martin K. Petersen
2022-04-07  3:07     ` [f2fs-dev] " Martin K. Petersen
2022-04-07  3:07     ` [Ocfs2-devel] " Martin K. Petersen via Ocfs2-devel
2022-04-06  6:04 ` [f2fs-dev] [PATCH 03/27] target: fix discard alignment on partitions Christoph Hellwig
2022-04-06  6:04   ` Christoph Hellwig
2022-04-06  6:04   ` [Ocfs2-devel] " Christoph Hellwig via Ocfs2-devel
2022-04-06  6:04   ` Christoph Hellwig
2022-04-06  6:04   ` [dm-devel] " Christoph Hellwig
2022-04-06  6:04   ` Christoph Hellwig
2022-04-07  3:14   ` [Ocfs2-devel] [dm-devel] " Martin K. Petersen via Ocfs2-devel
2022-04-07  3:14     ` Martin K. Petersen
2022-04-07  3:14     ` Martin K. Petersen
2022-04-07  3:14     ` [f2fs-dev] " Martin K. Petersen
2022-04-07  3:14     ` Martin K. Petersen
2022-04-07  3:14     ` Martin K. Petersen
2022-04-06  6:04 ` [f2fs-dev] [PATCH 04/27] drbd: remove assign_p_sizes_qlim Christoph Hellwig
2022-04-06  6:04   ` Christoph Hellwig
2022-04-06  6:04   ` [Ocfs2-devel] " Christoph Hellwig via Ocfs2-devel
2022-04-06  6:04   ` Christoph Hellwig
2022-04-06  6:04   ` [dm-devel] " Christoph Hellwig
2022-04-06  6:04   ` Christoph Hellwig
2022-04-06  9:09   ` [Drbd-dev] " Christoph Böhmwalder
2022-04-06  6:04 ` [f2fs-dev] [PATCH 05/27] drbd: use bdev based limit helpers in drbd_send_sizes Christoph Hellwig
2022-04-06  6:04   ` Christoph Hellwig
2022-04-06  6:04   ` [Ocfs2-devel] " Christoph Hellwig via Ocfs2-devel
2022-04-06  6:04   ` Christoph Hellwig
2022-04-06  6:04   ` [dm-devel] " Christoph Hellwig
2022-04-06  6:04   ` Christoph Hellwig
2022-04-06  9:20   ` [Drbd-dev] " Christoph Böhmwalder
2022-04-06  6:04 ` [f2fs-dev] [PATCH 06/27] drbd: cleanup decide_on_discard_support Christoph Hellwig
2022-04-06  6:04   ` Christoph Hellwig
2022-04-06  6:04   ` [dm-devel] " Christoph Hellwig
2022-04-06  6:04   ` [Ocfs2-devel] " Christoph Hellwig via Ocfs2-devel
2022-04-06  6:04   ` Christoph Hellwig
2022-04-06  6:04   ` Christoph Hellwig
2022-04-06  9:28   ` [Drbd-dev] " Christoph Böhmwalder
2022-04-06  6:04 ` [f2fs-dev] [PATCH 07/27] btrfs: use bdev_max_active_zones instead of open coding it Christoph Hellwig
2022-04-06  6:04   ` Christoph Hellwig
2022-04-06  6:04   ` [Ocfs2-devel] " Christoph Hellwig via Ocfs2-devel
2022-04-06  6:04   ` Christoph Hellwig
2022-04-06  6:04   ` [dm-devel] " Christoph Hellwig
2022-04-06  6:04   ` Christoph Hellwig
2022-04-06  9:56   ` [Ocfs2-devel] " Johannes Thumshirn via Ocfs2-devel
2022-04-06  9:56     ` [f2fs-dev] " Johannes Thumshirn via Linux-f2fs-devel
2022-04-06  9:56     ` Johannes Thumshirn
2022-04-06  9:56     ` [dm-devel] " Johannes Thumshirn
2022-04-06  9:56     ` Johannes Thumshirn
2022-04-07 15:20   ` David Sterba
2022-04-07 15:20     ` [dm-devel] " David Sterba
2022-04-07 15:20     ` [f2fs-dev] " David Sterba
2022-04-07 15:20     ` David Sterba
2022-04-07 15:20     ` David Sterba
2022-04-07 15:20     ` [Ocfs2-devel] " David Sterba via Ocfs2-devel
2022-04-07 15:26     ` Christoph Hellwig
2022-04-07 15:26       ` Christoph Hellwig
2022-04-07 15:26       ` [dm-devel] " Christoph Hellwig
2022-04-07 15:26       ` [f2fs-dev] " Christoph Hellwig
2022-04-07 15:26       ` Christoph Hellwig
2022-04-06  6:04 ` [PATCH 08/27] ntfs3: use bdev_logical_block_size " Christoph Hellwig
2022-04-06  6:04   ` Christoph Hellwig
2022-04-06  6:04   ` [dm-devel] " Christoph Hellwig
2022-04-06  6:04   ` [f2fs-dev] " Christoph Hellwig
2022-04-06  6:04   ` [Ocfs2-devel] " Christoph Hellwig via Ocfs2-devel
2022-04-06  6:04   ` Christoph Hellwig
2022-04-06  6:04 ` [PATCH 09/27] mm: use bdev_is_zoned in claim_swapfile Christoph Hellwig
2022-04-06  6:04   ` Christoph Hellwig
2022-04-06  6:04   ` [dm-devel] " Christoph Hellwig
2022-04-06  6:04   ` [f2fs-dev] " Christoph Hellwig
2022-04-06  6:04   ` [Ocfs2-devel] " Christoph Hellwig via Ocfs2-devel
2022-04-06  6:04   ` Christoph Hellwig
2022-04-06  6:04 ` [PATCH 10/27] block: add a bdev_nonrot helper Christoph Hellwig
2022-04-06  6:04   ` Christoph Hellwig
2022-04-06  6:04   ` [dm-devel] " Christoph Hellwig
2022-04-06  6:04   ` Christoph Hellwig
2022-04-06  6:04   ` [f2fs-dev] " Christoph Hellwig
2022-04-06  6:04   ` [Ocfs2-devel] " Christoph Hellwig via Ocfs2-devel
2022-04-07  3:16   ` Martin K. Petersen via Ocfs2-devel
2022-04-07  3:16     ` Martin K. Petersen
2022-04-07  3:16     ` [dm-devel] " Martin K. Petersen
2022-04-07  3:16     ` [f2fs-dev] " Martin K. Petersen
2022-04-07  3:16     ` Martin K. Petersen
2022-04-07  3:16     ` Martin K. Petersen
2022-04-07 15:21   ` David Sterba
2022-04-07 15:21     ` [dm-devel] " David Sterba
2022-04-07 15:21     ` [f2fs-dev] " David Sterba
2022-04-07 15:21     ` David Sterba
2022-04-07 15:21     ` David Sterba
2022-04-07 15:21     ` [Ocfs2-devel] " David Sterba via Ocfs2-devel
2022-04-06  6:05 ` [PATCH 11/27] block: add a bdev_write_cache helper Christoph Hellwig
2022-04-06  6:05   ` Christoph Hellwig
2022-04-06  6:05   ` [dm-devel] " Christoph Hellwig
2022-04-06  6:05   ` Christoph Hellwig
2022-04-06  6:05   ` [f2fs-dev] " Christoph Hellwig
2022-04-06  6:05   ` [Ocfs2-devel] " Christoph Hellwig via Ocfs2-devel
2022-04-07  3:17   ` Martin K. Petersen via Ocfs2-devel
2022-04-07  3:17     ` Martin K. Petersen
2022-04-07  3:17     ` [dm-devel] " Martin K. Petersen
2022-04-07  3:17     ` Martin K. Petersen
2022-04-07  3:17     ` Martin K. Petersen
2022-04-07  3:17     ` [f2fs-dev] " Martin K. Petersen
2022-04-07 15:21   ` David Sterba
2022-04-07 15:21     ` [dm-devel] " David Sterba
2022-04-07 15:21     ` [f2fs-dev] " David Sterba
2022-04-07 15:21     ` David Sterba
2022-04-07 15:21     ` David Sterba
2022-04-07 15:21     ` [Ocfs2-devel] " David Sterba via Ocfs2-devel
2022-04-06  6:05 ` [PATCH 12/27] block: add a bdev_fua helper Christoph Hellwig
2022-04-06  6:05   ` Christoph Hellwig
2022-04-06  6:05   ` [dm-devel] " Christoph Hellwig
2022-04-06  6:05   ` Christoph Hellwig
2022-04-06  6:05   ` [f2fs-dev] " Christoph Hellwig
2022-04-06  6:05   ` [Ocfs2-devel] " Christoph Hellwig via Ocfs2-devel
2022-04-07  3:17   ` Martin K. Petersen via Ocfs2-devel
2022-04-07  3:17     ` Martin K. Petersen
2022-04-07  3:17     ` [dm-devel] " Martin K. Petersen
2022-04-07  3:17     ` [f2fs-dev] " Martin K. Petersen
2022-04-07  3:17     ` Martin K. Petersen
2022-04-07  3:17     ` Martin K. Petersen
2022-04-06  6:05 ` [PATCH 13/27] block: add a bdev_stable_writes helper Christoph Hellwig
2022-04-06  6:05   ` Christoph Hellwig
2022-04-06  6:05   ` [dm-devel] " Christoph Hellwig
2022-04-06  6:05   ` Christoph Hellwig
2022-04-06  6:05   ` [f2fs-dev] " Christoph Hellwig
2022-04-06  6:05   ` [Ocfs2-devel] " Christoph Hellwig via Ocfs2-devel
2022-04-07  3:18   ` Martin K. Petersen
2022-04-07  3:18     ` Martin K. Petersen
2022-04-07  3:18     ` [dm-devel] " Martin K. Petersen
2022-04-07  3:18     ` Martin K. Petersen
2022-04-07  3:18     ` [f2fs-dev] " Martin K. Petersen
2022-04-07  3:18     ` Martin K. Petersen via Ocfs2-devel
2022-04-06  6:05 ` [PATCH 14/27] block: add a bdev_max_zone_append_sectors helper Christoph Hellwig
2022-04-06  6:05   ` Christoph Hellwig
2022-04-06  6:05   ` [dm-devel] " Christoph Hellwig
2022-04-06  6:05   ` Christoph Hellwig
2022-04-06  6:05   ` [f2fs-dev] " Christoph Hellwig
2022-04-06  6:05   ` [Ocfs2-devel] " Christoph Hellwig via Ocfs2-devel
2022-04-06  8:52   ` Damien Le Moal
2022-04-06  8:52     ` [f2fs-dev] " Damien Le Moal via Linux-f2fs-devel
2022-04-06  8:52     ` [dm-devel] " Damien Le Moal
2022-04-06  8:52     ` Damien Le Moal
2022-04-06  9:58   ` [Ocfs2-devel] " Johannes Thumshirn via Ocfs2-devel
2022-04-06  9:58     ` [f2fs-dev] " Johannes Thumshirn via Linux-f2fs-devel
2022-04-06  9:58     ` Johannes Thumshirn
2022-04-06  9:58     ` [dm-devel] " Johannes Thumshirn
2022-04-06  9:58     ` Johannes Thumshirn
2022-04-07  3:15   ` [Ocfs2-devel] [dm-devel] " Martin K. Petersen via Ocfs2-devel
2022-04-07  3:15     ` Martin K. Petersen
2022-04-07  3:15     ` Martin K. Petersen
2022-04-07  3:15     ` Martin K. Petersen
2022-04-07  3:15     ` [f2fs-dev] " Martin K. Petersen
2022-04-07  3:15     ` Martin K. Petersen
2022-04-06  6:05 ` [PATCH 15/27] block: use bdev_alignment_offset in part_alignment_offset_show Christoph Hellwig
2022-04-06  6:05   ` Christoph Hellwig
2022-04-06  6:05   ` [dm-devel] " Christoph Hellwig
2022-04-06  6:05   ` Christoph Hellwig
2022-04-06  6:05   ` [f2fs-dev] " Christoph Hellwig
2022-04-06  6:05   ` [Ocfs2-devel] " Christoph Hellwig via Ocfs2-devel
2022-04-07  3:19   ` Martin K. Petersen via Ocfs2-devel
2022-04-07  3:19     ` Martin K. Petersen
2022-04-07  3:19     ` [dm-devel] " Martin K. Petersen
2022-04-07  3:19     ` [f2fs-dev] " Martin K. Petersen
2022-04-07  3:19     ` Martin K. Petersen
2022-04-07  3:19     ` Martin K. Petersen
2022-04-06  6:05 ` [PATCH 16/27] drbd: use bdev_alignment_offset instead of queue_alignment_offset Christoph Hellwig
2022-04-06  6:05   ` Christoph Hellwig
2022-04-06  6:05   ` [dm-devel] " Christoph Hellwig
2022-04-06  6:05   ` Christoph Hellwig
2022-04-06  6:05   ` [f2fs-dev] " Christoph Hellwig
2022-04-06  6:05   ` [Ocfs2-devel] " Christoph Hellwig via Ocfs2-devel
2022-04-06  9:32   ` [Drbd-dev] " Christoph Böhmwalder
2022-04-06  6:05 ` [PATCH 17/27] block: use bdev_alignment_offset in disk_alignment_offset_show Christoph Hellwig
2022-04-06  6:05   ` Christoph Hellwig
2022-04-06  6:05   ` [dm-devel] " Christoph Hellwig
2022-04-06  6:05   ` Christoph Hellwig
2022-04-06  6:05   ` [f2fs-dev] " Christoph Hellwig
2022-04-06  6:05   ` [Ocfs2-devel] " Christoph Hellwig via Ocfs2-devel
2022-04-07  3:20   ` Martin K. Petersen via Ocfs2-devel
2022-04-07  3:20     ` Martin K. Petersen
2022-04-07  3:20     ` [dm-devel] " Martin K. Petersen
2022-04-07  3:20     ` Martin K. Petersen
2022-04-07  3:20     ` [f2fs-dev] " Martin K. Petersen
2022-04-07  3:20     ` Martin K. Petersen
2022-04-06  6:05 ` [PATCH 18/27] block: move bdev_alignment_offset and queue_limit_alignment_offset out of line Christoph Hellwig
2022-04-06  6:05   ` Christoph Hellwig
2022-04-06  6:05   ` [dm-devel] " Christoph Hellwig
2022-04-06  6:05   ` Christoph Hellwig
2022-04-06  6:05   ` [f2fs-dev] " Christoph Hellwig
2022-04-06  6:05   ` [Ocfs2-devel] " Christoph Hellwig via Ocfs2-devel
2022-04-07  3:21   ` Martin K. Petersen via Ocfs2-devel
2022-04-07  3:21     ` Martin K. Petersen
2022-04-07  3:21     ` [dm-devel] " Martin K. Petersen
2022-04-07  3:21     ` Martin K. Petersen
2022-04-07  3:21     ` [f2fs-dev] " Martin K. Petersen
2022-04-07  3:21     ` Martin K. Petersen
2022-04-06  6:05 ` [PATCH 19/27] block: remove queue_discard_alignment Christoph Hellwig
2022-04-06  6:05   ` Christoph Hellwig
2022-04-06  6:05   ` [dm-devel] " Christoph Hellwig
2022-04-06  6:05   ` Christoph Hellwig
2022-04-06  6:05   ` [f2fs-dev] " Christoph Hellwig
2022-04-06  6:05   ` [Ocfs2-devel] " Christoph Hellwig via Ocfs2-devel
2022-04-07  3:21   ` Martin K. Petersen via Ocfs2-devel
2022-04-07  3:21     ` Martin K. Petersen
2022-04-07  3:21     ` [dm-devel] " Martin K. Petersen
2022-04-07  3:21     ` [f2fs-dev] " Martin K. Petersen
2022-04-07  3:21     ` Martin K. Petersen
2022-04-07  3:21     ` Martin K. Petersen
2022-04-06  6:05 ` [PATCH 20/27] block: use bdev_discard_alignment in part_discard_alignment_show Christoph Hellwig
2022-04-06  6:05   ` Christoph Hellwig
2022-04-06  6:05   ` [dm-devel] " Christoph Hellwig
2022-04-06  6:05   ` [Ocfs2-devel] " Christoph Hellwig via Ocfs2-devel
2022-04-06  6:05   ` Christoph Hellwig
2022-04-06  6:05   ` [f2fs-dev] " Christoph Hellwig
2022-04-07  3:22   ` [Ocfs2-devel] " Martin K. Petersen via Ocfs2-devel
2022-04-07  3:22     ` Martin K. Petersen
2022-04-07  3:22     ` [dm-devel] " Martin K. Petersen
2022-04-07  3:22     ` Martin K. Petersen
2022-04-07  3:22     ` [f2fs-dev] " Martin K. Petersen
2022-04-07  3:22     ` Martin K. Petersen
2022-04-06  6:05 ` [PATCH 21/27] block: move {bdev,queue_limit}_discard_alignment out of line Christoph Hellwig
2022-04-06  6:05   ` [PATCH 21/27] block: move {bdev, queue_limit}_discard_alignment " Christoph Hellwig
2022-04-06  6:05   ` Christoph Hellwig
2022-04-06  6:05   ` [dm-devel] " Christoph Hellwig
2022-04-06  6:05   ` Christoph Hellwig
2022-04-06  6:05   ` [f2fs-dev] " Christoph Hellwig
2022-04-06  6:05   ` [Ocfs2-devel] " Christoph Hellwig via Ocfs2-devel
2022-04-07  3:27   ` Martin K. Petersen
2022-04-07  3:27     ` Martin K. Petersen
2022-04-07  3:27     ` [dm-devel] " Martin K. Petersen
2022-04-07  3:27     ` Martin K. Petersen via Ocfs2-devel
2022-04-07  3:27     ` Martin K. Petersen
2022-04-07  3:27     ` [f2fs-dev] " Martin K. Petersen
2022-04-06  6:05 ` [PATCH 22/27] block: refactor discard bio size limiting Christoph Hellwig
2022-04-06  6:05   ` Christoph Hellwig
2022-04-06  6:05   ` [dm-devel] " Christoph Hellwig
2022-04-06  6:05   ` [f2fs-dev] " Christoph Hellwig
2022-04-06  6:05   ` [Ocfs2-devel] " Christoph Hellwig via Ocfs2-devel
2022-04-06  6:05   ` Christoph Hellwig
2022-04-07  3:29   ` [Ocfs2-devel] " Martin K. Petersen
2022-04-07  3:29     ` Martin K. Petersen
2022-04-07  3:29     ` [dm-devel] " Martin K. Petersen
2022-04-07  3:29     ` Martin K. Petersen
2022-04-07  3:29     ` [f2fs-dev] " Martin K. Petersen
2022-04-07  3:29     ` Martin K. Petersen via Ocfs2-devel
2022-04-07  8:03   ` Coly Li
2022-04-07  8:03     ` Coly Li
2022-04-07  8:03     ` [dm-devel] " Coly Li
2022-04-07  8:03     ` [f2fs-dev] " Coly Li
2022-04-06  6:05 ` [Ocfs2-devel] [PATCH 23/27] block: add a bdev_max_discard_sectors helper Christoph Hellwig via Ocfs2-devel
2022-04-06  6:05   ` Christoph Hellwig
2022-04-06  6:05   ` [dm-devel] " Christoph Hellwig
2022-04-06  6:05   ` Christoph Hellwig
2022-04-06  6:05   ` Christoph Hellwig
2022-04-06  6:05   ` [f2fs-dev] " Christoph Hellwig
2022-04-06  9:36   ` [Drbd-dev] " Christoph Böhmwalder
2022-04-06  9:53   ` [Cluster-devel] " Andreas Gruenbacher
2022-04-06  9:53     ` [dm-devel] " Andreas Gruenbacher
2022-04-06  9:53     ` Andreas Gruenbacher
2022-04-06  9:53     ` [f2fs-dev] " Andreas Gruenbacher
2022-04-06  9:53     ` [Ocfs2-devel] " Andreas Gruenbacher via Ocfs2-devel
2022-04-06 17:28   ` Ryusuke Konishi
2022-04-06 17:28     ` [dm-devel] " Ryusuke Konishi
2022-04-06 17:28     ` Ryusuke Konishi
2022-04-06 17:28     ` [f2fs-dev] " Ryusuke Konishi
2022-04-07  3:30   ` [Ocfs2-devel] " Martin K. Petersen via Ocfs2-devel
2022-04-07  3:30     ` Martin K. Petersen
2022-04-07  3:30     ` [dm-devel] " Martin K. Petersen
2022-04-07  3:30     ` Martin K. Petersen
2022-04-07  3:30     ` [f2fs-dev] " Martin K. Petersen
2022-04-07  3:30     ` Martin K. Petersen
2022-04-07  5:16   ` Coly Li
2022-04-07  5:16     ` [dm-devel] " Coly Li
2022-04-07  5:16     ` Coly Li
2022-04-07  5:16     ` [f2fs-dev] " Coly Li
2022-04-07 15:22   ` David Sterba
2022-04-07 15:22     ` [dm-devel] " David Sterba
2022-04-07 15:22     ` [f2fs-dev] " David Sterba
2022-04-07 15:22     ` David Sterba
2022-04-07 15:22     ` [Ocfs2-devel] " David Sterba via Ocfs2-devel
2022-04-07 15:22     ` David Sterba
2022-04-06  6:05 ` [PATCH 24/27] block: add a bdev_discard_granularity helper Christoph Hellwig
2022-04-06  6:05   ` Christoph Hellwig
2022-04-06  6:05   ` [dm-devel] " Christoph Hellwig
2022-04-06  6:05   ` Christoph Hellwig
2022-04-06  6:05   ` [f2fs-dev] " Christoph Hellwig
2022-04-06  6:05   ` [Ocfs2-devel] " Christoph Hellwig via Ocfs2-devel
2022-04-06  9:40   ` [Drbd-dev] " Christoph Böhmwalder
2022-04-06 18:26   ` Ryusuke Konishi
2022-04-06 18:26     ` [dm-devel] " Ryusuke Konishi
2022-04-06 18:26     ` Ryusuke Konishi
2022-04-06 18:26     ` [f2fs-dev] " Ryusuke Konishi
2022-04-07  3:32   ` [Ocfs2-devel] " Martin K. Petersen via Ocfs2-devel
2022-04-07  3:32     ` Martin K. Petersen
2022-04-07  3:32     ` [dm-devel] " Martin K. Petersen
2022-04-07  3:32     ` Martin K. Petersen
2022-04-07  3:32     ` Martin K. Petersen
2022-04-07  3:32     ` [f2fs-dev] " Martin K. Petersen
2022-04-07 15:26   ` David Sterba via Ocfs2-devel
2022-04-07 15:26     ` [dm-devel] " David Sterba
2022-04-07 15:26     ` David Sterba
2022-04-07 15:26     ` [f2fs-dev] " David Sterba
2022-04-07 15:26     ` David Sterba
2022-04-07 15:26     ` David Sterba
2022-04-06  6:05 ` [Ocfs2-devel] [PATCH 25/27] block: remove QUEUE_FLAG_DISCARD Christoph Hellwig via Ocfs2-devel
2022-04-06  6:05   ` Christoph Hellwig
2022-04-06  6:05   ` [dm-devel] " Christoph Hellwig
2022-04-06  6:05   ` Christoph Hellwig
2022-04-06  6:05   ` Christoph Hellwig
2022-04-06  6:05   ` [f2fs-dev] " Christoph Hellwig
2022-04-06  9:43   ` Christoph Böhmwalder [this message]
2022-04-07  3:34   ` [Ocfs2-devel] " Martin K. Petersen via Ocfs2-devel
2022-04-07  3:34     ` Martin K. Petersen
2022-04-07  3:34     ` [dm-devel] " Martin K. Petersen
2022-04-07  3:34     ` [f2fs-dev] " Martin K. Petersen
2022-04-07  3:34     ` Martin K. Petersen
2022-04-07  3:34     ` Martin K. Petersen
2022-04-07  5:18   ` Coly Li
2022-04-07  5:18     ` [dm-devel] " Coly Li
2022-04-07  5:18     ` Coly Li
2022-04-07  5:18     ` [f2fs-dev] " Coly Li
2022-04-06  6:05 ` [Ocfs2-devel] [PATCH 26/27] block: uncouple REQ_OP_SECURE_ERASE from REQ_OP_DISCARD Christoph Hellwig via Ocfs2-devel
2022-04-06  6:05   ` Christoph Hellwig
2022-04-06  6:05   ` [dm-devel] " Christoph Hellwig
2022-04-06  6:05   ` Christoph Hellwig
2022-04-06  6:05   ` Christoph Hellwig
2022-04-06  6:05   ` [f2fs-dev] " Christoph Hellwig
2022-04-06  9:45   ` [Drbd-dev] " Christoph Böhmwalder
2022-04-07  3:36   ` [Ocfs2-devel] " Martin K. Petersen via Ocfs2-devel
2022-04-07  3:36     ` Martin K. Petersen
2022-04-07  3:36     ` [dm-devel] " Martin K. Petersen
2022-04-07  3:36     ` Martin K. Petersen
2022-04-07  3:36     ` [f2fs-dev] " Martin K. Petersen
2022-04-07  3:36     ` Martin K. Petersen
2022-04-07  6:09   ` Coly Li
2022-04-07  6:09     ` [dm-devel] " Coly Li
2022-04-07  6:09     ` Coly Li
2022-04-07  6:09     ` [f2fs-dev] " Coly Li
2022-04-07  7:01   ` Ryusuke Konishi
2022-04-07  7:01     ` [dm-devel] " Ryusuke Konishi
2022-04-07  7:01     ` Ryusuke Konishi
2022-04-07  7:01     ` [f2fs-dev] " Ryusuke Konishi
2022-04-07 15:26   ` [Ocfs2-devel] " David Sterba via Ocfs2-devel
2022-04-07 15:26     ` [dm-devel] " David Sterba
2022-04-07 15:26     ` David Sterba
2022-04-07 15:26     ` [f2fs-dev] " David Sterba
2022-04-07 15:26     ` David Sterba
2022-04-07 15:26     ` David Sterba
2022-04-11 18:24   ` [f2fs-dev] " Jaegeuk Kim
2022-04-11 18:24     ` [dm-devel] " Jaegeuk Kim
2022-04-11 18:24     ` Jaegeuk Kim
2022-04-11 18:24     ` Jaegeuk Kim
2022-04-11 18:24     ` [Ocfs2-devel] " Jaegeuk Kim via Ocfs2-devel
2022-04-06  6:05 ` [Ocfs2-devel] [PATCH 27/27] direct-io: remove random prefetches Christoph Hellwig via Ocfs2-devel
2022-04-06  6:05   ` Christoph Hellwig
2022-04-06  6:05   ` [dm-devel] " Christoph Hellwig
2022-04-06  6:05   ` Christoph Hellwig
2022-04-06  6:05   ` Christoph Hellwig
2022-04-06  6:05   ` [f2fs-dev] " Christoph Hellwig
     [not found] ` <0b7ae3df301c4fdd8d37f773d8d1eb93@FR3P281MB0843.DEUP281.PROD.OUTLOOK.COM>
2022-04-06  7:04   ` [PATCH 15/27] block: use bdev_alignment_offset in part_alignment_offset_show Alan Robinson
2022-04-06  7:04     ` [dm-devel] " Alan Robinson
2022-04-06  7:04     ` Alan Robinson
2022-04-06  7:04     ` [f2fs-dev] " Alan Robinson
2022-04-06  7:04     ` Alan Robinson

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=287eee33-720d-79f2-c1c6-862001164673@linbit.com \
    --to=christoph.boehmwalder@linbit.com \
    --cc=axboe@kernel.dk \
    --cc=drbd-dev@lists.linbit.com \
    --cc=hch@lst.de \
    --cc=linux-block@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.