[PULL 05/36] qemu-file: qemu_file_get_fd(): get byte in one shot

Fabiano Rosas posted 36 patches 1 day, 14 hours ago
Maintainers: Peter Xu <peterx@redhat.com>, Fabiano Rosas <farosas@suse.de>, Mark Kanda <mark.kanda@oracle.com>, Ben Chaney <bchaney@akamai.com>, Li Zhijian <lizhijian@fujitsu.com>, Eric Blake <eblake@redhat.com>, Markus Armbruster <armbru@redhat.com>, Laurent Vivier <lvivier@redhat.com>, Paolo Bonzini <pbonzini@redhat.com>
[PULL 05/36] qemu-file: qemu_file_get_fd(): get byte in one shot
Posted by Fabiano Rosas 1 day, 14 hours ago
From: Vladimir Sementsov-Ogievskiy <vsementsov@yandex-team.ru>

No reason to split into peek + get.

Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@yandex-team.ru>
Reviewed-by: Fabiano Rosas <farosas@suse.de>
Link: https://lore.kernel.org/qemu-devel/20260114064710.176268-4-vsementsov@yandex-team.ru
Signed-off-by: Fabiano Rosas <farosas@suse.de>
---
 migration/qemu-file.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/migration/qemu-file.c b/migration/qemu-file.c
index 1f3b4cf4c5..847694e2cf 100644
--- a/migration/qemu-file.c
+++ b/migration/qemu-file.c
@@ -396,8 +396,7 @@ int qemu_file_get_fd(QEMUFile *f)
         goto fail;
     }
 
-    /* Force the dummy byte and its fd passenger to appear. */
-    qemu_peek_byte(f, 0);
+    qemu_get_byte(f);
 
     fde = QTAILQ_FIRST(&f->fds);
     if (!fde) {
@@ -405,7 +404,6 @@ int qemu_file_get_fd(QEMUFile *f)
         goto fail;
     }
 
-    qemu_get_byte(f);       /* Drop the dummy byte */
     fd = fde->fd;
     QTAILQ_REMOVE(&f->fds, fde, entry);
     g_free(fde);
-- 
2.51.0