Re: [PATCH v4 2/2] block: better deal with the delayed not supported case in blk_cloned_rq_check_limits
From: Ritika Srivastava <hidden>
Date: 2020-08-26 17:03:53
From: Ritika Srivastava <hidden>
Date: 2020-08-26 17:03:53
Hi Jens, Can the following patches please be applied. [PATCH 1/2] block: Return blk_status_t instead of errno codes [PATCH v4 2/2] block: better deal with the delayed not supported case in blk_cloned_rq_check_limits Thanks, Ritika
On Aug 18, 2020, at 12:43 PM, Christoph Hellwig [off-list ref] wrote: On Tue, Aug 18, 2020 at 10:39:40AM -0700, Ritika Srivastava wrote:quoted
If WRITE_ZERO/WRITE_SAME operation is not supported by the storage, blk_cloned_rq_check_limits() will return IO error which will cause device-mapper to fail the paths. Instead, if the queue limit is set to 0, return BLK_STS_NOTSUPP. BLK_STS_NOTSUPP will be ignored by device-mapper and will not fail the paths. Suggested-by: Martin K. Petersen <martin.petersen@oracle.com> Signed-off-by: Ritika Srivastava <redacted> Reviewed-by: Martin K. Petersen <martin.petersen@oracle.com>Looks good, Reviewed-by: Christoph Hellwig <hch@lst.de>