Optimization #832
closed
clean up packet action macros
Added by Victor Julien over 11 years ago.
Updated over 11 years ago.
Description
Clean up is need for packet action macro's:
- PASS_PACKET, DROP_PACKET and the others should be wrappers to UPDATE_PACKET_ACTION
- Lets rename things to have PACKET_* prefix, as we use almost everywhere
- Consumers of these macro's should use PASS/DROP/etc macro's directly as much as possible, instead of the UPDATE macro
Some questions:
Point 2:
So we have
UPDATE_PACKET_ACTION > PACKET_UPDATE_ACTION
DROP_PACKET>PACKET_DROP
Point 3:
Currently, I've choose to use UPDATE_PACKET_ACTION where we add a "|= ACTION". In place with a simple =, I've used direct access to PASS/DROP macros. So the question is. Is there a reason to use |= when setting DROP or PASS ?
Point 2: ya.
Point 3: we need to review this carefully. For example in case of reject, we also set drop to that we both drop and send out a rst/icmp err. Some actions are obviously conflicting, like pass and drop.
- % Done changed from 0 to 90
- Status changed from Assigned to Closed
- % Done changed from 90 to 100
Also available in: Atom
PDF