[Qemu-devel] [PATCH] qtest: Use cpu address space instead of system memory

Julia Suvorova via Qemu-devel posted 1 patch 5 years, 8 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20180702065237.27899-1-jusual@mail.ru
Test checkpatch passed
Test docker-mingw@fedora passed
Test docker-quick@centos7 passed
Test s390x passed
qtest.c | 39 ++++++++++++++++++++++++++-------------
1 file changed, 26 insertions(+), 13 deletions(-)
[Qemu-devel] [PATCH] qtest: Use cpu address space instead of system memory
Posted by Julia Suvorova via Qemu-devel 5 years, 8 months ago
Some devices (like nvic in armv7m) are not accessable through
address_space_memory, therefore can not be tested with qtest.

Signed-off-by: Julia Suvorova <jusual@mail.ru>
---
 qtest.c | 39 ++++++++++++++++++++++++++-------------
 1 file changed, 26 insertions(+), 13 deletions(-)

diff --git a/qtest.c b/qtest.c
index cbbfb71114..69b9e9962b 100644
--- a/qtest.c
+++ b/qtest.c
@@ -387,19 +387,23 @@ static void qtest_process_command(CharBackend *chr, gchar **words)
 
         if (words[0][5] == 'b') {
             uint8_t data = value;
-            cpu_physical_memory_write(addr, &data, 1);
+            address_space_rw(first_cpu->as, addr, MEMTXATTRS_UNSPECIFIED,
+                             &data, 1, true);
         } else if (words[0][5] == 'w') {
             uint16_t data = value;
             tswap16s(&data);
-            cpu_physical_memory_write(addr, &data, 2);
+            address_space_rw(first_cpu->as, addr, MEMTXATTRS_UNSPECIFIED,
+                             (uint8_t *) &data, 2, true);
         } else if (words[0][5] == 'l') {
             uint32_t data = value;
             tswap32s(&data);
-            cpu_physical_memory_write(addr, &data, 4);
+            address_space_rw(first_cpu->as, addr, MEMTXATTRS_UNSPECIFIED,
+                             (uint8_t *) &data, 4, true);
         } else if (words[0][5] == 'q') {
             uint64_t data = value;
             tswap64s(&data);
-            cpu_physical_memory_write(addr, &data, 8);
+            address_space_rw(first_cpu->as, addr, MEMTXATTRS_UNSPECIFIED,
+                             (uint8_t *) &data, 8, true);
         }
         qtest_send_prefix(chr);
         qtest_send(chr, "OK\n");
@@ -417,18 +421,22 @@ static void qtest_process_command(CharBackend *chr, gchar **words)
 
         if (words[0][4] == 'b') {
             uint8_t data;
-            cpu_physical_memory_read(addr, &data, 1);
+            address_space_rw(first_cpu->as, addr, MEMTXATTRS_UNSPECIFIED,
+                             &data, 1, false);
             value = data;
         } else if (words[0][4] == 'w') {
             uint16_t data;
-            cpu_physical_memory_read(addr, &data, 2);
+            address_space_rw(first_cpu->as, addr, MEMTXATTRS_UNSPECIFIED,
+                             (uint8_t *) &data, 2, false);
             value = tswap16(data);
         } else if (words[0][4] == 'l') {
             uint32_t data;
-            cpu_physical_memory_read(addr, &data, 4);
+            address_space_rw(first_cpu->as, addr, MEMTXATTRS_UNSPECIFIED,
+                             (uint8_t *) &data, 4, false);
             value = tswap32(data);
         } else if (words[0][4] == 'q') {
-            cpu_physical_memory_read(addr, &value, 8);
+            address_space_rw(first_cpu->as, addr, MEMTXATTRS_UNSPECIFIED,
+                             (uint8_t *) &value, 8, false);
             tswap64s(&value);
         }
         qtest_send_prefix(chr);
@@ -448,7 +456,8 @@ static void qtest_process_command(CharBackend *chr, gchar **words)
         g_assert(len);
 
         data = g_malloc(len);
-        cpu_physical_memory_read(addr, data, len);
+        address_space_rw(first_cpu->as, addr, MEMTXATTRS_UNSPECIFIED,
+                         data, len, false);
 
         enc = g_malloc(2 * len + 1);
         for (i = 0; i < len; i++) {
@@ -473,7 +482,8 @@ static void qtest_process_command(CharBackend *chr, gchar **words)
         g_assert(ret == 0);
 
         data = g_malloc(len);
-        cpu_physical_memory_read(addr, data, len);
+        address_space_rw(first_cpu->as, addr, MEMTXATTRS_UNSPECIFIED,
+                         data, len, false);
         b64_data = g_base64_encode(data, len);
         qtest_send_prefix(chr);
         qtest_sendf(chr, "OK %s\n", b64_data);
@@ -507,7 +517,8 @@ static void qtest_process_command(CharBackend *chr, gchar **words)
                 data[i] = 0;
             }
         }
-        cpu_physical_memory_write(addr, data, len);
+        address_space_rw(first_cpu->as, addr, MEMTXATTRS_UNSPECIFIED,
+                         data, len, true);
         g_free(data);
 
         qtest_send_prefix(chr);
@@ -529,7 +540,8 @@ static void qtest_process_command(CharBackend *chr, gchar **words)
         if (len) {
             data = g_malloc(len);
             memset(data, pattern, len);
-            cpu_physical_memory_write(addr, data, len);
+            address_space_rw(first_cpu->as, addr, MEMTXATTRS_UNSPECIFIED,
+                             data, len, true);
             g_free(data);
         }
 
@@ -562,7 +574,8 @@ static void qtest_process_command(CharBackend *chr, gchar **words)
             out_len = MIN(out_len, len);
         }
 
-        cpu_physical_memory_write(addr, data, out_len);
+        address_space_rw(first_cpu->as, addr, MEMTXATTRS_UNSPECIFIED,
+                         data, len, true);
 
         qtest_send_prefix(chr);
         qtest_send(chr, "OK\n");
-- 
2.17.1


Re: [Qemu-devel] [PATCH] qtest: Use cpu address space instead of system memory
Posted by Stefan Hajnoczi 5 years, 8 months ago
On Mon, Jul 02, 2018 at 09:52:37AM +0300, Julia Suvorova via Qemu-devel wrote:
> Some devices (like nvic in armv7m) are not accessable through
> address_space_memory, therefore can not be tested with qtest.
> 
> Signed-off-by: Julia Suvorova <jusual@mail.ru>
> ---
>  qtest.c | 39 ++++++++++++++++++++++++++-------------
>  1 file changed, 26 insertions(+), 13 deletions(-)

Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
Re: [Qemu-devel] [PATCH] qtest: Use cpu address space instead of system memory
Posted by Paolo Bonzini 5 years, 8 months ago
On 02/07/2018 08:52, Julia Suvorova wrote:
> Some devices (like nvic in armv7m) are not accessable through
> address_space_memory, therefore can not be tested with qtest.
> 
> Signed-off-by: Julia Suvorova <jusual@mail.ru>
> ---
>  qtest.c | 39 ++++++++++++++++++++++++++-------------
>  1 file changed, 26 insertions(+), 13 deletions(-)
> 
> diff --git a/qtest.c b/qtest.c
> index cbbfb71114..69b9e9962b 100644
> --- a/qtest.c
> +++ b/qtest.c
> @@ -387,19 +387,23 @@ static void qtest_process_command(CharBackend *chr, gchar **words)
>  
>          if (words[0][5] == 'b') {
>              uint8_t data = value;
> -            cpu_physical_memory_write(addr, &data, 1);
> +            address_space_rw(first_cpu->as, addr, MEMTXATTRS_UNSPECIFIED,
> +                             &data, 1, true);
>          } else if (words[0][5] == 'w') {
>              uint16_t data = value;
>              tswap16s(&data);
> -            cpu_physical_memory_write(addr, &data, 2);
> +            address_space_rw(first_cpu->as, addr, MEMTXATTRS_UNSPECIFIED,
> +                             (uint8_t *) &data, 2, true);
>          } else if (words[0][5] == 'l') {
>              uint32_t data = value;
>              tswap32s(&data);
> -            cpu_physical_memory_write(addr, &data, 4);
> +            address_space_rw(first_cpu->as, addr, MEMTXATTRS_UNSPECIFIED,
> +                             (uint8_t *) &data, 4, true);
>          } else if (words[0][5] == 'q') {
>              uint64_t data = value;
>              tswap64s(&data);
> -            cpu_physical_memory_write(addr, &data, 8);
> +            address_space_rw(first_cpu->as, addr, MEMTXATTRS_UNSPECIFIED,
> +                             (uint8_t *) &data, 8, true);
>          }
>          qtest_send_prefix(chr);
>          qtest_send(chr, "OK\n");
> @@ -417,18 +421,22 @@ static void qtest_process_command(CharBackend *chr, gchar **words)
>  
>          if (words[0][4] == 'b') {
>              uint8_t data;
> -            cpu_physical_memory_read(addr, &data, 1);
> +            address_space_rw(first_cpu->as, addr, MEMTXATTRS_UNSPECIFIED,
> +                             &data, 1, false);
>              value = data;
>          } else if (words[0][4] == 'w') {
>              uint16_t data;
> -            cpu_physical_memory_read(addr, &data, 2);
> +            address_space_rw(first_cpu->as, addr, MEMTXATTRS_UNSPECIFIED,
> +                             (uint8_t *) &data, 2, false);
>              value = tswap16(data);
>          } else if (words[0][4] == 'l') {
>              uint32_t data;
> -            cpu_physical_memory_read(addr, &data, 4);
> +            address_space_rw(first_cpu->as, addr, MEMTXATTRS_UNSPECIFIED,
> +                             (uint8_t *) &data, 4, false);
>              value = tswap32(data);
>          } else if (words[0][4] == 'q') {
> -            cpu_physical_memory_read(addr, &value, 8);
> +            address_space_rw(first_cpu->as, addr, MEMTXATTRS_UNSPECIFIED,
> +                             (uint8_t *) &value, 8, false);
>              tswap64s(&value);
>          }
>          qtest_send_prefix(chr);
> @@ -448,7 +456,8 @@ static void qtest_process_command(CharBackend *chr, gchar **words)
>          g_assert(len);
>  
>          data = g_malloc(len);
> -        cpu_physical_memory_read(addr, data, len);
> +        address_space_rw(first_cpu->as, addr, MEMTXATTRS_UNSPECIFIED,
> +                         data, len, false);
>  
>          enc = g_malloc(2 * len + 1);
>          for (i = 0; i < len; i++) {
> @@ -473,7 +482,8 @@ static void qtest_process_command(CharBackend *chr, gchar **words)
>          g_assert(ret == 0);
>  
>          data = g_malloc(len);
> -        cpu_physical_memory_read(addr, data, len);
> +        address_space_rw(first_cpu->as, addr, MEMTXATTRS_UNSPECIFIED,
> +                         data, len, false);
>          b64_data = g_base64_encode(data, len);
>          qtest_send_prefix(chr);
>          qtest_sendf(chr, "OK %s\n", b64_data);
> @@ -507,7 +517,8 @@ static void qtest_process_command(CharBackend *chr, gchar **words)
>                  data[i] = 0;
>              }
>          }
> -        cpu_physical_memory_write(addr, data, len);
> +        address_space_rw(first_cpu->as, addr, MEMTXATTRS_UNSPECIFIED,
> +                         data, len, true);
>          g_free(data);
>  
>          qtest_send_prefix(chr);
> @@ -529,7 +540,8 @@ static void qtest_process_command(CharBackend *chr, gchar **words)
>          if (len) {
>              data = g_malloc(len);
>              memset(data, pattern, len);
> -            cpu_physical_memory_write(addr, data, len);
> +            address_space_rw(first_cpu->as, addr, MEMTXATTRS_UNSPECIFIED,
> +                             data, len, true);
>              g_free(data);
>          }
>  
> @@ -562,7 +574,8 @@ static void qtest_process_command(CharBackend *chr, gchar **words)
>              out_len = MIN(out_len, len);
>          }
>  
> -        cpu_physical_memory_write(addr, data, out_len);
> +        address_space_rw(first_cpu->as, addr, MEMTXATTRS_UNSPECIFIED,
> +                         data, len, true);
>  
>          qtest_send_prefix(chr);
>          qtest_send(chr, "OK\n");
> 

Queued, thanks.

Paolo