[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v9 5/7] multifd: multifd_send_sync_main now returns negative
From: |
Peter Xu |
Subject: |
Re: [PATCH v9 5/7] multifd: multifd_send_sync_main now returns negative on error |
Date: |
Tue, 26 Apr 2022 08:59:47 -0400 |
On Mon, Apr 25, 2022 at 06:50:54PM -0300, Leonardo Bras wrote:
> Even though multifd_send_sync_main() currently emits error_reports, it's
> callers don't really check it before continuing.
>
> Change multifd_send_sync_main() to return -1 on error and 0 on success.
> Also change all it's callers to make use of this change and possibly fail
> earlier.
>
> (This change is important to next patch on multifd zero copy
> implementation, to make it sure an error in zero-copy flush does not go
> unnoticed.
>
> Signed-off-by: Leonardo Bras <leobras@redhat.com>
Reviewed-by: Peter Xu <peterx@redhat.com>
--
Peter Xu
- [PATCH v9 0/7] MSG_ZEROCOPY + multifd, Leonardo Bras, 2022/04/25
- [PATCH v9 3/7] migration: Add zero-copy-send parameter for QMP/HMP for Linux, Leonardo Bras, 2022/04/25
- [PATCH v9 1/7] QIOChannel: Add flags on io_writev and introduce io_flush callback, Leonardo Bras, 2022/04/25
- [PATCH v9 5/7] multifd: multifd_send_sync_main now returns negative on error, Leonardo Bras, 2022/04/25
- [PATCH v9 4/7] migration: Add migrate_use_tls() helper, Leonardo Bras, 2022/04/25
- [PATCH v9 2/7] QIOChannelSocket: Implement io_writev zero copy flag & io_flush for CONFIG_LINUX, Leonardo Bras, 2022/04/25
- [PATCH v9 6/7] multifd: Send header packet without flags if zero-copy-send is enabled, Leonardo Bras, 2022/04/25
- [PATCH v9 7/7] multifd: Implement zero copy write in multifd migration (multifd-zero-copy), Leonardo Bras, 2022/04/25