[ovs-dev] [PATCH 13/15] doc: Convert DESIGN to rST
Russell Bryant
russell at ovn.org
Sat Oct 29 17:53:34 UTC 2016
On Tue, Oct 18, 2016 at 10:03 PM, Stephen Finucane <stephen at that.guru>
wrote:
> -```
> - master/
> - message and reason code other slave
> - ---------------------------------------- ------- -----
> - OFPT_PACKET_IN / NXT_PACKET_IN
> - OFPR_NO_MATCH yes ---
> - OFPR_ACTION yes ---
> - OFPR_INVALID_TTL --- ---
> - OFPR_ACTION_SET (OF1.4+) yes ---
> - OFPR_GROUP (OF1.4+) yes ---
> -
> - OFPT_FLOW_REMOVED / NXT_FLOW_REMOVED
> - OFPRR_IDLE_TIMEOUT yes ---
> - OFPRR_HARD_TIMEOUT yes ---
> - OFPRR_DELETE yes ---
> - OFPRR_GROUP_DELETE (OF1.4+) yes ---
> - OFPRR_METER_DELETE (OF1.4+) yes ---
> - OFPRR_EVICTION (OF1.4+) yes ---
> -
> - OFPT_PORT_STATUS
> - OFPPR_ADD yes yes
> - OFPPR_DELETE yes yes
> - OFPPR_MODIFY yes yes
> -
> - OFPT_ROLE_REQUEST / OFPT_ROLE_REPLY (OF1.4+)
> - OFPCRR_MASTER_REQUEST --- ---
> - OFPCRR_CONFIG --- ---
> - OFPCRR_EXPERIMENTER --- ---
> -
> - OFPT_TABLE_STATUS (OF1.4+)
> - OFPTR_VACANCY_DOWN --- ---
> - OFPTR_VACANCY_UP --- ---
> -
> - OFPT_REQUESTFORWARD (OF1.4+)
> - OFPRFR_GROUP_MOD --- ---
> - OFPRFR_METER_MOD --- ---
> -```
>
> ++-------------------------------------------+---------+-------+
> +| | master/ | |
> +| message and reason code | other | slave |
> ++===========================================+=========+=======+
> +| ``OFPT_PACKET_IN`` / ``NXT_PACKET_IN`` |
> ++-------------------------------------------+---------+-------+
> +| ``OFPR_NO_MATCH`` | yes | --- |
> ++-------------------------------------------+---------+-------+
> +| ``OFPR_ACTION`` | yes | --- |
> ++-------------------------------------------+---------+-------+
> +| ``OFPR_INVALID_TTL`` | --- | --- |
> ++-------------------------------------------+---------+-------+
> +| ``OFPR_ACTION_SET`` (OF1.4+) | yes | --- |
> ++-------------------------------------------+---------+-------+
> +| ``OFPR_GROUP`` (OF1.4+) | yes | --- |
> ++-------------------------------------------+---------+-------+
> +| ``OFPT_FLOW_REMOVED`` / ``NXT_FLOW_REMOVED`` |
> ++-------------------------------------------+---------+-------+
> +| ``OFPRR_IDLE_TIMEOUT`` | yes | --- |
> ++-------------------------------------------+---------+-------+
> +| ``OFPRR_HARD_TIMEOUT`` | yes | --- |
> ++-------------------------------------------+---------+-------+
> +| ``OFPRR_DELETE`` | yes | --- |
> ++-------------------------------------------+---------+-------+
> +| ``OFPRR_GROUP_DELETE`` (OF1.4+) | yes | --- |
> ++-------------------------------------------+---------+-------+
> +| ``OFPRR_METER_DELETE`` (OF1.4+) | yes | --- |
> ++-------------------------------------------+---------+-------+
> +| ``OFPRR_EVICTION`` (OF1.4+) | yes | --- |
> ++-------------------------------------------+---------+-------+
> +| ``OFPT_PORT_STATUS`` |
> ++-------------------------------------------+---------+-------+
> +| ``OFPPR_ADD`` | yes | yes |
> ++-------------------------------------------+---------+-------+
> +| ``OFPPR_DELETE`` | yes | yes |
> ++-------------------------------------------+---------+-------+
> +| ``OFPPR_MODIFY`` | yes | yes |
> ++-------------------------------------------+---------+-------+
> +| ``OFPT_ROLE_REQUEST`` / ``OFPT_ROLE_REPLY`` (OF1.4+) |
> ++-------------------------------------------+---------+-------+
> +| ``OFPCRR_MASTER_REQUEST`` | --- | --- |
> ++-------------------------------------------+---------+-------+
> +| ``OFPCRR_CONFIG`` | --- | --- |
> ++-------------------------------------------+---------+-------+
> +| ``OFPCRR_EXPERIMENTER`` | --- | --- |
> ++-------------------------------------------+---------+-------+
> +| ``OFPT_TABLE_STATUS`` (OF1.4+) |
> ++-------------------------------------------+---------+-------+
> +| ``OFPTR_VACANCY_DOWN`` | --- | --- |
> ++-------------------------------------------+---------+-------+
> +| ``OFPTR_VACANCY_UP`` | --- | --- |
> ++-------------------------------------------+---------+-------+
> +| ``OFPT_REQUESTFORWARD`` (OF1.4+) |
> ++-------------------------------------------+---------+-------+
> +| ``OFPRFR_GROUP_MOD`` | --- | --- |
> ++-------------------------------------------+---------+-------+
> +| ``OFPRFR_METER_MOD`` | --- | --- |
> ++-------------------------------------------+---------+-------+
>
>
I find the new formatting less clear than before. I think I'll leave it
for now, but I'd be interested in an alternative. Maybe each message
should have its own separate table?
--
Russell Bryant
More information about the dev
mailing list