Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Host reconnects when Python discovery controller sends Async event #323

Closed
leonidc opened this issue Nov 16, 2023 · 1 comment · Fixed by #326
Closed

Host reconnects when Python discovery controller sends Async event #323

leonidc opened this issue Nov 16, 2023 · 1 comment · Fixed by #326
Assignees
Labels
bug Something isn't working Discovery Controller

Comments

@leonidc
Copy link
Collaborator

leonidc commented Nov 16, 2023

1.Created persistent discovery connection.
2.Created new subsystem.
3.Python Discovery controller tries to send Async event status to the host :
error occurs in python code in conversion from byte-list to the integer : async_reply.dword0 = b'\x02\xf0\x70\x00'

After fixing the error using async_reply.dword0 = int.from_bytes (b'\x02\xf0\x70\x00' , byteorder='little')
Controller does not crash but host doesnt receive KA response and reconnects. No Async Event status is seen in the tcpdump.

dmesg:
[Wed Nov 15 11:26:07 2023] nvme nvme0: unsupported pdu type (0)
[Wed Nov 15 11:26:07 2023] nvme nvme0: receive failed: -22
[Wed Nov 15 11:26:07 2023] nvme nvme0: starting error recovery
[Wed Nov 15 11:26:07 2023] nvme0: Keep Alive(0x18), Host Aborted Command (sct 0x3 / sc 0x71)

@leonidc
Copy link
Collaborator Author

leonidc commented Nov 16, 2023

disc_no_keep_alive.log
this is actually the tcpdump .pcap file

@leonidc leonidc self-assigned this Nov 17, 2023
@caroav caroav assigned baum and unassigned leonidc Nov 19, 2023
@caroav caroav added bug Something isn't working Discovery Controller labels Nov 19, 2023
@baum baum mentioned this issue Nov 19, 2023
@baum baum closed this as completed in #326 Nov 19, 2023
@github-project-automation github-project-automation bot moved this from 🆕 New to ✅ Done in NVMe-oF Nov 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Discovery Controller
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants