mir/openvswitch
2
0
Fork 0
mirror of https://github.com/openvswitch/ovs synced 2025-10-11 13:57:52 +00:00
Code Issues Releases Wiki Activity

bridge: Correctly omit unsupported interface statistics from database.

Browse Source 
The database documentation says:

    If an interface does not support a given statistic, then that pair is
    omitted.

but in fact the implementation included the key-value pair for an
unsupported statistic with -1 as the value.

Found by inspection.

Signed-off-by: Ben Pfaff <blp@nicira.com>
Acked-by: Ethan Jackson <ethan@nicira.com>
This commit is contained in:
Ben Pfaff
2013-05-03 15:36:56 -07:00
parent 62d86422f9
commit c2553db908
1 changed files with 15 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

25
vswitchd/bridge.c
View File

@@ -1898,11 +1898,12 @@ iface_refresh_stats(struct iface *iface)
IFACE_STAT(rx_crc_errors, "rx_crc_err") \
IFACE_STAT(collisions, "collisions")
#define IFACE_STAT(MEMBER, NAME) NAME,
static char *keys[] = { IFACE_STATS };
#define IFACE_STAT(MEMBER, NAME) + 1
enum { N_IFACE_STATS = IFACE_STATS };
#undef IFACE_STAT
int64_t values[ARRAY_SIZE(keys)];
int i;
int64_t values[N_IFACE_STATS];
char *keys[N_IFACE_STATS];
int n;
struct netdev_stats stats;
@@ -1914,15 +1915,19 @@ iface_refresh_stats(struct iface *iface)
* all-1s, and we will deal with that correctly below. */
netdev_get_stats(iface->netdev, &stats);
/* Copy statistics into values[] array. */
i = 0;
#define IFACE_STAT(MEMBER, NAME) values[i++] = stats.MEMBER;
/* Copy statistics into keys[] and values[]. */
n = 0;
#define IFACE_STAT(MEMBER, NAME) \
if (stats.MEMBER != UINT64_MAX) { \
keys[n] = NAME; \
values[n] = stats.MEMBER; \
n++; \
}
IFACE_STATS;
#undef IFACE_STAT
ovs_assert(i == ARRAY_SIZE(keys));
ovs_assert(n <= N_IFACE_STATS);
ovsrec_interface_set_statistics(iface->cfg, keys, values,
ARRAY_SIZE(keys));
ovsrec_interface_set_statistics(iface->cfg, keys, values, n);
#undef IFACE_STATS
}