Re: [PATCH] tests/qtest/boot-serial-test: Add support on LoongArch system

[maobibo]

On 2024/5/8 下午5:00, Thomas Huth wrote:

On 08/05/2024 10.55, Bibo Mao wrote:

Add boot-serial-test test case support on LoongArch system.


... and also the filter tests?

yes, it is :) Will update changelog in next version.




Signed-off-by: Bibo Mao 
---
  tests/qtest/boot-serial-test.c | 10 ++
  tests/qtest/meson.build    |  4 
  2 files changed, 14 insertions(+)

diff --git a/tests/qtest/boot-serial-test.c 
b/tests/qtest/boot-serial-test.c

index e3b7d65fe5..631015e8c8 100644
--- a/tests/qtest/boot-serial-test.c
+++ b/tests/qtest/boot-serial-test.c
@@ -129,6 +129,14 @@ static const uint8_t kernel_stm32vldiscovery[] = {
  0x04, 0x38, 0x01, 0x40  /* 0x40013804 = USART1 
TXD */

  };
+static const uint8_t bios_loongarch64[] = {
+    0x0c, 0xc0, 0x3f, 0x14,    /* lu12i.w $t0,  0x1fe00    */
+    0x8c, 0x81, 0x87, 0x03,    /* ori $t0,  $t0, 0x1e0 */
+    0x0d, 0x50, 0x81, 0x03,    /* li.w    $t1,  'T'    */
+    0x8d, 0x01, 0x00, 0x29,    /* st.b    $t1, $t0,  0 */
+    0xff, 0xf3, 0xff, 0x53,    /*  b  -16  #loop   */
+};
+
  typedef struct testdef {
  const char *arch;   /* Target architecture */
  const char *machine;    /* Name of the machine */
@@ -181,6 +189,8 @@ static const testdef_t tests[] = {
  { "arm", "microbit", "", "T", sizeof(kernel_nrf51), kernel_nrf51 },
  { "arm", "stm32vldiscovery", "", "T",
    sizeof(kernel_stm32vldiscovery), kernel_stm32vldiscovery },
+    { "loongarch64", "virt", "-cpu max", "TT", sizeof(bios_loongarch64),
+  NULL, bios_loongarch64 },
  { NULL }
  };
diff --git a/tests/qtest/meson.build b/tests/qtest/meson.build
index 6f2f594ace..6619b630e6 100644
--- a/tests/qtest/meson.build
+++ b/tests/qtest/meson.build
@@ -256,6 +256,10 @@ qtests_s390x = \
  qtests_riscv32 = \
    (config_all_devices.has_key('CONFIG_SIFIVE_E_AON') ? 
['sifive-e-aon-watchdog-test'] : [])

+ qtests_loongarch64 = \
+  qtests_filter + \
+  ['boot-serial-test']


It's already a little bit messed up, but I think we originally had the 
entries in alphabetical order, so I'd like to suggest to add this 
between qtests_hppa and qtests_m68k instead.

sure, will do.

Regards
Bibo Mao


  Thomas



  qos_test_ss = ss.source_set()
  qos_test_ss.add(
    'ac97-test.c',

base-commit: 4e66a08546a2588a4667766a1edab9caccf24ce3

Re: [PATCH] tests/qtest/boot-serial-test: Add support on LoongArch system

[Thomas Huth]

On 08/05/2024 10.55, Bibo Mao wrote:

Add boot-serial-test test case support on LoongArch system.


... and also the filter tests?


Signed-off-by: Bibo Mao 
---
  tests/qtest/boot-serial-test.c | 10 ++
  tests/qtest/meson.build|  4 
  2 files changed, 14 insertions(+)

diff --git a/tests/qtest/boot-serial-test.c b/tests/qtest/boot-serial-test.c
index e3b7d65fe5..631015e8c8 100644
--- a/tests/qtest/boot-serial-test.c
+++ b/tests/qtest/boot-serial-test.c
@@ -129,6 +129,14 @@ static const uint8_t kernel_stm32vldiscovery[] = {
  0x04, 0x38, 0x01, 0x40  /* 0x40013804 = USART1 TXD */
  };
  
+static const uint8_t bios_loongarch64[] = {

+0x0c, 0xc0, 0x3f, 0x14, /* lu12i.w $t0, 0x1fe00*/
+0x8c, 0x81, 0x87, 0x03, /* ori $t0, $t0, 0x1e0 */
+0x0d, 0x50, 0x81, 0x03, /* li.w$t1, 'T'*/
+0x8d, 0x01, 0x00, 0x29, /* st.b$t1, $t0, 0 */
+0xff, 0xf3, 0xff, 0x53, /*  b  -16  # loop */
+};
+
  typedef struct testdef {
  const char *arch;   /* Target architecture */
  const char *machine;/* Name of the machine */
@@ -181,6 +189,8 @@ static const testdef_t tests[] = {
  { "arm", "microbit", "", "T", sizeof(kernel_nrf51), kernel_nrf51 },
  { "arm", "stm32vldiscovery", "", "T",
sizeof(kernel_stm32vldiscovery), kernel_stm32vldiscovery },
+{ "loongarch64", "virt", "-cpu max", "TT", sizeof(bios_loongarch64),
+  NULL, bios_loongarch64 },
  
  { NULL }

  };
diff --git a/tests/qtest/meson.build b/tests/qtest/meson.build
index 6f2f594ace..6619b630e6 100644
--- a/tests/qtest/meson.build
+++ b/tests/qtest/meson.build
@@ -256,6 +256,10 @@ qtests_s390x = \
  qtests_riscv32 = \
(config_all_devices.has_key('CONFIG_SIFIVE_E_AON') ? 
['sifive-e-aon-watchdog-test'] : [])
  
+ qtests_loongarch64 = \

+  qtests_filter + \
+  ['boot-serial-test']


It's already a little bit messed up, but I think we originally had the 
entries in alphabetical order, so I'd like to suggest to add this between 
qtests_hppa and qtests_m68k instead.


 Thomas



  qos_test_ss = ss.source_set()
  qos_test_ss.add(
'ac97-test.c',

base-commit: 4e66a08546a2588a4667766a1edab9caccf24ce3

[PATCH] tests/qtest/boot-serial-test: Add support on LoongArch system

[Bibo Mao]

Add boot-serial-test test case support on LoongArch system.

Signed-off-by: Bibo Mao 
---
 tests/qtest/boot-serial-test.c | 10 ++
 tests/qtest/meson.build|  4 
 2 files changed, 14 insertions(+)

diff --git a/tests/qtest/boot-serial-test.c b/tests/qtest/boot-serial-test.c
index e3b7d65fe5..631015e8c8 100644
--- a/tests/qtest/boot-serial-test.c
+++ b/tests/qtest/boot-serial-test.c
@@ -129,6 +129,14 @@ static const uint8_t kernel_stm32vldiscovery[] = {
 0x04, 0x38, 0x01, 0x40  /* 0x40013804 = USART1 TXD */
 };
 
+static const uint8_t bios_loongarch64[] = {
+0x0c, 0xc0, 0x3f, 0x14, /* lu12i.w $t0, 0x1fe00*/
+0x8c, 0x81, 0x87, 0x03, /* ori $t0, $t0, 0x1e0 */
+0x0d, 0x50, 0x81, 0x03, /* li.w$t1, 'T'*/
+0x8d, 0x01, 0x00, 0x29, /* st.b$t1, $t0, 0 */
+0xff, 0xf3, 0xff, 0x53, /*  b  -16  # loop */
+};
+
 typedef struct testdef {
 const char *arch;   /* Target architecture */
 const char *machine;/* Name of the machine */
@@ -181,6 +189,8 @@ static const testdef_t tests[] = {
 { "arm", "microbit", "", "T", sizeof(kernel_nrf51), kernel_nrf51 },
 { "arm", "stm32vldiscovery", "", "T",
   sizeof(kernel_stm32vldiscovery), kernel_stm32vldiscovery },
+{ "loongarch64", "virt", "-cpu max", "TT", sizeof(bios_loongarch64),
+  NULL, bios_loongarch64 },
 
 { NULL }
 };
diff --git a/tests/qtest/meson.build b/tests/qtest/meson.build
index 6f2f594ace..6619b630e6 100644
--- a/tests/qtest/meson.build
+++ b/tests/qtest/meson.build
@@ -256,6 +256,10 @@ qtests_s390x = \
 qtests_riscv32 = \
   (config_all_devices.has_key('CONFIG_SIFIVE_E_AON') ? 
['sifive-e-aon-watchdog-test'] : [])
 
+ qtests_loongarch64 = \
+  qtests_filter + \
+  ['boot-serial-test']
+
 qos_test_ss = ss.source_set()
 qos_test_ss.add(
   'ac97-test.c',

base-commit: 4e66a08546a2588a4667766a1edab9caccf24ce3
-- 
2.39.3