>> +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