From: Ben Pfaff <blp@nicira.com>
Date: Mon, 16 Mar 2009 23:27:39 +0000 (-0700)
Subject: secchan: Add explanatory comment.
X-Git-Url: https://pintos-os.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=d90c128a8101edfa9cbb2477a4bc7f5af8ef1a4c;p=openvswitch

secchan: Add explanatory comment.
---

diff --git a/secchan/ofproto.c b/secchan/ofproto.c
index c2a31837..8c62115b 100644
--- a/secchan/ofproto.c
+++ b/secchan/ofproto.c
@@ -109,6 +109,17 @@ struct rule {
     uint8_t ip_tos;             /* Last-seen IP type-of-service. */
     tag_type tags;              /* Tags (set only by hooks). */
 
+    /* A subrule with a known super-rule has a non-null 'super' other than
+     * UNKNOWN_SUPER, in which case 'list' is an element of the super-rule's
+     * list of subrules.
+     *
+     * A subrule with an unknown super-rule (which is an optimization that
+     * might well be better removed) has 'super' set to UNKNOWN_SUPER, in
+     * which case 'list' is unused.
+     *
+     * A super-rule has 'super' of NULL, in which case 'list' is the head of a
+     * list of sub-rules.  (A super-rule with no wildcards will never have any
+     * sub-rules.) */
     struct rule *super;
     struct list list;