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

Jinank Jain posted 5 patches 3 years, 5 months ago
There is a newer version of this series
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(-)
[PATCH v3 0/5] Add support running nested Microsoft Hypervisor
Posted by Jinank Jain 3 years, 5 months ago
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(-)

-- 
2.25.1
Re: [PATCH v3 0/5] Add support running nested Microsoft Hypervisor
Posted by Anirudh Rayabharam 3 years, 5 months ago
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
Re: [PATCH v3 0/5] Add support running nested Microsoft Hypervisor
Posted by Borislav Petkov 3 years, 5 months ago
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.

-- 
Regards/Gruss,
    Boris.

https://people.kernel.org/tglx/notes-about-netiquette
Re: [PATCH v3 0/5] Add support running nested Microsoft Hypervisor
Posted by Wei Liu 3 years, 4 months ago
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
>
Re: [PATCH v3 0/5] Add support running nested Microsoft Hypervisor
Posted by Jinank Jain 3 years, 4 months ago
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
>>