netdev - Re: [PATCH] Re: SFQ depth limit

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Thu, 24 Jul 2008 11:57:15 +0200
From:	Patrick McHardy <kaber@...sh.net>
To:	Jarek Poplawski <jarkao2@...il.com>
CC:	David Miller <davem@...emloft.net>, denys@...p.net.lb,
	netdev@...r.kernel.org
Subject: Re: [PATCH] Re: SFQ depth limit

Jarek Poplawski wrote:
> On Thu, Jul 24, 2008 at 10:45:39AM +0200, Patrick McHardy wrote:
>>
>> 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.

--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html