Re: [Minios-devel] [UNIKRAFT PATCH V3 0/6] Enable solo5 platform support

["Haibo Xu (Arm Technology China)" <Haibo.Xu@xxxxxxx>]

Hi Felipe,

I have verified the build with helloworld app, and it works well.
You can following the below steps to have a try.

Regards,
Haibo
---------------------------------------------------------------------------------------------------
1. Include Simon's patch to support external platform library
   
(https://lists.xenproject.org/archives/html/minios-devel/2019-05/msg00165.html)
2. Put this patch into a dedicated repo(as an external platform library)-> 
attachment 1.PNG
3. Change the helloworld app Makefile to include the external platform -> 
attachment 2.PNG
4. Configure the helloworld app to use "solo5" platform, and then build the app.
   In the helloworld/build directory, you will find the monitor and app binary. 
-> attachment 3.PNG


On 2019/8/21 18:34, Felipe Huici wrote:
> Hi Haibo,
>
> Thanks a lot for the patches. It tried compiling this but I get the following 
> linker error:
>
> # gcc  -nostdinc -nostdlib -Wl,--omagic -Wl,--build-id=none  
> -Wl,--gc-sections -no-pie  -Wl,-m,elf_x86_64 
> -Wl,-dT,/root/workspace/gitlab/unikraft/upstreaming/plats/solo5/link64.lds 
> -Wl,-T/root/workspace/gitlab/unikraft/upstreaming/unikraft/lib/ukdebug/extra.ld
>  
> -Wl,-T/root/workspace/gitlab/unikraft/upstreaming/unikraft/lib/vfscore/extra.ld
>  
> /root/workspace/gitlab/unikraft/upstreaming/apps/helloworld/build/helloworld_solo5-x86_64.o
>  
> /root/workspace/gitlab/unikraft/upstreaming/apps/helloworld/build/libsolo5/origin/solo5-0.4.1/bindings/hvt/solo5_hvt.o
>  -o 
> /root/workspace/gitlab/unikraft/upstreaming/apps/helloworld/build/helloworld_solo5-x86_64
> /usr/bin/ld: .comment not found for insert
>
> If I add a lilne to put that section in, I then get undefined reference 
> errors for _tls_start and _tls_end . Both these symbols and the .comment line 
> appear in the internal platforms' linker scripts. Does this actually build 
> for you?
>
> Thanks,
>
> -- Felipe
>
> On 16.08.19, 04:59, "Minios-devel on behalf of Haibo Xu" 
> <minios-devel-bounces@xxxxxxxxxxxxxxxxxxxx on behalf of haibo.xu@xxxxxxx> 
> wrote:
>
>     Hi,
>
>     As discussed with Simon and Ricardo in the email loop,
>     Solo5(formerly known as ukvm) would make most sense as
>     another and new platform library for the unikraft, this
>     patch set will try to enable the solo5 platform support.
>
>     Currently, the implementation has been tested with the
>     hello-world application on both x86-64 and arm64 machine.
>
>     v1->v2:
>             - add a menu option to solo5 platform that enable
>               build of solo5 monitor;
>             - change the solo5 src codes download method, and
>               use the unikraft build system "fetch" and "extract"
>               method.
>     v2->v3:
>             - put the solo5 as an external platform.
>
>     Regards,
>
>     Haibo
>
>     Haibo Xu (6):
>       Add basic documentations.
>       external plat: Add the solo5 API declaration in solo5.h
>       external plat: Add solo5 platform Config.uk file
>       external plat: Add solo5 platform link files.
>       external plat: Add solo5 platform Makefile.uk
>       external plat: Add platform interface implementation
>
>      CODING_STYLE.md       |   4 +
>      CONTRIBUTING.md       |   4 +
>      COPYING.md            |  46 +++++++
>      Config.uk             |  17 +++
>      Linker.uk             |  61 +++++++++
>      MAINTAINERS.md        |  11 ++
>      Makefile.uk           |  56 +++++++++
>      README.md             |   5 +
>      console.c             |  45 +++++++
>      include/solo5/setup.h |  49 ++++++++
>      include/solo5/solo5.h | 280 ++++++++++++++++++++++++++++++++++++++++++
>      io.c                  |  41 +++++++
>      irq.c                 |  28 +++++
>      lcpu.c                |  75 +++++++++++
>      link64.lds            | 119 ++++++++++++++++++
>      memory.c              |  81 ++++++++++++
>      setup.c               |  42 +++++++
>      shutdown.c            |  37 ++++++
>      time.c                |  56 +++++++++
>      19 files changed, 1057 insertions(+)
>      create mode 100644 CODING_STYLE.md
>      create mode 100644 CONTRIBUTING.md
>      create mode 100644 COPYING.md
>      create mode 100644 Config.uk
>      create mode 100644 Linker.uk
>      create mode 100644 MAINTAINERS.md
>      create mode 100644 Makefile.uk
>      create mode 100644 README.md
>      create mode 100644 console.c
>      create mode 100644 include/solo5/setup.h
>      create mode 100644 include/solo5/solo5.h
>      create mode 100644 io.c
>      create mode 100644 irq.c
>      create mode 100644 lcpu.c
>      create mode 100644 link64.lds
>      create mode 100644 memory.c
>      create mode 100644 setup.c
>      create mode 100644 shutdown.c
>      create mode 100644 time.c
>
>     --
>     2.17.1
>
>
>     _______________________________________________
>     Minios-devel mailing list
>     Minios-devel@xxxxxxxxxxxxxxxxxxxx
>     https://lists.xenproject.org/mailman/listinfo/minios-devel
>
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.

Attachment: 1.PNG
Description: 1.PNG

Attachment: 2.PNG
Description: 2.PNG

Attachment: 3.PNG
Description: 3.PNG

_______________________________________________
Minios-devel mailing list
Minios-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/minios-devel