Message ID | 20190910154238.9155-4-bob.beckett@collabora.com |
---|---|
State | Changes Requested, archived |
Headers | show |
Series | None | expand |
Context | Check | Description |
---|---|---|
robh/checkpatch | success |
On 9/10/19 8:41 AM, Robert Beckett wrote: > Document a new setting for Marvell switch chips to set the default queue > priorities per port. > > Signed-off-by: Robert Beckett <bob.beckett@collabora.com> > --- > Documentation/devicetree/bindings/net/dsa/marvell.txt | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/Documentation/devicetree/bindings/net/dsa/marvell.txt b/Documentation/devicetree/bindings/net/dsa/marvell.txt > index 6f9538974bb9..e097c3c52eac 100644 > --- a/Documentation/devicetree/bindings/net/dsa/marvell.txt > +++ b/Documentation/devicetree/bindings/net/dsa/marvell.txt > @@ -47,6 +47,10 @@ Optional properties: > bus. The node must contains a compatible string of > "marvell,mv88e6xxx-mdio-external" > > +Optional properties for ports: > +- defqpri=<n> : Enforced default queue priority for the given port. > + Valid range is 0..3 This is a vendor specific driver/property, marvell,default-queue-priority (which be cheapskate on words) would be more readable. But still, I have some more fundamental issues with the general approach, see my response in the cover letter.
Hi Robert, On Tue, 10 Sep 2019 09:42:24 -0700, Florian Fainelli <f.fainelli@gmail.com> wrote: > This is a vendor specific driver/property, > marvell,default-queue-priority (which be cheapskate on words) would be > more readable. But still, I have some more fundamental issues with the > general approach, see my response in the cover letter. As Florian said, the DT is unlikely to welcome vendor specific nodes for configuration which may be generic through standard network userspace tools. Thanks, Vivien
Hi guys, On 10/09/2019, Vivien Didelot <vivien.didelot@gmail.com> wrote: > Hi Robert, > > On Tue, 10 Sep 2019 09:42:24 -0700, Florian Fainelli <f.fainelli@gmail.com> > wrote: >> This is a vendor specific driver/property, >> marvell,default-queue-priority (which be cheapskate on words) would be >> more readable. But still, I have some more fundamental issues with the >> general approach, see my response in the cover letter. > > As Florian said, the DT is unlikely to welcome vendor specific nodes for > configuration which may be generic through standard network userspace > tools. > > > Thanks, > > Vivien > While I do agree that the DT bindings are a big no-no for QoS settings, the topic is interesting. What is the user space knob for configuring port-default priority (say RX queue)? Something like this maybe? (a very forced "matchall" with rxnfc) ethtool --config-nfc eth0 flow-type ether src 00:00:00:00:00:00 m 00:00:00:00:00:00 action 5 Regards, -Vladimir
diff --git a/Documentation/devicetree/bindings/net/dsa/marvell.txt b/Documentation/devicetree/bindings/net/dsa/marvell.txt index 6f9538974bb9..e097c3c52eac 100644 --- a/Documentation/devicetree/bindings/net/dsa/marvell.txt +++ b/Documentation/devicetree/bindings/net/dsa/marvell.txt @@ -47,6 +47,10 @@ Optional properties: bus. The node must contains a compatible string of "marvell,mv88e6xxx-mdio-external" +Optional properties for ports: +- defqpri=<n> : Enforced default queue priority for the given port. + Valid range is 0..3 + Example: mdio {
Document a new setting for Marvell switch chips to set the default queue priorities per port. Signed-off-by: Robert Beckett <bob.beckett@collabora.com> --- Documentation/devicetree/bindings/net/dsa/marvell.txt | 4 ++++ 1 file changed, 4 insertions(+)