On 4/19/2016 12:06 AM, Christoph Hellwig wrote:
iSER currently has a couple places that set max_sectors in either the host template or SCSI host, and all of them get it wrong. This patch instead uses a single assignment that (hopefully) gets it right: the max_sectors value must be derived from the number of segments in the FR or FRM structure, but actually be one lower than the page size multiplied
* FRM ==> FMR
by the number of sectors, as it has to handle the case of non-aligned I/O. Without this I get trivivial to reproduce hangs when running xfstests (on XFS) over iSER to Linux targets. Signed-off-by: Christoph Hellwig <hch@xxxxxx> --- drivers/infiniband/ulp/iser/iscsi_iser.c | 14 ++++++++++---- 1 file changed, 10 insertions(+), 4 deletions(-)
Looks good. Reviewed-by: Max Gurtovoy <maxg@xxxxxxxxxxxx> -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html