Re: [dm-devel] [RFC PATCH 2/2] dm: don't try to split REQ_OP_ZONE_APPEND bios

2020-06-22 Thread Johannes Thumshirn
On 22/06/2020 02:27, Damien Le Moal wrote: [...] > Just to be extra sure, I checked this again. Since for zoned block devices the > mapping of a target must be zoned aligned and a zone append command is always > fully contained within a zone, we do not need this check. The stacked limits > and > s

Re: [dm-devel] [RFC PATCH 2/2] dm: don't try to split REQ_OP_ZONE_APPEND bios

2020-06-21 Thread Damien Le Moal
On 2020/06/20 1:27, Mike Snitzer wrote: > On Fri, Jun 19 2020 at 3:54am -0400, > Damien Le Moal wrote: > >> On 2020/06/19 15:59, Johannes Thumshirn wrote: >>> REQ_OP_ZONE_APPEND bios cannot be split so return EIO if we can't fit it >>> into one IO. >>> >>> Signed-off-by: Johannes Thumshirn >>>

Re: [dm-devel] [RFC PATCH 2/2] dm: don't try to split REQ_OP_ZONE_APPEND bios

2020-06-19 Thread Mike Snitzer
On Fri, Jun 19 2020 at 3:54am -0400, Damien Le Moal wrote: > On 2020/06/19 15:59, Johannes Thumshirn wrote: > > REQ_OP_ZONE_APPEND bios cannot be split so return EIO if we can't fit it > > into one IO. > > > > Signed-off-by: Johannes Thumshirn > > --- > > drivers/md/dm.c | 3 +++ > > 1 file c

[dm-devel] [RFC PATCH 2/2] dm: don't try to split REQ_OP_ZONE_APPEND bios

2020-06-19 Thread Johannes Thumshirn
REQ_OP_ZONE_APPEND bios cannot be split so return EIO if we can't fit it into one IO. Signed-off-by: Johannes Thumshirn --- drivers/md/dm.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/md/dm.c b/drivers/md/dm.c index 058c34abe9d1..c720a7e3269a 100644 --- a/drivers/md/dm.c +++ b

Re: [dm-devel] [RFC PATCH 2/2] dm: don't try to split REQ_OP_ZONE_APPEND bios

2020-06-19 Thread Damien Le Moal
On 2020/06/19 15:59, Johannes Thumshirn wrote: > REQ_OP_ZONE_APPEND bios cannot be split so return EIO if we can't fit it > into one IO. > > Signed-off-by: Johannes Thumshirn > --- > drivers/md/dm.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/drivers/md/dm.c b/drivers/md/dm.c >