On Thu, 2015-05-28 at 11:19 -0700, Tom Herbert wrote: > @@ -566,11 +640,15 @@ static const struct flow_dissector_key > flow_keys_dissector_keys[] = { > }, > { > .key_id = FLOW_DISSECTOR_KEY_IPV4_ADDRS, > - .offset = offsetof(struct flow_keys, addrs), > + .offset = offsetof(struct flow_keys, addrs.v4addrs), > + }, > + { > + .key_id = FLOW_DISSECTOR_KEY_IPV6_ADDRS, > + .offset = offsetof(struct flow_keys, addrs.v6addrs), > }, > { > .key_id = FLOW_DISSECTOR_KEY_IPV6_HASH_ADDRS, > - .offset = offsetof(struct flow_keys, addrs), > + .offset = offsetof(struct flow_keys, addrs.v4addrs),
Shouldn't it be offsetof(struct flow_keys, addrs.v6addrs), ? -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html