Re: [PATCH v5 3/5] PCI/AER: Add sysfs attributes to provide breakdown of AERs

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

 



On 6/21/2018 5:25 PM, Rajat Jain wrote:
On Thu, Jun 21, 2018 at 11:48 AM, Bjorn Helgaas <helgaas@xxxxxxxxxx> wrote:
[+cc Tyler for AER dmesg decoding]

- Tyler posted a patch [1] to update those dmesg strings so they match
   the way lspci decodes them.  I really liked that update, but we
   never quite finished it.  If we're going to do that, it would be
   nice to do it first, so we don't publish new sysfs files, then
   immediately change the labels used in them.
Sure, I guess you can push them in the right order.
The way the prints are handled has already been unified in 4.18rc1:

https://elixir.bootlin.com/linux/v4.18-rc1/source/drivers/pci/pcie/aer.c#L636

So that patch isn't needed anymore in it's entirety.
- IIRC, Tyler's patch had the nice property of changing the strings so
   each error name had no spaces, which would make it a little easier
   to parse this sysfs file: each line would be a single identifier
   followed by a single number (I would probably remove the "=" from
   the middle).

Will do.
Would you like me to send a patch with just the string changes?

Thanks,
Tyler
[1] https://lkml.kernel.org/r/1518034285-3543-1-git-send-email-tbaicar@xxxxxxxxxxxxxx

--
Qualcomm Datacenter Technologies, Inc. as an affiliate of Qualcomm Technologies, Inc.
Qualcomm Technologies, Inc. is a member of the Code Aurora Forum,
a Linux Foundation Collaborative Project.




[Index of Archives]     [DMA Engine]     [Linux Coverity]     [Linux USB]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Greybus]

  Powered by Linux