Traffic shaping at the mean traffic level is the way to go.
So, I like the idea of Dshaper because it serves two purposes:
1 - to implement a fair use policy based at a reasonable level of mean
2 - to allow users to burst at maximum bandwidth during an configurable
This is my understanding after looking at the code, but, in my opinion,
the implementation is not (yet) mature enough to be included in m0n0wall.
That's because the configuration is unnecessary complex and the
shaper-help confuses me in understanding what it does. I will give
personal feedback on this later, to start further discussion.
I think it's better to skip a variable in the GUI-configuration and have a
close look at how to describe the burst characteristics (i.e. timing).
That hopefully makes it easier to use.