Thread (3 messages) 3 messages, 2 authors, 2020-08-26

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

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>
  
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help