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
