Two trivial comments I spot:
On Fri, Dec 13, 2024 at 03:08:44PM +0800, Chenyi Qiang wrote:
> +struct GuestMemfdManager {
> + Object parent;
> +
> + /* Managed memory region. */
> + MemoryRegion *mr;
> +
> + /*
> + * 1-setting of the bit represents the memory is populated (shared).
> + */
> + int32_t bitmap_size;
> + unsigned long *bitmap;
Might be clearer to name the bitmap directly as what it represents. E.g.,
shared_bitmap?
> +
> + /* block size and alignment */
> + uint64_t block_size;
Can we always fetch it from the MR/ramblock? If this is needed, better add
some comment explaining why.
> +
> + /* listeners to notify on populate/discard activity. */
> + QLIST_HEAD(, RamDiscardListener) rdl_list;
> +};
--
Peter Xu
