Thread (11 messages) read the whole thread 11 messages, 3 authors, 2023-03-31

Re: [RFC PATCH v3 1/4] vsock/vmci: convert VMCI error code to -ENOMEM on send

From: Vishnu Dasa <hidden>
Date: 2023-03-30 21:41:05
Also in: kvm, lkml, virtualization

On Mar 30, 2023, at 1:12 PM, Arseniy Krasnov [off-list ref] wrote:

!! External Email

This adds conversion of VMCI specific error code to general -ENOMEM. It
is needed, because af_vsock.c passes error value returned from transport
to the user, which does not expect to get VMCI_ERROR_* values.

Fixes: c43170b7e157 ("vsock: return errors other than -ENOMEM to socket")
Signed-off-by: Arseniy Krasnov <redacted>
Thanks, looks good to me.

Reviewed-by: Vishnu Dasa <redacted>
quoted hunk ↗ jump to hunk
---
net/vmw_vsock/vmci_transport.c | 8 +++++++-
1 file changed, 7 insertions(+), 1 deletion(-)
diff --git a/net/vmw_vsock/vmci_transport.c b/net/vmw_vsock/vmci_transport.c
index 36eb16a40745..95cc4d79ba29 100644
--- a/net/vmw_vsock/vmci_transport.c
+++ b/net/vmw_vsock/vmci_transport.c
@@ -1842,7 +1842,13 @@ static ssize_t vmci_transport_stream_enqueue(
       struct msghdr *msg,
       size_t len)
{
-       return vmci_qpair_enquev(vmci_trans(vsk)->qpair, msg, len, 0);
+       ssize_t err;
+
+       err = vmci_qpair_enquev(vmci_trans(vsk)->qpair, msg, len, 0);
+       if (err < 0)
+               err = -ENOMEM;
+
+       return err;
}

static s64 vmci_transport_stream_has_data(struct vsock_sock *vsk)
--
2.25.1

!! External Email: This email originated from outside of the organization. Do not click links or open attachments unless you recognize the sender.
  
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help