On 09/17/2015 02:42 AM, Sagi Grimberg wrote:
The new fast registration  verg ib_map_mr_sg receives a scatterlist

                             ^ verb ?

+/**
+ * ib_map_mr_sg() - Map a memory region with the largest prefix of
+ *     a dma mapped SG list

This description could be made more clear. How about the following:

Register the largest possible prefix of a DMA-mapped SG-list

+                       } else if (last_page_off + dma_len < mr->page_size) {
+                               /* chunk this fragment with the last */
+                               last_end_dma_addr += dma_len;
+                               last_page_off += dma_len;
+                               mr->length += dma_len;
+                               continue;

Shouldn't this code update last_page_addr ?

--
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