Skip to content

Add the Linux VSOCK transport.#4822

Merged
bluca merged 13 commits intozeromq:masterfrom
remijouannet:vsock
Mar 18, 2026
Merged

Add the Linux VSOCK transport.#4822
bluca merged 13 commits intozeromq:masterfrom
remijouannet:vsock

Conversation

@remijouannet
Copy link
Copy Markdown
Contributor

@remijouannet remijouannet commented Oct 29, 2025

Hello,
VSOCK transport allows communication between the Host and guests.
It requires linux on the host and Guest.

the code is mostly inspire from the tipc and vmci.

there is a conflict between VMCI and VSOCK header, i don't know how to treat this issue, do we support having both build ? or do we make exclusion between them ?
Fixes #4491

@bluca
Copy link
Copy Markdown
Member

bluca commented Oct 30, 2025

Thanks for the PR

i don't know how to treat this issue, do we support having both build ?

Yeah they should both work at the same time

@remijouannet
Copy link
Copy Markdown
Contributor Author

@bluca i'm not an expert in cpp, i've used preprocessor to rename the conflicted variables between vmci and vsock, i don't know if this is the right way to handle this.

@remijouannet
Copy link
Copy Markdown
Contributor Author

Hello @bluca, any chance to get this merge ? or there is anything i should improve ?

@bluca
Copy link
Copy Markdown
Member

bluca commented Mar 18, 2026

In file included from src/ctx.cpp:30:
/usr/include/linux/vm_sockets.h:178:8: error: redefinition of 'struct sockaddr_vm'
  178 | struct sockaddr_vm {
      |        ^~~~~~~~~~~
In file included from src/ctx.cpp:25:
/home/runner/work/libzmq/libzmq/libzmq/builds/coverage/vmci/vmci_sockets.h:506:8: note: previous definition of 'struct sockaddr_vm'
  506 | struct sockaddr_vm {
      |        ^~~~~~~~~~~

@remijouannet
Copy link
Copy Markdown
Contributor Author

sorry i was messing around in the wrong branch, it's fixed now

@bluca bluca merged commit 7d95ac0 into zeromq:master Mar 18, 2026
19 of 22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

VSOCK support

3 participants