On Thu, Jul 02, 2009 at 10:13:54AM +0200, Anders Broman wrote:
> I played with it a bit and loading large files is quite slow but
> removing the resorting for every row speeds it up to be faster than
> the original code I think. A 61Mb trace New packet list with patch
> ~11s, old WS (SVN Rev 27374) ~16s
As you can see, packet_list_append_record() in gtk/packet_list_store.c
does a g_renew() to add each additional record (packet) to the list
store. It would probably be useful to scan through a capture file for
the number of packets and then allocate enough memory so that we don't
have to renew it every time. Of course, when doing a live capture, this
wouldn't be possible.
> Memmory usage is much higher though 183 744Kb vs 252 388K. Should some
> columns be stored as numbers instead of strings?
I'm going to see (if someone doesn't beat me to it) how custom we can
make the values that are stored. Perhaps we can store IP addresses for
example in binary and have the packet list display them in a normal
format. Or... ?
Steve