Re: [PATCH v5 01/11] memory: Introduce RamDiscardMgr for RAM memory regions

[David Hildenbrand]

> +    /**
> +     * @is_populated:
> +     *
> +     * Check whether the given range within the #MemoryRegion is completely
> +     * populated (i.e., no parts are currently discarded). There are no
> +     * alignment requirements for the range.
> +     *
> +     * @rdm: the #RamDiscardMgr
> +     * @mr: the #MemoryRegion
> +     * @offset: offset into the #MemoryRegion
> +     * @size: size in the #MemoryRegion
> +     *
> +     * Returns whether the given range is completely populated.
> +     */
> +    bool (*is_populated)(const RamDiscardMgr *rdm, const MemoryRegion *mr,
> +                         ram_addr_t start, ram_addr_t offset);

Minor thing I noticed in the meanwhile: Variable naming here is wrong. 
Last line should be:

"ram_addr_t offset, ram_addr_t size"

--
Thanks,

David / dhildenb