qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [Qemu-devel] [PATCH] qemu-file: use fwrite() correctly (take II)
@ 2014-03-05 19:08 Juan Quintela
  2014-03-05 19:27 ` Eric Blake
  0 siblings, 1 reply; 2+ messages in thread
From: Juan Quintela @ 2014-03-05 19:08 UTC (permalink / raw)
  To: qemu-devel

Simple minds like mine can't understand neither fwrite, or comments
when there are 3 versions of a patch.  The integrated patch is missing
this bit.

Signed-off-by: Juan Quintela <quintela@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Reviewed-by: Markus Armbruster <armbru@redhat.com>
---

I added Reviewed-by: by Eric & Markus because they acked the version
with this change in.

 qemu-file.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/qemu-file.c b/qemu-file.c
index f074af1..e5ec798 100644
--- a/qemu-file.c
+++ b/qemu-file.c
@@ -105,7 +105,7 @@ static int stdio_put_buffer(void *opaque, const uint8_t *buf, int64_t pos,
     res = fwrite(buf, 1, size, s->stdio_file);

     if (res != size) {
-        return -EIO;	/* fake errno value */
+        return -errno;
     }
     return res;
 }
-- 
1.8.5.3

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [Qemu-devel] [PATCH] qemu-file: use fwrite() correctly (take II)
  2014-03-05 19:08 [Qemu-devel] [PATCH] qemu-file: use fwrite() correctly (take II) Juan Quintela
@ 2014-03-05 19:27 ` Eric Blake
  0 siblings, 0 replies; 2+ messages in thread
From: Eric Blake @ 2014-03-05 19:27 UTC (permalink / raw)
  To: Juan Quintela, qemu-devel

[-- Attachment #1: Type: text/plain, Size: 837 bytes --]

On 03/05/2014 12:08 PM, Juan Quintela wrote:
> Simple minds like mine can't understand neither fwrite, or comments
> when there are 3 versions of a patch.  The integrated patch is missing
> this bit.
> 
> Signed-off-by: Juan Quintela <quintela@redhat.com>
> Reviewed-by: Eric Blake <eblake@redhat.com>
> Reviewed-by: Markus Armbruster <armbru@redhat.com>
> ---
> 
> I added Reviewed-by: by Eric & Markus because they acked the version
> with this change in.

Also posted here:
https://lists.nongnu.org/archive/html/qemu-devel/2014-02/msg04902.html

but indeed, we want this patch in sooner rather than later, since
misapplied patches can be a nightmare to track down later (such as in
backporting efforts).

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 604 bytes --]

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2014-03-05 19:27 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-03-05 19:08 [Qemu-devel] [PATCH] qemu-file: use fwrite() correctly (take II) Juan Quintela
2014-03-05 19:27 ` Eric Blake

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).