openbmc.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
* Phosphor-fan-present only update fan Functional for xyz.openbmc_project.Inventory.Manager
@ 2020-12-16  3:01 Thu Nguyen
  2020-12-16  3:12 ` Matthew Barth
  0 siblings, 1 reply; 3+ messages in thread
From: Thu Nguyen @ 2020-12-16  3:01 UTC (permalink / raw)
  To: openbmc

Hi,

I would like to ask about current behavior of phosphor-fan-present.

When I unplug one fan, phosphor-fan-monitor will only update the 
Functional Dbus properties of /system/chassis/motherboard/FAN3_1/FAN3_1 
and /system/chassis/motherboard/FAN3_1 of inventory interface to false.

But not for "/xyz/openbmc_project/sensors/fan_tach/FAN3_1" of hwmon 
interface.

Below is test log:

~# busctl get-property xyz.openbmc_project.Inventory.Manager 
/xyz/openbmc_project/inventory/system/chassis/motherboard/FAN4_2/FAN4_2 
xyz.openbmc_project.State.Decorator.OperationalStatus Functional
b false
~# busctl get-property xyz.openbmc_project.Inventory.Manager 
/xyz/openbmc_project/inventory/system/chassis/motherboard/FAN4_2 
xyz.openbmc_project.State.Decorator.OperationalStatus Functional
b false
~# busctl get-property xyz.openbmc_project.Hwmon-1644477290.Hwmon1 
/xyz/openbmc_project/sensors/fan_tach/FAN4_2 
xyz.openbmc_project.State.Decorator.OperationalStatus Functional
b true

Is this the expected behavior?

Regards.

Thu Nguyen.


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: Phosphor-fan-present only update fan Functional for xyz.openbmc_project.Inventory.Manager
  2020-12-16  3:01 Phosphor-fan-present only update fan Functional for xyz.openbmc_project.Inventory.Manager Thu Nguyen
@ 2020-12-16  3:12 ` Matthew Barth
  2020-12-16  3:18   ` Thu Nguyen
  0 siblings, 1 reply; 3+ messages in thread
From: Matthew Barth @ 2020-12-16  3:12 UTC (permalink / raw)
  To: Thu Nguyen, openbmc


On 12/15/20 9:01 PM, Thu Nguyen wrote:
> Hi,
>
> I would like to ask about current behavior of phosphor-fan-present.
>
> When I unplug one fan, phosphor-fan-monitor will only update the 
> Functional Dbus properties of 
> /system/chassis/motherboard/FAN3_1/FAN3_1 and 
> /system/chassis/motherboard/FAN3_1 of inventory interface to false.
>
> But not for "/xyz/openbmc_project/sensors/fan_tach/FAN3_1" of hwmon 
> interface.
>
> Below is test log:
>
> ~# busctl get-property xyz.openbmc_project.Inventory.Manager 
> /xyz/openbmc_project/inventory/system/chassis/motherboard/FAN4_2/FAN4_2 
> xyz.openbmc_project.State.Decorator.OperationalStatus Functional
> b false
> ~# busctl get-property xyz.openbmc_project.Inventory.Manager 
> /xyz/openbmc_project/inventory/system/chassis/motherboard/FAN4_2 
> xyz.openbmc_project.State.Decorator.OperationalStatus Functional
> b false
> ~# busctl get-property xyz.openbmc_project.Hwmon-1644477290.Hwmon1 
> /xyz/openbmc_project/sensors/fan_tach/FAN4_2 
> xyz.openbmc_project.State.Decorator.OperationalStatus Functional
> b true
>
> Is this the expected behavior?
>
Yes. The phosphor-fan-monitor application is configured to only update 
the functional status of the fan (and/or individual fan rotors) within 
inventory. This is separate than the functional state of the device 
providing the tach sensors themselves since, for example, the sensor can 
be correctly reporting 0 RPMs for a feedback speed, however that should 
be shown as a nonfunctional fan needing to be replaced and not the 
device providing the tach sensors of the fan.

> Regards.
>
> Thu Nguyen.
>
Matt

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: Phosphor-fan-present only update fan Functional for xyz.openbmc_project.Inventory.Manager
  2020-12-16  3:12 ` Matthew Barth
@ 2020-12-16  3:18   ` Thu Nguyen
  0 siblings, 0 replies; 3+ messages in thread
From: Thu Nguyen @ 2020-12-16  3:18 UTC (permalink / raw)
  To: Matthew Barth, openbmc

On 12/16/20 10:12, Matthew Barth wrote:
>
> On 12/15/20 9:01 PM, Thu Nguyen wrote:
>> Hi,
>>
>> I would like to ask about current behavior of phosphor-fan-present.
>>
>> When I unplug one fan, phosphor-fan-monitor will only update the 
>> Functional Dbus properties of 
>> /system/chassis/motherboard/FAN3_1/FAN3_1 and 
>> /system/chassis/motherboard/FAN3_1 of inventory interface to false.
>>
>> But not for "/xyz/openbmc_project/sensors/fan_tach/FAN3_1" of hwmon 
>> interface.
>>
>> Below is test log:
>>
>> ~# busctl get-property xyz.openbmc_project.Inventory.Manager 
>> /xyz/openbmc_project/inventory/system/chassis/motherboard/FAN4_2/FAN4_2 
>> xyz.openbmc_project.State.Decorator.OperationalStatus Functional
>> b false
>> ~# busctl get-property xyz.openbmc_project.Inventory.Manager 
>> /xyz/openbmc_project/inventory/system/chassis/motherboard/FAN4_2 
>> xyz.openbmc_project.State.Decorator.OperationalStatus Functional
>> b false
>> ~# busctl get-property xyz.openbmc_project.Hwmon-1644477290.Hwmon1 
>> /xyz/openbmc_project/sensors/fan_tach/FAN4_2 
>> xyz.openbmc_project.State.Decorator.OperationalStatus Functional
>> b true
>>
>> Is this the expected behavior?
>>
> Yes. The phosphor-fan-monitor application is configured to only update 
> the functional status of the fan (and/or individual fan rotors) within 
> inventory. This is separate than the functional state of the device 
> providing the tach sensors themselves since, for example, the sensor 
> can be correctly reporting 0 RPMs for a feedback speed, however that 
> should be shown as a nonfunctional fan needing to be replaced and not 
> the device providing the tach sensors of the fan.
>
>> Regards.
>>
>> Thu Nguyen.
>>
> Matt

It is clear.


Thank Matthew,

Thu Nguyen.


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2020-12-16  3:20 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-12-16  3:01 Phosphor-fan-present only update fan Functional for xyz.openbmc_project.Inventory.Manager Thu Nguyen
2020-12-16  3:12 ` Matthew Barth
2020-12-16  3:18   ` Thu Nguyen

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).