Re: [PATCH 2/3] [VLAN]: Update iif when receiving via VLAN device
From: Patrick McHardy <hidden>
Date: 2006-06-26 18:24:06
David Miller wrote:
From: Patrick McHardy <redacted> Date: Mon, 26 Jun 2006 19:04:15 +0200quoted
I know this was discussed before, but I can't remember the exact outcome. Why don't we just unconditionally update iif in netif_receive_skb()?Software devices might have interesting semantics that would make not setting iif desirable.
Right, I remember now.
Once you set iif, you can't just undo it because the information is lost. I also would really prefer to set it unconditionally in netif_receive_skb(), but Jamal's concerns in this area are real. We really need to evaluate this on a case-by-case basis.
I still have a hard time imagining a case where this wouldn't fit, netfilter is perfectly happy with just using skb->dev as input device on the input path so far. I also think it will be confusing to the user to have different notions of what constitutes the input device. If anyone can think of an example where the user would really expect the input device to be something different than skb->dev I'd be really interested to hear it. BTW, this is not meant to be an objection to Thomas's patch, just me still wondering ..