]> www.infradead.org Git - users/willy/xarray.git/commitdiff
ice: implement full NVM read from ETHTOOL_GEEPROM
authorJesse Brandeburg <jesse.brandeburg@intel.com>
Thu, 12 Mar 2020 01:58:14 +0000 (18:58 -0700)
committerJeff Kirsher <jeffrey.t.kirsher@intel.com>
Sat, 21 Mar 2020 07:31:17 +0000 (00:31 -0700)
The current implementation of .get_eeprom only enables reading from the
Shadow RAM portion of the NVM contents. Implement support for reading
the entire flash contents instead of only the initial portion contained
in the Shadow RAM.

A complete dump can take several seconds, but the ETHTOOL_GEEPROM ioctl
is capable of reading only a limited portion at a time by specifying the
offset and length to read.

In order to perform the reads directly, several functions are made non
static. Additionally, the unused ice_read_sr_buf_aq and ice_read_sr_buf
functions are removed.

Signed-off-by: Jesse Brandeburg <jesse.brandeburg@intel.com>
Signed-off-by: Jacob Keller <jacob.e.keller@intel.com>
Tested-by: Andrew Bowers <andrewx.bowers@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
drivers/net/ethernet/intel/ice/ice_common.h
drivers/net/ethernet/intel/ice/ice_ethtool.c
drivers/net/ethernet/intel/ice/ice_nvm.c
drivers/net/ethernet/intel/ice/ice_nvm.h

index 8fb29657e316ba1a6a3d4bf9e3360d55381e5957..06fcd3b55570e9465a63b5b80fdd6227d15e50e1 100644 (file)
@@ -38,9 +38,6 @@ enum ice_status
 ice_alloc_hw_res(struct ice_hw *hw, u16 type, u16 num, bool btm, u16 *res);
 enum ice_status
 ice_free_hw_res(struct ice_hw *hw, u16 type, u16 num, u16 *res);
-enum ice_status ice_init_nvm(struct ice_hw *hw);
-enum ice_status
-ice_read_sr_buf(struct ice_hw *hw, u16 offset, u16 *words, u16 *data);
 enum ice_status
 ice_aq_alloc_free_res(struct ice_hw *hw, u16 num_entries,
                      struct ice_aqc_alloc_free_res_elem *buf, u16 buf_size,
index 75970bcfa6c0e0d7f86dd7634083f796b25165b0..593fb37bd59e0f1e8df1d6700adf0df545f49ebb 100644 (file)
@@ -245,7 +245,7 @@ static int ice_get_eeprom_len(struct net_device *netdev)
        struct ice_netdev_priv *np = netdev_priv(netdev);
        struct ice_pf *pf = np->vsi->back;
 
-       return (int)(pf->hw.nvm.sr_words * sizeof(u16));
+       return (int)pf->hw.nvm.flash_size;
 }
 
 static int
@@ -253,39 +253,46 @@ ice_get_eeprom(struct net_device *netdev, struct ethtool_eeprom *eeprom,
               u8 *bytes)
 {
        struct ice_netdev_priv *np = netdev_priv(netdev);
-       u16 first_word, last_word, nwords;
        struct ice_vsi *vsi = np->vsi;
        struct ice_pf *pf = vsi->back;
        struct ice_hw *hw = &pf->hw;
        enum ice_status status;
        struct device *dev;
        int ret = 0;
-       u16 *buf;
+       u8 *buf;
 
        dev = ice_pf_to_dev(pf);
 
        eeprom->magic = hw->vendor_id | (hw->device_id << 16);
+       netdev_dbg(netdev, "GEEPROM cmd 0x%08x, offset 0x%08x, len 0x%08x\n",
+                  eeprom->cmd, eeprom->offset, eeprom->len);
 
-       first_word = eeprom->offset >> 1;
-       last_word = (eeprom->offset + eeprom->len - 1) >> 1;
-       nwords = last_word - first_word + 1;
-
-       buf = devm_kcalloc(dev, nwords, sizeof(u16), GFP_KERNEL);
+       buf = kzalloc(eeprom->len, GFP_KERNEL);
        if (!buf)
                return -ENOMEM;
 
-       status = ice_read_sr_buf(hw, first_word, &nwords, buf);
+       status = ice_acquire_nvm(hw, ICE_RES_READ);
        if (status) {
-               dev_err(dev, "ice_read_sr_buf failed, err %d aq_err %d\n",
+               dev_err(dev, "ice_acquire_nvm failed, err %d aq_err %d\n",
                        status, hw->adminq.sq_last_status);
-               eeprom->len = sizeof(u16) * nwords;
                ret = -EIO;
                goto out;
        }
 
-       memcpy(bytes, (u8 *)buf + (eeprom->offset & 1), eeprom->len);
+       status = ice_read_flat_nvm(hw, eeprom->offset, &eeprom->len, buf,
+                                  false);
+       if (status) {
+               dev_err(dev, "ice_read_flat_nvm failed, err %d aq_err %d\n",
+                       status, hw->adminq.sq_last_status);
+               ret = -EIO;
+               goto release;
+       }
+
+       memcpy(bytes, buf, eeprom->len);
+release:
+       ice_release_nvm(hw);
 out:
-       devm_kfree(dev, buf);
+       kfree(buf);
        return ret;
 }
 
index 4cdce03709637c5ca9b08bab7f5063c3887f5aaf..08909d1c7cce6b28dc46d5c8a31ef601a1f1aa52 100644 (file)
@@ -107,70 +107,6 @@ ice_read_flat_nvm(struct ice_hw *hw, u32 offset, u32 *length, u8 *data,
        return status;
 }
 
-/**
- * ice_check_sr_access_params - verify params for Shadow RAM R/W operations.
- * @hw: pointer to the HW structure
- * @offset: offset in words from module start
- * @words: number of words to access
- */
-static enum ice_status
-ice_check_sr_access_params(struct ice_hw *hw, u32 offset, u16 words)
-{
-       if ((offset + words) > hw->nvm.sr_words) {
-               ice_debug(hw, ICE_DBG_NVM,
-                         "NVM error: offset beyond SR lmt.\n");
-               return ICE_ERR_PARAM;
-       }
-
-       if (words > ICE_SR_SECTOR_SIZE_IN_WORDS) {
-               /* We can access only up to 4KB (one sector), in one AQ write */
-               ice_debug(hw, ICE_DBG_NVM,
-                         "NVM error: tried to access %d words, limit is %d.\n",
-                         words, ICE_SR_SECTOR_SIZE_IN_WORDS);
-               return ICE_ERR_PARAM;
-       }
-
-       if (((offset + (words - 1)) / ICE_SR_SECTOR_SIZE_IN_WORDS) !=
-           (offset / ICE_SR_SECTOR_SIZE_IN_WORDS)) {
-               /* A single access cannot spread over two sectors */
-               ice_debug(hw, ICE_DBG_NVM,
-                         "NVM error: cannot spread over two sectors.\n");
-               return ICE_ERR_PARAM;
-       }
-
-       return 0;
-}
-
-/**
- * ice_read_sr_aq - Read Shadow RAM.
- * @hw: pointer to the HW structure
- * @offset: offset in words from module start
- * @words: number of words to read
- * @data: storage for the words read from Shadow RAM (Little Endian)
- * @last_command: tells the AdminQ that this is the last command
- *
- * Reads 16-bit Little Endian word buffers from the Shadow RAM using the admin
- * command.
- */
-static enum ice_status
-ice_read_sr_aq(struct ice_hw *hw, u32 offset, u16 words, __le16 *data,
-              bool last_command)
-{
-       enum ice_status status;
-
-       status = ice_check_sr_access_params(hw, offset, words);
-
-       /* values in "offset" and "words" parameters are sized as words
-        * (16 bits) but ice_aq_read_nvm expects these values in bytes.
-        * So do this conversion while calling ice_aq_read_nvm.
-        */
-       if (!status)
-               status = ice_aq_read_nvm(hw, 0, 2 * offset, 2 * words, data,
-                                        last_command, true, NULL);
-
-       return status;
-}
-
 /**
  * ice_read_sr_word_aq - Reads Shadow RAM via AQ
  * @hw: pointer to the HW structure
@@ -198,63 +134,6 @@ ice_read_sr_word_aq(struct ice_hw *hw, u16 offset, u16 *data)
        return 0;
 }
 
-/**
- * ice_read_sr_buf_aq - Reads Shadow RAM buf via AQ
- * @hw: pointer to the HW structure
- * @offset: offset of the Shadow RAM word to read (0x000000 - 0x001FFF)
- * @words: (in) number of words to read; (out) number of words actually read
- * @data: words read from the Shadow RAM
- *
- * Reads 16 bit words (data buf) from the SR using the ice_read_sr_aq
- * method. Ownership of the NVM is taken before reading the buffer and later
- * released.
- */
-static enum ice_status
-ice_read_sr_buf_aq(struct ice_hw *hw, u16 offset, u16 *words, u16 *data)
-{
-       enum ice_status status;
-       bool last_cmd = false;
-       u16 words_read = 0;
-       u16 i = 0;
-
-       do {
-               u16 read_size, off_w;
-
-               /* Calculate number of bytes we should read in this step.
-                * It's not allowed to read more than one page at a time or
-                * to cross page boundaries.
-                */
-               off_w = offset % ICE_SR_SECTOR_SIZE_IN_WORDS;
-               read_size = off_w ?
-                       min_t(u16, *words,
-                             (ICE_SR_SECTOR_SIZE_IN_WORDS - off_w)) :
-                       min_t(u16, (*words - words_read),
-                             ICE_SR_SECTOR_SIZE_IN_WORDS);
-
-               /* Check if this is last command, if so set proper flag */
-               if ((words_read + read_size) >= *words)
-                       last_cmd = true;
-
-               status = ice_read_sr_aq(hw, offset, read_size,
-                                       data + words_read, last_cmd);
-               if (status)
-                       goto read_nvm_buf_aq_exit;
-
-               /* Increment counter for words already read and move offset to
-                * new read location
-                */
-               words_read += read_size;
-               offset += read_size;
-       } while (words_read < *words);
-
-       for (i = 0; i < *words; i++)
-               data[i] = le16_to_cpu(((__force __le16 *)data)[i]);
-
-read_nvm_buf_aq_exit:
-       *words = words_read;
-       return status;
-}
-
 /**
  * ice_acquire_nvm - Generic request for acquiring the NVM ownership
  * @hw: pointer to the HW structure
@@ -262,7 +141,7 @@ read_nvm_buf_aq_exit:
  *
  * This function will request NVM ownership.
  */
-static enum ice_status
+enum ice_status
 ice_acquire_nvm(struct ice_hw *hw, enum ice_aq_res_access_type access)
 {
        if (hw->nvm.blank_nvm_mode)
@@ -277,7 +156,7 @@ ice_acquire_nvm(struct ice_hw *hw, enum ice_aq_res_access_type access)
  *
  * This function will release NVM ownership.
  */
-static void ice_release_nvm(struct ice_hw *hw)
+void ice_release_nvm(struct ice_hw *hw)
 {
        if (hw->nvm.blank_nvm_mode)
                return;
@@ -514,31 +393,6 @@ enum ice_status ice_init_nvm(struct ice_hw *hw)
        return 0;
 }
 
-/**
- * ice_read_sr_buf - Reads Shadow RAM buf and acquire lock if necessary
- * @hw: pointer to the HW structure
- * @offset: offset of the Shadow RAM word to read (0x000000 - 0x001FFF)
- * @words: (in) number of words to read; (out) number of words actually read
- * @data: words read from the Shadow RAM
- *
- * Reads 16 bit words (data buf) from the SR using the ice_read_nvm_buf_aq
- * method. The buf read is preceded by the NVM ownership take
- * and followed by the release.
- */
-enum ice_status
-ice_read_sr_buf(struct ice_hw *hw, u16 offset, u16 *words, u16 *data)
-{
-       enum ice_status status;
-
-       status = ice_acquire_nvm(hw, ICE_RES_READ);
-       if (!status) {
-               status = ice_read_sr_buf_aq(hw, offset, words, data);
-               ice_release_nvm(hw);
-       }
-
-       return status;
-}
-
 /**
  * ice_nvm_validate_checksum
  * @hw: pointer to the HW struct
index 4245ef988edf47914677974fa76df1ec40292e23..7375f6b969195efbbaecc26f00d26f15e3a2f45c 100644 (file)
@@ -4,8 +4,12 @@
 #ifndef _ICE_NVM_H_
 #define _ICE_NVM_H_
 
+enum ice_status
+ice_acquire_nvm(struct ice_hw *hw, enum ice_aq_res_access_type access);
+void ice_release_nvm(struct ice_hw *hw);
 enum ice_status
 ice_read_flat_nvm(struct ice_hw *hw, u32 offset, u32 *length, u8 *data,
                  bool read_shadow_ram);
+enum ice_status ice_init_nvm(struct ice_hw *hw);
 enum ice_status ice_read_sr_word(struct ice_hw *hw, u16 offset, u16 *data);
 #endif /* _ICE_NVM_H_ */