Re: [PATCH v4 07/15] Drivers: hv: Move hv_call_deposit_pages and hv_call_create_vp to common code

2023-10-02 Thread kernel test robot
Hi Nuno, kernel test robot noticed the following build errors: [auto build test ERROR on arnd-asm-generic/master] [also build test ERROR on arm64/for-next/core linus/master v6.6-rc4 next-20230929] [cannot apply to tip/x86/core] [If your patch is applied to the wrong git tree, kindly drop us a no

Re: [PATCH v4 14/15] asm-generic: hyperv: Use new Hyper-V headers conditionally.

2023-10-02 Thread Nuno Das Neves
Hi Alex, On 10/2/2023 12:35 PM, Alex Ionescu wrote: Hi Nuno, I understand the requirement to have undocumented/non-standard/non-TLFS-published information in the HDK headers, however, the current state of this patch is that for any other code that's not in the kernel today, or in this upcoming

RE: [PATCH 3/5] x86/mm: Mark CoCo VM pages not present while changing encrypted state

2023-10-02 Thread Michael Kelley (LINUX)
From: Tom Lendacky Sent: Monday, October 2, 2023 11:59 AM > > On 10/2/23 11:35, Tom Lendacky wrote: > > On 9/29/23 13:19, Michael Kelley wrote: > >> In a CoCo VM when a page transitions from encrypted to decrypted, or vice > >> versa, attributes in the PTE must be updated *and* the hypervisor mu

Re: [PATCH v4 14/15] asm-generic: hyperv: Use new Hyper-V headers conditionally.

2023-10-02 Thread Alex Ionescu
Hi Nuno, I understand the requirement to have undocumented/non-standard/non-TLFS-published information in the HDK headers, however, the current state of this patch is that for any other code that's not in the kernel today, or in this upcoming driver, the hyperv-tlfs definitions are incomplete, bec

Re: [PATCH v4 09/15] Drivers: hv: Introduce hv_output_arg_exists in hv_common.c

2023-10-02 Thread Alex Ionescu
Hi Nuno, Is it possible to simply change to always allocating the output page? For example, the output page could be needed in scenarios where Linux is not running as the root partition, since certain hypercalls that a guest can make will still require one (I realize that's not the case _today_, b

Re: [PATCH 3/5] x86/mm: Mark CoCo VM pages not present while changing encrypted state

2023-10-02 Thread Tom Lendacky
On 10/2/23 11:35, Tom Lendacky wrote: On 9/29/23 13:19, Michael Kelley wrote: In a CoCo VM when a page transitions from encrypted to decrypted, or vice versa, attributes in the PTE must be updated *and* the hypervisor must be notified of the change. Because there are two separate steps, there's

Re: [PATCH 3/5] x86/mm: Mark CoCo VM pages not present while changing encrypted state

2023-10-02 Thread Tom Lendacky
On 9/29/23 13:19, Michael Kelley wrote: In a CoCo VM when a page transitions from encrypted to decrypted, or vice versa, attributes in the PTE must be updated *and* the hypervisor must be notified of the change. Because there are two separate steps, there's a window where the settings are inconsi

Re: [PATCH 1/5] x86/coco: Use slow_virt_to_phys() in page transition hypervisor callbacks

2023-10-02 Thread Tom Lendacky
On 9/29/23 13:19, Michael Kelley wrote: In preparation for temporarily marking pages not present during a transition between encrypted and decrypted, use slow_virt_to_phys() in the hypervisor callbacks. As long as the PFN is correct, slow_virt_to_phys() works even if the leaf PTE is not present.

Re: [PATCH 0/5] x86/coco: Mark CoCo VM pages not present when changing encrypted state

2023-10-02 Thread Tom Lendacky
On 9/29/23 13:19, Michael Kelley wrote: In a CoCo VM when a page transitions from encrypted to decrypted, or vice versa, attributes in the PTE must be updated *and* the hypervisor must be notified of the change. Because there are two separate steps, there's a window where the settings are inconsi