Hi everyone,<= /p>

I=92m Tung Nguyen, developer fo= r AmpereComputing - Altra system. While working with the IPMI commands I ha= ve a concern when comparing the design document and the implementation like= the following:

The state-management-and-extern= al-interfaces.md

The full mapping of Redfish and IPMI to xyz.op= enbmc_project.State.* is as follows:

Redfish &nb= sp; IPMI &= nbsp; &nb= sp;xyz.openbmc_project.State.Transition

ForceOff = power down = Chassis.Off

ForceOn &nb= sp; power up &nb= sp; Host.On

ForceRestart &nbs= p; hard reset &n= bsp; Host.ForceWarmReboot

GracefulRestart &= nbsp; &nb= sp; Host.Gracefu= lWarmReboot

GracefulShutdown = soft off = Host.Off

On &n= bsp; = ; power up = ; Host.On

PowerCycle (host on) power c= ycle Host.Reboot=

PowerCycle (host off) &= nbsp; &nbs= p; Chassis.PowerCycle

the IPMI =96 chassishandler.cpp= :

ipmi::R= spType<> ipmiChassisControl(uint8_t chassisControl)=

{

int rc =3D 0;

switch= (chassisControl)

{

case CMD_POWER_ON:

rc =3D initiate_state_transi= tion(State::Host::Transition::On);

break<= /span>;

case CMD_POWER_OFF<= span class=3D"s3">:

// This path would be hit in= 2 conditions.

// 1: When user asks for pow= er off using ipmi chassis command 0x04

// 2: Host asking for power = off post shutting down.

// If it's a host requested = power off, then need to nudge Softoff

// application that it needs= to stop the watchdog timer if running.

// If it is a user requested= power off, then this is not really

// needed. But then we need = to differentiate between user and host

// calling this same command=

// For now, we are going ahe= ad with trying to nudge the soft off and

// interpret the failure to = do so as a non softoff case

rc =3D stop_soft_off_timer()= ;

// Only request the Off tran= sition if the soft power off

// application is not runnin= g

if (rc < 0)

{

&n= bsp; // First create a file to in= dicate to the soft off application

&n= bsp; // that it should not run. N= ot doing this will result in State

&n= bsp; // manager doing a default s= oft power off when asked for power

// off.

indicate_no_softoff_needed()= ;

&n= bsp; // Now request the shutdown<= o:p>

rc =3D initiate_state_transi= tion(State::Host::Transition::Off);

}

else

{

&n= bsp; log<level::INFO>(<= span style=3D"font-size:10.0pt">"Soft off is running, so let shutdown target "

&= nbsp; "sto= p the host");

}

break<= /span>;

The redfish =96 systems.hpp:

else if (resetType =3D=3D "ForceOff")

{

command =3D "xyz.openbmc_project.State.Chassis.Transition.Off";

hostCommand =3D false;

}

Although the indicate_no_softof= f_needed() can prevent the host from soft off, but it = seems like a mismatch b/w the design document and the IPMI implementation.<= o:p>

So, my question: is it reasonable for = IPMI command ?

Reference:

https://github.com/openbmc/docs= /blob/master/designs/state-management-and-external-interfaces.md=

Best regards,=

Tung