[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 22/43] windbg: implemented kd_api_read_virtual_m
From: |
Ladi Prosek |
Subject: |
Re: [Qemu-devel] [PATCH 22/43] windbg: implemented kd_api_read_virtual_memory and kd_api_read_virtual_memory |
Date: |
Tue, 3 Oct 2017 14:24:07 +0200 |
nit: The subject line has "kd_api_read_virtual_memory" twice in it.
Was probably meant to be
[PATCH 22/43] windbg: implemented kd_api_read_virtual_memory and
kd_api_write_virtual_memory
On Tue, Sep 26, 2017 at 1:06 PM, Mihail Abakumov
<address@hidden> wrote:
> Signed-off-by: Mihail Abakumov <address@hidden>
> Signed-off-by: Pavel Dovgalyuk <address@hidden>
> Signed-off-by: Dmitriy Koltunov <address@hidden>
> ---
> include/exec/windbgstub-utils.h | 2 ++
> windbgstub-utils.c | 47
> +++++++++++++++++++++++++++++++++++++++
> windbgstub.c | 8 +++++++
> 3 files changed, 57 insertions(+)
>
> diff --git a/include/exec/windbgstub-utils.h b/include/exec/windbgstub-utils.h
> index ca21af7c91..c1ff1cec57 100755
> --- a/include/exec/windbgstub-utils.h
> +++ b/include/exec/windbgstub-utils.h
> @@ -102,6 +102,8 @@ typedef struct SizedBuf {
> buf.size = 0; \
> } while (false)
>
> +void kd_api_read_virtual_memory(CPUState *cpu, PacketData *pd);
> +void kd_api_write_virtual_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 e7b381f829..1bde60a1e9 100755
> --- a/windbgstub-utils.c
> +++ b/windbgstub-utils.c
> @@ -30,6 +30,53 @@ typedef struct KDData {
>
> static KDData *kd;
>
> +void kd_api_read_virtual_memory(CPUState *cpu, PacketData *pd)
> +{
> + DBGKD_READ_MEMORY64 *mem = &pd->m64.u.ReadMemory;
> + uint32_t len;
> + target_ulong addr;
> + int err;
> +
> + len = MIN(ldl_p(&mem->TransferCount), PACKET_MAX_SIZE - M64_SIZE);
> + addr = ldtul_p(&mem->TargetBaseAddress);
> + err = cpu_memory_rw_debug(cpu, addr, pd->extra, len, 0);
> +
> + if (!err) {
> + pd->extra_size = len;
> + mem->ActualBytesRead = ldl_p(&len);
> + } else {
> + pd->extra_size = 0;
> + mem->ActualBytesRead = 0;
> + pd->m64.ReturnStatus = STATUS_UNSUCCESSFUL;
> +
> + WINDBG_DEBUG("read_virtual_memory: No physical page mapped: "
> FMT_ADDR,
> + (target_ulong) mem->TargetBaseAddress);
> + }
> +}
> +
> +void kd_api_write_virtual_memory(CPUState *cpu, PacketData *pd)
> +{
> + DBGKD_WRITE_MEMORY64 *mem = &pd->m64.u.WriteMemory;
> + uint32_t len;
> + target_ulong addr;
> + int err;
> +
> + len = MIN(ldl_p(&mem->TransferCount), pd->extra_size);
> + addr = ldtul_p(&mem->TargetBaseAddress);
> + err = cpu_memory_rw_debug(cpu, addr, pd->extra, len, 1);
> +
> + pd->extra_size = 0;
> + if (!err) {
> + mem->ActualBytesWritten = ldl_p(&len);
> + } else {
> + mem->ActualBytesWritten = 0;
> + pd->m64.ReturnStatus = STATUS_UNSUCCESSFUL;
> +
> + WINDBG_DEBUG("read_write_memory: No physical page mapped: " FMT_ADDR,
> + (target_ulong) mem->TargetBaseAddress);
> + }
> +}
> +
> void kd_api_unsupported(CPUState *cpu, PacketData *pd)
> {
> WINDBG_ERROR("Catched unimplemented api %s",
> diff --git a/windbgstub.c b/windbgstub.c
> index 50b4ef0dea..327fe83f70 100755
> --- a/windbgstub.c
> +++ b/windbgstub.c
> @@ -133,6 +133,14 @@ static void
> windbg_process_manipulate_packet(ParsingContext *ctx)
>
> switch (ctx->data.m64.ApiNumber) {
>
> + case DbgKdReadVirtualMemoryApi:
> + kd_api_read_virtual_memory(cpu, &ctx->data);
> + break;
> +
> + case DbgKdWriteVirtualMemoryApi:
> + kd_api_write_virtual_memory(cpu, &ctx->data);
> + break;
> +
> default:
> kd_api_unsupported(cpu, &ctx->data);
> break;
>
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Qemu-devel] [PATCH 22/43] windbg: implemented kd_api_read_virtual_memory and kd_api_read_virtual_memory,
Ladi Prosek <=