[Differential] D1944: PF and VIMAGE fixes
nvass-gmx.com (Nikos Vassiliadis)
phabric-noreply at FreeBSD.org
Sat May 7 11:34:52 UTC 2016
nvass-gmx.com added a comment.
Thanks for taking a look Kristof,
I believe all your points are valid. Thing is that all eventhandlers need a more thorough look. They don't work as they should; be it virtualized or not, be it a VIMAGE or GENERIC kernel. I am trying to take a look but I haven't thrown enough time to it.
INLINE COMMENTS
> @kristof wrote in sys/netpfil/pf/pf_if.c:130
> I don't understand why this is required. Surely if an ifnet lives in V_ifnet (so, lives in the current vnet) ifp->if_vnet is always going to be curvnet?
You're correct. It is plain wrong
REVISION DETAIL
https://reviews.freebsd.org/D1944
EMAIL PREFERENCES
https://reviews.freebsd.org/settings/panel/emailpreferences/
To: nvass-gmx.com, bz, trociny, kristof, gnn, zec, rodrigc, glebius, eri
Cc: ryan_timewasted.me, mmoll, javier_ovi_yahoo.com, farrokhi, julian, robak, freebsd-virtualization-list, freebsd-pf-list, freebsd-net-list
More information about the freebsd-virtualization
mailing list