On 04/10/2017 10:20 PM, Michael Roth wrote:
Quoting Philippe Mathieu-Daudé (2017-04-07 17:20:16)
Suggested-by: Michael Roth <mdr...@linux.vnet.ibm.com>
Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org>
Reviewed-by: Michael Roth <mdr...@linux.vnet.ibm.com>
---
Michael should I use Signed-off-by instead of the Suggested-by (since it is your
code)?
I suppose it could go either way depending on the circumstances
(assuming you've asked for / obtained their SoB obviously), but
speaking for myself here I don't think it's necessary.
Ok! I allowed myself to add your "Suggested-by" after reading Linux's
process/submitting-patches.rst doc, assuming no need to ask for it since
you answered in public ML. I'll diligently ask next time!
qga/commands-posix.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/qga/commands-posix.c b/qga/commands-posix.c
index fc45102a1e..ca5a24b2c9 100644
--- a/qga/commands-posix.c
+++ b/qga/commands-posix.c
@@ -2128,6 +2128,9 @@ static void transfer_memory_block(GuestMemoryBlock
*mem_blk, bool sys2memblk,
if (errno == ENOENT) {
result->response =
GUEST_MEMORY_BLOCK_RESPONSE_TYPE_OPERATION_NOT_SUPPORTED;
+ } else {
+ result->response =
+ GUEST_MEMORY_BLOCK_RESPONSE_TYPE_OPERATION_FAILED;
}
goto out1;
}
--
2.11.0