[PATCH v5 3/7] scsi_io_completion_nz_result function added

2018-03-28 Thread Douglas Gilbert
Break out several intertwined paths when cmd->result is non zero and place them in the scsi_io_completion_nz_result helper function. The logic is not changed. Signed-off-by: Douglas Gilbert --- A reviewer requested the original helper function's two return values be reduced to one: the blk_stat v

Re: [PATCH v5 3/7] scsi_io_completion_nz_result function added

2018-03-29 Thread Johannes Thumshirn
Looks good (as far as I can tell) Reviewed-by: Johannes Thumshirn -- Johannes Thumshirn Storage jthumsh...@suse.de+49 911 74053 689 SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg GF: Felix Imendörffer, Jane Smithard, Graham

Re: [PATCH v5 3/7] scsi_io_completion_nz_result function added

2018-03-29 Thread Bart Van Assche
On Thu, 2018-03-29 at 01:46 -0400, Douglas Gilbert wrote: > Break out several intertwined paths when cmd->result is non zero and > place them in the scsi_io_completion_nz_result helper function. The > logic is not changed. Reviewed-by: Bart Van Assche