On Mon, Oct 25, 2021 at 09:34:10AM +0200, Paolo Bonzini wrote:
On 24/10/21 18:49, Michael Roth wrote:
So test code would need to consider cases where addr_gpa2raw() needs to be used to set the C-bit (which is basically only when they want to mix usage of the vm_phy_page[s]_alloc with their own mapping of the guest page tables, which doesn't seem to be done in any existing tests anyway).
Yes, and it seems like a more rare case in general.
The library code would need these addr_gpa2raw() hooks in places where it calls virt_*map() internally. Probably just a handful of places though.
Either that, or you can have virt_*map that consults the encryption bitmap, and virt_*map_enc (or _raw, doesn't matter) that takes the encryption state explicitly as a bool.
That sounds promising. Will give that a shot. Thanks!