DPDK patches and discussions
 help / color / mirror / Atom feed
From: Rasesh Mody <rmody@marvell.com>
To: Kevin Traynor <ktraynor@redhat.com>,
	"dev@dpdk.org" <dev@dpdk.org>,
	"Jerin Jacob Kollanukkaran" <jerinj@marvell.com>,
	"ferruh.yigit@intel.com" <ferruh.yigit@intel.com>
Cc: GR-Everest-DPDK-Dev <GR-Everest-DPDK-Dev@marvell.com>
Subject: Re: [dpdk-dev] [PATCH 1/3] net/qede/base: fix for FW bug
Date: Mon, 21 Oct 2019 18:52:52 +0000	[thread overview]
Message-ID: <BYAPR18MB2838BF0B620585F0952641E2B5690@BYAPR18MB2838.namprd18.prod.outlook.com> (raw)

Hi Kevin,

>From: Kevin Traynor <ktraynor@redhat.com>
>Sent: Monday, October 21, 2019 3:58 AM
>
>----------------------------------------------------------------------
>On 20/10/2019 06:20, Rasesh Mody wrote:
>> In our testing we have identified a critical FW bug. Performance is
>> degraded significantly for certain packet sizes with 8.40.25.0 FW.
>>
>> This patch updates the FW to version 8.40.33.0. The updated FW has a
>> fix to performance issue.
>>
>> The patch also adds initialization for FW overlay RAM as part of
>> hardware initialization which is required by the new FW.
>>
>> Fixes: 066f0d380a2f ("net/qede/base: update FW to 8.40.25.0")
>>
>
>Hi Rasesh, it should be:
>
>Fixes: 21c959e2cb10 ("net/qede/base: update FW to 8.40.25.0")
>

I could not find the 21c959e2cb10 reference, can you please let me know which tree is it coming from?

>> Signed-off-by: Rasesh Mody <rmody@marvell.com>
>> ---
>>  doc/guides/nics/qede.rst                      |   4 +-
>>  drivers/net/qede/base/common_hsi.h            |   6 +-
>>  drivers/net/qede/base/ecore.h                 |   4 +-
>>  drivers/net/qede/base/ecore_dev.c             |  19 ++-
>>  drivers/net/qede/base/ecore_hsi_debug_tools.h |   3 +
>>  drivers/net/qede/base/ecore_hw.c              |   4 +-
>>  drivers/net/qede/base/ecore_init_fw_funcs.c   | 138
>++++++++++++++++++
>>  drivers/net/qede/base/ecore_init_fw_funcs.h   |  35 +++++
>>  drivers/net/qede/base/ecore_int.c             |  14 +-
>>  drivers/net/qede/base/ecore_iro_values.h      | 122 ++++++++--------
>>  drivers/net/qede/qede_ethdev.h                |   2 +-
>>  drivers/net/qede/qede_main.c                  |   2 +-
>>  12 files changed, 274 insertions(+), 79 deletions(-)
>>
>> diff --git a/doc/guides/nics/qede.rst b/doc/guides/nics/qede.rst index
>> 471d98014..2f4045795 100644
>> --- a/doc/guides/nics/qede.rst
>> +++ b/doc/guides/nics/qede.rst
>> @@ -78,9 +78,9 @@ Supported QLogic Adapters  Prerequisites
>>  -------------
>>
>> -- Requires storm firmware version **8.37.7.0**. Firmware may be
>> available
>> +- Requires storm firmware version **8.40.33.0**. Firmware may be
>> +available
>>    inbox in certain newer Linux distros under the standard directory
>> -  ``E.g. /lib/firmware/qed/qed_init_values-8.37.7.0.bin``.
>> +  ``E.g. /lib/firmware/qed/qed_init_values-8.40.33.0.bin``.
>
>Not available in latest Fedora and Ubuntu releases

That is right, it would take some time before its available in latest Fedora/Ubuntu releases.
>
>>    If the required firmware files are not available then download it from
>>    `linux-firmware git repository
>> <https://urldefense.proofpoint.com/v2/url?u=http-3A__git.kernel.org_pu
>> b_scm_linux_kernel_git_firmware_linux-
>2Dfirmware.git_tree_qed&d=DwIFaQ
>>
>&c=nKjWec2b6R0mOyPaz7xtfQ&r=9aB46H7c7TYTnBun6ODgtnNLQdw3jNiVKH
>bs9eOyBS
>>
>U&m=LjD8kHeZxFHw1ql6Jz5s_XJfxPOIaBqLpyP5FDBNHQc&s=YOhhxcxca9mX
>dFNN7HjG
>> djQF5G22nnsfj1fy4S4cOCM&e= >`_
>
>Not available here either

We are working on getting 8.40.33.0 version into linux-firmware git tree.

Thanks!
-Rasesh
>
>>    or `QLogic Driver Download Center
><http://driverdownloads.qlogic.com/QLogicDriverDownloads_UI/DefaultNew
>Search.aspx>`_.
>
>Maybe here? 8.37 is the latest elsewhere.


             reply	other threads:[~2019-10-21 18:52 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-10-21 18:52 Rasesh Mody [this message]
2019-10-21 20:10 ` Kevin Traynor
2019-10-22  8:45 ` Ferruh Yigit
2019-10-22  9:13   ` Kevin Traynor
2019-10-30 19:15 ` Rasesh Mody
  -- strict thread matches above, loose matches on Subject: below --
2019-10-21 20:46 Rasesh Mody
2019-10-21 21:41 ` Kevin Traynor
2019-10-20  5:20 Rasesh Mody
2019-10-21 10:58 ` Kevin Traynor
2019-10-30 11:25   ` Jerin Jacob

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=BYAPR18MB2838BF0B620585F0952641E2B5690@BYAPR18MB2838.namprd18.prod.outlook.com \
    --to=rmody@marvell.com \
    --cc=GR-Everest-DPDK-Dev@marvell.com \
    --cc=dev@dpdk.org \
    --cc=ferruh.yigit@intel.com \
    --cc=jerinj@marvell.com \
    --cc=ktraynor@redhat.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).