Damien, > good_bytes is already set to 0 at the beginning of sd_done() when > result != 0 and again within the initial switch-case for > REQ_OP_DISCARD and REQ_OP_WRITE_SAME cases, which are the only > commands that can lead to hitting that "else" part in the sense data > processing. I may be missing something, but I think that that > assignment is redundant. Fixed this up, thanks! -- Martin K. Petersen Oracle Linux Engineering