[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <f25514f8-2d34-5911-b29f-a520d1df3db4@huawei.com>
Date: Fri, 24 Jun 2022 11:25:10 +0800
From: "yekai(A)" <yekai13@...wei.com>
To: Greg KH <gregkh@...uxfoundation.org>
CC: <herbert@...dor.apana.org.au>, <linux-crypto@...r.kernel.org>,
<linux-accelerators@...ts.ozlabs.org>,
<linux-kernel@...r.kernel.org>, <linuxarm@...wei.com>,
<zhangfei.gao@...aro.org>, <wangzhou1@...ilicon.com>
Subject: Re: [PATCH v4 2/3] Documentation: add a isolation strategy sysfs node
for uacce
On 2022/6/23 17:01, Greg KH wrote:
> On Thu, Jun 23, 2022 at 02:14:51PM +0800, Kai Ye wrote:
>> Update documentation describing sysfs node that could help to
>> configure isolation method command for users in th user space.
>>
>> Signed-off-by: Kai Ye <yekai13@...wei.com>
>> ---
>> Documentation/ABI/testing/sysfs-driver-uacce | 18 ++++++++++++++++++
>> 1 file changed, 18 insertions(+)
>>
>> diff --git a/Documentation/ABI/testing/sysfs-driver-uacce b/Documentation/ABI/testing/sysfs-driver-uacce
>> index 08f2591138af..8784efa96e01 100644
>> --- a/Documentation/ABI/testing/sysfs-driver-uacce
>> +++ b/Documentation/ABI/testing/sysfs-driver-uacce
>> @@ -19,6 +19,24 @@ Contact: linux-accelerators@...ts.ozlabs.org
>> Description: Available instances left of the device
>> Return -ENODEV if uacce_ops get_available_instances is not provided
>>
>> +What: /sys/class/uacce/<dev_name>/isolate_strategy
>> +Date: Jun 2022
>> +KernelVersion: 5.20
>> +Contact: linux-accelerators@...ts.ozlabs.org
>> +Description: A sysfs node that used to configures the hardware error
>> + isolation method command. The command can be parsed
>> + in correct driver. e.g. If the device slot reset frequency
>> + exceeds the preset value in a time window, the device will be
>> + isolated.
>
> What is the "command"? What is being parsed? This needs to be
> documented a lot more here, this is very vague and not obvious at all.
>
>
This command is a string command issued by the user. After the command
is configured, the acc driver parses the command.
>> +
>> +What: /sys/class/uacce/<dev_name>/isolate
>> +Date: Jun 2022
>> +KernelVersion: 5.20
>> +Contact: linux-accelerators@...ts.ozlabs.org
>> +Description: A sysfs node that show the device isolated state. The value 0
>> + means that the device is working. The value 1 means that the
>> + device has been isolated.
>
> Are these read-only? Write only? read/write?
this node is read-only
>
> thanks,
>
> greg k-h
> .
>
Thanks
Kai
Powered by blists - more mailing lists