Wireshark mailing list archives
Re: Replacing wmem_packet_scope() with pinfo->pool?
From: Evan Huus <eapache () gmail com>
Date: Mon, 12 Jul 2021 14:13:41 -0400
On Mon, Jul 12, 2021 at 2:05 PM João Valverde via Wireshark-dev <wireshark-dev () wireshark org> wrote:
On 12/07/21 16:52, Evan Huus wrote:I've been thinking recently about starting the process of getting rid of the "global" wmem scope methods (wmem_packet_scope, wmem_file_scope, etc) in favour of passing them around in arguments (or in pinfo, or something). This would let us drop a bunch of in-scope/out-of-scope tracking and assertion, as well as make the code more amenable to future refactors like (potentially) concurrency. At a first glance, we already have pinfo->pool which maintains the lifetime of the packet_info object. As far as I can reason, this is almost/effectively the same as the existing wmem_packet_scope - it gets cleaned up later in the dissection flow, but there's still only ever one which gets reused for each packet. Is this correct? If so, does it make sense to start replacing `wmem_packet_scope()` calls with `pinfo->pool` when pinfo is already in scope?I think wmem_packet_scope() should return pinfo->pool.
It would have to be converted to a macro (or do a mass-replace anyway to take pinfo as an argument), so I figure using `pinfo->pool` directly in most cases ends up being simplest.
Other than that, I don't see a compelling reason to remove the global wmem scope methods.Thanks, Evan ___________________________________________________________________________ Sent via: Wireshark-dev mailing list <wireshark-dev () wireshark org> Archives: https://www.wireshark.org/lists/wireshark-dev Unsubscribe: https://www.wireshark.org/mailman/options/wireshark-dev mailto:wireshark-dev-request () wireshark org?subject=unsubscribe___________________________________________________________________________ Sent via: Wireshark-dev mailing list <wireshark-dev () wireshark org> Archives: https://www.wireshark.org/lists/wireshark-dev Unsubscribe: https://www.wireshark.org/mailman/options/wireshark-dev mailto:wireshark-dev-request () wireshark org?subject=unsubscribe
___________________________________________________________________________ Sent via: Wireshark-dev mailing list <wireshark-dev () wireshark org> Archives: https://www.wireshark.org/lists/wireshark-dev Unsubscribe: https://www.wireshark.org/mailman/options/wireshark-dev mailto:wireshark-dev-request () wireshark org?subject=unsubscribe
Current thread:
- Replacing wmem_packet_scope() with pinfo->pool? Evan Huus (Jul 12)
- Re: Replacing wmem_packet_scope() with pinfo->pool? Pascal Quantin (Jul 12)
- Re: Replacing wmem_packet_scope() with pinfo->pool? Dr. Matthias St. Pierre (Jul 12)
- Re: Replacing wmem_packet_scope() with pinfo->pool? Dr. Matthias St. Pierre (Jul 12)
- Re: Replacing wmem_packet_scope() with pinfo->pool? Dr. Matthias St. Pierre (Jul 12)
- Re: Replacing wmem_packet_scope() with pinfo->pool? João Valverde via Wireshark-dev (Jul 12)
- Re: Replacing wmem_packet_scope() with pinfo->pool? Evan Huus (Jul 12)
- Re: Replacing wmem_packet_scope() with pinfo->pool? João Valverde via Wireshark-dev (Jul 12)
- Re: Replacing wmem_packet_scope() with pinfo->pool? Evan Huus (Jul 12)
- Re: Replacing wmem_packet_scope() with pinfo->pool? João Valverde via Wireshark-dev (Jul 12)
- Re: Replacing wmem_packet_scope() with pinfo->pool? Evan Huus (Jul 12)
- Re: Replacing wmem_packet_scope() with pinfo->pool? Pascal Quantin (Jul 12)
- Re: Replacing wmem_packet_scope() with pinfo->pool? Moshe Kaplan (Jul 21)
- Re: Replacing wmem_packet_scope() with pinfo->pool? Pascal Quantin (Jul 21)