netdev - Re: [PATCH net-next 2/2] net: hns3: add vf fault process in hns3 ras

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <a5a603bb-ae04-f274-5d68-f8d63a4bf13b@huawei.com>
Date:   Tue, 17 Jan 2023 15:04:15 +0800
From:   "wangjie (L)" <wangjie125@...wei.com>
To:     Leon Romanovsky <leon@...nel.org>, Hao Lan <lanhao@...wei.com>
CC:     <davem@...emloft.net>, <kuba@...nel.org>,
        <yisen.zhuang@...wei.com>, <salil.mehta@...wei.com>,
        <edumazet@...gle.com>, <pabeni@...hat.com>,
        <richardcochran@...il.com>, <shenjian15@...wei.com>,
        <netdev@...r.kernel.org>
Subject: Re: [PATCH net-next 2/2] net: hns3: add vf fault process in hns3 ras



On 2023/1/13 14:51, Leon Romanovsky wrote:
> On Fri, Jan 13, 2023 at 10:08:29AM +0800, Hao Lan wrote:
>> From: Jie Wang <wangjie125@...wei.com>
>>
>> Currently hns3 driver supports vf fault detect feature. Several ras caused
>> by VF resources don't need to do PF function reset for recovery. The driver
>> only needs to reset the specified VF.
>>
>> So this patch adds process in ras module. New process will get detailed
>> information about ras and do the most correct measures based on these
>> accurate information.
>>
>> Signed-off-by: Jie Wang <wangjie125@...wei.com>
>> Signed-off-by: Hao Lan <lanhao@...wei.com>
>> ---
>>  drivers/net/ethernet/hisilicon/hns3/hnae3.h   |   1 +
>>  .../hns3/hns3_common/hclge_comm_cmd.h         |   1 +
>>  .../hisilicon/hns3/hns3pf/hclge_err.c         | 113 +++++++++++++++++-
>>  .../hisilicon/hns3/hns3pf/hclge_err.h         |   2 +
>>  .../hisilicon/hns3/hns3pf/hclge_main.c        |   3 +-
>>  .../hisilicon/hns3/hns3pf/hclge_main.h        |   1 +
>>  6 files changed, 115 insertions(+), 6 deletions(-)
>
> Why is it good idea to reset VF from PF?
> What will happen with driver bound to this VF?
> Shouldn't PCI recovery handle it?
>
> Thanks
> .
PF doesn't reset VF directly. These VF faults are detected by hardware,
and only reported to PF. PF get the VF id from firmware, then notify the 
VF that it needs reset. VF will do reset after receive the request.

These hardware faults are not standard PCI ras event, so we prefer to 
use MSIx path.
>