[ovs-dev] [PATCH] dpif-netdev: Use portable error code for zero rate meter band
Andy Zhou
azhou at ovn.org
Thu Sep 28 19:39:35 UTC 2017
'EBADRQC' is only defined on the Linux platform. Without this fix,
The travis MacOS build fails. Switching to using EDOM which is more
portable.
Fixes: 2029ce9ac3a601 (dpif-netdev: Fix a zero-rate bug for meter)
CC: Ali Volkan ATLI <volkan.atli at argela.com.tr>
Signed-off-by: Andy Zhou <azhou at ovn.org>
---
lib/dpif-netdev.c | 2 +-
ofproto/ofproto-dpif.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/lib/dpif-netdev.c b/lib/dpif-netdev.c
index 0fce94e0e409..d5eb8305c8a2 100644
--- a/lib/dpif-netdev.c
+++ b/lib/dpif-netdev.c
@@ -4285,7 +4285,7 @@ dpif_netdev_meter_set(struct dpif *dpif, ofproto_meter_id *meter_id,
/* Validate rates */
for (i = 0; i < config->n_bands; i++) {
if (config->bands[i].rate == 0) {
- return EBADRQC; /* rate must be non-zero */
+ return EDOM; /* rate must be non-zero */
}
}
diff --git a/ofproto/ofproto-dpif.c b/ofproto/ofproto-dpif.c
index d99dc9d88788..43d670a15c3f 100644
--- a/ofproto/ofproto-dpif.c
+++ b/ofproto/ofproto-dpif.c
@@ -5695,7 +5695,7 @@ meter_set(struct ofproto *ofproto_, ofproto_meter_id *meter_id,
return OFPERR_OFPMMFC_OUT_OF_BANDS;
case ENODEV: /* Unsupported band type */
return OFPERR_OFPMMFC_BAD_BAND;
- case EBADRQC: /* Rate must be non-zero */
+ case EDOM: /* Rate must be non-zero */
return OFPERR_OFPMMFC_BAD_RATE;
default:
return OFPERR_OFPMMFC_UNKNOWN;
--
1.8.3.1
More information about the dev
mailing list