From: Jie Hai <haijie1@huawei.com>
To: Ferruh Yigit <ferruh.yigit@amd.com>,
Dengdui Huang <huangdengdui@huawei.com>, <dev@dpdk.org>,
Thomas Monjalon <thomas@monjalon.net>,
David Marchand <david.marchand@redhat.com>
Cc: <stephen@networkplumber.org>, <lihuisong@huawei.com>,
<liuyonglong@huawei.com>, <fengchengwen@huawei.com>
Subject: Re: [PATCH 0/3] fix some bug for hns3
Date: Tue, 28 Nov 2023 11:33:44 +0800 [thread overview]
Message-ID: <f2a095f1-f2e8-f155-70f3-78496ce7e22d@huawei.com> (raw)
In-Reply-To: <a9423490-02d7-41b6-96f6-e8f4633e4317@amd.com>
On 2023/11/28 2:33, Ferruh Yigit wrote:
> On 11/27/2023 1:55 PM, Jie Hai wrote:
>> On 2023/11/27 21:39, Dengdui Huang wrote:
>>> Dengdui Huang (3):
>>> net/hns3: fix reset detect be ignored
>>> net/hns3: fix VF wrong clear reset status
>>> net/hns3: fix the VF reset interrupted possibly
>> Hi, Dengdui,
>>
>> Good fix.
>> For the patchset,
>> Acked-by: Jie Hai <haijie1@huawei.com>
>>
>
>
> Hi Jie, Dengdui, is this set for this release, which is a few days away?
> My suggestion is to wait next version as issues doesn't look critical.
>
>
> I applied set to next-net, but leaving decision to pull it for the
> release or not to Thomas/David.
>
> Applied to dpdk-next-net/main, thanks.
>
Hi, Ferruh, Thomas and David,
I'm sorry that this problem is a little serious.
a) Without [PATCH 2/3], when the VF executes the
hns3_clear_reset_event(), unexpected memory is written.
As a result, unexpected errors occur in the program or even the app may
crash.
b) Without [PATCH 3/3], there is a low probability that the app is
unavailable after the FLR reset is done.
c) In addition, some LTS versions have the same problem.
Therefore, we hope that the problem can be solved in the current version.
Thanks,
Jie Hai
> .
next prev parent reply other threads:[~2023-11-28 3:33 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-11-27 13:39 Dengdui Huang
2023-11-27 13:39 ` [PATCH 1/3] net/hns3: fix reset detect be ignored Dengdui Huang
2023-11-27 13:39 ` [PATCH 2/3] net/hns3: fix VF wrong clear reset status Dengdui Huang
2023-11-27 13:39 ` [PATCH 3/3] net/hns3: fix the VF reset interrupted possibly Dengdui Huang
2023-11-27 13:55 ` [PATCH 0/3] fix some bug for hns3 Jie Hai
2023-11-27 18:33 ` Ferruh Yigit
2023-11-28 3:33 ` Jie Hai [this message]
2023-11-28 10:31 ` Thomas Monjalon
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=f2a095f1-f2e8-f155-70f3-78496ce7e22d@huawei.com \
--to=haijie1@huawei.com \
--cc=david.marchand@redhat.com \
--cc=dev@dpdk.org \
--cc=fengchengwen@huawei.com \
--cc=ferruh.yigit@amd.com \
--cc=huangdengdui@huawei.com \
--cc=lihuisong@huawei.com \
--cc=liuyonglong@huawei.com \
--cc=stephen@networkplumber.org \
--cc=thomas@monjalon.net \
/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).