[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 06/42] esp: fix esp_reg_read() trace event
From: |
Laurent Vivier |
Subject: |
Re: [PATCH v2 06/42] esp: fix esp_reg_read() trace event |
Date: |
Mon, 1 Mar 2021 21:29:35 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.7.0 |
Le 09/02/2021 à 20:29, Mark Cave-Ayland a écrit :
> Move the trace event to the end of the function so that it correctly reports
> the returned value if it doesn't come directly from the rregs array.
>
> Signed-off-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
> Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
> ---
> hw/scsi/esp.c | 18 ++++++++++++------
> 1 file changed, 12 insertions(+), 6 deletions(-)
>
> diff --git a/hw/scsi/esp.c b/hw/scsi/esp.c
> index 9951472ee6..c36cb0f238 100644
> --- a/hw/scsi/esp.c
> +++ b/hw/scsi/esp.c
> @@ -595,9 +595,8 @@ static void parent_esp_reset(ESPState *s, int irq, int
> level)
>
> uint64_t esp_reg_read(ESPState *s, uint32_t saddr)
> {
> - uint32_t old_val;
> + uint32_t val;
>
> - trace_esp_mem_readb(saddr, s->rregs[saddr]);
> switch (saddr) {
> case ESP_FIFO:
> if ((s->rregs[ESP_RSTAT] & STAT_PIO_MASK) == 0) {
> @@ -612,13 +611,14 @@ uint64_t esp_reg_read(ESPState *s, uint32_t saddr)
> s->ti_rptr = 0;
> s->ti_wptr = 0;
> }
> + val = s->rregs[ESP_FIFO];
> break;
> case ESP_RINTR:
> /*
> * Clear sequence step, interrupt register and all status bits
> * except TC
> */
> - old_val = s->rregs[ESP_RINTR];
> + val = s->rregs[ESP_RINTR];
> s->rregs[ESP_RINTR] = 0;
> s->rregs[ESP_RSTAT] &= ~STAT_TC;
> s->rregs[ESP_RSEQ] = SEQ_CD;
> @@ -627,16 +627,22 @@ uint64_t esp_reg_read(ESPState *s, uint32_t saddr)
> esp_report_command_complete(s, s->deferred_status);
> s->deferred_complete = false;
> }
> - return old_val;
> + break;
> case ESP_TCHI:
> /* Return the unique id if the value has never been written */
> if (!s->tchi_written) {
> - return s->chip_id;
> + val = s->chip_id;
> + } else {
> + val = s->rregs[saddr];
> }
> + break;
> default:
> + val = s->rregs[saddr];
> break;
> }
> - return s->rregs[saddr];
> +
> + trace_esp_mem_readb(saddr, val);
> + return val;
> }
>
> void esp_reg_write(ESPState *s, uint32_t saddr, uint64_t val)
>
Reviewed-by: Laurent Vivier <laurent@vivier.eu>
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [PATCH v2 06/42] esp: fix esp_reg_read() trace event,
Laurent Vivier <=