Revision b5a0faa3572ac70bd374bd66190ac3ad4fddab20 authored by Ivan Khoronzhuk on 19 November 2019, 00:23:12 UTC, committed by David S. Miller on 19 November 2019, 23:23:15 UTC
The taprio qdisc allows to set mqprio setting but only once. In case
if mqprio settings are provided next time the error is returned as
it's not allowed to change traffic class mapping in-flignt and that
is normal. But if configuration is absolutely the same - no need to
return error. It allows to provide same command couple times,
changing only base time for instance, or changing only scheds maps,
but leaving mqprio setting w/o modification. It more corresponds the
message: "Changing the traffic mapping of a running schedule is not
supported", so reject mqprio if it's really changed.

Also corrected TC_BITMASK + 1 for consistency, as proposed.

Fixes: a3d43c0d56f1 ("taprio: Add support adding an admin schedule")
Reviewed-by: Vladimir Oltean <olteanv@gmail.com>
Tested-by: Vladimir Oltean <olteanv@gmail.com>
Acked-by: Vinicius Costa Gomes <vinicius.gomes@intel.com>
Signed-off-by: Ivan Khoronzhuk <ivan.khoronzhuk@linaro.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
1 parent d4ffb02
History
File Mode Size
keys
tpm
IMA-templates.rst -rw-r--r-- 4.6 KB
SCTP.rst -rw-r--r-- 14.0 KB
credentials.rst -rw-r--r-- 20.5 KB
index.rst -rw-r--r-- 230 bytes
lsm-development.rst -rw-r--r-- 738 bytes
lsm.rst -rw-r--r-- 10.5 KB
sak.rst -rw-r--r-- 3.0 KB
self-protection.rst -rw-r--r-- 13.5 KB
siphash.rst -rw-r--r-- 6.3 KB

back to top