Fault handling(Threshold exceeds/low) in Fan and NIC sensors

[Kumar Thangavel]

Classification: Internal

Hi Ed,

        In short, Our requirement is to take the actions when the fan fails. That action is platform specific.

        Fan failure :  This is based on Fan sensors. If fan sensor's tach values is less than 33%, will consider as a fan failure. So will take the actions to reduce the heat production in the system.
                                So that,  hosts, NIC and other power consuming modules.

        Dbus-sensor's already handles the threshold masking. We just use that threshold masking to take the platform specific actions.

        Please let us know if any clarifications needed.

Thanks,
Kumar.

-----Original Message-----
From: Ed Tanous <ed at tanous.net>
Sent: Saturday, November 14, 2020 2:14 AM
To: Kumar Thangavel <thangavel.k at hcl.com>
Cc: openbmc at lists.ozlabs.org; Velumani T-ERS,HCLTech <velumanit at hcl.com>; sdasari at fb.com; Patrick Williams <patrickw3 at fb.com>; Patrick Venture <venture at google.com>; Jae Hyun Yoo <jae.hyun.yoo at linux.intel.com>; Vernon Mauery <vernon.mauery at linux.intel.com>; Zhikui Ren <zhikui.ren at intel.com>
Subject: Re: Fault handling(Threshold exceeds/low) in Fan and NIC sensors

[CAUTION: This Email is from outside the Organization. Unless you trust the sender, Don’t click links or open attachments as it may be a Phishing email, which can steal your Information and compromise your Computer.]

On Fri, Nov 13, 2020 at 8:31 AM Kumar Thangavel <thangavel.k at hcl.com> wrote:
>
>          Could you please provide your suggestions on this.

I'm having a little trouble following your email.  Dbus-sensors has the ability to mask thresholds where appropriate, the platform specifics of which are already captured in the config file definition.
If there's some configurable masking needed that's new, we can certainly add it, but I'd recommend looking at the existing threshold masking before adding anything new to see if what's there meets your needs.  If you have some concrete things you'd like to see added, I'm happy to talk in more detail, just at this point, I have no idea what you're looking to solve, so you might want to be slightly more specific, and reference the existing threshold even masking in your proposed changes.

Cheers,

-Ed
::DISCLAIMER::
________________________________
The contents of this e-mail and any attachment(s) are confidential and intended for the named recipient(s) only. E-mail transmission is not guaranteed to be secure or error-free as information could be intercepted, corrupted, lost, destroyed, arrive late or incomplete, or may contain viruses in transmission. The e mail and its contents (with or without referred errors) shall therefore not attach any liability on the originator or HCL or its affiliates. Views or opinions, if any, presented in this email are solely those of the author and may not necessarily reflect the views or opinions of HCL or its affiliates. Any form of reproduction, dissemination, copying, disclosure, modification, distribution and / or publication of this message without the prior written consent of authorized representative of HCL is strictly prohibited. If you have received this email in error please delete it and notify the sender immediately. Before opening any email and/or attachments, please check them for viruses and other defects.
________________________________