[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH 1/2] tools/migration: Fix iovec handling in send_checkpoint_dirty_pfn_list()

To: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
From: Olaf Hering <olaf@xxxxxxxxx>
Date: Tue, 6 Jul 2021 14:20:10 +0200
Authentication-results: strato.com; dkim=none
Cc: Xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Ian Jackson <iwj@xxxxxxxxxxxxxx>, Wei Liu <wl@xxxxxxx>, Juergen Gross <jgross@xxxxxxxx>, Jan Beulich <JBeulich@xxxxxxxx>
Delivery-date: Tue, 06 Jul 2021 12:20:32 +0000
List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

Am Tue, 6 Jul 2021 12:23:31 +0100
schrieb Andrew Cooper <andrew.cooper3@xxxxxxxxxx>:

> We shouldn't be using two struct iovec's to write half of 'rec' each, and
> there is no need to malloc() for two struct iovec's at all.
> 
> Simplify down to just two - one covering the whole of 'rec', and one covering
> the pfns array.

Acked-by: Olaf Hering <olaf@xxxxxxxxx>


Olaf

Attachment: pgpU_zf0kbBh4.pgp
Description: Digitale Signatur von OpenPGP

References:
- [PATCH 0/2] tools/migration: Fixes in send_checkpoint_dirty_pfn_list()
  - From: Andrew Cooper
- [PATCH 1/2] tools/migration: Fix iovec handling in send_checkpoint_dirty_pfn_list()
  - From: Andrew Cooper

Prev by Date: Re: [PATCH v4] arm64: Change type of hsr, cpsr, spsr_el1 to uint64_t
Next by Date: Re: [RESEND PATCH V5 0/2] Virtio support for toolstack on Arm (Was "IOREQ feature (+ virtio-mmio) on Arm")
Previous by thread: Re: [PATCH 1/2] tools/migration: Fix iovec handling in send_checkpoint_dirty_pfn_list()
Next by thread: [PATCH 2/2] tools/migration: Fix potential overflow in send_checkpoint_dirty_pfn_list()
Index(es):
- Date
- Thread

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.