Re: [PATCH v6 6/9] misc: smpro-errmon: Add Ampere's SMpro error monitor driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 24/12/2021 15:32, Greg Kroah-Hartman wrote:
On Fri, Dec 24, 2021 at 11:13:49AM +0700, Quan Nguyen wrote:
This commit adds Ampere's SMpro error monitor driver for monitoring
and reporting RAS-related errors as reported by SMpro co-processor
found on Ampere's Altra processor family.

Signed-off-by: Quan Nguyen <quan@xxxxxxxxxxxxxxxxxxxxxx>
---
Change in v6:
   + First introduced in v6 [Quan]

  drivers/mfd/smpro-mfd.c     |   1 +
  drivers/misc/Kconfig        |   7 +
  drivers/misc/Makefile       |   1 +
  drivers/misc/smpro-errmon.c | 571 ++++++++++++++++++++++++++++++++++++
  4 files changed, 580 insertions(+)
  create mode 100644 drivers/misc/smpro-errmon.c

You need Documentation/ABI/ updates when you add sysfs files.  Please do
that for your next version of this patch.


Thanks greg for the review,
Will update Documentation/ABI/testing/sysfs-* file in next version.

Also remember that sysfs is only "one value per file", this driver seems
to violate that in huge ways and is not ok.


These sysfs's does not return "multi value per file" but a single record of data, one example is to return the 48-byte ARM RAS Vendor specific error record.

Thanks
- Quan



[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux FS]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux