1. 25 8月, 2019 1 次提交
  2. 24 8月, 2019 4 次提交
  3. 23 8月, 2019 19 次提交
  4. 22 8月, 2019 16 次提交
    • E
      net/mlx5e: Add mlx5e HV VHCA stats agent · cef35af3
      Eran Ben Elisha 提交于
      HV VHCA stats agent is responsible on running a preiodic rx/tx
      packets/bytes stats update. Currently the supported format is version
      MLX5_HV_VHCA_STATS_VERSION. Block ID 1 is dedicated for statistics data
      transfer from the VF to the PF.
      
      The reporter fetch the statistics data from all opened channels, fill it
      in a buffer and send it to mlx5_hv_vhca_write_agent.
      
      As the stats layer should include some metadata per block (sequence and
      offset), the HV VHCA layer shall modify the buffer before actually send it
      over block 1.
      Signed-off-by: NEran Ben Elisha <eranbe@mellanox.com>
      Signed-off-by: NSaeed Mahameed <saeedm@mellanox.com>
      Signed-off-by: NHaiyang Zhang <haiyangz@microsoft.com>
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      cef35af3
    • E
      net/mlx5: Add HV VHCA control agent · 29ddad43
      Eran Ben Elisha 提交于
      Control agent is responsible over of the control block (ID 0). It should
      update the PF via this block about every capability change. In addition,
      upon block 0 invalidate, it should activate all other supported agents
      with data requests from the PF.
      
      Upon agent create/destroy, the invalidate callback of the control agent
      is being called in order to update the PF driver about this change.
      
      The control agent is an integral part of HV VHCA and will be created
      and destroy as part of the HV VHCA init/cleanup flow.
      Signed-off-by: NEran Ben Elisha <eranbe@mellanox.com>
      Signed-off-by: NSaeed Mahameed <saeedm@mellanox.com>
      Signed-off-by: NHaiyang Zhang <haiyangz@microsoft.com>
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      29ddad43
    • E
      net/mlx5: Add HV VHCA infrastructure · 87175120
      Eran Ben Elisha 提交于
      HV VHCA is a layer which provides PF to VF communication channel based on
      HyperV PCI config channel. It implements Mellanox's Inter VHCA control
      communication protocol. The protocol contains control block in order to
      pass messages between the PF and VF drivers, and data blocks in order to
      pass actual data.
      
      The infrastructure is agent based. Each agent will be responsible of
      contiguous buffer blocks in the VHCA config space. This infrastructure will
      bind agents to their blocks, and those agents can only access read/write
      the buffer blocks assigned to them. Each agent will provide three
      callbacks (control, invalidate, cleanup). Control will be invoked when
      block-0 is invalidated with a command that concerns this agent. Invalidate
      callback will be invoked if one of the blocks assigned to this agent was
      invalidated. Cleanup will be invoked before the agent is being freed in
      order to clean all of its open resources or deferred works.
      
      Block-0 serves as the control block. All execution commands from the PF
      will be written by the PF over this block. VF will ack on those by
      writing on block-0 as well. Its format is described by struct
      mlx5_hv_vhca_control_block layout.
      Signed-off-by: NEran Ben Elisha <eranbe@mellanox.com>
      Signed-off-by: NSaeed Mahameed <saeedm@mellanox.com>
      Signed-off-by: NHaiyang Zhang <haiyangz@microsoft.com>
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      87175120
    • E
      net/mlx5: Add wrappers for HyperV PCIe operations · 913d14e8
      Eran Ben Elisha 提交于
      Add wrapper functions for HyperV PCIe read / write /
      block_invalidate_register operations.  This will be used as an
      infrastructure in the downstream patch for software communication.
      
      This will be enabled by default if CONFIG_PCI_HYPERV_INTERFACE is set.
      Signed-off-by: NEran Ben Elisha <eranbe@mellanox.com>
      Signed-off-by: NSaeed Mahameed <saeedm@mellanox.com>
      Signed-off-by: NHaiyang Zhang <haiyangz@microsoft.com>
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      913d14e8
    • H
      PCI: hv: Add a Hyper-V PCI interface driver for software backchannel interface · 348dd93e
      Haiyang Zhang 提交于
      This interface driver is a helper driver allows other drivers to
      have a common interface with the Hyper-V PCI frontend driver.
      Signed-off-by: NHaiyang Zhang <haiyangz@microsoft.com>
      Signed-off-by: NSaeed Mahameed <saeedm@mellanox.com>
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      348dd93e
    • D
      PCI: hv: Add a paravirtual backchannel in software · e5d2f910
      Dexuan Cui 提交于
      Windows SR-IOV provides a backchannel mechanism in software for communication
      between a VF driver and a PF driver.  These "configuration blocks" are
      similar in concept to PCI configuration space, but instead of doing reads and
      writes in 32-bit chunks through a very slow path, packets of up to 128 bytes
      can be sent or received asynchronously.
      
      Nearly every SR-IOV device contains just such a communications channel in
      hardware, so using this one in software is usually optional.  Using the
      software channel, however, allows driver implementers to leverage software
      tools that fuzz the communications channel looking for vulnerabilities.
      
      The usage model for these packets puts the responsibility for reading or
      writing on the VF driver.  The VF driver sends a read or a write packet,
      indicating which "block" is being referred to by number.
      
      If the PF driver wishes to initiate communication, it can "invalidate" one or
      more of the first 64 blocks.  This invalidation is delivered via a callback
      supplied by the VF driver by this driver.
      
      No protocol is implied, except that supplied by the PF and VF drivers.
      Signed-off-by: NJake Oshins <jakeo@microsoft.com>
      Signed-off-by: NDexuan Cui <decui@microsoft.com>
      Cc: Haiyang Zhang <haiyangz@microsoft.com>
      Cc: K. Y. Srinivasan <kys@microsoft.com>
      Cc: Stephen Hemminger <sthemmin@microsoft.com>
      Signed-off-by: NSaeed Mahameed <saeedm@mellanox.com>
      Signed-off-by: NHaiyang Zhang <haiyangz@microsoft.com>
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      e5d2f910
    • V
      net/mlx5e: Add trace point for neigh update · 5970882a
      Vlad Buslov 提交于
      Allow tracing neigh state during neigh update task that is executed on
      workqueue and is scheduled by neigh state change event.
      
      Usage example:
       ># cd /sys/kernel/debug/tracing
       ># echo mlx5:mlx5e_rep_neigh_update >> set_event
       ># cat trace
          ...
          kworker/u48:7-2221  [009] ...1  1475.387435: mlx5e_rep_neigh_update:
      netdev: ens1f0 MAC: 24:8a:07:9a:17:9a IPv4: 1.1.1.10 IPv6: ::ffff:1.1.1.10 neigh_connected=1
      
      Added corresponding documentation in
          Documentation/networking/device-driver/mellanox/mlx5.rst
      Signed-off-by: NVlad Buslov <vladbu@mellanox.com>
      Reviewed-by: NDmytro Linkin <dmitrolin@mellanox.com>
      Signed-off-by: NSaeed Mahameed <saeedm@mellanox.com>
      5970882a
    • V
      net/mlx5e: Add trace point for neigh used value update · c786fe59
      Vlad Buslov 提交于
      Allow tracing result of neigh used value update task that is executed
      periodically on workqueue.
      
      Usage example:
       ># cd /sys/kernel/debug/tracing
       ># echo mlx5:mlx5e_tc_update_neigh_used_value >> set_event
       ># cat trace
          ...
          kworker/u48:4-8806  [009] ...1 55117.882428: mlx5e_tc_update_neigh_used_value:
      netdev: ens1f0 IPv4: 1.1.1.10 IPv6: ::ffff:1.1.1.10 neigh_used=1
      
      Added corresponding documentation in
          Documentation/networking/device-driver/mellanox/mlx5.rst
      Signed-off-by: NVlad Buslov <vladbu@mellanox.com>
      Reviewed-by: NDmytro Linkin <dmitrolin@mellanox.com>
      Signed-off-by: NSaeed Mahameed <saeedm@mellanox.com>
      c786fe59
    • D
      net/mlx5e: Add tc flower tracepoints · 7a978759
      Dmytro Linkin 提交于
      Implemented following tracepoints:
      1. Configure flower (mlx5e_configure_flower)
      2. Delete flower (mlx5e_delete_flower)
      3. Stats flower (mlx5e_stats_flower)
      
      Usage example:
       ># cd /sys/kernel/debug/tracing
       ># echo mlx5:mlx5e_configure_flower >> set_event
       ># cat trace
          ...
          tc-6535  [019] ...1  2672.404466: mlx5e_configure_flower: cookie=0000000067874a55 actions= REDIRECT
      
      Added corresponding documentation in
          Documentation/networking/device-driver/mellanox/mlx5.rst
      Signed-off-by: NDmytro Linkin <dmitrolin@mellanox.com>
      Reviewed-by: NVlad Buslov <vladbu@mellanox.com>
      Signed-off-by: NSaeed Mahameed <saeedm@mellanox.com>
      7a978759
    • V
      net/mlx5e: Only access fully initialized flows in neigh update · 95435ad7
      Vlad Buslov 提交于
      To remove dependency on rtnl lock and prevent neigh update code from
      accessing uninitialized flows when executing concurrently with tc, extend
      mlx5e_tc_flow with 'init_done' completion. Modify helper
      mlx5e_take_all_encap_flows() to wait for flow completion after obtaining
      reference to it. Modify mlx5e_tc_encap_flows_del() and
      mlx5e_tc_encap_flows_add() to skip flows that don't have OFFLOADED flag
      set, which can happen if concurrent flow initialization failed.
      
      This commit finishes neigh update refactoring for concurrent execution
      started in previous change in this series.
      Signed-off-by: NVlad Buslov <vladbu@mellanox.com>
      Reviewed-by: NRoi Dayan <roid@mellanox.com>
      Signed-off-by: NSaeed Mahameed <saeedm@mellanox.com>
      95435ad7
    • V
      net/mlx5e: Refactor neigh update for concurrent execution · 2a1f1768
      Vlad Buslov 提交于
      In order to remove dependency on rtnl lock and allow neigh update workqueue
      task to execute concurrently with tc, refactor mlx5e_rep_neigh_update() for
      concurrent execution:
      
      - Lock encap table when accessing encap entry to prevent concurrent
        changes. To do this properly, the initial encap state check is moved from
        mlx5e_rep_neigh_update() into mlx5e_rep_update_flows() to be performed
        under encap_tbl_lock protection.
      
      - Wait for encap to be fully initialized before accessing it by means of
        'res_ready' completion.
      
      - Add mlx5e_take_all_encap_flows() helper which is used to construct a
        temporary list of flows and efi indexes that is used to access current
        encap data in flow which can be attached to multiple encaps
        simultaneously. Release the flows from temporary list after
        encap_tbl_lock critical section. This is necessary because
        mlx5e_flow_put() can't be called while holding encap_tbl_lock.
      
      - Modify mlx5e_tc_encap_flows_add() and mlx5e_tc_encap_flows_del() to work
        with user-provided list of flows built by mlx5e_take_all_encap_flows(),
        instead of traversing encap flow list directly.
      
      This is first step in complex neigh update refactoring, which is finished
      by following commit in this series.
      Signed-off-by: NVlad Buslov <vladbu@mellanox.com>
      Reviewed-by: NRoi Dayan <roid@mellanox.com>
      Signed-off-by: NSaeed Mahameed <saeedm@mellanox.com>
      2a1f1768
    • V
      net/mlx5e: Refactor neigh used value update for concurrent execution · 6a06c2f7
      Vlad Buslov 提交于
      In order to remove dependency on rtnl lock and allow neigh used value
      update workqueue task to execute concurrently with tc, refactor
      mlx5e_tc_update_neigh_used_value() for concurrent execution:
      
      - Lock encap table when accessing encap entry to prevent concurrent
        changes.
      
      - Save offloaded encap flows to temporary list and release them after encap
        entry is updated. Add mlx5e_put_encap_flow_list() helper which is
        intended to be shared with neigh update code in following patch in this
        series. This is necessary because mlx5e_flow_put() can't be called while
        holding encap_tbl_lock.
      Signed-off-by: NVlad Buslov <vladbu@mellanox.com>
      Reviewed-by: NRoi Dayan <roid@mellanox.com>
      Signed-off-by: NSaeed Mahameed <saeedm@mellanox.com>
      6a06c2f7
    • V
      net/mlx5e: Protect neigh hash encap list with spinlock and rcu · ac0d9176
      Vlad Buslov 提交于
      Rcu-ify mlx5e_neigh_hash_entry->encap_list by changing operations on encap
      list to their rcu counterparts and extending encap structure with rcu_head
      to free the encap instances after rcu grace period. Use rcu read lock when
      traversing encap list. Implement helper mlx5e_get_next_valid_encap()
      function that is used by mlx5e_tc_update_neigh_used_value() to safely
      iterate over valid entries of nhe->encap_list.
      Signed-off-by: NVlad Buslov <vladbu@mellanox.com>
      Reviewed-by: NJianbo Liu <jianbol@mellanox.com>
      Reviewed-by: NRoi Dayan <roid@mellanox.com>
      Signed-off-by: NSaeed Mahameed <saeedm@mellanox.com>
      ac0d9176
    • V
      net/mlx5e: Refactor mlx5e_neigh_update_table->encap_lock · 70e83bd3
      Vlad Buslov 提交于
      To remove dependency on rtnl lock, always take neigh update encap lock when
      modifying neigh update hash table and list. Originally, this lock was only
      used to synchronize with netevent handler function, which is called from bh
      context and cannot use rtnl lock for synchronization. Take lock in encap
      entry attach function to prevent concurrent modifications of neigh update
      hash table and list.
      
      Taking the encap lock when creating new nhe introduces a problem that we
      need to allocate new entry with sleeping GFP_KERNEL flag while holding a
      spinlock. However, since previous patch in this series has already
      converted lookup in netevent handler function to user rcu read lock instead
      of encap lock, we can safely convert the lock type to mutex.
      Signed-off-by: NVlad Buslov <vladbu@mellanox.com>
      Reviewed-by: NJianbo Liu <jianbol@mellanox.com>
      Reviewed-by: NRoi Dayan <roid@mellanox.com>
      Signed-off-by: NSaeed Mahameed <saeedm@mellanox.com>
      70e83bd3
    • V
      net/mlx5e: Extend neigh hash entry with rcu · 1216ce9d
      Vlad Buslov 提交于
      To remove dependency on rtnl lock and to allow unlocked iteration over list
      of neigh hash entries, extend nhe with rcu. Change operations on neigh list
      to their rcu counterparts and free neigh hash entry with rcu timeout.
      
      Introduce mlx5e_get_next_nhe() helper that is used to iterate over rcu
      neigh list with reference to nhe taken.
      Signed-off-by: NVlad Buslov <vladbu@mellanox.com>
      Reviewed-by: NJianbo Liu <jianbol@mellanox.com>
      Reviewed-by: NRoi Dayan <roid@mellanox.com>
      Signed-off-by: NSaeed Mahameed <saeedm@mellanox.com>
      1216ce9d
    • V
      net/mlx5e: Always take reference to neigh entry · 61081f9c
      Vlad Buslov 提交于
      Neigh entry has reference counter, however it is only used when scheduling
      neigh update event. In all other cases reference to neigh entry is not
      taken while working with it. Neigh code relies on synchronization provided
      by rtnl lock and uses encap list size as implicit reference counter.
      
      To remove dependency on rtnl lock, always take reference to neigh entry
      while using it. Remove neigh entry from hash table and delete it only when
      reference counter reaches zero. This can result spurious neigh update
      events, when there is an event on entry that has zero encaps attached.
      However, such events are rare and properly handled by neigh update handler.
      
      Extend encap entry with reference to neigh hash entry in order to be able
      to directly release it when encap is detached, instead of lookup nhe by key
      through hash table. Extend nhe with reference to device priv structure to
      guarantee correctness when nhe is used with stack devices, bond setup, in
      which case it is non-trivial to determine correct device when releasing the
      nhe.
      Signed-off-by: NVlad Buslov <vladbu@mellanox.com>
      Reviewed-by: NJianbo Liu <jianbol@mellanox.com>
      Reviewed-by: NRoi Dayan <roid@mellanox.com>
      Signed-off-by: NSaeed Mahameed <saeedm@mellanox.com>
      61081f9c