Well, we jump to NFQUEUE in tap chains.

If we ACCEPT at begin of forward, we bypass ip.
and we jump to NFQUEUE at begin of forward, we are going to ips for all vms (I 
want to enable it by vm)


I just notice a bug, if sourcevm out (ips:0) -> sourcevm in (ips:1)

it'll do an accept in tap-out, and bypass the ips.

I'll rework my patch.

(something like 
PVEFW-FORWARD -m conntrack --ctstate RELATED,ESTABLISHED -j IPSCHAIN

IPSCHAIN -m physdev --physdev-out tapxxxi0 --physdev-is-bridged -j NFQUEUE
IPSCHAIN -m physdev --physdev-out tapxxxi0 --physdev-is-bridged -j NFQUEUE
IPSCHAIN -j ACCEPT
)

should be faster too


----- Mail original ----- 

De: "Dietmar Maurer" <diet...@proxmox.com> 
À: "Alexandre Derumier" <aderum...@odiso.com>, pve-devel@pve.proxmox.com 
Envoyé: Lundi 17 Mars 2014 07:10:20 
Objet: RE: [pve-devel] [PATCH] add ips feature 

> # fixme: this is an optimization? if so, we should also drop INVALID 
> packages? 
> - ruleset_insertrule($ruleset, "PVEFW-FORWARD", "-m conntrack --ctstate 
> RELATED,ESTABLISHED -j ACCEPT"); 
> - 
> + if(!$ips_enable){ 
> + ruleset_insertrule($ruleset, "PVEFW-FORWARD", "-m conntrack -- 
> ctstate RELATED,ESTABLISHED -j ACCEPT"); 
> + } 

What happens here if ips is enabled? Don't we need to jump to NFQUEUE? 
_______________________________________________
pve-devel mailing list
pve-devel@pve.proxmox.com
http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Reply via email to