|To:||"David S. Miller" <davem@xxxxxxxxxxxxx>|
|Subject:||Re: netif_rx packet dumping|
|From:||Baruch Even <baruch@xxxxxxxxx>|
|Date:||Thu, 31 Mar 2005 17:33:41 +0100|
|Cc:||shemminger@xxxxxxxx, jheffner@xxxxxxx, netdev@xxxxxxxxxxx|
|References:||<20050303123811.4d934249@xxxxxxxxxxxxxxxxx> <42278122.6000000@xxxxxxxxx> <20050303133659.0d224e61.davem@xxxxxxxxxxxxx> <42278554.2090902@xxxxxxxxx> <20050303135718.2e1a0170.davem@xxxxxxxxxxxxx>|
|User-agent:||Debian Thunderbird 1.0 (X11/20050116)|
David S. Miller wrote:
On Thu, 03 Mar 2005 21:44:52 +0000 Baruch Even <baruch@xxxxxxxxx> wrote:The current linked list goes over all the packets, the linked list we add is for the packets that were not SACKed. The idea being that it is a lot faster since there are a lot less packets not SACKed compared to packets already SACKed (or never mentioned in SACKs).If you have a way around this I'd be happy to hear it.I'm sure you can find a way to steal sizeof(void *) from "struct tcp_skb_cb" :-) It is currently 36 bytes on both 32-bit and 64-bit platforms. This means if you can squeeze out 4 bytes (so that it fits in the skb->cb 40 byte area), you can fit a pointer in there for the linked list stuff.
I changed the code to use only the next pointer and dropped the prev. The cb still fits into 40 bytes for 32bit but for an em64t compile with gcc version 3.4.4 20041218 (prerelease) (Debian 3.4.3-6) the cb now requires 48 bytes. I haven't looked at the code emitted by I suspect it's an alignment that forced the pointer to start at 40 and then the size is 8 bytes.
From a quick test I did there was no performance hit and the numbers very very similar to those in the 32bit case. The machine is exactly the same (it's a 3Ghz xeon which I so-far only run as 32 bit, not suspecting I had a 64 bit test machine).
Baruchp.s. Thanks to whomever put the compile time test of this, it probably saved me quite a lot of time of hunting weird crashes.
|<Prev in Thread]||Current Thread||[Next in Thread>|
|Previous by Date:||Re: [Ksummit-2005-discuss] Summary of 2005 Kernel Summit Proposed Topics, Grant Grundler|
|Next by Date:||RE: [PATCH 188.8.131.52-bk2] e100: Use EEPROM config for Auto MDI/MDI-X, Venkatesan, Ganesh|
|Previous by Thread:||Re: netif_rx packet dumping, Thomas Graf|
|Next by Thread:||Re: netif_rx packet dumping, jamal|
|Indexes:||[Date] [Thread] [Top] [All Lists]|