Wireshark mailing list archives
Re: [Wireshark-commits] rev 52608: /trunk/epan/dissectors/ /trunk/epan/dissectors/: packet-vlan.c
From: Anders Broman <a.broman () bredband net>
Date: Tue, 15 Oct 2013 07:19:34 +0200
Maynard, Chris skrev 2013-10-15 00:47:
Why not add a fence so it's always visible then?
My point is that I don't want to write column info in the VLAN dissector to speed up dissection. If it could be arranged to only write the "last" entry that will actually be in the packet list or written out by tshark it would be much more efficient. One option is to only write to columns if next dissector isn't found in the cases where that is possible.
In the reference trace I'm pursuing col_add_fstr() costs 7.52 is called 4,7 million times, where of 2.8 million times is from the VLAN dissector.
Regards Anders
-----Original Message----- From: wireshark-commits-bounces () wireshark org [mailto:wireshark-commits-bounces () wireshark org] On Behalf Of etxrab () wireshark org Sent: Monday, October 14, 2013 5:51 PM To: wireshark-commits () wireshark org Subject: [Wireshark-commits] rev 52608: /trunk/epan/dissectors/ /trunk/epan/dissectors/: packet-vlan.c http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=52608 User: etxrab Date: 2013/10/14 09:50 PM Log: Reverting as requested by Jakub, eventhoug I don't think this prticular entry is useful as it's duplicated in the tree and almost certanly never vissible. Directory: /trunk/epan/dissectors/ Changes Path Action +0 -3 packet-vlan.c Modified
___________________________________________________________________________ Sent via: Wireshark-dev mailing list <wireshark-dev () wireshark org> Archives: http://www.wireshark.org/lists/wireshark-dev Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev mailto:wireshark-dev-request () wireshark org?subject=unsubscribe
Current thread:
- Re: [Wireshark-commits] rev 52608: /trunk/epan/dissectors/ /trunk/epan/dissectors/: packet-vlan.c Maynard, Chris (Oct 14)
- Re: [Wireshark-commits] rev 52608: /trunk/epan/dissectors/ /trunk/epan/dissectors/: packet-vlan.c Anders Broman (Oct 14)
- <Possible follow-ups>
- Re: [Wireshark-commits] rev 52608: /trunk/epan/dissectors/ /trunk/epan/dissectors/: packet-vlan.c Maynard, Chris (Oct 15)