> On Oct 22, 2015, at 8:43 PM, Andy Zhou <az...@nicira.com> wrote: > > Use the wording from RFC 5880 to describe the "diagnostic" and > "remote_diagnostic" fields. > > Reported-by: Justin Pettit <jpet...@nicira.com> > Signed-off-by: Andy Zhou <az...@nicira.com> > --- > vswitchd/vswitch.xml | 13 +++++++------ > 1 file changed, 7 insertions(+), 6 deletions(-) > > diff --git a/vswitchd/vswitch.xml b/vswitchd/vswitch.xml > index 4dbd9e3..395d8f2 100644 > --- a/vswitchd/vswitch.xml > +++ b/vswitchd/vswitch.xml > @@ -2578,9 +2578,10 @@ > </column> > > <column name="bfd_status" key="diagnostic"> > - In case of a problem, set to an error message that reports what the > - local BFD session thinks is wrong. The error messages are defined > - in section 4.1 of [RFC 5880]. > + A diagnostic code specifying the local system's reason for the > + last change in session state. It allows the remote system to > + determine the reason that the previous session failed. The error > + messages are defined in section 4.1 of [RFC 5880]. > </column>
Do you think the middle sentence is necessary? It sounds like the remote side would read this column, but I doubt that's the case. Acked-by: Justin Pettit <jpet...@nicira.com> --Justin _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
