Gabriel Krisman Bertazi <krisman@xxxxxxxxxxxxxxxxxx> writes: > My one concern about this patch is if remapping an arbitrary queue to > hctx_0 could result in outstanding requests getting submitted to the > wrong hctx. I couldn't observe this happening during tests, but I'm not > entirely sure it'll never happen. I believe the queue will be empty if > we are trying to allocate tags for it, unless it was using another alive > hctx queue and for some reason got reassigned to this new hctx. is this > possible at all? I no longer believe this case to be an issue for accepting this patch, since if a queue enters this path and get's remapped to hctx_0, it means the ctx would already be getting remaped to a new queue anyway, and it should use the requests that were just mapped for that queue. So, for a correctness standpoint it doesn't matter which queue we are remaping, the failed hctx or hctx_0, no request would get redirected to the wrong queue with this change. Jens, do you think this set is good for merging in 4.10? It fixes some issues for us on low memory conditions. -- Gabriel Krisman Bertazi -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
