[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v3 02/11] net: cadence_gem: Fix the queue address update duri
From: |
Edgar E. Iglesias |
Subject: |
Re: [PATCH v3 02/11] net: cadence_gem: Fix the queue address update during wrap around |
Date: |
Fri, 8 May 2020 13:25:56 +0200 |
User-agent: |
Mutt/1.10.1 (2018-07-13) |
On Fri, May 08, 2020 at 04:30:36PM +0530, Sai Pavan Boddu wrote:
> During wrap around and reset, queues are pointing to initial base
> address of queue 0, irrespective of what queue we are dealing with.
> Fix it by assigning proper base address every time.
Reviewed-by: Edgar E. Iglesias <address@hidden>
>
> Signed-off-by: Sai Pavan Boddu <address@hidden>
> ---
> hw/net/cadence_gem.c | 37 +++++++++++++++++++++++++++++++++----
> 1 file changed, 33 insertions(+), 4 deletions(-)
>
> diff --git a/hw/net/cadence_gem.c b/hw/net/cadence_gem.c
> index 5476c62..e6dc436 100644
> --- a/hw/net/cadence_gem.c
> +++ b/hw/net/cadence_gem.c
> @@ -845,6 +845,35 @@ static int get_queue_from_screen(CadenceGEMState *s,
> uint8_t *rxbuf_ptr,
> return 0;
> }
>
> +static uint32_t gem_get_queue_base_addr(CadenceGEMState *s, bool tx, int q)
> +{
> + uint32_t base_addr = 0;
> +
> + switch (q) {
> + case 0:
> + base_addr = s->regs[tx ? GEM_TXQBASE : GEM_RXQBASE];
> + break;
> + case 1 ... (MAX_PRIORITY_QUEUES - 1):
> + base_addr = s->regs[(tx ? GEM_TRANSMIT_Q1_PTR :
> + GEM_RECEIVE_Q1_PTR) + q - 1];
> + break;
> + default:
> + g_assert_not_reached();
> + };
> +
> + return base_addr;
> +}
> +
> +static uint32_t gem_get_tx_queue_base_addr(CadenceGEMState *s, int q)
> +{
> + return gem_get_queue_base_addr(s, true, q);
> +}
> +
> +static uint32_t gem_get_rx_queue_base_addr(CadenceGEMState *s, int q)
> +{
> + return gem_get_queue_base_addr(s, false, q);
> +}
> +
> static hwaddr gem_get_desc_addr(CadenceGEMState *s, bool tx, int q)
> {
> hwaddr desc_addr = 0;
> @@ -1043,7 +1072,7 @@ static ssize_t gem_receive(NetClientState *nc, const
> uint8_t *buf, size_t size)
> /* Next descriptor */
> if (rx_desc_get_wrap(s->rx_desc[q])) {
> DB_PRINT("wrapping RX descriptor list\n");
> - s->rx_desc_addr[q] = s->regs[GEM_RXQBASE];
> + s->rx_desc_addr[q] = gem_get_rx_queue_base_addr(s, q);
> } else {
> DB_PRINT("incrementing RX descriptor list\n");
> s->rx_desc_addr[q] += 4 * gem_get_desc_len(s, true);
> @@ -1199,7 +1228,7 @@ static void gem_transmit(CadenceGEMState *s)
> sizeof(desc_first));
> /* Advance the hardware current descriptor past this packet
> */
> if (tx_desc_get_wrap(desc)) {
> - s->tx_desc_addr[q] = s->regs[GEM_TXQBASE];
> + s->tx_desc_addr[q] = gem_get_tx_queue_base_addr(s, q);
> } else {
> s->tx_desc_addr[q] = packet_desc_addr +
> 4 * gem_get_desc_len(s, false);
> @@ -1251,7 +1280,7 @@ static void gem_transmit(CadenceGEMState *s)
> } else {
> packet_desc_addr = 0;
> }
> - packet_desc_addr |= s->regs[GEM_TXQBASE];
> + packet_desc_addr |= gem_get_tx_queue_base_addr(s, q);
> } else {
> packet_desc_addr += 4 * gem_get_desc_len(s, false);
> }
> @@ -1457,7 +1486,7 @@ static void gem_write(void *opaque, hwaddr offset,
> uint64_t val,
> if (!(val & GEM_NWCTRL_TXENA)) {
> /* Reset to start of Q when transmit disabled. */
> for (i = 0; i < s->num_priority_queues; i++) {
> - s->tx_desc_addr[i] = s->regs[GEM_TXQBASE];
> + s->tx_desc_addr[i] = gem_get_tx_queue_base_addr(s, i);
> }
> }
> if (gem_can_receive(qemu_get_queue(s->nic))) {
> --
> 2.7.4
>
- [PATCH v3 00/11] Cadence GEM Fixes, Sai Pavan Boddu, 2020/05/08
- [PATCH v3 01/11] net: cadence_gem: Fix debug statements, Sai Pavan Boddu, 2020/05/08
- [PATCH v3 03/11] net: cadence_gem: Fix irq update w.r.t queue, Sai Pavan Boddu, 2020/05/08
- [PATCH v3 02/11] net: cadence_gem: Fix the queue address update during wrap around, Sai Pavan Boddu, 2020/05/08
- Re: [PATCH v3 02/11] net: cadence_gem: Fix the queue address update during wrap around,
Edgar E. Iglesias <=
- [PATCH v3 04/11] net: cadence_gem: Define access permission for interrupt registers, Sai Pavan Boddu, 2020/05/08
- [PATCH v3 05/11] net: cadence_gem: Set ISR according to queue in use, Sai Pavan Boddu, 2020/05/08
- [PATCH v3 06/11] net: cadence_gem: Move tx/rx packet buffert to CadenceGEMState, Sai Pavan Boddu, 2020/05/08
- [PATCH v3 09/11] net: cadence_gem: Update the reset value for interrupt mask register, Sai Pavan Boddu, 2020/05/08
- [PATCH v3 07/11] net: cadence_gem: Add support for jumbo frames, Sai Pavan Boddu, 2020/05/08
- [PATCH v3 10/11] net: cadence_gem: TX_LAST bit should be set by guest, Sai Pavan Boddu, 2020/05/08