[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