[v3,0/5] Add support running nested Microsoft Hypervisor

Message ID cover.1667480257.git.jinankjain@linux.microsoft.com
Headers
Series Add support running nested Microsoft Hypervisor |

Message

Jinank Jain Nov. 3, 2022, 1:04 p.m. UTC
  This patch series plans to add support for running nested Microsoft
Hypervisor. In case of nested Microsoft Hypervisor there are few
privileged hypercalls which need to go L0 Hypervisor instead of L1
Hypervisor. This patches series basically identifies such hypercalls and
replace them with nested hypercalls.

Jinank Jain (5):
  x86/hyperv: Add support for detecting nested hypervisor
  Drivers: hv: Setup synic registers in case of nested root partition
  x86/hyperv: Add an interface to do nested hypercalls
  Drivers: hv: Enable vmbus driver for nested root partition
  x86/hyperv: Change interrupt vector for nested root partition

 arch/x86/include/asm/hyperv-tlfs.h | 17 +++++++-
 arch/x86/include/asm/idtentry.h    |  2 +
 arch/x86/include/asm/irq_vectors.h |  6 +++
 arch/x86/include/asm/mshyperv.h    | 68 ++++++++++++++++++++++++++++--
 arch/x86/kernel/cpu/mshyperv.c     | 22 ++++++++++
 arch/x86/kernel/idt.c              |  9 ++++
 drivers/hv/hv.c                    | 18 +++++---
 drivers/hv/hv_common.c             |  7 ++-
 drivers/hv/vmbus_drv.c             |  5 ++-
 include/asm-generic/hyperv-tlfs.h  |  1 +
 10 files changed, 141 insertions(+), 14 deletions(-)
  

Comments

Borislav Petkov Nov. 3, 2022, 2 p.m. UTC | #1
On Thu, Nov 03, 2022 at 01:04:02PM +0000, Jinank Jain wrote:
> ...

Please stop spamming people with patches every day:

From: Documentation/process/submitting-patches.rst

Don't get discouraged - or impatient
------------------------------------

After you have submitted your change, be patient and wait.  Reviewers are
busy people and may not get to your patch right away.

Once upon a time, patches used to disappear into the void without comment,
but the development process works more smoothly than that now.  You should
receive comments within a week or so; if that does not happen, make sure
that you have sent your patches to the right place.  Wait for a minimum of
						     ^^^^^^^^^^^^^^^^^^^^^
one week before resubmitting or pinging reviewers - possibly longer during
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

busy times like merge windows.
  
Anirudh Rayabharam Nov. 3, 2022, 3:42 p.m. UTC | #2
On Thu, Nov 03, 2022 at 01:04:02PM +0000, Jinank Jain wrote:
> This patch series plans to add support for running nested Microsoft
> Hypervisor. In case of nested Microsoft Hypervisor there are few
> privileged hypercalls which need to go L0 Hypervisor instead of L1
> Hypervisor. This patches series basically identifies such hypercalls and
> replace them with nested hypercalls.

In the future, please include a changelog in your cover letter
explaining what changed in each version of the series.

Anirudh.

> 
> Jinank Jain (5):
>   x86/hyperv: Add support for detecting nested hypervisor
>   Drivers: hv: Setup synic registers in case of nested root partition
>   x86/hyperv: Add an interface to do nested hypercalls
>   Drivers: hv: Enable vmbus driver for nested root partition
>   x86/hyperv: Change interrupt vector for nested root partition
> 
>  arch/x86/include/asm/hyperv-tlfs.h | 17 +++++++-
>  arch/x86/include/asm/idtentry.h    |  2 +
>  arch/x86/include/asm/irq_vectors.h |  6 +++
>  arch/x86/include/asm/mshyperv.h    | 68 ++++++++++++++++++++++++++++--
>  arch/x86/kernel/cpu/mshyperv.c     | 22 ++++++++++
>  arch/x86/kernel/idt.c              |  9 ++++
>  drivers/hv/hv.c                    | 18 +++++---
>  drivers/hv/hv_common.c             |  7 ++-
>  drivers/hv/vmbus_drv.c             |  5 ++-
>  include/asm-generic/hyperv-tlfs.h  |  1 +
>  10 files changed, 141 insertions(+), 14 deletions(-)
> 
> -- 
> 2.25.1
  
Wei Liu Nov. 16, 2022, 12:12 p.m. UTC | #3
On Thu, Nov 03, 2022 at 01:04:02PM +0000, Jinank Jain wrote:
> This patch series plans to add support for running nested Microsoft
> Hypervisor. In case of nested Microsoft Hypervisor there are few
> privileged hypercalls which need to go L0 Hypervisor instead of L1
> Hypervisor. This patches series basically identifies such hypercalls and
> replace them with nested hypercalls.
> 
> Jinank Jain (5):
>   x86/hyperv: Add support for detecting nested hypervisor

I see `__weak hv_nested` in this patch.

I guess this version has fixed ARM64 build?

>   Drivers: hv: Setup synic registers in case of nested root partition
>   x86/hyperv: Add an interface to do nested hypercalls
>   Drivers: hv: Enable vmbus driver for nested root partition
>   x86/hyperv: Change interrupt vector for nested root partition
> 
>  arch/x86/include/asm/hyperv-tlfs.h | 17 +++++++-
>  arch/x86/include/asm/idtentry.h    |  2 +
>  arch/x86/include/asm/irq_vectors.h |  6 +++
>  arch/x86/include/asm/mshyperv.h    | 68 ++++++++++++++++++++++++++++--
>  arch/x86/kernel/cpu/mshyperv.c     | 22 ++++++++++
>  arch/x86/kernel/idt.c              |  9 ++++
>  drivers/hv/hv.c                    | 18 +++++---
>  drivers/hv/hv_common.c             |  7 ++-
>  drivers/hv/vmbus_drv.c             |  5 ++-
>  include/asm-generic/hyperv-tlfs.h  |  1 +
>  10 files changed, 141 insertions(+), 14 deletions(-)
> 
> -- 
> 2.25.1
>
  
Jinank Jain Nov. 17, 2022, 4:04 a.m. UTC | #4
On 11/16/2022 5:42 PM, Wei Liu wrote:
> On Thu, Nov 03, 2022 at 01:04:02PM +0000, Jinank Jain wrote:
>> This patch series plans to add support for running nested Microsoft
>> Hypervisor. In case of nested Microsoft Hypervisor there are few
>> privileged hypercalls which need to go L0 Hypervisor instead of L1
>> Hypervisor. This patches series basically identifies such hypercalls and
>> replace them with nested hypercalls.
>>
>> Jinank Jain (5):
>>    x86/hyperv: Add support for detecting nested hypervisor
> I see `__weak hv_nested` in this patch.
>
> I guess this version has fixed ARM64 build?

It is fixed in v4 of the patch series.

Regards,

Jinank


>
>>    Drivers: hv: Setup synic registers in case of nested root partition
>>    x86/hyperv: Add an interface to do nested hypercalls
>>    Drivers: hv: Enable vmbus driver for nested root partition
>>    x86/hyperv: Change interrupt vector for nested root partition
>>
>>   arch/x86/include/asm/hyperv-tlfs.h | 17 +++++++-
>>   arch/x86/include/asm/idtentry.h    |  2 +
>>   arch/x86/include/asm/irq_vectors.h |  6 +++
>>   arch/x86/include/asm/mshyperv.h    | 68 ++++++++++++++++++++++++++++--
>>   arch/x86/kernel/cpu/mshyperv.c     | 22 ++++++++++
>>   arch/x86/kernel/idt.c              |  9 ++++
>>   drivers/hv/hv.c                    | 18 +++++---
>>   drivers/hv/hv_common.c             |  7 ++-
>>   drivers/hv/vmbus_drv.c             |  5 ++-
>>   include/asm-generic/hyperv-tlfs.h  |  1 +
>>   10 files changed, 141 insertions(+), 14 deletions(-)
>>
>> -- 
>> 2.25.1
>>