From patchwork Thu Jun 3 12:27:21 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Numan Siddique X-Patchwork-Id: 1487203 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Authentication-Results: ozlabs.org; spf=pass (sender SPF authorized) smtp.mailfrom=openvswitch.org (client-ip=2605:bc80:3010::133; helo=smtp2.osuosl.org; envelope-from=ovs-dev-bounces@openvswitch.org; receiver=) Received: from smtp2.osuosl.org (smtp2.osuosl.org [IPv6:2605:bc80:3010::133]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id 4FwlXc51Vsz9s1l for ; Thu, 3 Jun 2021 22:27:44 +1000 (AEST) Received: from localhost (localhost [127.0.0.1]) by smtp2.osuosl.org (Postfix) with ESMTP id 54A71400EB; Thu, 3 Jun 2021 12:27:42 +0000 (UTC) X-Virus-Scanned: amavisd-new at osuosl.org Received: from smtp2.osuosl.org ([127.0.0.1]) by localhost (smtp2.osuosl.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id l_1Wr4YJ1dHm; Thu, 3 Jun 2021 12:27:38 +0000 (UTC) Received: from lists.linuxfoundation.org (lf-lists.osuosl.org [IPv6:2605:bc80:3010:104::8cd3:938]) by smtp2.osuosl.org (Postfix) with ESMTP id C6284400CA; Thu, 3 Jun 2021 12:27:37 +0000 (UTC) Received: from lf-lists.osuosl.org (localhost [127.0.0.1]) by lists.linuxfoundation.org (Postfix) with ESMTP id 9ABC0C000D; Thu, 3 Jun 2021 12:27:37 +0000 (UTC) X-Original-To: dev@openvswitch.org Delivered-To: ovs-dev@lists.linuxfoundation.org Received: from smtp4.osuosl.org (smtp4.osuosl.org [140.211.166.137]) by lists.linuxfoundation.org (Postfix) with ESMTP id 06BEDC0001 for ; Thu, 3 Jun 2021 12:27:36 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by smtp4.osuosl.org (Postfix) with ESMTP id E173D404E3 for ; Thu, 3 Jun 2021 12:27:35 +0000 (UTC) X-Virus-Scanned: amavisd-new at osuosl.org Received: from smtp4.osuosl.org ([127.0.0.1]) by localhost (smtp4.osuosl.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 0N5ZZM9JPjw0 for ; Thu, 3 Jun 2021 12:27:32 +0000 (UTC) X-Greylist: domain auto-whitelisted by SQLgrey-1.8.0 Received: from relay6-d.mail.gandi.net (relay6-d.mail.gandi.net [217.70.183.198]) by smtp4.osuosl.org (Postfix) with ESMTPS id A3026404C9 for ; Thu, 3 Jun 2021 12:27:31 +0000 (UTC) Received: (Authenticated sender: numans@ovn.org) by relay6-d.mail.gandi.net (Postfix) with ESMTPSA id 11189C0002; Thu, 3 Jun 2021 12:27:27 +0000 (UTC) From: numans@ovn.org To: dev@openvswitch.org Date: Thu, 3 Jun 2021 08:27:21 -0400 Message-Id: <20210603122721.2066524-1-numans@ovn.org> X-Mailer: git-send-email 2.31.1 MIME-Version: 1.0 Subject: [ovs-dev] [PATCH ovn v9 0/5] ovn-controller: Split logical flow and physical flow processing X-BeenThere: ovs-dev@openvswitch.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: ovs-dev-bounces@openvswitch.org Sender: "dev" From: Numan Siddique This series splits the logical flow and physical flow processing and also handles the runtime data changes for the ct zone engine data. New patch p2 is added in v9 to handle sbrec_chassis changes in pflow_output and lflow_output engine nodes. The patches 2-4 added in v8 are now 3-5 in v9. v8 -> v9 ---- * Addressed review comments from Han. Removed the noop handlers for some of the engine inputs. * Added a new patch - p2 to handle sbrec_chassis changes in pflow_output and lflow_output engine nodes. v7 -> v8 ---- * Added the ct zones I-P for datapath and runtime data changes. * Removed the noop_handler for runtime data changes in pflow engine node. Now there is no handler for runtime data changes for the pflow engine node. v6 -> v7 ---- * Added comments on usage of noop_handler for a couple of engine inputs as suggested by Han. * Addressed other review comments from Han. * Added the check to handle the flow changes if 'skipped_last_time' is true in ofctrl_put(). v5 -> v6 ---- * Missed out checking in the uncommitted code in ofctrl.c in v4. v5 fixes it. * v5 accidently modified ovs submodule commit id. v6 reverts it. v4 -> v5 ----- * Addressed Han's comments. v3 -> v4 ----- * Addressed Mark G's comments. * Rebased to resolve conflicts. v2 -> v3 ----- * Rebased to resolve conflicts. v1 -> v2 ----- * Rebased to resolve conflicts. Numan Siddique (5): ovn-controller: Split logical flow and physical flow processing. controller: Handle sbrec_chassis changes in lflow and pflow output engines. ovn-controller: Handle datapath changes incrementally for ct zone I-P engine node. physical: Set the port binding uuid as cookie for flows where relevant. controller I-P: ct zone runtime data handler. TODO.rst | 6 + controller/ofctrl.c | 99 +++-- controller/ofctrl.h | 6 +- controller/ovn-controller.c | 812 ++++++++++++++++++++---------------- controller/physical.c | 68 ++- controller/physical.h | 4 - tests/ovn-performance.at | 26 ++ 7 files changed, 581 insertions(+), 440 deletions(-) Acked-by: Mark Michelson Acked-by: Mark Michelson Acked-by: Han Zhou