Handshake with PF kernel driver to check driver version compatibility.

Signed-off-by: Shijith Thotton <shijith.thot...@caviumnetworks.com>
Signed-off-by: Jerin Jacob <jerin.ja...@caviumnetworks.com>
Signed-off-by: Derek Chickles <derek.chick...@caviumnetworks.com>
Signed-off-by: Venkat Koppula <venkat.kopp...@caviumnetworks.com>
Signed-off-by: Srisivasubramanian S <ssriniva...@caviumnetworks.com>
Signed-off-by: Mallesham Jatharakonda <mjatharako...@oneconvergence.com>
---
 drivers/net/liquidio/base/lio_23xx_vf.c | 96 +++++++++++++++++++++++++++++++++
 drivers/net/liquidio/base/lio_23xx_vf.h |  4 ++
 drivers/net/liquidio/base/lio_hw_defs.h |  3 ++
 drivers/net/liquidio/base/lio_mbox.h    |  1 +
 drivers/net/liquidio/lio_ethdev.c       | 21 ++++++++
 drivers/net/liquidio/lio_struct.h       | 45 ++++++++++++++++
 6 files changed, 170 insertions(+)

diff --git a/drivers/net/liquidio/base/lio_23xx_vf.c 
b/drivers/net/liquidio/base/lio_23xx_vf.c
index 70faa9b..6270af5 100644
--- a/drivers/net/liquidio/base/lio_23xx_vf.c
+++ b/drivers/net/liquidio/base/lio_23xx_vf.c
@@ -255,6 +255,102 @@
        return 0;
 }
 
+static void
+cn23xx_pfvf_hs_callback(struct lio_device *lio_dev,
+                       struct lio_mbox_cmd *cmd, void *arg)
+{
+       uint32_t major = 0;
+
+       PMD_INIT_FUNC_TRACE();
+
+       rte_memcpy((uint8_t *)&lio_dev->pfvf_hsword, cmd->msg.s.params, 6);
+       if (cmd->recv_len > 1) {
+               struct lio_version *lio_ver = (struct lio_version *)cmd->data;
+
+               major = lio_ver->major;
+               major = major << 16;
+       }
+
+       rte_atomic64_set((rte_atomic64_t *)arg, major | 1);
+}
+
+int
+cn23xx_pfvf_handshake(struct lio_device *lio_dev)
+{
+       struct lio_mbox_cmd mbox_cmd;
+       struct lio_version *lio_ver = (struct lio_version *)&mbox_cmd.data[0];
+       rte_atomic64_t status;
+       uint32_t count = 0;
+       uint32_t pfmajor;
+       uint32_t vfmajor;
+       uint32_t ret;
+
+       PMD_INIT_FUNC_TRACE();
+
+       /* Sending VF_ACTIVE indication to the PF driver */
+       lio_dev_dbg(lio_dev, "requesting info from PF\n");
+
+       mbox_cmd.msg.mbox_msg64 = 0;
+       mbox_cmd.msg.s.type = LIO_MBOX_REQUEST;
+       mbox_cmd.msg.s.resp_needed = 1;
+       mbox_cmd.msg.s.cmd = LIO_VF_ACTIVE;
+       mbox_cmd.msg.s.len = 2;
+       mbox_cmd.data[0] = 0;
+       lio_ver->major = LIO_BASE_MAJOR_VERSION;
+       lio_ver->minor = LIO_BASE_MINOR_VERSION;
+       lio_ver->micro = LIO_BASE_MICRO_VERSION;
+       mbox_cmd.q_no = 0;
+       mbox_cmd.recv_len = 0;
+       mbox_cmd.recv_status = 0;
+       mbox_cmd.fn = (lio_mbox_callback)cn23xx_pfvf_hs_callback;
+       mbox_cmd.fn_arg = (void *)&status;
+
+       if (lio_mbox_write(lio_dev, &mbox_cmd)) {
+               lio_dev_err(lio_dev, "Write to mailbox failed\n");
+               return -1;
+       }
+
+       rte_atomic64_set(&status, 0);
+
+       do {
+               rte_delay_ms(1);
+       } while ((rte_atomic64_read(&status) == 0) && (count++ < 10000));
+
+       ret = rte_atomic64_read(&status);
+       if (ret == 0) {
+               lio_dev_err(lio_dev, "cn23xx_pfvf_handshake timeout\n");
+               return -1;
+       }
+
+       vfmajor = LIO_BASE_MAJOR_VERSION;
+       pfmajor = ret >> 16;
+       if (pfmajor != vfmajor) {
+               lio_dev_err(lio_dev,
+                           "VF LiquidIO driver (major version %d) is not 
compatible with LiquidIO PF driver (major version %d)\n",
+                           vfmajor, pfmajor);
+               ret = -EPERM;
+       } else {
+               lio_dev_dbg(lio_dev,
+                           "VF LiquidIO driver (major version %d), LiquidIO PF 
driver (major version %d)\n",
+                           vfmajor, pfmajor);
+               ret = 0;
+       }
+
+       return ret;
+}
+
+void
+cn23xx_vf_handle_mbox(struct lio_device *lio_dev)
+{
+       uint64_t mbox_int_val;
+
+       /* read and clear by writing 1 */
+       mbox_int_val = rte_read64(lio_dev->mbox[0]->mbox_int_reg);
+       rte_write64(mbox_int_val, lio_dev->mbox[0]->mbox_int_reg);
+       if (lio_mbox_read(lio_dev->mbox[0]))
+               lio_mbox_process_message(lio_dev->mbox[0]);
+}
+
 int
 cn23xx_vf_setup_device(struct lio_device *lio_dev)
 {
diff --git a/drivers/net/liquidio/base/lio_23xx_vf.h 
b/drivers/net/liquidio/base/lio_23xx_vf.h
index 1af09d0..83dc053 100644
--- a/drivers/net/liquidio/base/lio_23xx_vf.h
+++ b/drivers/net/liquidio/base/lio_23xx_vf.h
@@ -87,5 +87,9 @@
 
 #define CN23XX_VF_BUSY_READING_REG_LOOP_COUNT  100000
 
+int cn23xx_pfvf_handshake(struct lio_device *lio_dev);
+
 int cn23xx_vf_setup_device(struct lio_device  *lio_dev);
+
+void cn23xx_vf_handle_mbox(struct lio_device *lio_dev);
 #endif /* _LIO_23XX_VF_H_  */
diff --git a/drivers/net/liquidio/base/lio_hw_defs.h 
b/drivers/net/liquidio/base/lio_hw_defs.h
index a2654cd..9282068 100644
--- a/drivers/net/liquidio/base/lio_hw_defs.h
+++ b/drivers/net/liquidio/base/lio_hw_defs.h
@@ -77,6 +77,9 @@ enum lio_card_type {
 #define LIO_23XX_NAME "23xx"
 
 #define LIO_DEVICE_NAME_LEN            32
+#define LIO_BASE_MAJOR_VERSION         1
+#define LIO_BASE_MINOR_VERSION         5
+#define LIO_BASE_MICRO_VERSION         1
 
 /* Routines for reading and writing CSRs */
 #ifdef RTE_LIBRTE_LIO_DEBUG_REGS
diff --git a/drivers/net/liquidio/base/lio_mbox.h 
b/drivers/net/liquidio/base/lio_mbox.h
index 28c9e1a..f1c5b8e 100644
--- a/drivers/net/liquidio/base/lio_mbox.h
+++ b/drivers/net/liquidio/base/lio_mbox.h
@@ -42,6 +42,7 @@
 
 #define LIO_MBOX_DATA_MAX                      32
 
+#define LIO_VF_ACTIVE                          0x1
 #define LIO_CORES_CRASHED                      0x3
 
 /* Macro for Read acknowledgment */
diff --git a/drivers/net/liquidio/lio_ethdev.c 
b/drivers/net/liquidio/lio_ethdev.c
index 5ee1bb5..bebe0e8 100644
--- a/drivers/net/liquidio/lio_ethdev.c
+++ b/drivers/net/liquidio/lio_ethdev.c
@@ -40,6 +40,20 @@
 #include "lio_23xx_vf.h"
 #include "lio_ethdev.h"
 
+static void
+lio_check_pf_hs_response(void *lio_dev)
+{
+       struct lio_device *dev = lio_dev;
+
+       /* check till response arrives */
+       if (dev->pfvf_hsword.coproc_tics_per_us)
+               return;
+
+       cn23xx_vf_handle_mbox(dev);
+
+       rte_eal_alarm_set(1, lio_check_pf_hs_response, lio_dev);
+}
+
 /**
  * \brief Identify the LIO device and to map the BAR address space
  * @param lio_dev lio device
@@ -91,6 +105,13 @@
                goto error;
        }
 
+       /* Check PF response */
+       lio_check_pf_hs_response((void *)lio_dev);
+
+       /* Do handshake and exit if incompatible PF driver */
+       if (cn23xx_pfvf_handshake(lio_dev))
+               goto error;
+
        if (cn23xx_vf_set_io_queues_off(lio_dev)) {
                lio_dev_err(lio_dev, "Setting io queues off failed\n");
                goto error;
diff --git a/drivers/net/liquidio/lio_struct.h 
b/drivers/net/liquidio/lio_struct.h
index 01b5716..e8b6e1d 100644
--- a/drivers/net/liquidio/lio_struct.h
+++ b/drivers/net/liquidio/lio_struct.h
@@ -43,6 +43,13 @@
 
 #include "lio_hw_defs.h"
 
+struct lio_version {
+       uint16_t major;
+       uint16_t minor;
+       uint16_t micro;
+       uint16_t reserved;
+};
+
 struct lio_device;
 struct lio_fn_list {
        int (*setup_mbox)(struct lio_device *);
@@ -51,6 +58,42 @@ struct lio_fn_list {
        int (*setup_device_regs)(struct lio_device *);
 };
 
+struct lio_pf_vf_hs_word {
+#if RTE_BYTE_ORDER == RTE_LITTLE_ENDIAN
+       /** PKIND value assigned for the DPI interface */
+       uint64_t pkind : 8;
+
+       /** OCTEON core clock multiplier */
+       uint64_t core_tics_per_us : 16;
+
+       /** OCTEON coprocessor clock multiplier */
+       uint64_t coproc_tics_per_us : 16;
+
+       /** app that currently running on OCTEON */
+       uint64_t app_mode : 8;
+
+       /** RESERVED */
+       uint64_t reserved : 16;
+
+#elif RTE_BYTE_ORDER == RTE_BIG_ENDIAN
+
+       /** RESERVED */
+       uint64_t reserved : 16;
+
+       /** app that currently running on OCTEON */
+       uint64_t app_mode : 8;
+
+       /** OCTEON coprocessor clock multiplier */
+       uint64_t coproc_tics_per_us : 16;
+
+       /** OCTEON core clock multiplier */
+       uint64_t core_tics_per_us : 16;
+
+       /** PKIND value assigned for the DPI interface */
+       uint64_t pkind : 8;
+#endif
+};
+
 struct lio_sriov_info {
        /** Number of rings assigned to VF */
        uint32_t rings_per_vf;
@@ -129,6 +172,8 @@ struct lio_device {
 
        struct lio_sriov_info sriov_info;
 
+       struct lio_pf_vf_hs_word pfvf_hsword;
+
        /** Mail Box details of each lio queue. */
        struct lio_mbox **mbox;
 
-- 
1.8.3.1

Reply via email to