On 10/21/2019 02:56 PM, Josef Bacik wrote: > We noticed a problem where NBD sometimes double completes the same request > when > things go wrong and we time out the request. If the other side goes out to > lunch but happens to reply just as we're timing out the requests we can end up > with a double completion on the request. > > We already keep track of the command status, we just need to make sure we > protect all cases where we set cmd->status with the cmd->lock, which is patch > #1. Patch #2 is the fix for the problem, which catches the case where we race > with the timeout handler and the reply handler. Thanks, >
Patches look ok and tested ok for me. Reviewed-by: Mike Christie <[email protected]>
