[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4 02/11] sdcard: replace DPRINTF() by trace eve
From: |
Alistair Francis |
Subject: |
Re: [Qemu-devel] [PATCH v4 02/11] sdcard: replace DPRINTF() by trace events |
Date: |
Fri, 16 Feb 2018 13:01:34 -0800 |
On Thu, Feb 15, 2018 at 2:05 PM, Philippe Mathieu-Daudé <address@hidden> wrote:
> Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
Reviewed-by: Alistair Francis <address@hidden>
Alistair
> ---
> hw/sd/sd.c | 32 ++++++++++++++++++++++++++------
> hw/sd/trace-events | 6 ++++++
> 2 files changed, 32 insertions(+), 6 deletions(-)
>
> diff --git a/hw/sd/sd.c b/hw/sd/sd.c
> index ce1f2fdf76..72e9b47e34 100644
> --- a/hw/sd/sd.c
> +++ b/hw/sd/sd.c
> @@ -40,6 +40,7 @@
> #include "qemu/error-report.h"
> #include "qemu/timer.h"
> #include "qemu/log.h"
> +#include "trace.h"
>
> //#define DEBUG_SD 1
>
> @@ -132,6 +133,26 @@ struct SDState {
> bool cmd_line;
> };
>
> +static const char *sd_state_name(enum SDCardStates state)
> +{
> + static const char *state_name[] = {
> + [sd_idle_state] = "idle",
> + [sd_ready_state] = "ready",
> + [sd_identification_state] = "identification",
> + [sd_standby_state] = "standby",
> + [sd_transfer_state] = "transfer",
> + [sd_sendingdata_state] = "sendingdata",
> + [sd_receivingdata_state] = "receivingdata",
> + [sd_programming_state] = "programming",
> + [sd_disconnect_state] = "disconnect",
> + };
> + if (state == sd_inactive_state) {
> + return "inactive";
> + }
> + assert(state <= ARRAY_SIZE(state_name));
> + return state_name[state];
> +}
> +
> static uint8_t sd_get_dat_lines(SDState *sd)
> {
> return sd->enable ? sd->dat_lines : 0;
> @@ -776,6 +797,8 @@ static sd_rsp_type_t sd_normal_command(SDState *sd,
> uint32_t rca = 0x0000;
> uint64_t addr = (sd->ocr & (1 << 30)) ? (uint64_t) req.arg << 9 :
> req.arg;
>
> + trace_sdcard_normal_command(req.cmd, req.arg, sd_state_name(sd->state));
> +
> /* Not interpreting this as an app command */
> sd->card_status &= ~APP_CMD;
>
> @@ -790,7 +813,6 @@ static sd_rsp_type_t sd_normal_command(SDState *sd,
> sd->multi_blk_cnt = 0;
> }
>
> - DPRINTF("CMD%d 0x%08x state %d\n", req.cmd, req.arg, sd->state);
> switch (req.cmd) {
> /* Basic commands (Class 0 and Class 1) */
> case 0: /* CMD0: GO_IDLE_STATE */
> @@ -1310,8 +1332,6 @@ static sd_rsp_type_t sd_normal_command(SDState *sd,
> return sd_r1;
>
> case 56: /* CMD56: GEN_CMD */
> - fprintf(stderr, "SD: GEN_CMD 0x%08x\n", req.arg);
> -
> switch (sd->state) {
> case sd_transfer_state:
> sd->data_offset = 0;
> @@ -1345,7 +1365,7 @@ static sd_rsp_type_t sd_normal_command(SDState *sd,
> static sd_rsp_type_t sd_app_command(SDState *sd,
> SDRequest req)
> {
> - DPRINTF("ACMD%d 0x%08x\n", req.cmd, req.arg);
> + trace_sdcard_app_command(req.cmd, req.arg);
> sd->card_status |= APP_CMD;
> switch (req.cmd) {
> case 6: /* ACMD6: SET_BUS_WIDTH */
> @@ -1606,8 +1626,7 @@ send_response:
>
> static void sd_blk_read(SDState *sd, uint64_t addr, uint32_t len)
> {
> - DPRINTF("sd_blk_read: addr = 0x%08llx, len = %d\n",
> - (unsigned long long) addr, len);
> + trace_sdcard_read_block(addr, len);
> if (!sd->blk || blk_pread(sd->blk, addr, sd->data, len) < 0) {
> fprintf(stderr, "sd_blk_read: read error on host side\n");
> }
> @@ -1615,6 +1634,7 @@ static void sd_blk_read(SDState *sd, uint64_t addr,
> uint32_t len)
>
> static void sd_blk_write(SDState *sd, uint64_t addr, uint32_t len)
> {
> + trace_sdcard_write_block(addr, len);
> if (!sd->blk || blk_pwrite(sd->blk, addr, sd->data, len, 0) < 0) {
> fprintf(stderr, "sd_blk_write: write error on host side\n");
> }
> diff --git a/hw/sd/trace-events b/hw/sd/trace-events
> index 0f8536db32..75dac5a2cd 100644
> --- a/hw/sd/trace-events
> +++ b/hw/sd/trace-events
> @@ -23,6 +23,12 @@ sdhci_read_dataport(uint16_t data_count) "all %u bytes of
> data have been read fr
> sdhci_write_dataport(uint16_t data_count) "write buffer filled with %u bytes
> of data"
> sdhci_capareg(const char *desc, uint16_t val) "%s: %u"
>
> +# hw/sd/sd.c
> +sdcard_normal_command(uint8_t cmd, uint32_t arg, const char *state) "CMD%d
> arg 0x%08x (state %s)"
> +sdcard_app_command(uint8_t acmd, uint32_t arg) "ACMD%d arg 0x%08x"
> +sdcard_read_block(uint64_t addr, uint32_t len) "addr 0x%" PRIx64 " size 0x%x"
> +sdcard_write_block(uint64_t addr, uint32_t len) "addr 0x%" PRIx64 " size
> 0x%x"
> +
> # hw/sd/milkymist-memcard.c
> milkymist_memcard_memory_read(uint32_t addr, uint32_t value) "addr 0x%08x
> value 0x%08x"
> milkymist_memcard_memory_write(uint32_t addr, uint32_t value) "addr 0x%08x
> value 0x%08x"
> --
> 2.16.1
>
>
- [Qemu-devel] [PATCH v4 00/11] SDCard: housekeeping, add tracing (part 4), Philippe Mathieu-Daudé, 2018/02/15
- [Qemu-devel] [PATCH v4 01/11] sdcard: reorder SDState struct members, Philippe Mathieu-Daudé, 2018/02/15
- [Qemu-devel] [PATCH v4 02/11] sdcard: replace DPRINTF() by trace events, Philippe Mathieu-Daudé, 2018/02/15
- Re: [Qemu-devel] [PATCH v4 02/11] sdcard: replace DPRINTF() by trace events,
Alistair Francis <=
- [Qemu-devel] [PATCH v4 03/11] sdcard: add a trace event for command responses, Philippe Mathieu-Daudé, 2018/02/15
- [Qemu-devel] [PATCH v4 04/11] sdcard: replace fprintf() by qemu_hexdump(), Philippe Mathieu-Daudé, 2018/02/15
- [Qemu-devel] [PATCH v4 05/11] sdcard: add more trace events, Philippe Mathieu-Daudé, 2018/02/15
- [Qemu-devel] [PATCH v4 06/11] sdcard: do not trace CMD55 when expecting ACMD, Philippe Mathieu-Daudé, 2018/02/15
- [Qemu-devel] [PATCH v4 07/11] sdcard: define SDMMC_CMD_MAX instead of using the magic '64', Philippe Mathieu-Daudé, 2018/02/15
- [Qemu-devel] [PATCH v4 08/11] sdcard: display command name when tracing CMD/ACMD, Philippe Mathieu-Daudé, 2018/02/15
- [Qemu-devel] [PATCH v4 09/11] sdcard: display protocol used when tracing, Philippe Mathieu-Daudé, 2018/02/15