From: Peter Xu <peterx@redhat.com>
To: Vladimir Sementsov-Ogievskiy <vsementsov@yandex-team.ru>
Cc: jasowang@redhat.com, mst@redhat.com, armbru@redhat.com,
farosas@suse.de, raphael.s.norwitz@gmail.com, bchaney@akamai.com,
qemu-devel@nongnu.org, berrange@redhat.com, pbonzini@redhat.com,
yc-core@yandex-team.ru, mark.caveayland@nutanix.com
Subject: Re: [PATCH v21 11/16] migration/fd: add errp parameter to migration_fd_valid()
Date: Wed, 19 Aug 2026 14:54:33 -0400 [thread overview]
Message-ID: <aoX76TtzqcTMrYhY@x1.local> (raw)
In-Reply-To: <20260819180201.1970193-12-vsementsov@yandex-team.ru>
On Wed, Aug 19, 2026 at 09:01:54PM +0300, Vladimir Sementsov-Ogievskiy wrote:
> Let migration_fd_valid() report the error itself, instead of
> duplicating the error message in both callers. This also prepares
> for adding one more check (with a different error message) to the
> function.
>
> Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@yandex-team.ru>
Reviewed-by: Peter Xu <peterx@redhat.com>
--
Peter Xu
next prev parent reply other threads:[~2026-08-19 18:54 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20260819180201.1970193-1-vsementsov@yandex-team.ru>
2026-08-19 18:01 ` [PATCH v21 01/16] net/tap: rework tap_parse_script Vladimir Sementsov-Ogievskiy
2026-08-19 18:01 ` [PATCH v21 02/16] net/tap: improve script/downscript options documentation Vladimir Sementsov-Ogievskiy
2026-08-19 18:01 ` [PATCH v21 03/16] net/tap: deprecate "no" as special value for script/downscript Vladimir Sementsov-Ogievskiy
2026-08-19 18:01 ` [PATCH v21 05/16] net/tap: move vhost initialization to tap_setup_vhost() Vladimir Sementsov-Ogievskiy
2026-08-19 18:01 ` [PATCH v21 06/16] net/tap: use container_of instead of DO_UPCAST Vladimir Sementsov-Ogievskiy
2026-08-19 18:01 ` [PATCH v21 07/16] net/tap: QOMify tap backend Vladimir Sementsov-Ogievskiy
2026-08-19 18:01 ` [PATCH v21 10/16] migration/fd: fix fd leak when fd is not valid for migration Vladimir Sementsov-Ogievskiy
2026-08-19 18:53 ` Peter Xu
2026-08-19 18:01 ` [PATCH v21 11/16] migration/fd: add errp parameter to migration_fd_valid() Vladimir Sementsov-Ogievskiy
2026-08-19 18:54 ` Peter Xu [this message]
2026-08-19 18:01 ` [PATCH v21 12/16] migration: check that transfer is UNIX socket when "local" set Vladimir Sementsov-Ogievskiy
2026-08-19 18:59 ` Peter Xu
2026-08-19 18:01 ` [PATCH v21 13/16] virtio-net: support local migration of backend Vladimir Sementsov-Ogievskiy
2026-08-19 18:01 ` [PATCH v21 15/16] net/tap: support local migration with virtio-net Vladimir Sementsov-Ogievskiy
2026-08-19 18:01 ` [PATCH v21 16/16] tests/functional: add test_tap_migration Vladimir Sementsov-Ogievskiy
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=aoX76TtzqcTMrYhY@x1.local \
--to=peterx@redhat.com \
--cc=armbru@redhat.com \
--cc=bchaney@akamai.com \
--cc=berrange@redhat.com \
--cc=farosas@suse.de \
--cc=jasowang@redhat.com \
--cc=mark.caveayland@nutanix.com \
--cc=mst@redhat.com \
--cc=pbonzini@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=raphael.s.norwitz@gmail.com \
--cc=vsementsov@yandex-team.ru \
--cc=yc-core@yandex-team.ru \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.