[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-arm] [PATCH v2 02/12] net: cadence_gem: Announce availability
From: |
Edgar E. Iglesias |
Subject: |
Re: [Qemu-arm] [PATCH v2 02/12] net: cadence_gem: Announce availability of priority queues |
Date: |
Wed, 17 Oct 2018 15:30:25 +0200 |
User-agent: |
Mutt/1.9.4 (2018-02-28) |
On Tue, Oct 16, 2018 at 05:15:36PM +0100, Peter Maydell wrote:
> On 11 October 2018 at 03:19, Edgar E. Iglesias <address@hidden> wrote:
> > From: "Edgar E. Iglesias" <address@hidden>
> >
> > Announce the availability of the various priority queues.
> > This fixes an issue where guest kernels would miss to
> > configure secondary queues due to inproper feature bits.
> >
> > Reviewed-by: Alistair Francis <address@hidden>
> > Signed-off-by: Edgar E. Iglesias <address@hidden>
> > ---
> > hw/net/cadence_gem.c | 6 +++++-
> > 1 file changed, 5 insertions(+), 1 deletion(-)
> >
> > diff --git a/hw/net/cadence_gem.c b/hw/net/cadence_gem.c
> > index e560b7a142..901c173970 100644
> > --- a/hw/net/cadence_gem.c
> > +++ b/hw/net/cadence_gem.c
> > @@ -1213,6 +1213,7 @@ static void gem_reset(DeviceState *d)
> > int i;
> > CadenceGEMState *s = CADENCE_GEM(d);
> > const uint8_t *a;
> > + uint32_t queues_mask;
> >
> > DB_PRINT("\n");
> >
> > @@ -1229,7 +1230,10 @@ static void gem_reset(DeviceState *d)
> > s->regs[GEM_DESCONF] = 0x02500111;
> > s->regs[GEM_DESCONF2] = 0x2ab13fff;
> > s->regs[GEM_DESCONF5] = 0x002f2045;
> > - s->regs[GEM_DESCONF6] = 0x00000200;
> > + s->regs[GEM_DESCONF6] = 0x0;
> > +
> > + queues_mask = MAKE_64BIT_MASK(1, s->num_priority_queues - 1);
> > + s->regs[GEM_DESCONF6] |= queues_mask;
>
> This causes the clang runtime sanitizer to complain:
> /aarch64/qom/xilinx-zynq-a9:
> /home/petmay01/linaro/qemu-for-merges/hw/net/cadence_gem.c:1306:19:
> runtime error: shift exponent 64 is too large for 64-bit type
> 'unsigned long long'
Argh, my bad.
>
> because s->num_priority_queues is 1, and so you've
> asked for a zero-width mask, which MAKE_64BIT_MASK
> doesn't permit. What was the intention of the code here?
The intention was a mask with zero bits set when only 1 queue is available.
DESCONF6 has a bit set per queue that is available,
except for queue 0, for which bit 0 is always 0.
For example configuration with 1 queue:
..000000
2 queues:
..000010
3 queues:
..000110
4 queues:
..001110
etc.
I'll post a v3 with this fixed.
> I don't think any value of num_priority_queues can
> produce the old 0x200 value for this ID register.
Right, the 0x200 value was bogus.
>
> I'm going to drop this patch and "net: cadence_gem:
> Announce 64bit addressing support" (which depends
> on this one) from my pullreq.
Best regards,
Edgar
- [Qemu-arm] [PATCH v2 00/12] arm: Add first models of Xilinx Versal SoC, Edgar E. Iglesias, 2018/10/10
- [Qemu-arm] [PATCH v2 01/12] net: cadence_gem: Disable TSU feature bit, Edgar E. Iglesias, 2018/10/10
- [Qemu-arm] [PATCH v2 02/12] net: cadence_gem: Announce availability of priority queues, Edgar E. Iglesias, 2018/10/10
- [Qemu-arm] [PATCH v2 03/12] net: cadence_gem: Use uint32_t for 32bit descriptor words, Edgar E. Iglesias, 2018/10/10
- [Qemu-arm] [PATCH v2 04/12] net: cadence_gem: Add macro with max number of descriptor words, Edgar E. Iglesias, 2018/10/10
- [Qemu-arm] [PATCH v2 09/12] target-arm: powerctl: Enable HVC when starting CPUs to EL2, Edgar E. Iglesias, 2018/10/10
- [Qemu-arm] [PATCH v2 05/12] net: cadence_gem: Add support for extended descriptors, Edgar E. Iglesias, 2018/10/10
- [Qemu-arm] [PATCH v2 10/12] target/arm: Add the Cortex-A72, Edgar E. Iglesias, 2018/10/10
- [Qemu-arm] [PATCH v2 07/12] net: cadence_gem: Implement support for 64bit descriptor addresses, Edgar E. Iglesias, 2018/10/10
- [Qemu-arm] [PATCH v2 08/12] net: cadence_gem: Announce 64bit addressing support, Edgar E. Iglesias, 2018/10/10
- [Qemu-arm] [PATCH v2 06/12] net: cadence_gem: Add support for selecting the DMA MemoryRegion, Edgar E. Iglesias, 2018/10/10
- [Qemu-arm] [PATCH v2 11/12] hw/arm: versal: Add a model of Xilinx Versal SoC, Edgar E. Iglesias, 2018/10/10