From 5fc270bf780b3a64a0544764d3baaedca8a96622 Mon Sep 17 00:00:00 2001 From: Ben Pfaff Date: Thu, 8 Apr 2010 10:30:11 -0700 Subject: [PATCH] xfif, wdp: Clarify comment. Suggested by partner. --- lib/xfif-provider.h | 9 +++++---- ofproto/wdp-provider.h | 9 +++++---- 2 files changed, 10 insertions(+), 8 deletions(-) diff --git a/lib/xfif-provider.h b/lib/xfif-provider.h index d469269a5..8730c5331 100644 --- a/lib/xfif-provider.h +++ b/lib/xfif-provider.h @@ -54,10 +54,11 @@ static inline void xfif_assert_class(const struct xfif *xfif, * These functions return 0 if successful or a positive errno value on failure, * except where otherwise noted. * - * These functions are expected to execute synchronously, that is, to block as - * necessary to obtain a result. Thus, they may not return EAGAIN or - * EWOULDBLOCK or EINPROGRESS. We may relax this requirement in the future if - * and when we encounter performance problems. */ + * Most of these functions are expected to execute synchronously, that is, to + * block as necessary to obtain a result. Thus, these functions may return + * EAGAIN (or EWOULDBLOCK or EINPROGRESS) only where the function descriptions + * explicitly say those errors are a possibility. We may relax this + * requirement in the future if and when we encounter performance problems. */ struct xfif_class { /* Type of xfif in this class, e.g. "system", "netdev", etc. * diff --git a/ofproto/wdp-provider.h b/ofproto/wdp-provider.h index 9e7d229b5..8c52295cf 100644 --- a/ofproto/wdp-provider.h +++ b/ofproto/wdp-provider.h @@ -61,10 +61,11 @@ static inline void wdp_assert_class(const struct wdp *wdp, * These functions return 0 if successful or a positive errno value on failure, * except where otherwise noted. * - * These functions are expected to execute synchronously, that is, to block as - * necessary to obtain a result. Thus, they may not return EAGAIN or - * EWOULDBLOCK or EINPROGRESS. We may relax this requirement in the future if - * and when we encounter performance problems. */ + * Most of these functions are expected to execute synchronously, that is, to + * block as necessary to obtain a result. Thus, these functions may return + * EAGAIN (or EWOULDBLOCK or EINPROGRESS) only where the function descriptions + * explicitly say those errors are a possibility. We may relax this + * requirement in the future if and when we encounter performance problems. */ struct wdp_class { /* Type of wdp in this class, e.g. "system", "netdev", etc. * -- 2.47.0