Thread (3 messages) 3 messages, 2 authors, 2016-08-08
DORMANTno replies

[PATCH RFC] nvmet-rdma: +1 to *queue_size from hsqsize/hrqsize

From: J Freyensee <hidden>
Date: 2016-08-08 17:48:17

On Mon, 2016-08-08@16:09 +0000, Verkamp, Daniel wrote:
Hi Jay,

On Fri, 2016-08-05@20:00 -0700, Jay Freyensee wrote:
quoted
-	queue->recv_queue_size = le16_to_cpu(req->hsqsize);
-	queue->send_queue_size = le16_to_cpu(req->hrqsize);
+	queue->recv_queue_size = le16_to_cpu(req->hsqsize)+1;
+	queue->send_queue_size = le16_to_cpu(req->hrqsize)+1;
I think the +1 needs to be inside the le16_to_cpu() in both cases.
Thanks.

This patch will be dependent on the "nvme-rdma: sqsize/hsqsize/hrqsize
is 0-based val" patch, which reading the spec again I still think it's
0-based.
-- Daniel
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help