Re: [PATCH net-2.6] [NET] The world is not perfect patch.

Previous thread: [PATCH] x86: add save_stack_trace_bp() for tracing from a specific stack frame by Vegard Nossum on Tuesday, May 20, 2008 - 2:28 am. (1 message)

Next thread: [PATCH] [.gitignore] match ncscope.out by Jike Song on Wednesday, May 21, 2008 - 6:23 pm. (5 messages)
From: Rami Rosen
Date: Tuesday, May 20, 2008 - 2:29 am

Hi,

There are three methods in the network stack which start with
#ifndef I_WISH_WORLD_WERE_PERFECT.

These methods are:
ipip_err() in net/ipv4/ipip.c
ipgre_err() in net/ipv4/ip_gre.c
ipip6_err() in net/ipv6/sit.c

When building the kernel with -DI_WISH_WORLD_WERE_PERFECT we have
various compilation
errors, mostly due to the fact that the corresponding code was not
updated recently.

(As we all know, indeed the world is not perfect, unluckily...)

This patch fixes these errors and enables building the kernel
successfully with -DI_WISH_WORLD_WERE_PERFECT.

(Note : I am not fully sure about correctness of len = skb->len, which I added;
feedback on this point is welcomed).


Regards,
Rami Rosen


Signed-off-by: Rami Rosen <ramirose@gmail.com>
From: David Miller
Date: Tuesday, May 20, 2008 - 2:33 pm

From: "Rami Rosen" <ramirose@gmail.com>

I think it is better to delete this code.  It is impossible
for anyone to build test it without making source modifications,
therefore it will continute to break over and over again over
time.
--

From: Rami Rosen
Date: Wednesday, May 21, 2008 - 12:33 am

Hi,
  Unless there will be any objection here, I suggest consider the
following patch which simply removes the code for the
-DI_WISH_WORLD_WERE_PERFECT in the three methods which use it.

The compilation errors we get when using -DI_WISH_WORLD_WERE_PERFECT
show that this code was not built and not used for really a long time.

Regards,
Rami Rosen


Signed-off-by: Rami Rosen <ramirose@gmail.com>


From: David Miller
Date: Wednesday, May 21, 2008 - 5:48 pm

From: "Rami Rosen" <ramirose@gmail.com>

Applied, thanks for following up on this Rami.
--

Previous thread: [PATCH] x86: add save_stack_trace_bp() for tracing from a specific stack frame by Vegard Nossum on Tuesday, May 20, 2008 - 2:28 am. (1 message)

Next thread: [PATCH] [.gitignore] match ncscope.out by Jike Song on Wednesday, May 21, 2008 - 6:23 pm. (5 messages)