diff mbox series

[42/43] windbg: implemented kd_api_fill_memory

Message ID 150642408350.3900.16985989168069353852.stgit@Misha-PC.lan02.inno
State New
Headers show
Series Windbg supporting | expand

Commit Message

Mikhail Abakumov Sept. 26, 2017, 11:08 a.m. UTC
Signed-off-by: Mihail Abakumov <mikhail.abakumov@ispras.ru>
Signed-off-by: Pavel Dovgalyuk <dovgaluk@ispras.ru>
Signed-off-by: Dmitriy Koltunov <koltunov@ispras.ru>
---
 include/exec/windbgstub-utils.h |    1 +
 windbgstub-utils.c              |   36 ++++++++++++++++++++++++++++++++++++
 windbgstub.c                    |    4 ++++
 3 files changed, 41 insertions(+)

Comments

Ladi Prosek Oct. 3, 2017, 2 p.m. UTC | #1
On Tue, Sep 26, 2017 at 1:08 PM, Mihail Abakumov
<mikhail.abakumov@ispras.ru> wrote:
> Signed-off-by: Mihail Abakumov <mikhail.abakumov@ispras.ru>
> Signed-off-by: Pavel Dovgalyuk <dovgaluk@ispras.ru>
> Signed-off-by: Dmitriy Koltunov <koltunov@ispras.ru>
> ---
>  include/exec/windbgstub-utils.h |    1 +
>  windbgstub-utils.c              |   36 ++++++++++++++++++++++++++++++++++++
>  windbgstub.c                    |    4 ++++
>  3 files changed, 41 insertions(+)
>
> diff --git a/include/exec/windbgstub-utils.h b/include/exec/windbgstub-utils.h
> index ba001ddb42..d1a10c862e 100755
> --- a/include/exec/windbgstub-utils.h
> +++ b/include/exec/windbgstub-utils.h
> @@ -119,6 +119,7 @@ void kd_api_get_version(CPUState *cpu, PacketData *pd);
>  void kd_api_read_msr(CPUState *cpu, PacketData *pd);
>  void kd_api_write_msr(CPUState *cpu, PacketData *pd);
>  void kd_api_search_memory(CPUState *cpu, PacketData *pd);
> +void kd_api_fill_memory(CPUState *cpu, PacketData *pd);
>  void kd_api_unsupported(CPUState *cpu, PacketData *pd);
>
>  SizedBuf kd_gen_exception_sc(CPUState *cpu);
> diff --git a/windbgstub-utils.c b/windbgstub-utils.c
> index 2d3fbcc41c..3832e0b116 100755
> --- a/windbgstub-utils.c
> +++ b/windbgstub-utils.c
> @@ -1570,6 +1570,42 @@ void kd_api_search_memory(CPUState *cpu, PacketData *pd)
>      pd->extra_size = 0;
>  }
>
> +void kd_api_fill_memory(CPUState *cpu, PacketData *pd)
> +{
> +    DBGKD_FILL_MEMORY *m64c = &pd->m64.u.FillMemory;
> +
> +    uint32_t len = ldl_p(&m64c->Length);
> +    target_ulong addr = ldq_p(&m64c->Address);
> +    uint16_t pattern = ldl_p(&m64c->PatternLength);
> +    uint16_t flags = ldl_p(&m64c->Flags);
> +
> +    uint8_t mem[len];

Same as patch 41, break len into, say, page-sized regions and call
cpu_memory_rw_debug in a loop.

> +    int i, err;
> +    for (i = 0; i < len; ++i) {
> +        mem[i] = pd->extra[i % pattern];
> +    }
> +
> +    switch (flags) {
> +    case DBGKD_FILL_MEMORY_VIRTUAL:
> +        err = cpu_memory_rw_debug(cpu, addr, mem, len, 1);
> +        if (err) {
> +            pd->m64.ReturnStatus = STATUS_UNSUCCESSFUL;
> +            WINDBG_DEBUG("fill_memory: No physical page mapped: " FMT_ADDR,
> +                        addr);
> +        }
> +        break;
> +
> +    case DBGKD_FILL_MEMORY_PHYSICAL:
> +        cpu_physical_memory_rw(addr, mem, len, 1);
> +        break;
> +
> +    default:
> +        break;
> +    }
> +
> +    pd->extra_size = 0;
> +}
> +
>  void kd_api_unsupported(CPUState *cpu, PacketData *pd)
>  {
>      WINDBG_ERROR("Catched unimplemented api %s",
> diff --git a/windbgstub.c b/windbgstub.c
> index 7cff6bc514..b01ddcf009 100755
> --- a/windbgstub.c
> +++ b/windbgstub.c
> @@ -209,6 +209,10 @@ static void windbg_process_manipulate_packet(ParsingContext *ctx)
>          kd_api_search_memory(cpu, &ctx->data);
>          break;
>
> +    case DbgKdFillMemoryApi:
> +        kd_api_fill_memory(cpu, &ctx->data);
> +        break;
> +
>      default:
>          kd_api_unsupported(cpu, &ctx->data);
>          break;
>
diff mbox series

Patch

diff --git a/include/exec/windbgstub-utils.h b/include/exec/windbgstub-utils.h
index ba001ddb42..d1a10c862e 100755
--- a/include/exec/windbgstub-utils.h
+++ b/include/exec/windbgstub-utils.h
@@ -119,6 +119,7 @@  void kd_api_get_version(CPUState *cpu, PacketData *pd);
 void kd_api_read_msr(CPUState *cpu, PacketData *pd);
 void kd_api_write_msr(CPUState *cpu, PacketData *pd);
 void kd_api_search_memory(CPUState *cpu, PacketData *pd);
+void kd_api_fill_memory(CPUState *cpu, PacketData *pd);
 void kd_api_unsupported(CPUState *cpu, PacketData *pd);
 
 SizedBuf kd_gen_exception_sc(CPUState *cpu);
diff --git a/windbgstub-utils.c b/windbgstub-utils.c
index 2d3fbcc41c..3832e0b116 100755
--- a/windbgstub-utils.c
+++ b/windbgstub-utils.c
@@ -1570,6 +1570,42 @@  void kd_api_search_memory(CPUState *cpu, PacketData *pd)
     pd->extra_size = 0;
 }
 
+void kd_api_fill_memory(CPUState *cpu, PacketData *pd)
+{
+    DBGKD_FILL_MEMORY *m64c = &pd->m64.u.FillMemory;
+
+    uint32_t len = ldl_p(&m64c->Length);
+    target_ulong addr = ldq_p(&m64c->Address);
+    uint16_t pattern = ldl_p(&m64c->PatternLength);
+    uint16_t flags = ldl_p(&m64c->Flags);
+
+    uint8_t mem[len];
+    int i, err;
+    for (i = 0; i < len; ++i) {
+        mem[i] = pd->extra[i % pattern];
+    }
+
+    switch (flags) {
+    case DBGKD_FILL_MEMORY_VIRTUAL:
+        err = cpu_memory_rw_debug(cpu, addr, mem, len, 1);
+        if (err) {
+            pd->m64.ReturnStatus = STATUS_UNSUCCESSFUL;
+            WINDBG_DEBUG("fill_memory: No physical page mapped: " FMT_ADDR,
+                        addr);
+        }
+        break;
+
+    case DBGKD_FILL_MEMORY_PHYSICAL:
+        cpu_physical_memory_rw(addr, mem, len, 1);
+        break;
+
+    default:
+        break;
+    }
+
+    pd->extra_size = 0;
+}
+
 void kd_api_unsupported(CPUState *cpu, PacketData *pd)
 {
     WINDBG_ERROR("Catched unimplemented api %s",
diff --git a/windbgstub.c b/windbgstub.c
index 7cff6bc514..b01ddcf009 100755
--- a/windbgstub.c
+++ b/windbgstub.c
@@ -209,6 +209,10 @@  static void windbg_process_manipulate_packet(ParsingContext *ctx)
         kd_api_search_memory(cpu, &ctx->data);
         break;
 
+    case DbgKdFillMemoryApi:
+        kd_api_fill_memory(cpu, &ctx->data);
+        break;
+
     default:
         kd_api_unsupported(cpu, &ctx->data);
         break;