Re: [PATCH 4/4] platform/x86: intel_telemetry: report debugfs failure

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

 





On 26-Sep-18 7:26 PM, Andy Shevchenko wrote:
On Mon, Sep 3, 2018 at 9:05 PM Rajneesh Bhardwaj
<rajneesh.bhardwaj@xxxxxxxxxxxxxxx> wrote:
On some Goldmont based systems such as ASRock J3455M the BIOS may not
enable the IPC1 device that provides access to the PMC and PUNIT. In
such scenarios, the ioss and pss resources from the platform device can
IOSS
PSS

Fine.


not be obtained and result in a invalid telemetry_plt_config.
What is telemetry_plt_config?

Internal data structure that holds platform config, maintained by the telemetry platform driver.


This is also applicable to the platforms where the BIOS supports IPC1
device under debug configurations but IPC1 is disabled by user or the
policy.

This change allows user to know the reason for not seeing entries under
/sys/kernel/debug/telemetry/* when there is no apparent failure at boot.

Cc: Matt Turner <matt.turner@xxxxxxxxx>
Cc: Len Brown <len.brown@xxxxxxxxx>
Cc: Souvik Kumar Chakravarty <souvik.k.chakravarty@xxxxxxxxx>
Cc: Kuppuswamy Sathyanarayanan <sathyanarayanan.kuppuswamy@xxxxxxxxx>

Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=198779

There should be not a blank line.

OK.


Acked-by: Matt Turner <matt.turner@xxxxxxxxx>
Signed-off-by: Rajneesh Bhardwaj <rajneesh.bhardwaj@xxxxxxxxxxxxxxx>
+exit:
+       pr_debug(pr_fmt(DRIVER_NAME) " Failed\n");
Completely useless.

Device core does it in generic way.

If i remove this print then perhaps there is no need of this patch. Reason to print this is that the platform driver / core driver does not show any error. In-fact they are even loaded in module table. OTOH, this debugfs interface fails. This is very confusing to the users if they check the lsmod output so i feel this print might help.






[Index of Archives]     [Linux Kernel Development]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux