On Mon, Jun 16, 2025 at 08:43:22AM +0200, Daniel Gomez wrote: > >> This comment does not match with blk_rq_dma_map_iter_start(). It returns false > >> and status is BLK_STS_INVAL. > > > > I went over you comment a few times and still don't understand it. > > The way I read the comment is that status is only valid when > blk_rq_dma_map_iter_* returns false. Yes. > But blk_rq_dma_map_iter_start() can return false and an invalid status (in the > default switch case). The status field is valid. Your patch below leaves it uninitialized instead, which leaves me even more confused than the previous comment.
