Jarek Poplawski wrote:

On Thu, Jul 24, 2008 at 10:45:39AM +0200, Patrick McHardy wrote:

quoted

I'm not sure we should do this, this removes the symetry between
->init/change and ->dump. Its not a big deal in this case since
flows is unused in ->init anyway, but still its a not so nice
precedent. Such things should normally be put in the statistics.

Hmm... I didn't even notice this. But I guess, this patch didn't
introduce this precedent since this non-config "flow" was dumped
before. 

Yes, thats true, but see below.

Btw., I can see e.g. direct_pkts in htb_dump() which looks
like similar precedent.

Ugh, yes, that also should have been in the statistics.

IMHO, printing the same thing 2x under
different names is worse precedent. Anyway, this patch isn't any
big deal, so I can send a revert.

I think its not exactly 2x the same thing. "flows" has a different
meaning than limit (its the depth), but because its hardcoded its
not parsed in ->init. And because of implementation details "flows"
is limited by "limit", which itself is limited by the hardcoded
depth, so they just happen to always have the same value. Using
the current number of *active* flows doesn't really match the
original meaning anymore. IIRC the ESFQ for SFQ patches even use
"flows" in ->init, so it might make things a bit more ugly if we
ever merge those patches.