[v2] migration: cleanup TLS channel referencing

[PATCH v2 0/5] migration: cleanup TLS channel referencing

peterx@redhat.com posted 5 patches 9 months, 1 week ago

Diff against v1
Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20240222095301.171137-1-peterx@redhat.com

Maintainers: Peter Xu <peterx@redhat.com>, Fabiano Rosas <farosas@suse.de>

migration/multifd.h |  2 --
migration/socket.h  |  3 ++-
migration/multifd.c | 58 +++++++++++++++++++++++----------------------
migration/socket.c  | 19 +++++++--------
4 files changed, 40 insertions(+), 42 deletions(-)

Expand all Fold all

[PATCH v2 0/5] migration: cleanup TLS channel referencing

Posted by peterx@redhat.com 9 months, 1 week ago

From: Peter Xu <peterx@redhat.com>

v2:
- add patches
  - migration/multifd: Make multifd_channel_connect() return void
  - migration/multifd: Cleanup outgoing_args in state destroy
  - migration/multifd: Drop unnecessary helper to destroy IOC
- fix spelling

This is a small cleanup patchset to firstly cleanup tls iochannel deref on
error paths, then further remove one unused var on yank if the cleanup
applies.  In v2 three more small cleanups on top as suggested by reviewers.

Please feel free to have a look, thanks.

Peter Xu (5):
  migration/multifd: Cleanup TLS iochannel referencing
  migration/multifd: Drop registered_yank
  migration/multifd: Make multifd_channel_connect() return void
  migration/multifd: Cleanup outgoing_args in state destroy
  migration/multifd: Drop unnecessary helper to destroy IOC

 migration/multifd.h |  2 --
 migration/socket.h  |  3 ++-
 migration/multifd.c | 58 +++++++++++++++++++++++----------------------
 migration/socket.c  | 19 +++++++--------
 4 files changed, 40 insertions(+), 42 deletions(-)

-- 
2.43.0

Re: [PATCH v2 0/5] migration: cleanup TLS channel referencing

Posted by Peter Xu 9 months ago

On Thu, Feb 22, 2024 at 05:52:56PM +0800, peterx@redhat.com wrote:
> From: Peter Xu <peterx@redhat.com>
> 
> v2:
> - add patches
>   - migration/multifd: Make multifd_channel_connect() return void
>   - migration/multifd: Cleanup outgoing_args in state destroy
>   - migration/multifd: Drop unnecessary helper to destroy IOC
> - fix spelling
> 
> This is a small cleanup patchset to firstly cleanup tls iochannel deref on
> error paths, then further remove one unused var on yank if the cleanup
> applies.  In v2 three more small cleanups on top as suggested by reviewers.
> 
> Please feel free to have a look, thanks.
> 
> Peter Xu (5):
>   migration/multifd: Cleanup TLS iochannel referencing
>   migration/multifd: Drop registered_yank
>   migration/multifd: Make multifd_channel_connect() return void
>   migration/multifd: Cleanup outgoing_args in state destroy
>   migration/multifd: Drop unnecessary helper to destroy IOC

queued.

-- 
Peter Xu