[PATCH v9 0/2] Basic early_printk and smoke test implementation

Oleksii Kurochko posted 2 patches 1 year, 2 months ago
Test gitlab-ci passed
Patches applied successfully (tree, apply log)
git fetch https://gitlab.com/xen-project/patchew/xen tags/patchew/cover.1675772422.git.oleksii.kurochko@gmail.com
There is a newer version of this series
automation/gitlab-ci/test.yaml            | 20 ++++++++++++++
automation/scripts/qemu-smoke-riscv64.sh  | 20 ++++++++++++++
xen/arch/riscv/Kconfig.debug              |  5 ++++
xen/arch/riscv/Makefile                   |  1 +
xen/arch/riscv/early_printk.c             | 33 +++++++++++++++++++++++
xen/arch/riscv/include/asm/early_printk.h | 12 +++++++++
xen/arch/riscv/setup.c                    |  4 +++
7 files changed, 95 insertions(+)
create mode 100755 automation/scripts/qemu-smoke-riscv64.sh
create mode 100644 xen/arch/riscv/early_printk.c
create mode 100644 xen/arch/riscv/include/asm/early_printk.h
[PATCH v9 0/2] Basic early_printk and smoke test implementation
Posted by Oleksii Kurochko 1 year, 2 months ago
The patch series introduces the following:
- the minimal set of headers and changes inside them.
- SBI (RISC-V Supervisor Binary Interface) things necessary for basic
  early_printk implementation.
- things needed to set up the stack.
- early_printk() function to print only strings.
- RISC-V smoke test which checks if  "Hello from C env" message is
  present in serial.tmp

The patch series is rebased on top of the patch "include/types: move
stddef.h-kind types to common header" [1]

[1] https://lore.kernel.org/xen-devel/5a0a9e2a-c116-21b5-8081-db75fe4178d7@suse.com/

---
Changes in V9:
 - Bring "cmodel=medany" check back to be sure that C function is safe to be called
   in early boot when MMU is off as 
---
Changes in V8:
 - Set "needs: archlinux-current-gcc-riscv64-debug" in test.yaml
   for RISCV job as CONFIG_EARLY_PRINTK is available only when
   CONFIG_DEBUG is available.
---
Changes in V7:
 - Fix dependecy for qemu-smoke-riscv64-gcc job
---
Changes in V6:
 - Rename container name in test.yaml for .qemu-riscv64.
---
Changes in V5:
  - Nothing changed
---
Changes in V4:
  - Nothing changed
---
Changes in V3:
  - Nothing changed
  - All mentioned comments by Stefano in Xen mailing list will be
    fixed as a separate patch out of this patch series.
---

Oleksii Kurochko (2):
  xen/riscv: introduce early_printk basic stuff
  automation: add RISC-V smoke test

 automation/gitlab-ci/test.yaml            | 20 ++++++++++++++
 automation/scripts/qemu-smoke-riscv64.sh  | 20 ++++++++++++++
 xen/arch/riscv/Kconfig.debug              |  5 ++++
 xen/arch/riscv/Makefile                   |  1 +
 xen/arch/riscv/early_printk.c             | 33 +++++++++++++++++++++++
 xen/arch/riscv/include/asm/early_printk.h | 12 +++++++++
 xen/arch/riscv/setup.c                    |  4 +++
 7 files changed, 95 insertions(+)
 create mode 100755 automation/scripts/qemu-smoke-riscv64.sh
 create mode 100644 xen/arch/riscv/early_printk.c
 create mode 100644 xen/arch/riscv/include/asm/early_printk.h

-- 
2.39.0
Re: [PATCH v9 0/2] Basic early_printk and smoke test implementation
Posted by Oleksii 1 year, 2 months ago
Hi all,

Please look at V10 and skip V9 as I missed to add the check with
medany.

Sorry.

~ Oleksii
On Tue, 2023-02-07 at 14:41 +0200, Oleksii Kurochko wrote:
> The patch series introduces the following:
> - the minimal set of headers and changes inside them.
> - SBI (RISC-V Supervisor Binary Interface) things necessary for basic
>   early_printk implementation.
> - things needed to set up the stack.
> - early_printk() function to print only strings.
> - RISC-V smoke test which checks if  "Hello from C env" message is
>   present in serial.tmp
> 
> The patch series is rebased on top of the patch "include/types: move
> stddef.h-kind types to common header" [1]
> 
> [1]
> https://lore.kernel.org/xen-devel/5a0a9e2a-c116-21b5-8081-db75fe4178d7@suse.com/
> 
> ---
> Changes in V9:
>  - Bring "cmodel=medany" check back to be sure that C function is
> safe to be called
>    in early boot when MMU is off as 
> ---
> Changes in V8:
>  - Set "needs: archlinux-current-gcc-riscv64-debug" in test.yaml
>    for RISCV job as CONFIG_EARLY_PRINTK is available only when
>    CONFIG_DEBUG is available.
> ---
> Changes in V7:
>  - Fix dependecy for qemu-smoke-riscv64-gcc job
> ---
> Changes in V6:
>  - Rename container name in test.yaml for .qemu-riscv64.
> ---
> Changes in V5:
>   - Nothing changed
> ---
> Changes in V4:
>   - Nothing changed
> ---
> Changes in V3:
>   - Nothing changed
>   - All mentioned comments by Stefano in Xen mailing list will be
>     fixed as a separate patch out of this patch series.
> ---
> 
> Oleksii Kurochko (2):
>   xen/riscv: introduce early_printk basic stuff
>   automation: add RISC-V smoke test
> 
>  automation/gitlab-ci/test.yaml            | 20 ++++++++++++++
>  automation/scripts/qemu-smoke-riscv64.sh  | 20 ++++++++++++++
>  xen/arch/riscv/Kconfig.debug              |  5 ++++
>  xen/arch/riscv/Makefile                   |  1 +
>  xen/arch/riscv/early_printk.c             | 33
> +++++++++++++++++++++++
>  xen/arch/riscv/include/asm/early_printk.h | 12 +++++++++
>  xen/arch/riscv/setup.c                    |  4 +++
>  7 files changed, 95 insertions(+)
>  create mode 100755 automation/scripts/qemu-smoke-riscv64.sh
>  create mode 100644 xen/arch/riscv/early_printk.c
>  create mode 100644 xen/arch/riscv/include/asm/early_printk.h
>