diff options
author | David S. Miller <davem@davemloft.net> | 2012-07-19 15:31:33 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2012-07-20 16:31:21 -0400 |
commit | f5b0a8743601a4477419171f5046bd07d1c080a0 (patch) | |
tree | 92e17ec293c3d0e35a0deda5d3bbf43710fb96ff /net/core | |
parent | f8126f1d5136be1ca1a3536d43ad7a710b5620f8 (diff) |
net: Document dst->obsolete better.
Add a big comment explaining how the field works, and use defines
instead of magic constants for the values assigned to it.
Suggested by Joe Perches.
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/core')
-rw-r--r-- | net/core/dst.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/net/core/dst.c b/net/core/dst.c index 07bacff84aa4..069d51d29414 100644 --- a/net/core/dst.c +++ b/net/core/dst.c | |||
@@ -94,7 +94,7 @@ loop: | |||
94 | * But we do not have state "obsoleted, but | 94 | * But we do not have state "obsoleted, but |
95 | * referenced by parent", so it is right. | 95 | * referenced by parent", so it is right. |
96 | */ | 96 | */ |
97 | if (dst->obsolete > 1) | 97 | if (dst->obsolete > 0) |
98 | continue; | 98 | continue; |
99 | 99 | ||
100 | ___dst_free(dst); | 100 | ___dst_free(dst); |
@@ -202,7 +202,7 @@ static void ___dst_free(struct dst_entry *dst) | |||
202 | */ | 202 | */ |
203 | if (dst->dev == NULL || !(dst->dev->flags&IFF_UP)) | 203 | if (dst->dev == NULL || !(dst->dev->flags&IFF_UP)) |
204 | dst->input = dst->output = dst_discard; | 204 | dst->input = dst->output = dst_discard; |
205 | dst->obsolete = 2; | 205 | dst->obsolete = DST_OBSOLETE_DEAD; |
206 | } | 206 | } |
207 | 207 | ||
208 | void __dst_free(struct dst_entry *dst) | 208 | void __dst_free(struct dst_entry *dst) |