From: Aaron Conole <aconole@redhat.com>
To: Ting Xu <ting.xu@intel.com>
Cc: dev@dpdk.org, wenzhuo.lu@intel.com, qiming.yang@intel.com,
qi.z.zhang@intel.com
Subject: Re: [dpdk-dev] [PATCH v2] net/ice: support device-specific DDP package loading
Date: Tue, 17 Sep 2019 08:47:26 -0400 [thread overview]
Message-ID: <f7to8zj144h.fsf@dhcp-25.97.bos.redhat.com> (raw)
In-Reply-To: <20190830141014.73749-1-ting.xu@intel.com> (Ting Xu's message of "Fri, 30 Aug 2019 14:10:14 +0000")
Ting Xu <ting.xu@intel.com> writes:
> This patch adds the feature that supports loading DDP package
> according to the device serial number. Prior to loading the
> default DDP package (ice.pkg), the driver will check for the
> presence of a device-specific DDP package with the name containing
> 64-bit PCIe Device Serial Number (ice-xxxxxxxxxxxxxxxx.pkg)
> during initialization. Users can use "lspci -vs" to get the device
> serial number.
> The pkg search path are /lib/firmware/updates/intel/ice/ddp/ and
> /lib/firmware/intel/ice/ddp/. If the package exists, the driver
> will download it to the device instead of the default one. The
> loaded package type (OS default or Comms) will be stored in
> ice_adapter->active_pkg_type. The package version is stored in
> ice_hw->active_pkg_ver. These fields can be used in other features.
>
> Signed-off-by: Ting Xu <ting.xu@intel.com>
> ---
This patch will fail when building the bnx2x driver:
In file included from ../drivers/net/bnx2x/bnx2x.h:45:0,
from ../drivers/net/bnx2x/bnx2x_ethdev.c:8:
/usr/include/linux/pci_regs.h:642:0: note: this is the location of the previous definition
> drivers/net/ice/ice_ethdev.c | 131 ++++++++++++++++++++++++++++++++++-
> drivers/net/ice/ice_ethdev.h | 8 +++
> 2 files changed, 138 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/net/ice/ice_ethdev.c b/drivers/net/ice/ice_ethdev.c
> index 686d6f00f..5d0689df0 100644
> --- a/drivers/net/ice/ice_ethdev.c
> +++ b/drivers/net/ice/ice_ethdev.c
> @@ -28,7 +28,15 @@ static const char * const ice_valid_args[] = {
> };
>
> #define ICE_DFLT_OUTER_TAG_TYPE ICE_AQ_VSI_OUTER_TAG_VLAN_9100
> +
> +/* DDP package search path */
> #define ICE_DFLT_PKG_FILE "/lib/firmware/intel/ice/ddp/ice.pkg"
> +#define ICE_PKG_FILE_SEARCH_PATH_DEFAULT "/lib/firmware/intel/ice/ddp/"
> +#define ICE_PKG_FILE_SEARCH_PATH_UPDATES "/lib/firmware/updates/intel/ice/ddp/"
> +
> +#define ICE_OS_DEFAULT_PKG_NAME "ICE OS Default Package"
> +#define ICE_COMMS_PKG_NAME "ICE COMMS Package"
> +#define ICE_MAX_PKG_FILENAME_SIZE 256
>
> int ice_logtype_init;
> int ice_logtype_driver;
> @@ -1265,15 +1273,133 @@ ice_pf_setup(struct ice_pf *pf)
> return 0;
> }
>
> +/* PCIe configuration space setting */
> +#define PCI_CFG_SPACE_SIZE 256
> +#define PCI_CFG_SPACE_EXP_SIZE 4096
> +#define PCI_EXT_CAP_ID(header) (int)((header) & 0x0000ffff)
> +#define PCI_EXT_CAP_NEXT(header) (((header) >> 20) & 0xffc)
> +#define PCI_EXT_CAP_ID_DSN 0x03
> +
> +static int
> +ice_pci_find_next_ext_capability(struct rte_pci_device *dev, int cap)
> +{
> + uint32_t header;
> + int ttl;
> + int pos = PCI_CFG_SPACE_SIZE;
> +
> + /* minimum 8 bytes per capability */
> + ttl = (PCI_CFG_SPACE_EXP_SIZE - PCI_CFG_SPACE_SIZE) / 8;
> +
> + if (rte_pci_read_config(dev, &header, 4, pos) < 0) {
> + PMD_INIT_LOG(ERR, "ice error reading extended capabilities\n");
> + return -1;
> + }
> +
> + /*
> + * If we have no capabilities, this is indicated by cap ID,
> + * cap version and next pointer all being 0.
> + */
> + if (header == 0)
> + return 0;
> +
> + while (ttl-- > 0) {
> + if (PCI_EXT_CAP_ID(header) == cap)
> + return pos;
> +
> + pos = PCI_EXT_CAP_NEXT(header);
> +
> + if (pos < PCI_CFG_SPACE_SIZE)
> + break;
> +
> + if (rte_pci_read_config(dev, &header, 4, pos) < 0) {
> + PMD_INIT_LOG(ERR, "ice error reading extended capabilities\n");
> + return -1;
> + }
> + }
> +
> + return 0;
> +}
> +
> +/* Extract device serial number from PCIe Configuration Space and
> + * determine the pkg file path according to the DSN.
> + */
> +static int
> +ice_pkg_file_search_path(struct rte_pci_device *pci_dev, char *pkg_file)
> +{
> + int pos;
> + char opt_ddp_filename[ICE_MAX_PKG_FILENAME_SIZE];
> + uint32_t dword;
> + uint32_t dsn_low, dsn_high;
> +
> + pos = ice_pci_find_next_ext_capability(pci_dev, PCI_EXT_CAP_ID_DSN);
> +
> + if (pos) {
> + rte_pci_read_config(pci_dev, &dword, 4, pos + 4);
> + dsn_low = dword;
> + rte_pci_read_config(pci_dev, &dword, 4, pos + 8);
> + dsn_high = dword;
> + snprintf(opt_ddp_filename, ICE_MAX_PKG_FILENAME_SIZE,
> + "ice-%08x%08x.pkg", dsn_high, dsn_low);
> + } else {
> + PMD_INIT_LOG(INFO, "Failed to read device serial number\n");
> + strncpy(pkg_file, ICE_DFLT_PKG_FILE, ICE_MAX_PKG_FILENAME_SIZE);
> +
> + return 0;
> + }
> +
> + strncpy(pkg_file, ICE_PKG_FILE_SEARCH_PATH_UPDATES,
> + ICE_MAX_PKG_FILENAME_SIZE);
> + if (!access(strncat(pkg_file, opt_ddp_filename,
> + ICE_MAX_PKG_FILENAME_SIZE - strlen(pkg_file) - 1), 0))
> + return 0;
> +
> + strncpy(pkg_file, ICE_PKG_FILE_SEARCH_PATH_DEFAULT,
> + ICE_MAX_PKG_FILENAME_SIZE);
> + if (!access(strncat(pkg_file, opt_ddp_filename,
> + ICE_MAX_PKG_FILENAME_SIZE - strlen(pkg_file) - 1), 0))
> + return 0;
> +
> + strncpy(pkg_file, ICE_DFLT_PKG_FILE, ICE_MAX_PKG_FILENAME_SIZE);
> + return 0;
> +}
> +
> +static enum ice_pkg_type
> +ice_get_pkg_type(struct ice_hw *hw)
> +{
> + enum ice_pkg_type package_type;
> +
> + /* store the activated package type (OS default or Comms) */
> + if (!strncmp((char *)hw->active_pkg_name, ICE_OS_DEFAULT_PKG_NAME,
> + ICE_PKG_NAME_SIZE))
> + package_type = ICE_PKG_TYPE_OS_DEFAULT;
> + else if (!strncmp((char *)hw->active_pkg_name, ICE_COMMS_PKG_NAME,
> + ICE_PKG_NAME_SIZE))
> + package_type = ICE_PKG_TYPE_COMMS;
> + else
> + package_type = ICE_PKG_TYPE_UNKNOWN;
> +
> + PMD_INIT_LOG(INFO, "Active package is: %d.%d.%d.%d, %s\n",
> + hw->active_pkg_ver.major, hw->active_pkg_ver.minor,
> + hw->active_pkg_ver.update, hw->active_pkg_ver.draft,
> + hw->active_pkg_name);
> +
> + return package_type;
> +}
> +
> static int ice_load_pkg(struct rte_eth_dev *dev)
> {
> struct ice_hw *hw = ICE_DEV_PRIVATE_TO_HW(dev->data->dev_private);
> - const char *pkg_file = ICE_DFLT_PKG_FILE;
> + char pkg_file[ICE_MAX_PKG_FILENAME_SIZE];
> int err;
> uint8_t *buf;
> int buf_len;
> FILE *file;
> struct stat fstat;
> + struct rte_pci_device *pci_dev = RTE_DEV_TO_PCI(dev->device);
> + struct ice_adapter *ad =
> + ICE_DEV_PRIVATE_TO_ADAPTER(dev->data->dev_private);
> +
> + ice_pkg_file_search_path(pci_dev, pkg_file);
>
> file = fopen(pkg_file, "rb");
> if (!file) {
> @@ -1313,6 +1439,9 @@ static int ice_load_pkg(struct rte_eth_dev *dev)
> PMD_INIT_LOG(ERR, "ice_copy_and_init_hw failed: %d\n", err);
> goto fail_exit;
> }
> +
> + ad->active_pkg_type = ice_get_pkg_type(hw);
> +
> err = ice_init_hw_tbls(hw);
> if (err) {
> PMD_INIT_LOG(ERR, "ice_init_hw_tbls failed: %d\n", err);
> diff --git a/drivers/net/ice/ice_ethdev.h b/drivers/net/ice/ice_ethdev.h
> index 5063960a8..d1d07641d 100644
> --- a/drivers/net/ice/ice_ethdev.h
> +++ b/drivers/net/ice/ice_ethdev.h
> @@ -124,6 +124,13 @@
> #define ICE_ETH_OVERHEAD \
> (RTE_ETHER_HDR_LEN + RTE_ETHER_CRC_LEN + ICE_VLAN_TAG_SIZE * 2)
>
> +/* DDP package type */
> +enum ice_pkg_type {
> + ICE_PKG_TYPE_UNKNOWN,
> + ICE_PKG_TYPE_OS_DEFAULT,
> + ICE_PKG_TYPE_COMMS,
> +};
> +
> struct ice_adapter;
>
> /**
> @@ -296,6 +303,7 @@ struct ice_adapter {
> uint32_t ptype_tbl[ICE_MAX_PKT_TYPE] __rte_cache_min_aligned;
> bool is_safe_mode;
> struct ice_devargs devargs;
> + enum ice_pkg_type active_pkg_type; /* loaded ddp package type */
> };
>
> struct ice_vsi_vlan_pvid_info {
prev parent reply other threads:[~2019-09-17 12:47 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-08-30 14:10 Ting Xu
2019-09-17 12:47 ` Aaron Conole [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=f7to8zj144h.fsf@dhcp-25.97.bos.redhat.com \
--to=aconole@redhat.com \
--cc=dev@dpdk.org \
--cc=qi.z.zhang@intel.com \
--cc=qiming.yang@intel.com \
--cc=ting.xu@intel.com \
--cc=wenzhuo.lu@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).