On Fri, 2020-11-27 at 21:21 -0800, Stephen Hemminger wrote:
> On Fri, 27 Nov 2020 15:26:25 +0000
> Luca Boccassi <bl...@debian.org> wrote:
>
> > As reported by a Debian user, mqprio output in json mode is
> > invalid:
> >
> > {
> > "kind": "mqprio",
> > "handle": "8021:",
> > "dev": "enp1s0f0",
> > "root": true,
> > "options": { tc 2 map 0 0 0 1 0 1 0 0 0 0 0 0 0 0 0 0
> > queues:(0:3) (4:7)
> > mode:channel
> > shaper:dcb}
> > }
> >
> > json-ify it, while trying to maintain the same formatting
> > for standard output.
> >
> > https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=972784
> >
> > Signed-off-by: Luca Boccassi <bl...@debian.org>
>
> Did you try feeding that into the python parser?
> What is before/after?
Uh, which python parser?
The reporter tested the patch, there's a small error. Before is above,
after is:
{
"kind": "mqprio",
"handle": "8001:",
"root": true,
"options": {
"tc": 2,
"map": [ 0,0,0,1,0,1,0,0,0,0,0,0,0,0,0,0 ],
"queues": [
"": [ 0,3 ],
"": [ 4,7 ] ],
"mode": "channel",
"shaper": "dcb"
}
}
Will fix the "queues" to remove the empty name in v2.
--
Kind regards,
Luca Boccassi
signature.asc
Description: This is a digitally signed message part
