>> +static  int window __read_mostly = 8;
>
> This is a (readonly) module parameter, but really is a constant (if you
> ever try to change it dynamically, bad things will happen in current
> code)
>
> const int window = 8;

It cannot be adjusted dynamically, but we can override the initial
window parameter when loading tcp_cdg as a module. Imho, this is a
good enough interface when considering that tuning CDG's parameters
are for special interest groups in the first place.

window, backoff_factor and backoff_beta are the most important knobs
that influence CDG's backoff behavior. We could remove window as a
parameter (make it constant), although that would make tuning it
rather tedious. We could also make window adjustable dynamically, but
I opt on waiting until someone needs this functionality before adding
the code. Or am I missing something?
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to