diff mbox

[7/9] rpc: protocol: Clarify VIR_NET_ERROR usage with streams

Message ID dfc278f30f2ea7ee9383d328531c2be1ed68bdc7.1461609759.git.crobinso@redhat.com
State Accepted
Commit 8958dde506bab3c854acd8b66d422406191bd3c9
Headers show

Commit Message

Cole Robinson April 25, 2016, 6:46 p.m. UTC 
The described protocol semantics really only apply to server initiated
stream messages. Document the semantics for client messages.
---
 src/rpc/virnetprotocol.x | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

-- 
2.7.3

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
diff mbox

Patch

diff --git a/src/rpc/virnetprotocol.x b/src/rpc/virnetprotocol.x
index 327a334..9ce33b0 100644
--- a/src/rpc/virnetprotocol.x
+++ b/src/rpc/virnetprotocol.x
@@ -104,7 +104,9 @@  const VIR_NET_MESSAGE_NUM_FDS_MAX = 32;
  *  - type == VIR_NET_STREAM
  *     * VIR_NET_CONTINUE if more data is following
  *     * VIR_NET_OK if stream is complete
- *     * VIR_NET_ERROR if stream had an error
+ *     * VIR_NET_ERROR
+ *         server message: stream had an error
+ *         client message: client aborted the stream
  *
  * Payload varies according to type and status:
  *
@@ -125,7 +127,8 @@  const VIR_NET_MESSAGE_NUM_FDS_MAX = 32;
  *     * status == VIR_NET_CONTINUE
  *          byte[]       raw stream data
  *     * status == VIR_NET_ERROR
- *          remote_error error information
+ *          server message: remote_error error information
+ *          client message: <empty>
  *     * status == VIR_NET_OK
  *          <empty>
  *