On Saturday 11 September 2010 11:26:37 Sven Eckelmann wrote:
Is it possible to split the patch in those parts? It would make it easier to read it and understand the patches later.
I'm not sure that will do much good. He managed to reorganize the code and thereby remove redundancies. The second patch would probably be no bigger than 3-5 lines of code.
- /* packet for me */
- if (is_my_mac(unicast_packet->dest)) {
interface_rx(recv_if->soft_iface, skb, hdr_size);return NET_RX_SUCCESS;- }
There are different parts of the patch which makes ma a little bit curious
- for example: why it is possible to drop that check entirely? Could that
be an extra patch with an explanation why that can be dropped? Is it only valid in context of the new fragmentation handling? ...
I ran into the same question as it looks a bit odd here but if you apply the patch it all looks good. As I said: it seems he managed to purge quite a bit of redundancy (fragmented and non-fragmented packets are almost identical after all).
Cheers, Marek