> @@ -692,14 +692,13 @@ repoll:
> -             wc->sl             = cqe->sl >> 4;
> +             wc->sl             = be16_to_cpu(cqe->sl_vid >> 12);
> 
> I wasn't sure if/why a conversion from network order to host order is
neeed here, can you clarify that?

This commit has an endianess bug, that was fixed in commit f781a22f.
The cqe->sl_vid field is a be16, so we needed to convert the sl value to
host order.
Before the commit this field was two u8 fields, so no conversion was
needed.
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to