Commit 4650b751 authored by David S. Miller's avatar David S. Miller

Merge branch 'nfp-ethtool-flash-updates'

Jakub Kicinski says:

====================
nfp: ethtool flash updates

Dirk says:

This series adds the ability to update the control FW with ethtool.

It should be noted that the locking scheme here is to release the RTNL
lock before the flashing operation and to take it again afterwards to
ensure consistent state from the core code point of view. In this time,
we take a reference to the device to prevent the device being freed
while its being flashed.

This provides protection for the device being flashed while at the same
time not holding up any networking related functions which would
otherwise be locked out due to RTNL being held.
====================
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parents d31d38a0 7a741565
...@@ -47,6 +47,7 @@ ...@@ -47,6 +47,7 @@
#include <linux/interrupt.h> #include <linux/interrupt.h>
#include <linux/pci.h> #include <linux/pci.h>
#include <linux/ethtool.h> #include <linux/ethtool.h>
#include <linux/firmware.h>
#include "nfpcore/nfp.h" #include "nfpcore/nfp.h"
#include "nfpcore/nfp_nsp.h" #include "nfpcore/nfp_nsp.h"
...@@ -1269,6 +1270,57 @@ static int nfp_net_set_channels(struct net_device *netdev, ...@@ -1269,6 +1270,57 @@ static int nfp_net_set_channels(struct net_device *netdev,
return nfp_net_set_num_rings(nn, total_rx, total_tx); return nfp_net_set_num_rings(nn, total_rx, total_tx);
} }
static int
nfp_net_flash_device(struct net_device *netdev, struct ethtool_flash *flash)
{
const struct firmware *fw;
struct nfp_app *app;
struct nfp_nsp *nsp;
struct device *dev;
int err;
if (flash->region != ETHTOOL_FLASH_ALL_REGIONS)
return -EOPNOTSUPP;
app = nfp_app_from_netdev(netdev);
if (!app)
return -EOPNOTSUPP;
dev = &app->pdev->dev;
nsp = nfp_nsp_open(app->cpp);
if (IS_ERR(nsp)) {
err = PTR_ERR(nsp);
dev_err(dev, "Failed to access the NSP: %d\n", err);
return err;
}
err = request_firmware_direct(&fw, flash->data, dev);
if (err)
goto exit_close_nsp;
dev_info(dev, "Please be patient while writing flash image: %s\n",
flash->data);
dev_hold(netdev);
rtnl_unlock();
err = nfp_nsp_write_flash(nsp, fw);
if (err < 0) {
dev_err(dev, "Flash write failed: %d\n", err);
goto exit_rtnl_lock;
}
dev_info(dev, "Finished writing flash image\n");
exit_rtnl_lock:
rtnl_lock();
dev_put(netdev);
release_firmware(fw);
exit_close_nsp:
nfp_nsp_close(nsp);
return err;
}
static const struct ethtool_ops nfp_net_ethtool_ops = { static const struct ethtool_ops nfp_net_ethtool_ops = {
.get_drvinfo = nfp_net_get_drvinfo, .get_drvinfo = nfp_net_get_drvinfo,
.get_link = ethtool_op_get_link, .get_link = ethtool_op_get_link,
...@@ -1279,6 +1331,7 @@ static const struct ethtool_ops nfp_net_ethtool_ops = { ...@@ -1279,6 +1331,7 @@ static const struct ethtool_ops nfp_net_ethtool_ops = {
.get_sset_count = nfp_net_get_sset_count, .get_sset_count = nfp_net_get_sset_count,
.get_rxnfc = nfp_net_get_rxnfc, .get_rxnfc = nfp_net_get_rxnfc,
.set_rxnfc = nfp_net_set_rxnfc, .set_rxnfc = nfp_net_set_rxnfc,
.flash_device = nfp_net_flash_device,
.get_rxfh_indir_size = nfp_net_get_rxfh_indir_size, .get_rxfh_indir_size = nfp_net_get_rxfh_indir_size,
.get_rxfh_key_size = nfp_net_get_rxfh_key_size, .get_rxfh_key_size = nfp_net_get_rxfh_key_size,
.get_rxfh = nfp_net_get_rxfh, .get_rxfh = nfp_net_get_rxfh,
...@@ -1304,6 +1357,7 @@ const struct ethtool_ops nfp_port_ethtool_ops = { ...@@ -1304,6 +1357,7 @@ const struct ethtool_ops nfp_port_ethtool_ops = {
.get_strings = nfp_port_get_strings, .get_strings = nfp_port_get_strings,
.get_ethtool_stats = nfp_port_get_stats, .get_ethtool_stats = nfp_port_get_stats,
.get_sset_count = nfp_port_get_sset_count, .get_sset_count = nfp_port_get_sset_count,
.flash_device = nfp_net_flash_device,
.set_dump = nfp_app_set_dump, .set_dump = nfp_app_set_dump,
.get_dump_flag = nfp_app_get_dump_flag, .get_dump_flag = nfp_app_get_dump_flag,
.get_dump_data = nfp_app_get_dump_data, .get_dump_data = nfp_app_get_dump_data,
......
...@@ -51,6 +51,9 @@ ...@@ -51,6 +51,9 @@
#include "nfp_cpp.h" #include "nfp_cpp.h"
#include "nfp_nsp.h" #include "nfp_nsp.h"
#define NFP_NSP_TIMEOUT_DEFAULT 30
#define NFP_NSP_TIMEOUT_BOOT 30
/* Offsets relative to the CSR base */ /* Offsets relative to the CSR base */
#define NSP_STATUS 0x00 #define NSP_STATUS 0x00
#define NSP_STATUS_MAGIC GENMASK_ULL(63, 48) #define NSP_STATUS_MAGIC GENMASK_ULL(63, 48)
...@@ -93,6 +96,7 @@ enum nfp_nsp_cmd { ...@@ -93,6 +96,7 @@ enum nfp_nsp_cmd {
SPCODE_FW_LOAD = 6, /* Load fw from buffer, len in option */ SPCODE_FW_LOAD = 6, /* Load fw from buffer, len in option */
SPCODE_ETH_RESCAN = 7, /* Rescan ETHs, write ETH_TABLE to buf */ SPCODE_ETH_RESCAN = 7, /* Rescan ETHs, write ETH_TABLE to buf */
SPCODE_ETH_CONTROL = 8, /* Update media config from buffer */ SPCODE_ETH_CONTROL = 8, /* Update media config from buffer */
SPCODE_NSP_WRITE_FLASH = 11, /* Load and flash image from buffer */
SPCODE_NSP_SENSORS = 12, /* Read NSP sensor(s) */ SPCODE_NSP_SENSORS = 12, /* Read NSP sensor(s) */
SPCODE_NSP_IDENTIFY = 13, /* Read NSP version */ SPCODE_NSP_IDENTIFY = 13, /* Read NSP version */
}; };
...@@ -260,10 +264,10 @@ u16 nfp_nsp_get_abi_ver_minor(struct nfp_nsp *state) ...@@ -260,10 +264,10 @@ u16 nfp_nsp_get_abi_ver_minor(struct nfp_nsp *state)
} }
static int static int
nfp_nsp_wait_reg(struct nfp_cpp *cpp, u64 *reg, nfp_nsp_wait_reg(struct nfp_cpp *cpp, u64 *reg, u32 nsp_cpp, u64 addr,
u32 nsp_cpp, u64 addr, u64 mask, u64 val) u64 mask, u64 val, u32 timeout_sec)
{ {
const unsigned long wait_until = jiffies + 30 * HZ; const unsigned long wait_until = jiffies + timeout_sec * HZ;
int err; int err;
for (;;) { for (;;) {
...@@ -285,12 +289,13 @@ nfp_nsp_wait_reg(struct nfp_cpp *cpp, u64 *reg, ...@@ -285,12 +289,13 @@ nfp_nsp_wait_reg(struct nfp_cpp *cpp, u64 *reg,
} }
/** /**
* nfp_nsp_command() - Execute a command on the NFP Service Processor * __nfp_nsp_command() - Execute a command on the NFP Service Processor
* @state: NFP SP state * @state: NFP SP state
* @code: NFP SP Command Code * @code: NFP SP Command Code
* @option: NFP SP Command Argument * @option: NFP SP Command Argument
* @buff_cpp: NFP SP Buffer CPP Address info * @buff_cpp: NFP SP Buffer CPP Address info
* @buff_addr: NFP SP Buffer Host address * @buff_addr: NFP SP Buffer Host address
* @timeout_sec:Timeout value to wait for completion in seconds
* *
* Return: 0 for success with no result * Return: 0 for success with no result
* *
...@@ -300,10 +305,11 @@ nfp_nsp_wait_reg(struct nfp_cpp *cpp, u64 *reg, ...@@ -300,10 +305,11 @@ nfp_nsp_wait_reg(struct nfp_cpp *cpp, u64 *reg,
* -ENODEV if the NSP is not a supported model * -ENODEV if the NSP is not a supported model
* -EBUSY if the NSP is stuck * -EBUSY if the NSP is stuck
* -EINTR if interrupted while waiting for completion * -EINTR if interrupted while waiting for completion
* -ETIMEDOUT if the NSP took longer than 30 seconds to complete * -ETIMEDOUT if the NSP took longer than @timeout_sec seconds to complete
*/ */
static int nfp_nsp_command(struct nfp_nsp *state, u16 code, u32 option, static int
u32 buff_cpp, u64 buff_addr) __nfp_nsp_command(struct nfp_nsp *state, u16 code, u32 option, u32 buff_cpp,
u64 buff_addr, u32 timeout_sec)
{ {
u64 reg, ret_val, nsp_base, nsp_buffer, nsp_status, nsp_command; u64 reg, ret_val, nsp_base, nsp_buffer, nsp_status, nsp_command;
struct nfp_cpp *cpp = state->cpp; struct nfp_cpp *cpp = state->cpp;
...@@ -341,8 +347,8 @@ static int nfp_nsp_command(struct nfp_nsp *state, u16 code, u32 option, ...@@ -341,8 +347,8 @@ static int nfp_nsp_command(struct nfp_nsp *state, u16 code, u32 option,
return err; return err;
/* Wait for NSP_COMMAND_START to go to 0 */ /* Wait for NSP_COMMAND_START to go to 0 */
err = nfp_nsp_wait_reg(cpp, &reg, err = nfp_nsp_wait_reg(cpp, &reg, nsp_cpp, nsp_command,
nsp_cpp, nsp_command, NSP_COMMAND_START, 0); NSP_COMMAND_START, 0, NFP_NSP_TIMEOUT_DEFAULT);
if (err) { if (err) {
nfp_err(cpp, "Error %d waiting for code 0x%04x to start\n", nfp_err(cpp, "Error %d waiting for code 0x%04x to start\n",
err, code); err, code);
...@@ -350,8 +356,8 @@ static int nfp_nsp_command(struct nfp_nsp *state, u16 code, u32 option, ...@@ -350,8 +356,8 @@ static int nfp_nsp_command(struct nfp_nsp *state, u16 code, u32 option,
} }
/* Wait for NSP_STATUS_BUSY to go to 0 */ /* Wait for NSP_STATUS_BUSY to go to 0 */
err = nfp_nsp_wait_reg(cpp, &reg, err = nfp_nsp_wait_reg(cpp, &reg, nsp_cpp, nsp_status, NSP_STATUS_BUSY,
nsp_cpp, nsp_status, NSP_STATUS_BUSY, 0); 0, timeout_sec);
if (err) { if (err) {
nfp_err(cpp, "Error %d waiting for code 0x%04x to complete\n", nfp_err(cpp, "Error %d waiting for code 0x%04x to complete\n",
err, code); err, code);
...@@ -374,9 +380,18 @@ static int nfp_nsp_command(struct nfp_nsp *state, u16 code, u32 option, ...@@ -374,9 +380,18 @@ static int nfp_nsp_command(struct nfp_nsp *state, u16 code, u32 option,
return ret_val; return ret_val;
} }
static int nfp_nsp_command_buf(struct nfp_nsp *nsp, u16 code, u32 option, static int
const void *in_buf, unsigned int in_size, nfp_nsp_command(struct nfp_nsp *state, u16 code, u32 option, u32 buff_cpp,
void *out_buf, unsigned int out_size) u64 buff_addr)
{
return __nfp_nsp_command(state, code, option, buff_cpp, buff_addr,
NFP_NSP_TIMEOUT_DEFAULT);
}
static int
__nfp_nsp_command_buf(struct nfp_nsp *nsp, u16 code, u32 option,
const void *in_buf, unsigned int in_size, void *out_buf,
unsigned int out_size, u32 timeout_sec)
{ {
struct nfp_cpp *cpp = nsp->cpp; struct nfp_cpp *cpp = nsp->cpp;
unsigned int max_size; unsigned int max_size;
...@@ -429,7 +444,8 @@ static int nfp_nsp_command_buf(struct nfp_nsp *nsp, u16 code, u32 option, ...@@ -429,7 +444,8 @@ static int nfp_nsp_command_buf(struct nfp_nsp *nsp, u16 code, u32 option,
return err; return err;
} }
ret = nfp_nsp_command(nsp, code, option, cpp_id, cpp_buf); ret = __nfp_nsp_command(nsp, code, option, cpp_id, cpp_buf,
timeout_sec);
if (ret < 0) if (ret < 0)
return ret; return ret;
...@@ -442,12 +458,23 @@ static int nfp_nsp_command_buf(struct nfp_nsp *nsp, u16 code, u32 option, ...@@ -442,12 +458,23 @@ static int nfp_nsp_command_buf(struct nfp_nsp *nsp, u16 code, u32 option,
return ret; return ret;
} }
static int
nfp_nsp_command_buf(struct nfp_nsp *nsp, u16 code, u32 option,
const void *in_buf, unsigned int in_size, void *out_buf,
unsigned int out_size)
{
return __nfp_nsp_command_buf(nsp, code, option, in_buf, in_size,
out_buf, out_size,
NFP_NSP_TIMEOUT_DEFAULT);
}
int nfp_nsp_wait(struct nfp_nsp *state) int nfp_nsp_wait(struct nfp_nsp *state)
{ {
const unsigned long wait_until = jiffies + 30 * HZ; const unsigned long wait_until = jiffies + NFP_NSP_TIMEOUT_BOOT * HZ;
int err; int err;
nfp_dbg(state->cpp, "Waiting for NSP to respond (30 sec max).\n"); nfp_dbg(state->cpp, "Waiting for NSP to respond (%u sec max).\n",
NFP_NSP_TIMEOUT_BOOT);
for (;;) { for (;;) {
const unsigned long start_time = jiffies; const unsigned long start_time = jiffies;
...@@ -488,6 +515,17 @@ int nfp_nsp_load_fw(struct nfp_nsp *state, const struct firmware *fw) ...@@ -488,6 +515,17 @@ int nfp_nsp_load_fw(struct nfp_nsp *state, const struct firmware *fw)
fw->size, NULL, 0); fw->size, NULL, 0);
} }
int nfp_nsp_write_flash(struct nfp_nsp *state, const struct firmware *fw)
{
/* The flash time is specified to take a maximum of 70s so we add an
* additional factor to this spec time.
*/
u32 timeout_sec = 2.5 * 70;
return __nfp_nsp_command_buf(state, SPCODE_NSP_WRITE_FLASH, fw->size,
fw->data, fw->size, NULL, 0, timeout_sec);
}
int nfp_nsp_read_eth_table(struct nfp_nsp *state, void *buf, unsigned int size) int nfp_nsp_read_eth_table(struct nfp_nsp *state, void *buf, unsigned int size)
{ {
return nfp_nsp_command_buf(state, SPCODE_ETH_RESCAN, size, NULL, 0, return nfp_nsp_command_buf(state, SPCODE_ETH_RESCAN, size, NULL, 0,
......
...@@ -48,6 +48,7 @@ u16 nfp_nsp_get_abi_ver_minor(struct nfp_nsp *state); ...@@ -48,6 +48,7 @@ u16 nfp_nsp_get_abi_ver_minor(struct nfp_nsp *state);
int nfp_nsp_wait(struct nfp_nsp *state); int nfp_nsp_wait(struct nfp_nsp *state);
int nfp_nsp_device_soft_reset(struct nfp_nsp *state); int nfp_nsp_device_soft_reset(struct nfp_nsp *state);
int nfp_nsp_load_fw(struct nfp_nsp *state, const struct firmware *fw); int nfp_nsp_load_fw(struct nfp_nsp *state, const struct firmware *fw);
int nfp_nsp_write_flash(struct nfp_nsp *state, const struct firmware *fw);
int nfp_nsp_mac_reinit(struct nfp_nsp *state); int nfp_nsp_mac_reinit(struct nfp_nsp *state);
static inline bool nfp_nsp_has_mac_reinit(struct nfp_nsp *state) static inline bool nfp_nsp_has_mac_reinit(struct nfp_nsp *state)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment