Skip to content

: sch_ingress cleanups

Patrick McHardy netdev 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest9 patches 4 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/9: sch_ingress: formatting fixesnet/sched/sch_ingress.c+39 -640 replies
2/9: sch_ingress: remove excessive debuggingnet/sched/sch_ingress.c+3 -760 replies
3/9: sch_ingress: remove qdisc_priv() wrappernet/sched/sch_ingress.c+4 -60 replies
4/9: sch_ingress: remove unused inner qdiscnet/sched/sch_ingress.c+0 -40 replies
5/9: sch_ingress: return proper error code in ingress_graft()net/sched/sch_ingress.c+1 -10 replies
6/9: sch_ingress: remove unnecessary opsnet/sched/sch_ingress.c+0 -240 replies
7/9: sch_ingress: move dependencies to Kconfignet/sched/Kconfig net/sched/sch_ingress.c+2 -100 replies
8/9: sch_ingress: avoid a fewnet/sched/sch_ingress.c+4 -120 replies
9/9: sch_ingress: remove useless printknet/sched/sch_ingress.c+1 -80 replies

Reviews trailers per patch

None of the 9 patches has a Reviewed-by or Acked-by of their own; the series has one for all of them.

PatchJamal Hadi Salim
SeriesA-b
1/9
2/9
3/9
4/9
5/9
6/9
7/9
8/9
9/9

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v1 thread

Cover letter [NET_SCHED 00/09]: sch_ingress cleanups

These patches clean up sch_ingress by reformatting the code to match the usual kernel style, removing lots of debugging statements that seem unnecessary after years of stable operation, removing some "future use" stuff that doesn't seem likely to ever get used and some minor additional cleanups.

 net/sched/Kconfig       |    1 +
 net/sched/sch_ingress.c |  232 ++++++++---------------------------------------
 2 files changed, 41 insertions(+), 192 deletions(-)

Patrick McHardy (9):
[NET_SCHED]: sch_ingress: formatting fixes
[NET_SCHED]: sch_ingress: remove excessive debugging [NET_SCHED]: sch_ingress: remove qdisc_priv() wrapper [NET_SCHED]: sch_ingress: remove unused inner qdisc [NET_SCHED]: sch_ingress: return proper error code in ingress_graft() [NET_SCHED]: sch_ingress: remove unnecessary ops [NET_SCHED]: sch_ingress: move dependencies to Kconfig [NET_SCHED]: sch_ingress: avoid a few #ifdefs [NET_SCHED]: sch_ingress: remove useless printk