On 27/06/2013 14:46, Or Gerlitz wrote:
On 26/06/2013 11:15, Nicholas A. Bellinger wrote:
Hi Or & Co,
Ok, sendtargets discovery is now functioning over iser. The updated
target patches + v2 changelogs are in for-next commits here:
http://git.kernel.org/cgit/linux/kernel/git/nab/target-pending.git/commit/?h=for-next&id=889c8a68b8483a8b3482ac440af3ad7368c58555
http://git.kernel.org/cgit/linux/kernel/git/nab/target-pending.git/commit/?h=for-next&id=4ebc2e95f49aca3114acb13b15c3e6f769ee6300
http://git.kernel.org/cgit/linux/kernel/git/nab/target-pending.git/commit/?h=for-next&id=58203ef7ae06f17f7ee238491cd7301abe3dfc19
The running initiator + target discovery output is below. Please go
ahead and give it a shot on your setup.
Nic, using your for-next branch, this works for me too! I am using the
below initiator patch
and looking on what does it take to have things go correctly. We do
need to fix some issues related
to negotiation, but I prefer/hope to make discovery work independent
of these fixes.
index b6d81a8..cec5023 100644
--- a/drivers/infiniband/ulp/iser/iser_initiator.c
+++ b/drivers/infiniband/ulp/iser/iser_initiator.c
@@ -248,6 +248,8 @@ static int iser_post_rx_bufs(struct iscsi_conn
*conn, struct iscsi_hdr *req)
WARN_ON(iser_conn->ib_conn->post_recv_buf_count != 1);
WARN_ON(atomic_read(&iser_conn->ib_conn->post_send_buf_count) != 0);
+ iser_post_recvl(iser_conn->ib_conn);
+
iser_dbg("Initially post: %d\n", ISER_MIN_POSTED_RX);
/* Initial post receive buffers */
if (iser_post_recvm(iser_conn->ib_conn, ISER_MIN_POSTED_RX))
Things don't shine yet, from time to time LIO crashes, I sent Nic the
oops dump in private as I have it with JPG only.
Anyway, I think I have the correct iser kernel patch that would allow
for this to work once I have the user space patch that
does this set param after binding the connection in discovery and before
sending the login.
Or.
--
To unsubscribe from this list: send the line "unsubscribe target-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
