[ovs-dev] [PATCH 1/2] ovs-ctl.in: Don't save kernel datapath config.

Ben Pfaff blp at nicira.com
Wed Nov 14 17:58:34 UTC 2012


On Tue, Nov 06, 2012 at 11:11:30AM -0800, Gurucharan Shetty wrote:
> Now that we have a single datapath and the datapath
> port number has no relation to openflow port numbers,
> it does not make sense to save and restore kernel
> datapath configuration for the purpose of restoring
> openflow flows.
> 
> Signed-off-by: Gurucharan Shetty <gshetty at nicira.com>

Looks good, thank you.

I might make the commit message more detailed, maybe like this:

    Commit a41754333f6 (ovs-ctl.in: Ability to save flows and kernel
    datapath config.) made ovs-ctl able to usefully save and restore
    the flow table across loading and unloading the OVS kernel module
    and stopping and starting the OVS daemons.  To ensure that the
    flow table was still meaningful, it ensured that the datapath port
    numbers were the same with the old and new versions of the kernel
    module.

    However, later commit e1b1d06afde (Separate OpenFlow port numbers
    from datapath ones.) changed the OpenFlow implementation so that
    keeping the same datapath port numbers no longer ensured that the
    OpenFlow port numbers would be the same.  This caused a regression
    in saving and restoring the flow table.  Although the flow table
    was still saved and restored, it was no longer useful, since the
    datapath port numbers might change.

    This commit does not fix the regression, but it does drop the code
    that saves and restores the port datapath numbers, since it is no
    longer useful.  The following commit fixes the regression.

Thanks,

Ben.



More information about the dev mailing list