[PATCH] realtek: correct egress frame port verification

Sander Vanheule sander at svanheule.net
Sun Jun 19 04:38:16 PDT 2022


Hi Birger,

On Sun, 2022-06-19 at 11:40 +0200, Birger Koblitz wrote:
> Hi,
> 
> On 19.06.22 10:56, Sander Vanheule wrote:
> 
> > -       h->cpu_tag[1] = h->cpu_tag[2] = 0;
> > -       if (prio >= 0)
> > -               h->cpu_tag[2] = BIT(13) | prio << 8; // Enable and set Priority Queue
> > +       h->cpu_tag[1] = 0;
> > +       /* Enable (AS_QID) and set Priority Queue (QID) */
> > +       h->cpu_tag[2] = (BIT(5) | (prio & 0x1f)) << 8;
> You are removing the possibility to let the SoC choose a Queue on its own based on congestion,
> if you always enable AS_QID. There was a reason that there were negative Queue-ID values, which
> denoted allowing the SoC to choose, i.e. AS_QID was not set.

I changed the code this way because prio was always positive, or made so by masking out the upper
bits (rtl8380/rtl8390). The passed value (sk_buff::priority >> 1) positive (unsigned) too. There
should be no change in behaviour with this patch.

Currently preparing a second version (series) of these changes, which I will send shortly. Please
let me know how this changes (hopefully improves) behaviour on the affected platforms, as I won't
have time to test this in the next days.

Best,
Sander



More information about the openwrt-devel mailing list