On 10/26/2011 09:11 AM, Stefan Berger wrote:
Apologies for delaying so long on reviewing this.
> This patch adds support for filtering of VLAN (802.1Q) traffic to the
> parser and makes us of the ebtables support for VLAN filtering. This code
> now enables the filtering of traffic in chains with prefix 'vlan'.
>
> Signed-off-by: Stefan Berger <stefanb@xxxxxxxxxxxxxxxxxx>
>
> ---
> docs/schemas/nwfilter.rng | 47 +++++++++++++
> src/conf/nwfilter_conf.c | 102 ++++++++++++++++++++++++++++++
> src/conf/nwfilter_conf.h | 17 +++++
> src/nwfilter/nwfilter_ebiptables_driver.c | 35 ++++++++++
> 4 files changed, 201 insertions(+)
>
This didn't apply cleanly for me. Am I missing the review of another
pre-requisite series? Or is it just something where you need to rebase
and post a v2 for easier review?
>
> +static bool
> +checkVlanVlanID(enum attrDatatype datatype, union data *value,
> + virNWFilterRuleDefPtr nwf ATTRIBUTE_UNUSED,
ATTRIBUTE_UNUSED not necessary here, since...
> + nwItemDesc *item ATTRIBUTE_UNUSED)
> +{
> + int32_t res;
> +
> + res = value->ui;
> + if (res < 0 || res > 4095) {
> + res = -1;
> + }
> +
> + if (res != -1) {
> + nwf->p.vlanHdrFilter.dataVlanID.u.u16 = res;
...this uses nwf. Similar comment applies elsewhere in the patch.
Overall, it looks sane, but I didn't compile test it.
--
Eric Blake eblake@xxxxxxxxxx +1-919-301-3266
Libvirt virtualization library http://libvirt.org
Attachment:
signature.asc
Description: OpenPGP digital signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list