Re: [PATCH v4 2/7] net: remove old tcp_optlen function

From: William Allen Simpson
Date: Sat Mar 13 2010 - 06:12:27 EST


On 3/13/10 4:11 AM, Eric Dumazet wrote:
David already pointed out fact that this code path is not used in
forwardind / routing path. Your assumptions are clearly wrong.

Can you sit down and understand this difference ?

Only *locally* generated trafic by linux kernel can enter this path.

Since you agree with David, perhaps you could kindly point out exactly
how your statement is true? Proof by assertion is generally not good
argumentation. I'll start a new thread for you to discuss.

More importantly, how is this relevant to this patch?

This patch removes a seldom used function that generates bad code, and
MUST NOT be used elsewhere in the code base -- an attractive nuisance,
speaking in legal terms.

This patch is also more elegant, and generates faster code.

Are you saying this patch is incorrect in any way?

Please point to the bad line of code (not David's comments or diatribe).
Certainly, I'll be happy to correct it!


And if a bug in linux core network stack can feed any driver a buggy
skb, bad things can happen, even if a driver is perfect.

Certainly. But a driver cannot depend on the entirety of a large
project being perfect. It *should* be perfect in and of itself!


Please point out _this_ bug _if_ it really exists, so that we can
correct this bug instead of hiding it in one thousand of drivers.

Again, I'll start a new thread for your edification.


Your attacks make no sense, you know nothing about linux kernel
internals and assume it was written like other projects you were
involved to.

I've made no "attacks". The above is a personal /ad hominem/ attack,
evidently the raison d'etre for Linux email lists. :-(

Yes, I've been involved in many other projects, and am arguably the
most widely experienced TCP/IP implementer in the world. I've been
programming in C since 1977, and my first TCP/IP implementation was
started in 1979 (in assembly). Probably before you were born.

Why are you so threatened by that?

Why are you insulting and trying to drive me away?

Why, oh why, is there such a lack of community spirit here?!?!
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/