Fixes for 'make UNATTENDED=yes CC=clang CXX=clang verify'
[vpp.git] / src / vpp-api / vom / acl_binding.hpp
index faf9b24..89db0ee 100644 (file)
@@ -25,8 +25,8 @@
 #include "vom/interface.hpp"
 #include "vom/object_base.hpp"
 #include "vom/om.hpp"
-#include "vom/rpc_cmd.hpp"
 #include "vom/singular_db.hpp"
+#include "vom/singular_db_funcs.hpp"
 
 namespace VOM {
 namespace ACL {
@@ -34,14 +34,14 @@ namespace ACL {
  * A binding between an ACL and an interface.
  * A representation of the application of the ACL to the interface.
  */
-template <typename LIST, typename BIND, typename DUMP>
+template <typename LIST>
 class binding : public object_base
 {
 public:
   /**
    * The key for a binding is the direction and the interface
    */
-  typedef std::pair<direction_t, interface::key_type> key_t;
+  typedef std::pair<direction_t, interface::key_t> key_t;
 
   /**
    * Construct a new object matching the desried state
@@ -50,7 +50,7 @@ public:
     : m_direction(direction)
     , m_itf(itf.singular())
     , m_acl(acl.singular())
-    , m_binding(0)
+    , m_binding(false)
   {
     m_evh.order();
   }
@@ -62,7 +62,7 @@ public:
     : m_direction(o.m_direction)
     , m_itf(o.m_itf)
     , m_acl(o.m_acl)
-    , m_binding(0)
+    , m_binding(o.m_binding)
   {
   }
 
@@ -98,158 +98,7 @@ public:
    */
   static void dump(std::ostream& os) { m_db.dump(os); }
 
-  /**
-   * A command class that binds the ACL to the interface
-   */
-  class bind_cmd : public rpc_cmd<HW::item<bool>, rc_t, BIND>
-  {
-  public:
-    /**
-     * Constructor
-     */
-    bind_cmd(HW::item<bool>& item,
-             const direction_t& direction,
-             const handle_t& itf,
-             const handle_t& acl)
-      : rpc_cmd<HW::item<bool>, rc_t, BIND>(item)
-      , m_direction(direction)
-      , m_itf(itf)
-      , m_acl(acl)
-    {
-    }
-
-    /**
-     * Issue the command to VPP/HW
-     */
-    rc_t issue(connection& con);
-
-    /**
-     * convert to string format for debug purposes
-     */
-    std::string to_string() const
-    {
-      std::ostringstream s;
-      s << "acl-bind:[" << m_direction.to_string()
-        << " itf:" << m_itf.to_string() << " acl:" << m_acl.to_string() << "]";
-
-      return (s.str());
-    }
-
-    /**
-     * Comparison operator - only used for UT
-     */
-    bool operator==(const bind_cmd& other) const
-    {
-      return ((m_itf == other.m_itf) && (m_acl == m_acl));
-    }
-
-  private:
-    /**
-     * The direction of the binding
-     */
-    const direction_t m_direction;
-
-    /**
-     * The interface to bind to
-     */
-    const handle_t m_itf;
-
-    /**
-     * The ACL to bind
-     */
-    const handle_t m_acl;
-  };
-
-  /**
-   * A command class that binds the ACL to the interface
-   */
-  class unbind_cmd : public rpc_cmd<HW::item<bool>, rc_t, BIND>
-  {
-  public:
-    /**
-     * Constructor
-     */
-    unbind_cmd(HW::item<bool>& item,
-               const direction_t& direction,
-               const handle_t& itf,
-               const handle_t& acl)
-      : rpc_cmd<HW::item<bool>, rc_t, BIND>(item)
-      , m_direction(direction)
-      , m_itf(itf)
-      , m_acl(acl)
-    {
-    }
-
-    /**
-     * Issue the command to VPP/HW
-     */
-    rc_t issue(connection& con);
-
-    /**
-     * convert to string format for debug purposes
-     */
-    std::string to_string() const
-    {
-      std::ostringstream s;
-      s << "acl-unbind:[" << m_direction.to_string()
-        << " itf:" << m_itf.to_string() << " acl:" << m_acl.to_string() << "]";
-
-      return (s.str());
-    }
-
-    /**
-     * Comparison operator - only used for UT
-     */
-    bool operator==(const unbind_cmd& other) const
-    {
-      return ((m_itf == other.m_itf) && (m_acl == m_acl));
-    }
-
-  private:
-    /**
-     * The direction of the binding
-     */
-    const direction_t m_direction;
-
-    /**
-     * The interface to bind to
-     */
-    const handle_t m_itf;
-
-    /**
-     * The ACL to bind
-     */
-    const handle_t m_acl;
-  };
-
-  /**
-   * A cmd class that Dumps all the ACLs
-   */
-  class dump_cmd : public VOM::dump_cmd<DUMP>
-  {
-  public:
-    /**
-     * Constructor
-     */
-    dump_cmd() = default;
-    dump_cmd(const dump_cmd& d) = default;
-
-    /**
-     * Issue the command to VPP/HW
-     */
-    rc_t issue(connection& con);
-
-    /**
-     * convert to string format for debug purposes
-     */
-    std::string to_string() const { return ("acl-bind-dump"); }
-
-  private:
-    /**
-     * HW reutrn code
-     */
-    HW::item<bool> item;
-  };
+  static dependency_t order() { return m_evh.order(); }
 
 private:
   /**
@@ -258,11 +107,8 @@ private:
   class event_handler : public OM::listener, public inspect::command_handler
   {
   public:
-    event_handler()
-    {
-      OM::register_listener(this);
-      inspect::register_handler({ "acl-binding" }, "ACL bindings", this);
-    }
+    event_handler();
+
     virtual ~event_handler() = default;
 
     /**
@@ -278,7 +124,7 @@ private:
     /**
      * Show the object in the Singular DB
      */
-    void show(std::ostream& os) { m_db.dump(os); }
+    void show(std::ostream& os) { db_dump(m_db, os); }
 
     /**
      * Get the sortable Id of the listener
@@ -294,14 +140,7 @@ private:
   /**
    * Enquue commonds to the VPP command Q for the update
    */
-  void update(const binding& obj)
-  {
-    if (!m_binding) {
-      HW::enqueue(
-        new bind_cmd(m_binding, m_direction, m_itf->handle(), m_acl->handle()));
-    }
-    HW::write();
-  }
+  void update(const binding& obj);
 
   /**
    * Find or Add the instance in the DB
@@ -325,25 +164,12 @@ private:
   /**
    * Sweep/reap the object if still stale
    */
-  void sweep(void)
-  {
-    if (m_binding) {
-      HW::enqueue(new unbind_cmd(m_binding, m_direction, m_itf->handle(),
-                                 m_acl->handle()));
-    }
-    HW::write();
-  }
+  void sweep(void);
 
   /**
    * Replay the objects state to HW
    */
-  void replay(void)
-  {
-    if (m_binding) {
-      HW::enqueue(
-        new bind_cmd(m_binding, m_direction, m_itf->handle(), m_acl->handle()));
-    }
-  }
+  void replay(void);
 
   /**
    * The direction the of the packets on which to apply the ACL
@@ -380,35 +206,31 @@ private:
 /**
  * Typedef the L3 binding type
  */
-typedef binding<l3_list,
-                vapi::Acl_interface_add_del,
-                vapi::Acl_interface_list_dump>
-  l3_binding;
+typedef binding<l3_list> l3_binding;
 
 /**
  * Typedef the L2 binding type
  */
-typedef binding<l2_list,
-                vapi::Macip_acl_interface_add_del,
-                vapi::Macip_acl_interface_list_dump>
-  l2_binding;
+typedef binding<l2_list> l2_binding;
 
 /**
  * Definition of the static Singular DB for ACL bindings
  */
-template <typename LIST, typename BIND, typename DUMP>
-singular_db<typename ACL::binding<LIST, BIND, DUMP>::key_t,
-            ACL::binding<LIST, BIND, DUMP>>
-  binding<LIST, BIND, DUMP>::m_db;
-
-template <typename LIST, typename BIND, typename DUMP>
-typename ACL::binding<LIST, BIND, DUMP>::event_handler
-  binding<LIST, BIND, DUMP>::m_evh;
+template <typename LIST>
+singular_db<typename ACL::binding<LIST>::key_t, ACL::binding<LIST>>
+  binding<LIST>::m_db;
+
+template <typename LIST>
+typename ACL::binding<LIST>::event_handler binding<LIST>::m_evh;
+
+namespace {
+const static dependency_t __attribute__((unused)) l2o = l2_binding::order();
+const static dependency_t __attribute__((unused)) l3o = l3_binding::order();
+};
 };
 
-std::ostream& operator<<(
-  std::ostream& os,
-  const std::pair<direction_t, interface::key_type>& key);
+std::ostream& operator<<(std::ostream& os,
+                         const std::pair<direction_t, interface::key_t>& key);
 };
 
 /*