On Mon, Jun 17, 2019 at 04:27:51PM -0700, Darrick J. Wong wrote: > Ugh, I don't really like this awkwardly split else/#endif construction > here. Can we simply do: > > error = blk_status_to_errno(iclog->ic_bio.bi_status); > > #ifdef DEBUG > if (iclog->ic_fail_crc) > error = -EIO; > #endif > > instead? Fine with me.