CloudHv:arm: Enable direct kernel boot

[edk2-devel] [PATCH 0/3] CloudHv:arm: Enable direct kernel boot

Posted by Jianyong Wu 1 year, 7 months ago

Direct kernel boot removes the dependency of retrieving kernel image
from block device. For Cloud Hypervisor, we use the following way to
support it.

1. Cloud Hypervisor store kernel image into memory and put kernel info,
   including the memory base and size, into DT;
2. When init memory in edk2, the kernel memory region is retrieved from
   DT and set it as read only memory region;
3. Edk2 fetches kernel from memory and prepare a image handle;
4. Load kernel using LoadImage in the end.

1 is done in Cloud Hypervisor, 2 and 3 is done in this patch set, 4 is
not affected.

github PR link: https://github.com/tianocore/edk2/pull/3339

Signed-off-by: Jianyong Wu <jianyong.wu@arm.com>

Jianyong Wu (3):
  CloudHv:arm: add kernel load fs driver
  CloudHv:arm: build hob for kernel image memory as read-only
  CloudHv:arm: add kernel loader lib dsc/fdf

 ArmVirtPkg/ArmVirtCloudHv.dsc                 |   8 +-
 ArmVirtPkg/ArmVirtCloudHv.fdf                 |   1 +
 .../CloudHvKernelLoaderFsDxe.c                | 969 ++++++++++++++++++
 .../CloudHvKernelLoaderFsDxe.inf              |  55 +
 .../CloudHvVirtMemInfoLib.c                   |  66 +-
 5 files changed, 1094 insertions(+), 5 deletions(-)
 create mode 100644 ArmVirtPkg/CloudHvKernelLoaderFsDxe/CloudHvKernelLoaderFsDxe.c
 create mode 100644 ArmVirtPkg/CloudHvKernelLoaderFsDxe/CloudHvKernelLoaderFsDxe.inf

-- 
2.17.1



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#93882): https://edk2.groups.io/g/devel/message/93882
Mute This Topic: https://groups.io/mt/93715215/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [edk2-devel] [PATCH 0/3] CloudHv:arm: Enable direct kernel boot

Posted by Jianyong Wu 1 year, 5 months ago

Hello,

Long time silent. Any comments?

Thanks
Jianyong

> -----Original Message-----
> From: Jianyong Wu <jianyong.wu@arm.com>
> Sent: Friday, September 16, 2022 10:46 AM
> To: devel@edk2.groups.io; Sami Mujawar <Sami.Mujawar@arm.com>
> Cc: ardb+tianocore@kernel.org; Justin He <Justin.He@arm.com>; Jianyong
> Wu <Jianyong.Wu@arm.com>
> Subject: [PATCH 0/3] CloudHv:arm: Enable direct kernel boot
>
> Direct kernel boot removes the dependency of retrieving kernel image from
> block device. For Cloud Hypervisor, we use the following way to support it.
>
> 1. Cloud Hypervisor store kernel image into memory and put kernel info,
>    including the memory base and size, into DT; 2. When init memory in edk2,
> the kernel memory region is retrieved from
>    DT and set it as read only memory region; 3. Edk2 fetches kernel from
> memory and prepare a image handle; 4. Load kernel using LoadImage in the
> end.
>
> 1 is done in Cloud Hypervisor, 2 and 3 is done in this patch set, 4 is not
> affected.
>
> github PR link: https://github.com/tianocore/edk2/pull/3339
>
> Signed-off-by: Jianyong Wu <jianyong.wu@arm.com>
>
> Jianyong Wu (3):
>   CloudHv:arm: add kernel load fs driver
>   CloudHv:arm: build hob for kernel image memory as read-only
>   CloudHv:arm: add kernel loader lib dsc/fdf
>
>  ArmVirtPkg/ArmVirtCloudHv.dsc                 |   8 +-
>  ArmVirtPkg/ArmVirtCloudHv.fdf                 |   1 +
>  .../CloudHvKernelLoaderFsDxe.c                | 969 ++++++++++++++++++
>  .../CloudHvKernelLoaderFsDxe.inf              |  55 +
>  .../CloudHvVirtMemInfoLib.c                   |  66 +-
>  5 files changed, 1094 insertions(+), 5 deletions(-)  create mode 100644
> ArmVirtPkg/CloudHvKernelLoaderFsDxe/CloudHvKernelLoaderFsDxe.c
>  create mode 100644
> ArmVirtPkg/CloudHvKernelLoaderFsDxe/CloudHvKernelLoaderFsDxe.inf
>
> --
> 2.17.1

IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#96376): https://edk2.groups.io/g/devel/message/96376
Mute This Topic: https://groups.io/mt/93715215/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [edk2-devel] [PATCH 0/3] CloudHv:arm: Enable direct kernel boot

Posted by Sami Mujawar 1 year, 5 months ago

Hi Jianyong,

Apologies this slipped off my radar. 
I will start reviewing your patches this week.

Regards,

Sami Mujawar

On 15/11/2022, 03:00, "Jianyong Wu" <Jianyong.Wu@arm.com> wrote:

    Hello,

    Long time silent. Any comments?

    Thanks
    Jianyong

    > -----Original Message-----
    > From: Jianyong Wu <jianyong.wu@arm.com>
    > Sent: Friday, September 16, 2022 10:46 AM
    > To: devel@edk2.groups.io; Sami Mujawar <Sami.Mujawar@arm.com>
    > Cc: ardb+tianocore@kernel.org; Justin He <Justin.He@arm.com>; Jianyong
    > Wu <Jianyong.Wu@arm.com>
    > Subject: [PATCH 0/3] CloudHv:arm: Enable direct kernel boot
    >
    > Direct kernel boot removes the dependency of retrieving kernel image from
    > block device. For Cloud Hypervisor, we use the following way to support it.
    >
    > 1. Cloud Hypervisor store kernel image into memory and put kernel info,
    >    including the memory base and size, into DT; 2. When init memory in edk2,
    > the kernel memory region is retrieved from
    >    DT and set it as read only memory region; 3. Edk2 fetches kernel from
    > memory and prepare a image handle; 4. Load kernel using LoadImage in the
    > end.
    >
    > 1 is done in Cloud Hypervisor, 2 and 3 is done in this patch set, 4 is not
    > affected.
    >
    > github PR link: https://github.com/tianocore/edk2/pull/3339
    >
    > Signed-off-by: Jianyong Wu <jianyong.wu@arm.com>
    >
    > Jianyong Wu (3):
    >   CloudHv:arm: add kernel load fs driver
    >   CloudHv:arm: build hob for kernel image memory as read-only
    >   CloudHv:arm: add kernel loader lib dsc/fdf
    >
    >  ArmVirtPkg/ArmVirtCloudHv.dsc                 |   8 +-
    >  ArmVirtPkg/ArmVirtCloudHv.fdf                 |   1 +
    >  .../CloudHvKernelLoaderFsDxe.c                | 969 ++++++++++++++++++
    >  .../CloudHvKernelLoaderFsDxe.inf              |  55 +
    >  .../CloudHvVirtMemInfoLib.c                   |  66 +-
    >  5 files changed, 1094 insertions(+), 5 deletions(-)  create mode 100644
    > ArmVirtPkg/CloudHvKernelLoaderFsDxe/CloudHvKernelLoaderFsDxe.c
    >  create mode 100644
    > ArmVirtPkg/CloudHvKernelLoaderFsDxe/CloudHvKernelLoaderFsDxe.inf
    >
    > --
    > 2.17.1




-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#96397): https://edk2.groups.io/g/devel/message/96397
Mute This Topic: https://groups.io/mt/93715215/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [edk2-devel] [PATCH 0/3] CloudHv:arm: Enable direct kernel boot

Posted by Jianyong Wu 1 year, 5 months ago

Thanks Sami!

> -----Original Message-----
> From: Sami Mujawar <Sami.Mujawar@arm.com>
> Sent: Tuesday, November 15, 2022 4:52 PM
> To: Jianyong Wu <Jianyong.Wu@arm.com>; devel@edk2.groups.io
> Cc: ardb+tianocore@kernel.org; Justin He <Justin.He@arm.com>; nd
> <nd@arm.com>
> Subject: Re: [PATCH 0/3] CloudHv:arm: Enable direct kernel boot
> 
> Hi Jianyong,
> 
> Apologies this slipped off my radar.
> I will start reviewing your patches this week.
> 
> Regards,
> 
> Sami Mujawar
> 
> On 15/11/2022, 03:00, "Jianyong Wu" <Jianyong.Wu@arm.com> wrote:
> 
>     Hello,
> 
>     Long time silent. Any comments?
> 
>     Thanks
>     Jianyong
> 
>     > -----Original Message-----
>     > From: Jianyong Wu <jianyong.wu@arm.com>
>     > Sent: Friday, September 16, 2022 10:46 AM
>     > To: devel@edk2.groups.io; Sami Mujawar <Sami.Mujawar@arm.com>
>     > Cc: ardb+tianocore@kernel.org; Justin He <Justin.He@arm.com>;
> Jianyong
>     > Wu <Jianyong.Wu@arm.com>
>     > Subject: [PATCH 0/3] CloudHv:arm: Enable direct kernel boot
>     >
>     > Direct kernel boot removes the dependency of retrieving kernel image
> from
>     > block device. For Cloud Hypervisor, we use the following way to support
> it.
>     >
>     > 1. Cloud Hypervisor store kernel image into memory and put kernel info,
>     >    including the memory base and size, into DT; 2. When init memory in
> edk2,
>     > the kernel memory region is retrieved from
>     >    DT and set it as read only memory region; 3. Edk2 fetches kernel from
>     > memory and prepare a image handle; 4. Load kernel using LoadImage in
> the
>     > end.
>     >
>     > 1 is done in Cloud Hypervisor, 2 and 3 is done in this patch set, 4 is not
>     > affected.
>     >
>     > github PR link: https://github.com/tianocore/edk2/pull/3339
>     >
>     > Signed-off-by: Jianyong Wu <jianyong.wu@arm.com>
>     >
>     > Jianyong Wu (3):
>     >   CloudHv:arm: add kernel load fs driver
>     >   CloudHv:arm: build hob for kernel image memory as read-only
>     >   CloudHv:arm: add kernel loader lib dsc/fdf
>     >
>     >  ArmVirtPkg/ArmVirtCloudHv.dsc                 |   8 +-
>     >  ArmVirtPkg/ArmVirtCloudHv.fdf                 |   1 +
>     >  .../CloudHvKernelLoaderFsDxe.c                | 969 ++++++++++++++++++
>     >  .../CloudHvKernelLoaderFsDxe.inf              |  55 +
>     >  .../CloudHvVirtMemInfoLib.c                   |  66 +-
>     >  5 files changed, 1094 insertions(+), 5 deletions(-)  create mode 100644
>     > ArmVirtPkg/CloudHvKernelLoaderFsDxe/CloudHvKernelLoaderFsDxe.c
>     >  create mode 100644
>     > ArmVirtPkg/CloudHvKernelLoaderFsDxe/CloudHvKernelLoaderFsDxe.inf
>     >
>     > --
>     > 2.17.1
> 
> 



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#96399): https://edk2.groups.io/g/devel/message/96399
Mute This Topic: https://groups.io/mt/93715215/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-