Re: [PATCH net-next v6 0/9] vsock: add namespace support to vhost-vsock
From: Bobby Eshleman
Date: Wed Sep 17 2025 - 12:34:06 EST
On Wed, Sep 17, 2025 at 05:19:28PM +0100, Simon Horman wrote:
> On Tue, Sep 16, 2025 at 04:43:44PM -0700, Bobby Eshleman wrote:
>
> ...
>
> > base-commit: 949ddfb774fe527cebfa3f769804344940f7ed2e
>
> Hi Bobby,
>
> This series does not seem to compile when applied to the commit above.
> Likewise when applied to current net-next (which is now slightly newer).
>
> hyperv_transport.c: In function ‘hvs_open_connection’:
> hyperv_transport.c:316:14: error: too few arguments to function ‘vsock_find_bound_socket’
> 316 | sk = vsock_find_bound_socket(&addr, vsock_global_dummy_net());
> | ^~~~~~~~~~~~~~~~~~~~~~~
> In file included from hyperv_transport.c:15:
> /home/horms/projects/linux/linux/include/net/af_vsock.h:218:14: note: declared here
> 218 | struct sock *vsock_find_bound_socket(struct sockaddr_vm *addr, struct net *net,
> | ^~~~~~~~~~~~~~~~~~~~~~~
>
> --
> pw-bot: changes-requested
Ah dang it, looks like I had hvc disabled when I build tested it.
Thanks for the catch, I'll fix this in the next rev.
Best,
Bobby